| Summary: | Don't crash on empty yaml sysprefs | ||
|---|---|---|---|
| Product: | Koha | Reporter: | David Gustafsson <glasklas> |
| Component: | Architecture, internals, and plumbing | Assignee: | Bugs List <koha-bugs> |
| Status: | CLOSED INVALID | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | ||
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: | Bug 32615: Don't crash on empty yaml sysprefs | ||
|
Description
David Gustafsson
2023-01-12 16:43:50 UTC
Created attachment 145239 [details] [review] Bug 32615: Don't crash on empty yaml sysprefs To test: 1) Apply patch 2) Verify tests in t/Context.t pass I now realized it's only when adding a new syspref, setting the initial value to NULL that this is an issue. So very much of an edge case. I still don't see the harm of fixing this, but it's a very minor issue indeed. Closing this one, I made a mistake testing this and appears undef/null values actually don't cause any issues. |