Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #5533 - Null checker on EditorHostSetting #5535

Merged

Conversation

erw13n
Copy link
Contributor

@erw13n erw13n commented Feb 7, 2023

Summary

Fix #5533 - Error happen due filteredSettings is null.

Copy link
Contributor

@mitchelsellers mitchelsellers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks reasonable to me

@valadas valadas added this to the 9.11.1 milestone Feb 8, 2023
Copy link
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me and tested the CI build, I confirm it fixes both the server browser and the editor manger. Thanks! Releasing an RC3 in a few minutes...

@valadas valadas merged commit 32576bb into dnnsoftware:develop Feb 8, 2023
@erw13n erw13n deleted the feature/5533-CKE-Settings-Null-Checker branch February 8, 2023 05:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DNN v9.11.1-rc2 file browsing bug
3 participants