[dev] Ansel storeImage, numImages, getImageUrl,
getDefaultImage API calls
Chuck Hagenbuch
chuck at horde.org
Mon Oct 11 21:00:33 PDT 2004
Quoting Duck <duck at obala.net>:
> yes. I'm using it in an article management and user profiles module, where I
> sore article image in Ansel and user image.
>
> An basic example how I use it in the article manager. (Using VFS direct).
Cool, I like the general idea. Before committing it, I'd like to see it work a
tad differently, though. I'll keep the code around and might take a
shot at it,
but if you want to submit an updated patch, that'd be great.
- it should work more closely with gallery ids, instead of the $app parameter.
getDefaultImage(), for instance, should only need to take a gallery id.
Gallery
ids can be nested, though.
- split out dual purpose methods - again, see getDefaultImage. That should be
two methods - getDefaultImage, returning the id, and getImagePath (or
something
like that) returning the path of any image, given a gallery and an id.
- The parameters are kind of a mess. I guess if $image is from
Horde_Form, that
makes sense. But gallery_data and default should be combined if possible.
Etc. Make sense?
-chuck
--
"But she goes not abroad in search of monsters to destroy." - John
Quincy Adams
More information about the dev
mailing list