[geeklog-devel] [geeklog-cvs] Geeklog-1.x/system/classes config.class.php, 1.16, 1.17
devel at portalparts.com
Sun Jan 20 17:05:58 EST 2008
I was wondering why we need a new config type and not just check for the
existance of a configmanager_fieldname function?
I was thinking we need to check for a function to feed the values to a
config parm as well as validate upon save. The validate function could
do any post processing like convert days to seconds (if a field wanted
to present a update frequency in days and field needed to be saved as
Most likely the preprocess function would only be used for checkboxes,
select and radio fields but postprocess may be used or any config parm.
Dirk Haun wrote:
> Dirk Haun wrote:
>> Log Message:
>> Introduced a new config type 'fn:' that calls a specified function to
>> provide contents of a dropdown at runtime, e.g. for the language and
>> theme selection
> Can I have some opinions on this, please? Aaron, Blaine, anyone?
> In a nutshell, this introduces a new config type 'fn:' which, when
> written as "fn:something" calls a function configmanager_something,
> expecting it to return the selection array. In other words, it's a
> dynamic version of the 'select' type.
> I've already implemented it for the two obvious candidates: The site's
> default language and theme. As others have pointed out, we already have
> functions that provide dropdowns for those, so why not use them in the
> configuration (well, they both needed small wrapper functions ...).
> Is the function name okay or should it start with "CONF_" instead? Any
> room for improvements?
> bye, Dirk
More information about the geeklog-devel