[dev] [cvs] commit: klutz savecomic.php klutz/config conf.xml
Chuck Hagenbuch
chuck at horde.org
Mon Jun 11 04:35:08 UTC 2007
Quoting Michael Rubinsky <mike at theupstairsroom.com>:
> Log:
> Add config setting to set a default gallery title. If this string
> matches the title of a gallery the user has PERMS_EDIT on, then this
> gallery will be selected by default on the saveimage form.
I understand the motivation here, but this should be a preference, not
a config setting, and it should pick from a list of ids, not a string
- using a string is fragile and perhaps prone to complicated trickery,
though it's hard to see it being too harmful.
-chuck
More information about the dev
mailing list