Bug 13757: (QA followup) Make opac-memberentry.pl handle attrs deletion
authorTomas Cohen Arazi <tomascohen@theke.io>
Mon, 30 Jan 2017 15:36:57 +0000 (12:36 -0300)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 24 Mar 2017 18:45:13 +0000 (18:45 +0000)
commitc44a377d9cc44538b6a7134f48763184c9a110ba
tree5bd14273195dfe2d560ff76b69e544ca06dfc894
parentb6797fdc8708e73eee0cfd7562e55543ed4f5e3b
Bug 13757: (QA followup) Make opac-memberentry.pl handle attrs deletion

The original code on this bug skipped empty-valued attributes. But
emptying attributes is the only way to tell the controller script that
the user wants to delete them.

This patch makes opac-memberentry.pl check the existence of attributes
sharing the code of the empty for the given patron, and it stores the
deletion on the Koha::Patron::Modification as needed. Otherwise
deletions got skipped.

To test:
- Verify setting/deleting attributes that are opac-editable and verify
the results are sound.

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

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
opac/opac-memberentry.pl