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

Update subsite check to ensure the DO has the field #79

Merged

Conversation

satrun77
Copy link
Contributor

@satrun77 satrun77 commented Sep 13, 2022

Instead of checking for the value of the SubsiteID, we are checking if the SubsiteID field exists in the object. We want to check if the DataObject implements the subsite functionality or not.

Objects that do not implement subsite and objects such as SiteTree that implement the subsite have the same value 0 when the SiteTree is created for the Main Site.

@chrispenny chrispenny merged commit 079a81f into silverstripe:2 Feb 15, 2023
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

Successfully merging this pull request may close these issues.

None yet

2 participants