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

System/Context settings #13659

Open
Oetzie opened this issue Oct 27, 2017 · 8 comments
Open

System/Context settings #13659

Oetzie opened this issue Oct 27, 2017 · 8 comments
Labels
area-core bug The issue in the code or project, which should be addressed.

Comments

@Oetzie
Copy link
Contributor

Oetzie commented Oct 27, 2017

Summary

The configuration widget from the dashboard does not check the 'site_start' and 'error_page' at context level. This is also a bug with the resource remove processor?.

Step to reproduce

Duplicate the home page of the web context, change the 'site_start' context setting to the new ID. Delete the old homepage and an error like 'Your site start does not exists' will appear in the configuration widget.

Expected behavior

The context settings needs to overrule the system settings.

Environment

MODX version 2.5.8

@OptimusCrime
Copy link
Contributor

I think the consensus is that the default settings should be in the system settings. You can override the settings individually in the contexts, but the system settings should contain valid default values.

If I understood the issue correctly?

@OptimusCrime
Copy link
Contributor

Ping @Oetzie

@Oetzie
Copy link
Contributor Author

Oetzie commented Feb 9, 2018

Sorry for the late reply. But why should the systems settings contain valid default values? I override them at context level. So in that cause the system settings aren't needed any more.

@Mark-H
Copy link
Collaborator

Mark-H commented Feb 9, 2018

It can still be fallen back to in contexts that don't have it specifically set.

@OptimusCrime
Copy link
Contributor

You would usually not need to specify the site_start as a context setting for the default context (web). This is also the case for almost every other setting that interacts with specific contexts. I think this is intended behavior.

@Ibochkarev
Copy link
Collaborator

@alroniks @JoshuaLuckers this fix was merged in #14108 .

Please close issue.

@JoshuaLuckers JoshuaLuckers added bug The issue in the code or project, which should be addressed. area-core labels Jan 12, 2019
@Jako Jako closed this as completed Jan 12, 2019
@Jako Jako reopened this Jan 12, 2019
@Jako
Copy link
Collaborator

Jako commented Jan 12, 2019

The configuration widget should check all contexts for site_start/error_page.

The other part (message when deleting the page targeted by the site_start system setting) should be an intended behavior.

@JoshuaLuckers
Copy link
Contributor

JoshuaLuckers commented Jan 12, 2019

Context settings are not taken into consideration in various functionalities like checks that prevent deleting the resource marked as site_start. I was able to set site_start to an empty value in system settings. The web context did have the site_start value and was not checked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core bug The issue in the code or project, which should be addressed.
Projects
None yet
Development

No branches or pull requests

6 participants