Summary: | Can't add new patron categories | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | System Administration | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | critical | ||
Priority: | P5 - low | CC: | gmcharlt, jonathan.druart, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | 6254 | ||
Bug Blocks: | |||
Attachments: |
Bug 12454: Fix for adding new patron categories
[SIGNED-OFF] Bug 12454: Can't add new patron categories Bug 12454: Can't add new patron categories |
Description
Katrin Fischer
2014-06-19 15:38:50 UTC
Created attachment 29053 [details] [review] Bug 12454: Fix for adding new patron categories To test: - Add a new patron category - After saving, the new patron category won't show up in the list - Apply patch - Try to add a new patron category again - It should now save correctly and show up in the list of patron categories. Created attachment 29057 [details] [review] [SIGNED-OFF] Bug 12454: Can't add new patron categories To test: - Add a new patron category - After saving, the new patron category won't show up in the list - Apply patch - Try to add a new patron category again - It should now save correctly and show up in the list of patron categories. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 29090 [details] [review] Bug 12454: Can't add new patron categories To test: - Add a new patron category - After saving, the new patron category won't show up in the list - Apply patch - Try to add a new patron category again - It should now save correctly and show up in the list of patron categories. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Patch pushed to master. Thanks Katrina! Note: I'd like to see all this business logic be put in a lib in the Koha namespace we can have unit tests written for it. I'm sure this would belong to another bug report anyway. |