From a8f68074774e61941d9caff0aec52c893ff6fa21 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 24 Nov 2014 07:48:02 -0500 Subject: [PATCH] [PASSED QA] Bug 10363: [QA Followup] Signed-off-by: Tomas Cohen Arazi Signed-off-by: Katrin Fischer --- Koha/AuthorisedValues.pm | 2 +- admin/authorised_values.pl | 8 +++++--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/Koha/AuthorisedValues.pm b/Koha/AuthorisedValues.pm index ec201c3..55c7266 100644 --- a/Koha/AuthorisedValues.pm +++ b/Koha/AuthorisedValues.pm @@ -63,7 +63,7 @@ sub search { ->search( { %$params, %$or, }, $join ); my $class = ref($self); - return wantarray ? $self->_wrap( $rs->all() ) : $class->new_from_dbic($rs); + return wantarray ? $self->_wrap( $rs->all() ) : $class->_new_from_dbic($rs); } sub categories { diff --git a/admin/authorised_values.pl b/admin/authorised_values.pl index 022348e..1cf683d 100755 --- a/admin/authorised_values.pl +++ b/admin/authorised_values.pl @@ -101,7 +101,7 @@ if ($op eq 'add_form') { my $imageurl = $input->param( 'imageurl' ) || ''; $imageurl = '' if $imageurl =~ /removeImage/; my $duplicate_entry = 0; - my @branches = $input->param('branches'); + my @branches = grep { $_ ne q{} } $input->param('branches'); if ( $id ) { # Update my $av = Koha::AuthorisedValues->new->find( $id ); @@ -113,7 +113,7 @@ if ($op eq 'add_form') { $av->imageurl( $imageurl ); eval{ $av->store; - $av->branch_limitations( \@branches ); + $av->replace_branch_limitations( \@branches ); }; if ( $@ ) { push @messages, {type => 'error', code => 'error_on_update' }; @@ -129,10 +129,12 @@ if ($op eq 'add_form') { lib_opac => $input->param('lib_opac') || undef, imageurl => $imageurl, } ); + eval { $av->store; - $av->branch_limitations( \@branches ); + $av->replace_branch_limitations( \@branches ); }; + if ( $@ ) { push @messages, {type => 'error', code => 'error_on_insert' }; } else { -- 1.9.1