[dev] [cvs] commit: ansel/scripts/upgrades 2008-04-28_sql_share_migration.php

[Jan Schneider]

Zitat von "Michael Rubinsky" <mrubinsk at horde.org>:

> Quoting "Jan Schneider" <jan at horde.org>:
>
>> Zitat von "Michael Rubinsky" <mike at theupstairsroom.com>:
>>
>>> Quoting "Jan Schneider" <jan at horde.org>:
>>>
>>>> jan         2008-06-06 12:06:54 EDT
>>>>
>>>> Modified files:
>>>>   scripts/upgrades     2008-04-28_sql_share_migration.php
>>>> Log:
>>>> The old shares didn't have categories stored, but the must be not
>>>> null in the
>>>> backend.
>>>
>>> The categories were stored in the *original* datatree
>>> structure...then  were migrated to the gallery table with the
>>> 2008_01_07_sql_galleries.php migration script...and this script
>>> already checks for an empty or null value when migrating the
>>> ansel_gallery table data.
>>
>> Where ever they were supposed to be stored, I got NOT NULL constraint
>> violations, because the attribute_category values haven't been
>> retrieved or set anywhere in this script.
>
> Right, but unless I'm missing something in your patch, I don't see  
> how  setting the value where you did would make any difference? $row  
>  wouldn't persist past that one loop, right?

That's absolutely correct. Since I already migrated my galleries after  
the initial patch, I didn't notice that, thanks for the pointer.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/