[Koha-bugs] [Bug 11313] empty value in mandatory combobox in record edition

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11313

David Cook <dcook at prosentient.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #23196|0                           |1
        is obsolete|                            |

--- Comment #2 from David Cook <dcook at prosentient.com.au> ---
Created attachment 24560
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24560&action=edit
Bug 11313 - empty value in mandatory combobox in record edition

In biblio (or authority) record edition or creation,
Bug 3727 added an empty value to combobox for subfield using itemtype
authorized values, which is mandatory and does not have a default value. This
to avoid selecting arbitrarily the first value of combo.

This patch extends this behavior to normal authorized values like CCODE,
COUNTRY ...
Also, when record already exists and has a defined value for the subfield this
value will be selected in combobox so there is no need for an empty value.

Test plan :
- In a biblio framework, set a subfield mandatory and using itemtypes
authorised values
- Set a default value for this subfield
- Create a new record with the framework
=> You see the subfield with a combobox, the first empty value is selected
- Go to biblio framework and set a default value for the subfield
- Create a new record with the framework
=> You see the subfield with a combobox, the default value is selected and
there is no empty value
- Select an other value and save the record
- Edit this record
=> You see the subfield with a combobox, the saved value is selected and there
is no empty value

Perform same tests with a normal authorized values categorie (instead of
itemtypes).
Perform same tests with with authority creation/edition
Perform same tests with a normal authorized values categorie and authority
creation/edition

Signed-off-by: David Cook <dcook at prosentient.com.au>

Works as described.

Fridolyn: I had a lot of difficulty understanding your test plan,
especially since the second line should not be in the test plan
(since if there is a default value, no empty value should appear).

Perhaps it would be useful to use more whitespace to separate
steps and the things for which a tester should check.

Also, it might be useful to specify what testers should check for
before and after applying the patch, so that they're sure that the
behaviour has been corrected.

In any case, the patch works! Certainly a helpful addition.

-- 
You are receiving this mail because:
You are watching all bug changes.