Skip to content

Return empty options if master root is null#264

Open
zonky2 wants to merge 1 commit intoterminal42:mainfrom
zonky2:patch-1
Open

Return empty options if master root is null#264
zonky2 wants to merge 1 commit intoterminal42:mainfrom
zonky2:patch-1

Conversation

@zonky2
Copy link

@zonky2 zonky2 commented Jan 9, 2026

No description provided.

@aschempp
Copy link
Member

aschempp commented Feb 3, 2026

Can you explain why that would be necessary? How can you not have a fallback page on a website you're currently working on?

@zonky2
Copy link
Author

zonky2 commented Feb 3, 2026

Phew, that was a month ago – I'll have to look up which project that happened in...

But if it weren't possible, I wouldn't have made the mistake of $masterRoot = null.

@aschempp
Copy link
Member

aschempp commented Feb 5, 2026

I understand that, but addint a fix for something that might be a configuration issue in your setup isn't really worth having and needing maintenance etc... or we might want to throw an exception so users becore aware of the configuration issue. But all of that we can only know with a description (would have been good to have one in the first place 😉)

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.

2 participants