[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         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | Unification of instant messenger attributes
  Queue              | Turba
  Version            | Git master
  Type               | Bug
-State              | Feedback
+State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
-Owners             |
+Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2013-04-17 16:28) 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.