[ansel] Problem with sub-galleries and XP-publishing???

[Heath S. Hendrickson]

The problem wasn't so much one of looks as it was ordering.  For 
whatever reason on my system the galleries aren't shown in the proper 
parent->child order...

Here's a screenshot of what I was looking at:

http://www.outerspaceconsultants.com/Files/xppub.jpg

The correct relationship for the galleries is:

Collection
Anna's Galleries
    -> Winter 2003
    -> Thanksgiving 2003
    -> Picture of the Day
PrivateGalleries
    ->Laetitia Casta
    ->Foxes


h

Ben Chavet wrote:

> This has been committed, thanks!
>
> Also, I opted not to use your templates/xppublish/list.inc patch for 
> the sake of
> simplicity.  That, and I personally think the indented subgalleries 
> look better
> than the parent:subgallery scheme.
>
> --Ben
>
> Quoting "Heath S. Hendrickson" <heath at outerspaceconsultants.com>:
>
>>>>> Also, XP-publishing is still broken for me.  I'm not sure if it's 
>>>>> related to the same thing, but my guess is that it is...  I'll see 
>>>>> if I can do a quick fix, otherwise I'll leave it up to Ben to fix.
>>>>>
>>>>> h
>>>>
>>>>
>>>> Attached are patches to xppublish.php and 
>>>> templates/xppublish/list.inc that accomodate the new output of 
>>>> Ansel::listGalleries().  I had to add a little extra code to deal 
>>>> with the different array format for groupby=owner, but not that much.
>>>
>>>
>>> Okay, scap that patch... I missed another place where I needed to 
>>> make the modifications... I'm thinking it'll just be easier to 
>>> create a new Ansel function called checkPerms($galleryId, $perm) 
>>> that will return True/False.  It'll save on processing overhead of 
>>> calling Ansel::listGalleries(PERM_EDIT) several times when all you 
>>> need is to know if the user has a certain permission for the gallery.
>>>
>>> h
>>>
>> Attached is a revised patch that makes use of the
>> $gallery->hasPermission() call to check the permissions before selecting
>> the gallery and subsequently publishing the photos... I tested it on my
>> system and it works.  This way there is no klunky code to deal with
>> groupby differences in listGalleries().
>>
>> The previous patch for templates/xppublish/list.inc is still valid and
>> needs to be applied, however.
>>
>> h
>
>
>