[dev] [commits] Horde branch master updated. 447de1bf8b7baa20fdc96649ab6c9af44bcd8208

Jan Schneider jan at horde.org
Tue Nov 6 21:20:29 UTC 2012


Zitat von Michael M Slusarz <slusarz at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> Zitat von Michael M Slusarz <slusarz at horde.org>:
>>
>>> Quoting Jan Schneider <jan at horde.org>:
>>>
>>>> Zitat von Michael M Slusarz <slusarz at horde.org>:
>>>>
>>>>> The branch "master" has been updated.
>>>>> The following is a summary of the commits.
>>>>>
>>>>> from: ceb6993f44e840dfe5bd20ad64cd529773482050
>>>>>
>>>>> 447de1b Add H5 upgrade task to remove existing sending_charset pref
>>>>>
>>>>> -----------------------------------------------------------------------
>>>>>
>>>>> commit 447de1bf8b7baa20fdc96649ab6c9af44bcd8208
>>>>> Author: Michael M Slusarz <slusarz at horde.org>
>>>>> Date:   Tue Nov 6 11:55:19 2012 -0700
>>>>>
>>>>> Add H5 upgrade task to remove existing sending_charset pref
>>>>>
>>>>> horde/lib/LoginTasks/SystemTask/Upgrade.php |   15 ++++++++++++++-
>>>>> 1 files changed, 14 insertions(+), 1 deletions(-)
>>>>>
>>>>> http://git.horde.org/horde-git/-/commit/447de1bf8b7baa20fdc96649ab6c9af44bcd8208
>>>>
>>>> Changelog?
>>>
>>> This is another one that doesn't make any sense.  People upgrading  
>>> to 5.0.1 WON'T see this, so putting a changelog entry in there is  
>>> just confusing.
>>>
>>> And this is really behind the scenes stuff anyway (after the  
>>> discussion on the mailing list today, realized that this 100% must  
>>> be reset or else things won't work properly).
>>>
>>> Those that are already upgraded are screwed.  They will need to  
>>> manually change this preference value if it is set in the storage  
>>> backend.
>>
>> If this isn't triggered for people who already upgrade, and if we  
>> add this to 5.0.1 anyway, why don't you set the upgrade version to  
>> 5.0.1?
>
> There is no way to catch those who have explicitly changed this  
> value since 5.0.  In other words, you install 5.0, you change this  
> value, you upgrade to 5.0.1 - sending_charset value must not be  
> deleted.

The question is whether there are more users who already upgraded from  
H4 to 5.0.0 than users that did a fresh install of 5.0.0, unlocked the  
preference, and changed its value. I would say the former sounds more  
likely.
-- 
Jan Schneider
The Horde Project
http://www.horde.org/



More information about the dev mailing list