| Summary | Re-fix setting initial_page value in lib/Dynamic/Mailbox.php |
| Queue | IMP |
| Queue Version | Git master |
| Type | Bug |
| State | Resolved |
| Priority | 1. Low |
| Owners | slusarz (at) horde (dot) org |
| Requester | mm (at) freebsd (dot) org |
| Created | 07/17/2013 (4322 days ago) |
| Due | |
| Updated | 07/22/2013 (4317 days ago) |
| Assigned | 07/17/2013 (4322 days ago) |
| Resolved | 07/22/2013 (4317 days ago) |
| Milestone | |
| Patch | Yes |
State ⇒ Resolved
Priority ⇒ 1. Low
commit ba3bfe767a746ae17a9a993d6b3d2ac573348de2
Author: Michael M Slusarz <slusarz@horde.org>
Date: Wed Jul 17 16:18:01 2013 -0600
Bug #12468imp/lib/Dynamic/Mailbox.php | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/ba3bfe767a746ae17a9a993d6b3d2ac573348de2
Assigned to Michael Slusarz
Priority ⇒ 2. Medium
New Attachment: Mailbox.php.patch
Patch ⇒ Yes
Milestone ⇒
Summary ⇒ Re-fix setting initial_page value in lib/Dynamic/Mailbox.php
Type ⇒ Bug
State ⇒ Unconfirmed
Queue ⇒ IMP
File: imp/lib/Dynamic/Mailbox.php comment:
Comment: "Fix setting initial_page value in dynamic view if it is set
to the folders page"
This commit does not correctly fix the issue. The code:
'initial_page' => is_null($initial_page =
IMP_Auth::getInitialPage()->mbox) ?: $initial_page->form_to,
actually means:
'initial_page' => is_null($initial_page =
IMP_Auth::getInitialPage()->mbox) ? true : $initial_page->form_to,
and this is what you probably wanted:
'initial_page' => is_null($initial_page =
IMP_Auth::getInitialPage()->mbox) ? null : $initial_page->form_to,