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

Ah, that makes sense :)

I committed a slight variation (indented instead of parent:child), thanks!

--Ben

Quoting "Heath S. Hendrickson" <[email protected]>:

> 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" <[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]