-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
INI shouldn't be loaded from same directory as code #149
Comments
Thanks. I was thinking about this today while we taught a client how to turn off debug mode via toolkit.ini. |
proposed search order and names:
Should it be overrriden in code too? |
realizing this could be from code too for i.e app specific |
Toolkit install dir could contain a symlink to new location |
I will say that CONFIG_FILE is theoretically settable by defining it before Toolkit.php is loaded, but this seems greasy in a world where autoloaders exist. The fact the INI is basically handled by static resources sucks, and that could require a refactor to make instance-local. |
Not sure is this should be it exactly, but per-site configuration and system-wide (i.e managed by RPM) are common use cases; there may be more. We should accomodate these when possible, because user configuration in a source code directory is counter-intuitive and inflexible. Fixes #149
Reasoning:
The text was updated successfully, but these errors were encountered: