Revisions for ansel/gallery.php

From Duck: Add default gallery pref (Bug: 7016)
From Me: Fix some variable names, add a required.png image before the
gallery name field. Probably should make this a Horde_Form form eventually

Allow Ansel::selectGalleries() to take a gallery id that should be ignored
when building the gallery select menu. This prevents the users from trying
to assign a gallery to an inappropriate parent...like one of it's own children.

pretty urls wherever possible

This is not a function, cannot return

fix variable name - fixes creation of sub galleries when navigating to the
gallery creation form from the "New Gallery" menu item.

...more language tweaking for gettext strings

More gettext string clean up, standardization etc...

Missed one...clear out the gallery tree cache, and prune some dead code

- Delegate the rendering of gallery views to a Ansel_View_Renderer_* class
and make Ansel_Gallery_View responsible for knowing what Renderer to
instantiate based on the style we are requesting. This helps keep the URLs
clean and pretty as we no longer have to pass around the view in the url
when not using the "default" gallery view. (GalleryLightbox for instance)

- Pass a callback function to the Pager that re-writes the urls to something
like: horde/ansel/gallery/1/page/1 instead of tacking the page onto the URL
as a GET parameter. If not using "pretty" url generation, the callback is
not set, and of course, a callback can also be specified via the api.

make sure $old_gallery_id is defined.

Initialize variable

Allow changing the gallery parent from the properties page

Remove references to deprecated list.php.

wrap

Move the createGallery() method to the Ansel_Storage class where it really
belongs.

ws

We might be going back to the List view, in which case we don't
want to execute this bit...

If changing the gallery style of an existing gallery, make sure to replace
the 'view' url parameter with the correct view type for the gallery style
before redirecting.

No idea what this line was doing here

- Fix adding gallery_download attribute to storage
- Some tweaks and improvements merged from Duck's Ansel_Storage overrides

Allow control over who can download 'full' images.

Bug: 6406
Submitted by: Duck

Ability to limit gallery viewers by age

Bug: 6436
Submitted by: Duck

No need for a function call to get the gallery id now that galleries are
seperate from their share object. Remove Ansel_Gallery::getId() and replace
with id instance variable. Saves a bunch of function calls, and is now another
similarity between Ansel_Image and Ansel_Gallery objects.

Some refactoring. Move ajax request stuff to Ansel_XRequest classes,
similar to Imples.  Still needs some work with the stuff related to
tags, but I plan on refactoring the various widgets anyway...

Add gallery action to reset all of the current gallery's thumbnails to
allow them to be regenerated.

Clear gallery stacks when:
A new image is uploaded and becomes the gallery's default image (the new
image becomes the default, and is placed on the top of the stack)...
...and when a new image is uploaded and the gallery count is still < 5
(so the stack is not full yet)

Also, refactor a bit, adding a Ansel_Gallery::clearStacks() method.

Don't explode() the tags until we really need to in case there's an error
submitting the form - when the form would be rendered again.

Add ajax check for slug uniqueness.  I wanted to enable/disable the
submit button based on the validity of the slug, but I can't seem to
get the submit button to disable...

Check for error on gallery save, and provide a meaningful error message
when it fails.

Don't attempt to redirect to image upload if $gallery is a PEAR_Error,
plus some wrapping nits

Initial slug support. Basic CRUD stuff only for now. Still need uniqueness,
allowed character check etc...

ws

Fix redirect to last view after gallery properties are saved.

Don't decrease the gallery's stored image count when deleting a
gallery 'stack'

Statically cache Ansel_Storage::getImage() calls since they can often
be called multiple times per page load and from different scopes. Also
have to add reference operators now to be sure that only a single copy of
the image object is ever operated on per page load in PHP4.

Bug: 6219
Submitted by: Duck

Redirect to upload page after successful gallery creation.

Bug: 6196
Submitted by: Duck

The mother of all Ansel commits.  This removes all the gallery attributes
from the datatree, leaving only permissions and hierarchical information in
datatree.

To upgrade, first run the 2008_01_07_add_galleries.sql script to create
the gallery table and then run the sql_galleries.php script.  The script
does *NOT* delete any data from the datatree - it only creates the new
structure. Another (upcoming) upgrade script will remove the data from
the datatree once this is more thoroughly tested.

Bug: 6101

Make sure we delete all the gallery's default image thumbnails  from the
backend, not just the currently selected style's version of it.

This also fixes an issue that caused the image's entry in the DB to not
be removed.

Happy New Year

Standardize on using 'photos' instead of 'images'.

Fix adding/editing gallery categories

Chage "Regenerate" to "Reset"

Since we can now (or will shortly) have multiple types of default gallery images
it doesn't make sense to regenerate the stack here, since the gallery's style
doesn't determine what the stack will look like, it's the parent gallery's style
or default style etc... Instead, just clear the default images to force them to
be regenerated on the next view request.  Suggestions as to what to rename the menu action / notification text are welcome.

