Bug 11184: correct attribute cloning for the patron editor
authorSrdjan <srdjan@catalyst.net.nz>
Fri, 1 Nov 2013 08:24:11 +0000 (21:24 +1300)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 9 Apr 2014 22:42:54 +0000 (22:42 +0000)
commit3334f03d779dc41bac3fe6e25a0d08733fc030ca
tree9284096082d84ab9f8d7f0ba687b73fd8df46ef2
parent9094b4a1bbff95e0f8e5747a805dacede165bc9a
Bug 11184: correct attribute cloning for the patron editor

This patch fixes Perl warnings logged when setting up the
patron attribute form in the patron editor.

To test - Patron details entry page:
* Have  ExtendedPatronAttributes enabled. Check that "Additional
  attributes and identifiers" section behaves.
* Verify that editing and saving a patron record does not
  result in the following sorts of entries in the Apache log:

  se of uninitialized value $_ in hash element at memberentry.pl line 798
  Use of uninitialized value in anonymous hash ({}) at memberentry.pl line 798

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested with different types of patron attributes:
- repeatable
- linked to an authorized value
- free text
Tested editing, adding, removing one of multiple, adding multiple,
etc. No regressions found.
Passes all tests and QA script.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
members/memberentry.pl