Attachment 24590 Details for Bug 11313 – [PASSED QA] Bug 11313 - empty value in mandatory combobox in record edition

[patch] [PASSED QA] Bug 11313 - empty value in mandatory combobox in record edition

PASSED-QA-Bug-11313---empty-value-in-mandatory-com.patch (text/plain), 5.49 KB, created by Katrin Fischer on 2014-01-21 06:02:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2014-01-21 06:02:29 UTC

Size: 5.49 KB

patch

obsolete

>From d102bf174f669fe6e9ce9a969f07a875fba98807 Mon Sep 17 00:00:00 2001
>From: Fridolyn SOMERS <fridolyn.somers@biblibre.com>
>Date: Wed, 27 Nov 2013 16:40:09 +0100
>Subject: [PATCH] [PASSED QA] 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@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.
>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Passes all tests and QA script.
>
>1) Adding a bibliographic record
>Subfield is mandatory, no default value:
>- adding record: no value is preselected, empty value exists
>- editing record: saved value is shown,  empty value exists
>Subfield is mandatory, defined default value:
>- adding record: default is preselected, no empty value
>- editing record: saved value is shown, no empty value exists
>Subfield is not mandatory, defined default value:
>- adding record: default is preselected, empty value exists
>- editing record: saved value is shown, emtpy value exists
>Subfield is not mandatory, no default value:
>- adding record: empty value is preselected
>- editing record: saved value is shown, empty value exists
>
>2) Adding an authority record
>Repeating tests from above.
>- Subfield is mandatory, no default value: OK
>- Subfield is mandatory, defined default value: OK
>- Subfield is not mandatory, defined default value: OK
>- Subfield is not mandatory, no default value: OK
>---
> authorities/authorities.pl | 6 ++++--
> cataloguing/addbiblio.pl   | 6 ++++--
> 2 files changed, 8 insertions(+), 4 deletions(-)
>
>diff --git a/authorities/authorities.pl b/authorities/authorities.pl
>index bb527f6..de8bbf8 100755
>--- a/authorities/authorities.pl
>+++ b/authorities/authorities.pl
>@@ -92,7 +92,8 @@ sub build_authorized_values_list {
>             "select itemtype,description from itemtypes order by description");
>         $sth->execute;
>         push @authorised_values, ""
>-        unless ( $tagslib->{$tag}->{$subfield}->{mandatory} );
>+          unless ( $tagslib->{$tag}->{$subfield}->{mandatory}
>+            && ( $value || $tagslib->{$tag}->{$subfield}->{defaultvalue} ) );
>         
>         my $itemtype;
>         
>@@ -109,7 +110,8 @@ sub build_authorized_values_list {
>             $tagslib->{$tag}->{$subfield}->{authorised_value} );
> 
>         push @authorised_values, ""
>-        unless ( $tagslib->{$tag}->{$subfield}->{mandatory} );
>+          unless ( $tagslib->{$tag}->{$subfield}->{mandatory}
>+            && ( $value || $tagslib->{$tag}->{$subfield}->{defaultvalue} ) );
> 
>         while ( my ( $value, $lib ) = $authorised_values_sth->fetchrow_array ) {
>             push @authorised_values, $value;
>diff --git a/cataloguing/addbiblio.pl b/cataloguing/addbiblio.pl
>index 24be291..118bfc2 100755
>--- a/cataloguing/addbiblio.pl
>+++ b/cataloguing/addbiblio.pl
>@@ -192,7 +192,8 @@ sub build_authorized_values_list {
>             "select itemtype,description from itemtypes order by description");
>         $sth->execute;
>         push @authorised_values, ""
>-          unless ( $tagslib->{$tag}->{$subfield}->{defaultvalue} and $tagslib->{$tag}->{$subfield}->{mandatory} );
>+          unless ( $tagslib->{$tag}->{$subfield}->{mandatory}
>+            && ( $value || $tagslib->{$tag}->{$subfield}->{defaultvalue} ) );
>           
>         my $itemtype;
>         
>@@ -229,7 +230,8 @@ sub build_authorized_values_list {
>         );
> 
>         push @authorised_values, ""
>-          unless ( $tagslib->{$tag}->{$subfield}->{mandatory} );
>+          unless ( $tagslib->{$tag}->{$subfield}->{mandatory}
>+            && ( $value || $tagslib->{$tag}->{$subfield}->{defaultvalue} ) );
> 
>         while ( my ( $value, $lib ) = $authorised_values_sth->fetchrow_array ) {
>             push @authorised_values, $value;
>-- 
>1.8.3.2

Actions: View | Diff | Splinter Review

Attachments on bug 11313: 23196 | 24560 | 24590