Re: Problem with sub-galleries and XP-publishing???

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" <[email protected]>:

>>>> 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


-- 
ansel mailing list - Join the hunt: http://horde.org/bounties/#ansel
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: [email protected]