Looks like the url form field is being passed as an empty string, and Util::getFormData()
doesn't revert to the default in that case...jan, not sure if there was another intent
here, but this fixes adding subgalleries at the moment.

Move all gallery actions to the actions drop down (regenerate default image,
empty gallery), and let the gallery properties form only do exactly that.
Pass "emtpy gallery" action to gallery/delete.php, including confirmation
screen.
Show message whether stack regeneration succeeded.

Return to last view after editing gallery properties.

Redirect to index.php.

Default to parent gallery's style when creating a subgallery

Fix variable name.

Properly delete any existing photo stack when rebuiding.

Add error check here

Continue on our quest to add the idea of gallery styles to Ansel.
This commit adds:
- A config/styles.php.dist file to describe each available style. (Nothing other than thumbnail style so far, but will add bg/fg colors etc...)

- api calls default to ansel_default style, but can override this if desired and the ansel Blocks always use ansel_default for consistency.

Start slowly moving towards allowing galleries to specify named styles.
This commit doesn't add much functionality...just renaming some variables and methods to better reflect where we are going.

Remove any existing default-pretty image if we are regenerating one.

Add per-gallery option to select thumbnail style. The option is there, but it's not checked for anywhere in the display code yet...

- Don't assume we always want a prettythumb just because we're capable of them. (In particular for rss feeds we always return a 'normal' thumb).

- Use a Polaroid stack effect for default gallery images if we are capable of using prettythumbs...provide ability for user to regenerate the gallery default image from the gallery property page.

Probably still some tweaking to do to image sizes and things, plus I want to add per-gallery settings for prettythumb vs thumb but I wanted to get this into CVS before I went too much further :)

Another missing $gallery_tags <sigh>

Fix typo and make sure that $gallery_tags is always defined.

Allow adding tags to galleries as well.
Some minor variable/parameter renaming to reflect the fact that a tagged resource can be a gallery or an image.

Happy New Year

Assign copyright to The Horde Project for those copyright holders that already agreed to it.

revert & fix it right.

Add error check when downloading zip

A few more fixes as I find them.

Well, here goes. Images are now stored in a set of SQL tables instead
of in the DataTree. See prior commit message for upgrade instructions,
and let me know what you run in to. I'm sure I'll be tweaking things
myself as I upgrade my other galleries and use it more, but for now,
things seem to work.

actually we know we have a gallery object here.

define $name in this error message.

- Give gallery deletion a seperate page, not just a js popup
- gallery deletion requires POST
- all gallery-level actions are protected a bit in a <select>

Happy new year\!

We have to delete images before deleting the gallery; otherwise
deleting the images will fail.

Fix form field to focus on

Remove the autoname preference; since we use gallery ids in URLs now
instead of names, there's no point in picking the gallery datatree
name.

Fix quote style.

- Don't highlight New Gallery when editing an existing gallery
- Don't show Empty Gallery when creating a new gallery
- HTML cleanups, tweaks.

use countByAttributes to determine the number of images in a gallery

Happy New Year

Use new menu, remove view/* files that weren't being maintained.

fix subgallery creation

This set of patches brings a basic link to CategoryManager in Ansel.
So we can set and list/group galleries by their content.
For example Sport, 3D, Clipart...

Submitted by: Duck <duck@obala.net>

More name -> ID changes.  I think that's everything.

More name -> ID changes

Start modifications to require referrencing galleries & images by ID.

At this point, viewing & creating images & galleries works with no errors.
I'm sure there are other features that are broke now, but they should be
fairly straight-forward to fix.  I'm out of time tonight, but this is a
good start.  I should have more time tomorrow night if nobody beats me
to it.

Registry::getParam() -> Registry::get().

Use application permissions to restrict creating of galleries.
Bug: 612

use getGallery() instead of getShare()

Start consolidating some duplicate code with Ansel::createGallery() and
Ansel::getImageFromFile().

(Vijay Mahrra <webmaster@stain.net>).

Fix logic error

No more HORDE_LIBS.

Update some API function calls.

Display an error message if we aren't able to retrieve the gallery object
in the 'empty' action case.

We want to pull the error message out of $id, not $galleryId (which we
just set to null).

Add a $conf option for what kind of images to generate.

explicitly set counter to 0 after emptying a gallery.

minor CS and whitespace.

actually delete images when a gallery is deleted

Add 'empty gallery' functionallity.

Use javascript notification instead of $js_onLoad.

All footers look the same, so why not use only one.

More unnessecary counting

New backend for storing images.  They are no longer stored as part of a
gallery, but are now individual datatree objects.  This structure should
allow for better performance.

Fix Share:: method names.

Try to ditch a bunch of duplicative Horde_Share:: methods.