[Tickets #12468] Re-fix setting initial_page value in lib/Dynamic/Mailbox.php

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12468
------------------------------------------------------------------------------
  Ticket             | 12468
  Created By         | mm at freebsd.org
  Summary            | Re-fix setting initial_page value in
                     | lib/Dynamic/Mailbox.php
  Queue              | IMP
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 2. Medium
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


mm at freebsd.org (2013-07-17 07:55) wrote:

Commit: de142d0993ee46770562da9b9d38a02cece42d0d
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,



mm at freebsd.org (2013-07-17 07:55) uploaded: Mailbox.php.patch

http://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=Mailbox.php.patch&ticket=12468&fn=%2FMailbox.php.patch