]> git.koha-community.org Git - koha.git/commit
Bug 13757: Make Koha::Patron::Modification->store del empty attrs
authorTomas Cohen Arazi <tomascohen@theke.io>
Tue, 31 Jan 2017 15:41:51 +0000 (12:41 -0300)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 24 Mar 2017 18:45:11 +0000 (18:45 +0000)
commitb6797fdc8708e73eee0cfd7562e55543ed4f5e3b
tree0dc294dd64cdcb38640297ce11431378e5d778a3
parentbb039e0328e0151b78c752e7a7be2c6a1c4f7d3a
Bug 13757: Make Koha::Patron::Modification->store del empty attrs

This patch makes Koha::Patron::Modification->store delete the passed
attributes that contain empty values.

This is the way it currently works, as all opac-editable attributes are
presented to the end-user and they are allowed to delete them, and the
best way I found to represent the deletion on the modification request
is by setting it to the empty string. I chose this way because it is how
the staff interface handles it, so it is consistent.

To test:
- Apply this patch
- Run:
  $ prove t/db_dependent/Koha/Patron/Modifications.t
=> SUCCESS: This time tests pass!
- Verify comment #70 on the bug is fixed now
- Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
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>
Koha/Patron/Modification.pm