- Issue created by @Anybody
- ππΊHungary nevergone NyΓregyhΓ‘za, Hungary, Europe
Can you bring it up on a brand new install?
- π©πͺGermany Anybody Porta Westfalica
I'll ask @Grevil to try it. We're running into this in a client project. This happens with and without the throttling used. All kind of errors coming in:
- Error in the database log: php
- Error in the database log: client error
- Error in the database log: page not found
- Error in the database log: ...
- Assigned to Grevil
- First commit to issue fork.
- π©πͺGermany Grevil
Yes, this is easily recreated on a brand new install.
- π©πͺGermany Grevil
Fixed the issue! Simple logic error. Please review!
- Merge request !16Issue #3440614 by Grevil, Anybody: Logs "Page not found" errors, if just "Send all PHP entries" is enabled β (Open) created by Grevil
- Issue was unassigned.
- Status changed to Needs review
about 1 year ago 7:37am 18 April 2024 - π©πͺGermany Anybody Porta Westfalica
Update: This has nothing to do with the "Send all PHP entries" option, it's just what the module does in this condition, with or without the option:
$level <= RfcLogLevel::WARNING
This leads to logging the other errors, that are at least a warning!
The UX is just confusing...
- π©πͺGermany Anybody Porta Westfalica
The correct fix here would be to expose checkboxes to select the levels you'd like to be notified about in general.
- π©πͺGermany Anybody Porta Westfalica
Okay as the change would be a breaking change, we'll skip this fix and instead proceed with a new UI / UX in:
β¨ Add multiple emails for email report. Active which will also include π Settings form refactoring Active .This isn't a bug, it's just caused by the currently confusing UX and missing documentation in the settings form, what happens if you don't configure anything additionally.