-
Notifications
You must be signed in to change notification settings - Fork 85
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
Added paragraph about siteaccess scopes are merged #2475
base: 4.6
Are you sure you want to change the base?
Conversation
Thank you Vidar! So for scalar values (string, int) the "higher level" scope simply overwrites the lower levels - but in terms of arrays they get merged together? |
That is correct |
This is more complicated than that. It depends on specific set of settings and their nesting level. In case of languages this is true, but for each case I'd check the container. We had and probably still have some config merging issues in some places. |
@alongosz already approved this one, so I assume it can be merged? |
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.
Thank you Vidar!
A couple suggestions from me to make it simpler - I'd also mention the ibexa:debug:config
command to make sure people know how to check this themselves
languages: ['ger-DE'] | ||
``` | ||
|
||
The language setting for the `de` siteaccess will in this case be `['ger-DE','eng-GB']`, not `['ger-DE']`. |
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.
The language setting for the `de` siteaccess will in this case be `['ger-DE','eng-GB']`, not `['ger-DE']`. | |
The final value of the `languages` setting for the `de` SiteAccess is `['ger-DE','eng-GB']`, not `['ger-DE']`. | |
You can always check the final values by running the `ibexa:debug:config` console command. | |
For this example, the command looks like this: | |
``` bash | |
php bin/console ibexa:debug:config --siteaccess=de languages | |
``` |
Be aware of the fact that configurations in the different scopes are merged, not overwritten. This is important to have | ||
in mind when dealing with arrays: |
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.
Be aware of the fact that configurations in the different scopes are merged, not overwritten. This is important to have | |
in mind when dealing with arrays: | |
When working with arrays, configurations from different scopes are merged together. | |
See the following example: |
(modified to take into account Julita's suggestion from below)
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.
Just a small remark: "See the following (...)" - new line.
Added paragraph about settings for the different siteaccess scopes are merged
Checklist