https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18507

Marcel de Rooy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         QA Contact|[email protected] |[email protected]
                   |y.org                       |

--- Comment #5 from Marcel de Rooy <[email protected]> ---
Seems we have a bit of code duplication here between the add and modify case.
Can we slide that into one routine ? No blocker on feature slush, open a new
report please.
Do we need to check somewhere if the key is a valid patron column btw? Is not
done on create, so no blocker too.
Looks good enough to go along now. Feature can simply be config disabled, so no
harm. Only reviewed, not tested. But has been confirmed to work.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to