[commits] Horde branch master updated. fb5ad87109f0e9edd89faa09b2a1905131c608d2

Michael J. Rubinsky mrubinsk at horde.org
Fri Aug 30 17:25:43 UTC 2013


The branch "master" has been updated.
The following is a summary of the commits.

from: ad96aa8222d281186237f06f92c28599cd15442f

3688d8c Fix 'activesync_replyposition' pref readout
fb5ad87 No need for this parameter, this is Core, we can always get from prefs.

-----------------------------------------------------------------------

commit 3688d8c13448e002c04b80e8491cedb73366636b
Author: Thomas Jarosch <thomas.jarosch at intra2net.com>
Date:   Wed Aug 28 19:21:35 2013 +0200

    Fix 'activesync_replyposition' pref readout
    
    The moment we read out the 'activesync_replyposition'
    in the constructor was before the user is authenticated.
    
    Therefore the prefs backend will return the default value,
    which is 'bottom'. Setting the pref to 'top' via the UI never worked.
    
    Fix it by delaying the read out until the reply/forward action.
    Keep "reply_top" parameter handling as compatible as possible.
    
    Bug 12636
    
    Signed-off-by: Michael J Rubinsky <mrubinsk at horde.org>

 framework/Core/lib/Horde/Core/ActiveSync/Driver.php         |   14 ++++++----
 framework/Core/lib/Horde/Core/Factory/ActiveSyncBackend.php |    3 +-
 2 files changed, 9 insertions(+), 8 deletions(-)

http://git.horde.org/horde-git/-/commit/3688d8c13448e002c04b80e8491cedb73366636b

-----------------------------------------------------------------------

commit fb5ad87109f0e9edd89faa09b2a1905131c608d2
Author: Michael J Rubinsky <mrubinsk at horde.org>
Date:   Fri Aug 30 13:24:58 2013 -0400

    No need for this parameter, this is Core, we can always get from prefs.

 framework/Core/lib/Horde/Core/ActiveSync/Driver.php |    9 ++-------
 1 files changed, 2 insertions(+), 7 deletions(-)

http://git.horde.org/horde-git/-/commit/fb5ad87109f0e9edd89faa09b2a1905131c608d2




More information about the commits mailing list