[dev] [commits] Horde branch master updated. d08d48fc2bd3e825764bd142926a3c1215a1dc88

[Michael M Slusarz]

Quoting Michael J Rubinsky <mrubinsk at horde.org>:

> Quoting Michael J Rubinsky <mrubinsk at horde.org>:
>
>> Quoting Michael J Rubinsky <mrubinsk at horde.org>:
>>
>>> Quoting Michael M Slusarz <slusarz at horde.org>:
>>>
>>>> Quoting "Michael J. Rubinsky" <mrubinsk at horde.org>:
>>>>
>>>>> The branch "master" has been updated.
>>>>> The following is a summary of the commits.
>>>>>
>>>>> from: d69fdd081aab00f0a24fd4d4edcfc6bd5f5c9d8d
>>>>>
>>>>> d08d48f Fix method name.
>>>>>
>>>>> -----------------------------------------------------------------------
>>>>>
>>>>> commit d08d48fc2bd3e825764bd142926a3c1215a1dc88
>>>>> Author: Michael J Rubinsky <mrubinsk at horde.org>
>>>>> Date:   Wed Jul 24 16:41:17 2013 -0400
>>>>>
>>>>> Fix method name.
>>>>>
>>>>> Sigh.
>>>>>
>>>>> .../Horde/History/4_horde_history_add_compositeindex.php          
>>>>>  |    2 +-
>>>>> .../Horde/History/5_horde_history_remove_compositeindex.php       
>>>>>  |    2 +-
>>>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>>>>
>>>>> http://git.horde.org/horde-git/-/commit/d08d48fc2bd3e825764bd142926a3c1215a1dc88
>>>>
>>>> STOP!  I already did this and released a new version of the package.
>>>
>>> Your fix removed the wrong indexes. This removed the two  
>>> individual indexes on each column, not the composite index  
>>> consisting of both columns.
>>
>> I've got a fix locally, I'll commit shortly.
>
> sigh. It's already released. I'll have to add a new migration to  
> remove the correct index and re-add the erroneously removed indexes.

I blame Jan for not being around :)

michael

___________________________________
Michael Slusarz [slusarz at horde.org]