-
Notifications
You must be signed in to change notification settings - Fork 75
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
Docker: Put the docker specific settings in a XML file #123
Labels
Milestone
Comments
The branch for this feature was misnamed as issue-113-Config_pm instead of issue-123-Config_pm |
bschmalhofer
added a commit
that referenced
this issue
Jul 10, 2020
This setting will be activated in the installer, when ES is available.
bschmalhofer
added a commit
that referenced
this issue
Jul 10, 2020
Merged
bschmalhofer
added a commit
that referenced
this issue
Jul 17, 2020
And use the autoclose of lexical file handles.
bschmalhofer
added a commit
that referenced
this issue
Jul 17, 2020
Issue #123: fix stupid syntax error concerning scope of $FH.
bschmalhofer
added a commit
that referenced
this issue
Jul 17, 2020
And use the autoclose of lexical file handles.
Not really sure what the intention was. Closing the issue as it doesn't seem to be important. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now the docker specific settings are set in Kernel/Config.pm. This means that they can't be changed by SysConfig.
A better approach is to move the settings into an .xml file in Kernel/Config/Files/XML.
The sub
Kernel::System::SysConfig::ConfigurationXML2DB()
declares an order of the config files based on the attributeotobo.ìnit
in the XML file:So adding an override file with
otobo_config.init = 'Config'
should do the trick.The text was updated successfully, but these errors were encountered: