Skip to content
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

Unwind $GLOBALS['CONFIG'] #27

Closed
MaZderMind opened this issue Dec 2, 2016 · 1 comment
Closed

Unwind $GLOBALS['CONFIG'] #27

MaZderMind opened this issue Dec 2, 2016 · 1 comment
Milestone

Comments

@MaZderMind
Copy link
Contributor

In the early days, there was only one conference and so configuring it into $GLOBALS was fine. With the introduction of the multi-conference feature, hacks like this became necessary to remain compatibility with the old config format:

if(isset($GLOBALS['CONFIG']))

While preparing #20 this now -again- leads to the issue, that it's not possible to access all conferences' config options in one place, so now is probably the time to clean the $GLOBLAS thing up.

@MaZderMind MaZderMind added this to the 33C3 milestone Dec 2, 2016
@MaZderMind
Copy link
Contributor Author

fixed in 51ee823

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant