]> git.koha-community.org Git - koha.git/commit
Bug 13465: Correct the field prefix ambiguity
authorJonathan Druart <jonathan.druart@biblibre.com>
Wed, 17 Dec 2014 09:43:34 +0000 (10:43 +0100)
committerChris Cormack <chris@bigballofwax.co.nz>
Fri, 19 Dec 2014 23:28:10 +0000 (12:28 +1300)
commite8eb28df56a5b1811a2c89f1365d9716561da5de
tree0924ccb39089b0552d984828f65ef42079a7dbac
parentbe1d1df9c5b9b4768822e79672cd093cb93d5678
Bug 13465: Correct the field prefix ambiguity

This is introduced by Bug 12874.

Without this patch, it's not possible to clear (set to an empty string)
an item field.
This appended for field linked to an AV list but even if it's not.

The regex tried to prefix 'my_field' with 'items.' to have
'items.my_field'. It wanted to take care of the case where the prefix
already exists (Actually only 1: 'items.cn_source').
The regex is changed to: "add the prefix only if the string does not
contain a dot".

Moreover an ambiguity existed on the prefix: in marc_subfield_structure,
the kohafield is prefixed, but not in the key of the hash sent to
ModItemFromMarc.

Test plan:
- edit an item, set a status that is controlled by an authorized value
examples tested: damaged, not for loan
- check the status saved correctly
- edit the item again, reset the status to empty
- check the status saved correctly
- edit the item again, reset fields, edit fields
- check the fields saved correctly

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
(cherry picked from commit 2f266d94be06af73e72a807d9408136278e8fed4)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
C4/Items.pm