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

CONTENTBOX-1085 #430

Merged
merged 1 commit into from Oct 25, 2018
Merged

CONTENTBOX-1085 #430

merged 1 commit into from Oct 25, 2018

Conversation

gpickin
Copy link
Contributor

@gpickin gpickin commented Oct 25, 2018

Fixes #1085
Ensures that the setting name ( slug ) is unique, and cannot be duplicated.
Without this, if a weird cache race condition is found, the Preflight check would insert 135 duplicate settings, causing every ORM findWhere to return more than 1 record for a setting name, stopping ColdBox from Initiliazing, resulting in many strange and not so wonderful errors.

https://ortussolutions.atlassian.net/browse/CONTENTBOX-1085

Fixes #1085 
Ensures that the setting name ( slug ) is unique, and cannot be duplicated. 
Without this, if a weird cache race condition is found, the Preflight check would insert 135 duplicate settings, causing every ORM findWhere to return more than 1 record for a setting name, stopping ColdBox from Initiliazing, resulting in many strange and not so wonderful errors. 

https://ortussolutions.atlassian.net/browse/CONTENTBOX-1085
@gpickin gpickin requested a review from lmajano October 25, 2018 05:51
@lmajano lmajano merged commit 6a750d7 into development Oct 25, 2018
@lmajano lmajano deleted the CONTENTBOX-1085-GP branch December 7, 2020 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants