Skip to content

Update Module.php#9

Open
C-Norton wants to merge 1 commit intoarepr:mainfrom
rochester-rcl:main
Open

Update Module.php#9
C-Norton wants to merge 1 commit intoarepr:mainfrom
rochester-rcl:main

Conversation

@C-Norton
Copy link
Copy Markdown

Added an extra null check.

For whatever reason, I encountered a null error on this line during a previously working configuration. This code fixed it. Since this seems relatively benign, and it did fix the issue, I figure I'd send it upstream

Added an extra null check
@ivyrze
Copy link
Copy Markdown
Collaborator

ivyrze commented Apr 21, 2025

I’m curious to know what the pattern is that makes $locale be null?

@C-Norton
Copy link
Copy Markdown
Author

I'm actually not sure; this was on a working transcript page, no changes as far as I'm aware, and it started happening. Unfortunately, this is on our dev instance, so I can't show you the pagez as you're not on our VPN. If you let me know what info you'd need, I can collect it for you.

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