[Koha-patches] [PATCH] (bug #3057) fix the subfield edition

[Nahuel ANGELINETTI]

This patch must not be pushed, it implements another bug.

Le Wed, 25 Mar 2009 13:55:16 +0100,
Nahuel ANGELINETTI <nahuel.angelinetti at biblibre.com> a écrit :

> this patch modify the generation of inputs, and field "name" value of
> input fields. ---
>  admin/marc_subfields_structure.pl |   22 +++++++++++-----------
>  1 files changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/admin/marc_subfields_structure.pl
> b/admin/marc_subfields_structure.pl index 3c15291..1c96e32 100755
> --- a/admin/marc_subfields_structure.pl
> +++ b/admin/marc_subfields_structure.pl
> @@ -178,7 +178,7 @@ if ( $op eq 'add_form' ) {
>          }
>          $row_data{defaultvalue} = $data->{defaultvalue};
>          $row_data{tab} = CGI::scrolling_list(
> -            -name   => 'tab',
> +            -name   => 'tab$i',
>              -id     => "tab$i",
>              -values =>
>                [ '-1', '0', '1', '2', '3', '4', '5', '6', '7', '8',
> '9', '10' ], @@ -211,7 +211,7 @@ if ( $op eq 'add_form' ) {
>          $row_data{libopac}      =
> CGI::escapeHTML( $data->{'libopac'} ); $row_data{seealso}      =
> CGI::escapeHTML( $data->{'seealso'} ); $row_data{kohafield}    =
> CGI::scrolling_list(
> -            -name     => "kohafield",
> +            -name     => "kohafield$i",
>              -id       => "kohafield$i",
>              -values   => \@kohafields,
>              -default  => "$data->{'kohafield'}",
> @@ -219,7 +219,7 @@ if ( $op eq 'add_form' ) {
>              -multiple => 0,
>          );
>          $row_data{authorised_value} = CGI::scrolling_list(
> -            -name     => "authorised_value",
> +            -name     => "authorised_value$i",
>              -id       => "authorised_value$i",
>              -values   => \@authorised_values,
>              -default  => $data->{'authorised_value'},
> @@ -227,7 +227,7 @@ if ( $op eq 'add_form' ) {
>              -multiple => 0,
>          );
>          $row_data{value_builder} = CGI::scrolling_list(
> -            -name     => "value_builder",
> +            -name     => "value_builder$i",
>              -id       => "value_builder$i",
>              -values   => \@value_builder,
>              -default  => $data->{'value_builder'},
> @@ -235,7 +235,7 @@ if ( $op eq 'add_form' ) {
>              -multiple => 0,
>          );
>          $row_data{authtypes} = CGI::scrolling_list(
> -            -name     => "authtypecode",
> +            -name     => "authtypecode$i",
>              -id       => "authtypecode$i",
>              -values   => \@authtypes,
>              -default  => $data->{'authtypecode'},
> @@ -250,7 +250,7 @@ if ( $op eq 'add_form' ) {
>              -id       => "repeatable$i"
>          );
>          $row_data{mandatory} = CGI::checkbox(
> -            -name     => "mandatory",
> +            -name     => "mandatory$i",
>              -checked  => $data->{'mandatory'} ? 'checked' : '',
>              -value    => 1,
>              -label    => '',
> @@ -278,7 +278,7 @@ if ( $op eq 'add_form' ) {
>          $row_data{'subfieldcode'} = '';
>  
>          $row_data{tab} = CGI::scrolling_list(
> -            -name   => 'tab',
> +            -name   => 'tab$j',
>              -id     => "tab$j",
>              -values =>
>                [ '-1', '0', '1', '2', '3', '4', '5', '6', '7', '8',
> '9', '10' ], @@ -308,7 +308,7 @@ if ( $op eq 'add_form' ) {
>          $row_data{libopac}      = "";
>          $row_data{seealso}      = "";
>          $row_data{kohafield}    = CGI::scrolling_list(
> -            -name     => 'kohafield',
> +            -name     => 'kohafield$j',
>              -id       => "kohafield$j",
>              -values   => \@kohafields,
>              -default  => "",
> @@ -338,7 +338,7 @@ if ( $op eq 'add_form' ) {
>              -label    => ''
>          );
>          $row_data{value_builder} = CGI::scrolling_list(
> -            -name     => "value_builder",
> +            -name     => "value_builder$j",
>              -id       => "value_builder$j",
>              -values   => \@value_builder,
>              -default  => $data->{'value_builder'},
> @@ -346,14 +346,14 @@ if ( $op eq 'add_form' ) {
>              -multiple => 0,
>          );
>          $row_data{authorised_value} = CGI::scrolling_list(
> -            -name     => "authorised_value",
> +            -name     => "authorised_value$j",
>              -id       => "authorised_value$j",
>              -values   => \@authorised_values,
>              -size     => 1,
>              -multiple => 0,
>          );
>          $row_data{authtypes} = CGI::scrolling_list(
> -            -name     => "authtypecode",
> +            -name     => "authtypecode$j",
>              -id       => "authtypecode$j",
>              -values   => \@authtypes,
>              -size     => 1,


-- 
Nahuel ANGELINETTI