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

[Michael Rubinsky]

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?


Thanks,
mike

--
The Horde Project (www.horde.org)
mrubinsk at horde.org

"Time just hates me. That's why it made me an adult." - Josh Joplin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: PGP Digital Signature
Url : http://lists.horde.org/archives/dev/attachments/20080606/dbe976a5/attachment.bin