]> git.koha-community.org Git - koha.git/commit
Bug 35044: (QA follow-up): Allow for value '0' in additional fields
authorPedro Amorim <pedro.amorim@ptfs-europe.com>
Wed, 24 Jul 2024 18:15:05 +0000 (18:15 +0000)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 23 Aug 2024 16:21:26 +0000 (18:21 +0200)
commit36ebff29195da5f8961681db1e586566a6484c48
tree62e19baf80351a2cd5edaa9223ff4baa7fe1e8a1
parentc8cc04a397a10d7151b7ddc4a718695e6761b8a8
Bug 35044: (QA follow-up): Allow for value '0' in additional fields

This now allows for value '0' for an additional field (repeatable or
not).
This still does not allow for an empty '' field in additional fields.
This is consistent with borrower patron attributes, i.e. if you submit
an empty borrower patron attribute entry, it does not store it as empty.
Additionally, if you have a value in a borrower patron attribute but
then submit the form with that field as empty, it gets deleted from the
database.

I agree not being able to save the value '0' is a blocker, but not the
empty.
For consistency sake I'm keeping this as is, I don't see a use-case
where a row with an empty field value is preferred over it not existing.

If we allow for '' values of additional fields, the 'clear' button for
repeatable fields would have to remove the input instead of clearing the
value (or else how would the user remove repeatable fields entries?).
If this is done, it's inconsistent with the repeatable fields from
patron attribute types, as in that instance the 'clear' button only
removes the input contents, not the input element itself.

prove t/db_dependent/Koha/Object/Mixin/AdditionalFields.t

Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Koha/Object/Mixin/AdditionalFields.pm