-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Conversation
and changed the glob line for config files
$appConfig = include 'config/application.config.php'; | ||
|
||
$listenerOptions = new Zend\Module\Listener\ListenerOptions($appConfig['module_listener_options']); | ||
$defaultListeners = new Zend\Module\Listener\DefaultListenerAggregate($listenerOptions); | ||
$defaultListeners->getConfigListener()->addConfigGlobPath('config/autoload/*.config.php'); | ||
$defaultListeners->getConfigListener()->addConfigGlobPath("config/autoload/*.{global,$env}.php"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To keep or not to keep .config.php
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed it to be {global,$env}.config.php now
$appConfig = include 'config/application.config.php'; | ||
|
||
$listenerOptions = new Zend\Module\Listener\ListenerOptions($appConfig['module_listener_options']); | ||
$defaultListeners = new Zend\Module\Listener\DefaultListenerAggregate($listenerOptions); | ||
$defaultListeners->getConfigListener()->addConfigGlobPath('config/autoload/*.config.php'); | ||
$defaultListeners->getConfigListener()->addConfigGlobPath("config/autoload/*.{global,$env}..configphp"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be *{global,$env}.config.php
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeh I noticed my mess up after going to bed and tried to fix it on my phone, which ended in disaster. Should be sorted now.
@Ocramius the commits could probably be squashed into 1 so you only see the final |
Why worry? It doesn't really hurt. Though you could probably squash. |
…o facilitate local changes pulled in by index.php glob config
Added env check and changed glob line
added environment check for application_env and changed the glob line for config files