The warning reads like this: "The attribute value 1 is already is use by another patron record"
Created attachment 19500 [details] [review] Proposed patch Fairly trivial patch To Test on a system which includes a unique extended patron attribute Copy or create a patron duplicating the value of the unique attribute without this patch The error message appears as recorded in the bug With this patch the attribute and its value appears instead of 1 e.g. The attribute value MAGICCODE/BOB2 is already is use by another patron record.
Created attachment 19503 [details] [review] [SIGNED-OFF] Bug 10507 Fix namespace collision in patron entry If a warning about a duplicated patron attribute is returned the value was being passed back for display in the warning message with the same name as the variable in errors as the message is displayed in the context of errors the error value 1 was being displayed not the value duplicated as intended. Pass the value to the template with a unique name Signed-off-by: Owen Leonard <oleonard@myacpl.org> I fixed another tiny typo while signing off: "is already in use" instead of "is already is use."
Created attachment 19608 [details] [review] Bug 10507 Fix namespace collision in patron entry If a warning about a duplicated patron attribute is returned the value was being passed back for display in the warning message with the same name as the variable in errors as the message is displayed in the context of errors the error value 1 was being displayed not the value duplicated as intended. Pass the value to the template with a unique name Signed-off-by: Owen Leonard <oleonard@myacpl.org> I fixed another tiny typo while signing off: "is already in use" instead of "is already is use." Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Passes koha-qa.pl, works as advertised.
Pushed to master. Thanks, Colin!
This patch has been pushed to 3.12.x, will be in 3.12.2. Thanks Colin!
Pushed to 3.10.x, will be in 3.10.10
Pushed to 3.8.x, will be in 3.8.18