X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=admin%2Fmarc_subfields_structure.pl;h=bc6636a2885f5babc4a15ee70fd769f8efe24855;hb=e4c1e89691920a048d702c0d9b35b6499416ff27;hp=bae9c6501eed6f6d5eccc2ae6c9b3a7fce30487c;hpb=bce3e2e619e3efffdd01caf560ce53a8ba903df2;p=koha.git
diff --git a/admin/marc_subfields_structure.pl b/admin/marc_subfields_structure.pl
index bae9c6501e..bc6636a288 100755
--- a/admin/marc_subfields_structure.pl
+++ b/admin/marc_subfields_structure.pl
@@ -76,7 +76,7 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
debug => 1,
}
);
-my $pagesize = 30;
+
my $op = $input->param('op');
$tagfield =~ s/\,//g;
@@ -179,6 +179,7 @@ if ( $op eq 'add_form' ) {
while ( $data = $sth->fetchrow_hashref ) {
my %row_data; # get a fresh hash for the row data
$row_data{defaultvalue} = $data->{defaultvalue};
+ $row_data{maxlength} = $data->{maxlength};
$row_data{tab} = CGI::scrolling_list(
-name => 'tab',
-id => "tab$i",
@@ -273,14 +274,13 @@ if ( $op eq 'add_form' ) {
}
# add more_subfields empty lines for add if needed
- for ( my $j = 1 ; $j <= 1 ; $j++ ) {
my %row_data; # get a fresh hash for the row data
$row_data{'new_subfield'} = 1;
$row_data{'subfieldcode'} = '';
$row_data{tab} = CGI::scrolling_list(
-name => 'tab',
- -id => "tab$j",
+ -id => "tab$i",
-values =>
[ '-1', '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '10' ],
-labels => {
@@ -310,7 +310,7 @@ if ( $op eq 'add_form' ) {
$row_data{seealso} = "";
$row_data{kohafield} = CGI::scrolling_list(
-name => 'kohafield',
- -id => "kohafield$j",
+ -id => "kohafield$i",
-values => \@kohafields,
-default => "",
-size => 1,
@@ -318,29 +318,29 @@ if ( $op eq 'add_form' ) {
);
$row_data{hidden} = "";
$row_data{repeatable} = CGI::checkbox(
- -name => "repeatable$j",
- -id => "repeatable$j",
+ -name => "repeatable$i",
+ -id => "repeatable$i",
-checked => '',
-value => 1,
-label => ''
);
$row_data{mandatory} = CGI::checkbox(
- -name => "mandatory$j",
- -id => "mandatory$j",
+ -name => "mandatory$i",
+ -id => "mandatory$i",
-checked => '',
-value => 1,
-label => ''
);
$row_data{isurl} = CGI::checkbox(
- -name => "isurl$j",
- -id => "isurl$j",
+ -name => "isurl$i",
+ -id => "isurl$i",
-checked => '',
-value => 1,
-label => ''
);
$row_data{value_builder} = CGI::scrolling_list(
-name => "value_builder",
- -id => "value_builder$j",
+ -id => "value_builder$i",
-values => \@value_builder,
-default => $data->{'value_builder'},
-size => 1,
@@ -348,24 +348,24 @@ if ( $op eq 'add_form' ) {
);
$row_data{authorised_value} = CGI::scrolling_list(
-name => "authorised_value",
- -id => "authorised_value$j",
+ -id => "authorised_value$i",
-values => \@authorised_values,
-size => 1,
-multiple => 0,
);
$row_data{authtypes} = CGI::scrolling_list(
-name => "authtypecode",
- -id => "authtypecode$j",
+ -id => "authtypecode$i",
-values => \@authtypes,
-size => 1,
-multiple => 0,
);
$row_data{link} = CGI::escapeHTML( $data->{'link'} );
- $row_data{row} = $j;
+ $row_data{row} = $i;
push( @loop_data, \%row_data );
- }
- $template->param( 'use-heading-flags-p' => 1 );
- $template->param( 'heading-edit-subfields-p' => 1 );
+
+ $template->param( 'use_heading_flags_p' => 1 );
+ $template->param( 'heading_edit_subfields_p' => 1 );
$template->param(
action => "Edit subfields",
tagfield => $tagfield,
@@ -386,11 +386,11 @@ elsif ( $op eq 'add_validate' ) {
# values (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)"
# );
my $sth_insert = $dbh->prepare(qq{
- insert into marc_subfield_structure (tagfield,tagsubfield,liblibrarian,libopac,repeatable,mandatory,kohafield,tab,seealso,authorised_value,authtypecode,value_builder,hidden,isurl,frameworkcode, link,defaultvalue)
- values (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)
+ insert into marc_subfield_structure (tagfield,tagsubfield,liblibrarian,libopac,repeatable,mandatory,kohafield,tab,seealso,authorised_value,authtypecode,value_builder,hidden,isurl,frameworkcode, link,defaultvalue,maxlength)
+ values (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)
});
my $sth_update = $dbh->prepare(qq{
- update marc_subfield_structure set tagfield=?, tagsubfield=?, liblibrarian=?, libopac=?, repeatable=?, mandatory=?, kohafield=?, tab=?, seealso=?, authorised_value=?, authtypecode=?, value_builder=?, hidden=?, isurl=?, frameworkcode=?, link=?, defaultvalue=?
+ update marc_subfield_structure set tagfield=?, tagsubfield=?, liblibrarian=?, libopac=?, repeatable=?, mandatory=?, kohafield=?, tab=?, seealso=?, authorised_value=?, authtypecode=?, value_builder=?, hidden=?, isurl=?, frameworkcode=?, link=?, defaultvalue=?, maxlength=?
where tagfield=? and tagsubfield=? and frameworkcode=?
});
my @tagsubfield = $input->param('tagsubfield');
@@ -405,6 +405,7 @@ elsif ( $op eq 'add_validate' ) {
my @value_builder = $input->param('value_builder');
my @link = $input->param('link');
my @defaultvalue = $input->param('defaultvalue');
+ my @maxlength = $input->param('maxlength');
for ( my $i = 0 ; $i <= $#tagsubfield ; $i++ ) {
my $tagfield = $input->param('tagfield');
@@ -425,8 +426,9 @@ elsif ( $op eq 'add_validate' ) {
my $isurl = $input->param("isurl$i") ? 1 : 0;
my $link = $link[$i];
my $defaultvalue = $defaultvalue[$i];
+ my $maxlength = $maxlength[$i];
- if ($liblibrarian) {
+ if (defined($liblibrarian) && $liblibrarian ne "") {
unless ( C4::Context->config('demo') eq 1 ) {
if (marc_subfield_structure_exists($tagfield, $tagsubfield, $frameworkcode)) {
$sth_update->execute(
@@ -447,6 +449,7 @@ elsif ( $op eq 'add_validate' ) {
$frameworkcode,
$link,
$defaultvalue,
+ $maxlength,
(
$tagfield,
$tagsubfield,
@@ -472,6 +475,7 @@ elsif ( $op eq 'add_validate' ) {
$frameworkcode,
$link,
$defaultvalue,
+ $maxlength,
);
}
}
@@ -531,12 +535,7 @@ elsif ( $op eq 'delete_confirmed' ) {
else { # DEFAULT
my ( $count, $results ) = string_search( $tagfield, $frameworkcode );
my @loop_data = ();
- for (
- my $i = $offset ;
- $i < ( $offset + $pagesize < $count ? $offset + $pagesize : $count ) ;
- $i++
- )
- {
+ for ( my $i = 0; $i < $count; $i++ ) {
my %row_data; # get a fresh hash for the row data
$row_data{tagfield} = $results->[$i]{'tagfield'};
$row_data{tagsubfield} = $results->[$i]{'tagsubfield'};
@@ -552,10 +551,6 @@ else { # DEFAULT
$row_data{hidden} = $results->[$i]{'hidden'};
$row_data{isurl} = $results->[$i]{'isurl'};
$row_data{link} = $results->[$i]{'link'};
- $row_data{delete} =
-"$script_name?op=delete_confirm&tagfield=$tagfield&tagsubfield="
- . $results->[$i]{'tagsubfield'}
- . "&frameworkcode=$frameworkcode";
if ( $row_data{tab} eq -1 ) {
$row_data{subfield_ignored} = 1;
@@ -569,16 +564,6 @@ else { # DEFAULT
edit_frameworkcode => $frameworkcode
);
- if ( $offset > 0 ) {
- my $prevpage = $offset - $pagesize;
- $template->param(
- prev => "" );
- }
- if ( $offset + $pagesize < $count ) {
- my $nextpage = $offset + $pagesize;
- $template->param(
- next => "" );
- }
} #---- END $OP eq DEFAULT
output_html_with_http_headers $input, $cookie, $template->output;