[Tickets #12636] Core: Fix 'activesync_replyposition' pref readout

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12636
------------------------------------------------------------------------------
  Ticket             | 12636
  Created By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | Core: Fix 'activesync_replyposition' pref readout
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2013-08-28 17:27) wrote:

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.