[Tickets #12180] Re: Unification of instant messenger attributes

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12180
------------------------------------------------------------------------------
  Ticket             | 12180
  Updated By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | Unification of instant messenger attributes
  Queue              | Turba
  Version            | Git master
  Type               | Bug
  State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2013-04-18 07:16) wrote:

>>> At the very least, this will also need to be documented in UPGRADING.
>>
>> Exactly. That's why I mentioned it :) Let's just drop the hunk from
>> the patch that removes the attribute for the FRAMEWORK_5_0 branch.
>> The rename inside the Kolab section is safe.
>
> Well, none of this will be in FRAMEWORK_5_0, and dropping just the  
> deletion of the attribute isn't enough to avoid sort-of-bc-issues  
> because we are changing the name of the attribute when it is used in  
> code. I.e., keeping it attributes might avoid undefined index errors  
> (or whatever), but the attribute won't be used for anything in the  
> code.
>
> In other words, we should do all of it in master, and document the  
> need to change the attribute in any custom backend configurations in  
> UPGRADING.

Sounds with me, it's a rather cosmetic issue anyway.

For my users, I'll keep it in my local FRAMEWORK_5_0 branch
since I have full control of all the packages / configs involved.