Summary: | Suspected typo in patron self registration code | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Patrons | Assignee: | Kyle M Hall <kyle.m.hall> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | minor | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Bug 9997 - Suspected typo in patron self registration code
[SIGNED-OFF] Bug 9997 - Suspected typo in patron self registration code [PASSED QA] Bug 9997 - Suspected typo in patron self registration code |
Description
Katrin Fischer
2013-04-08 08:04:02 UTC
Created attachment 17290 [details] [review] Bug 9997 - Suspected typo in patron self registration code String fix. The field name is also incorrect, causing the alternate contact country to not be stored. Created attachment 17293 [details] [review] [SIGNED-OFF] Bug 9997 - Suspected typo in patron self registration code String fix. The field name is also incorrect, causing the alternate contact country to not be stored. To test: Submit an update to an existing patron record an include a change to the alternate contact's country field. Before the patch this line would not appear in the staff client for approval. After the patch it should. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Reminder: Even small patches require test plans. I have added one which I think covers the changes in the patch. Created attachment 17324 [details] [review] [PASSED QA] Bug 9997 - Suspected typo in patron self registration code String fix. The field name is also incorrect, causing the alternate contact country to not be stored. To test: Submit an update to an existing patron record an include a change to the alternate contact's country field. Before the patch this line would not appear in the staff client for approval. After the patch it should. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Reminder: Even small patches require test plans. I have added one which I think covers the changes in the patch. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All tests and QA script pass. This patch has been pushed to master and 3.12.x. |