[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