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

--- Comment #91 from Laura Escamilla <[email protected]> ---
Created attachment 206861
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=206861&action=edit
Bug 21860: Address additional QA findings

Addressed the additional QA findings.
* Updated update_field indicator handling so indicators are treated as values
to update rather than match criteria. Existing fields are now updated in place
rather than creating a duplicate when the requested indicator differs.
* Preserved the "add new" behavior when the target field/subfield does not
exist.
* Moved the _ to MARC blank indicator conversion until after form validation
succeeds, so failed validation does not alter the displayed indicator values.
* Added/updated regression coverage for updating indicators on an existing
field.
Manually tested both update and add-new behavior in the staff interface,
including the reported 650 _0 → 650 70 case and the failed-validation _
behavior.
QA tools and tests pass:
* t/SimpleMARC.t
* t/db_dependent/MarcModificationTemplates.t
* node --check
* git diff --check
* qa

Signed-off-by: Matt Blenkinsop <[email protected]>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to