search

bcosca / fatfree

A powerful yet easy-to-use PHP micro-framework designed to help you build dynamic and robust Web applications - fast!
2.66k stars 446 forks source link

tidying up the system variables #116

Closed ikkez closed 11 years ago

ikkez commented 12 years ago

here is a little enhancement for the next major version.

i think, some of the so called system variables are not necessarily a system var, but more a parameter for some methods..

like BGCOLOR, FGTRANS and FONTS are only for captchas, and therefore could be another arguments for Graphics::captcha().

furthermore, it's maybe better to group some vars of the same kind for better understanding. like: 

PATH.TEMP
PATH.GUI
PATH.PLUGINS
PATH.LOGS
PATH.LOCALES

CAPTCHA.BGCOLOR
CAPTCHA.FGTRANS
CAPTCHA.FONTS

IDENTICON.IBLOCKS
IDENTICON.IPIXELS

otherwise it's getting hard to figure out all relevant variables for a certain task, especially when the systems grows and more vars are coming. just to think about it ;)

another thing: TZ, ESCAPE and PATTERN are still not mentioned in the http://bcosca.github.com/fatfree/#quick-reference

telephone commented 12 years ago

:+1:

KOTRET commented 12 years ago

:+1: for all vars that belong to Plugins, base-vars can be prefixed too

eazuka commented 12 years ago

+1, yes i support this idea from ikkez

bcosca commented 11 years ago

3.0 architecture has taken this into consideration. So plugin-related variables are now restricted to their respective classes. This approach eliminates global clutter.