#
# This file is part of Koha.
#
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
#
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along
-# with Koha; if not, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
-use strict;
-use warnings;
+use Modern::Perl;
-use CGI;
-use C4::Auth;
-use C4::Branch;
-use C4::Context;
-use C4::Koha;
-use C4::Output;
+use CGI qw ( -utf8 );
+use List::MoreUtils qw( any );
+use C4::Auth qw( get_template_and_user );
+use C4::Context;
+use C4::Koha qw( getitemtypeimagelocation );
+use C4::Output qw( output_html_with_http_headers );
-sub AuthorizedValuesForCategory {
- my ($searchstring) = shift or return;
- my $dbh = C4::Context->dbh;
- $searchstring=~ s/\'/\\\'/g;
- my @data=split(' ',$searchstring);
- my $sth=$dbh->prepare('
- SELECT id, category, authorised_value, lib, lib_opac, imageurl
- FROM authorised_values
- WHERE (category = ?)
- ORDER BY category, authorised_value
- ');
- $sth->execute("$data[0]");
- return $sth->fetchall_arrayref({});
-}
+use Koha::AuthorisedValues;
+use Koha::AuthorisedValueCategories;
+use Koha::Libraries;
-my $input = new CGI;
+my $input = CGI->new;
my $id = $input->param('id');
-my $op = $input->param('op') || '';
-our $offset = $input->param('offset') || 0;
-our $searchfield = $input->param('searchfield');
+my $op = $input->param('op') || 'list';
+my $searchfield = $input->param('searchfield');
$searchfield = '' unless defined $searchfield;
$searchfield =~ s/\,//g;
-our $script_name = "/cgi-bin/koha/admin/authorised_values.pl";
-our $dbh = C4::Context->dbh;
+my @messages;
our ($template, $borrowernumber, $cookie)= get_template_and_user({
- template_name => "admin/authorised_values.tmpl",
- authnotrequired => 0,
- flagsrequired => {parameters => 'parameters_remaining_permissions'},
+ template_name => "admin/authorised_values.tt",
+ flagsrequired => {parameters => 'manage_auth_values'},
query => $input,
type => "intranet",
- debug => 1,
});
-$template->param( script_name => $script_name,
- ($op||'else') => 1 );
################## ADD_FORM ##################################
# called by default. Used to create form to add or modify a record
if ($op eq 'add_form') {
- my $data;
- my @selected_branches;
- if ($id) {
- my $sth=$dbh->prepare("select id, category, authorised_value, lib, lib_opac, imageurl from authorised_values where id=?");
- $sth->execute($id);
- $data=$sth->fetchrow_hashref;
- $sth = $dbh->prepare("SELECT b.branchcode, b.branchname FROM authorised_values_branches AS avb, branches AS b WHERE avb.branchcode = b.branchcode AND avb.av_id = ?;");
- $sth->execute( $id );
- while ( my $branch = $sth->fetchrow_hashref ) {
- push @selected_branches, $branch;
- }
- } else {
- $data->{'category'} = $input->param('category');
- }
+ my ( @selected_branches, $category, $av );
+ if ($id) {
+ $av = Koha::AuthorisedValues->new->find( $id );
+ @selected_branches = $av->library_limits ? $av->library_limits->as_list : ();
+ } else {
+ $category = $input->param('category');
+ }
- my $branches = GetBranches;
+ my $branches = Koha::Libraries->search( {}, { order_by => ['branchname'] } );
my @branches_loop;
-
- foreach my $branch (sort keys %$branches) {
- my $selected = ( grep {$_->{branchcode} eq $branch} @selected_branches ) ? 1 : 0;
+ while ( my $branch = $branches->next ) {
push @branches_loop, {
- branchcode => $branches->{$branch}{branchcode},
- branchname => $branches->{$branch}{branchname},
- selected => $selected,
+ branchcode => $branch->branchcode,
+ branchname => $branch->branchname,
+ selected => any {$_->branchcode eq $branch->branchcode} @selected_branches,
};
}
if ($id) {
$template->param(action_modify => 1);
- $template->param('heading_modify_authorized_value_p' => 1);
- } elsif ( ! $data->{'category'} ) {
+ } elsif ( ! $category ) {
$template->param(action_add_category => 1);
- $template->param('heading_add_new_category_p' => 1);
} else {
$template->param(action_add_value => 1);
- $template->param('heading_add_authorized_value_p' => 1);
}
- $template->param('use_heading_flags_p' => 1);
- $template->param( category => $data->{'category'},
- authorised_value => $data->{'authorised_value'},
- lib => $data->{'lib'},
- lib_opac => $data->{'lib_opac'},
- id => $data->{'id'},
- imagesets => C4::Koha::getImageSets( checked => $data->{'imageurl'} ),
- offset => $offset,
- branches_loop => \@branches_loop,
- );
-
-################## ADD_VALIDATE ##################################
-# called by add_form, used to insert/modify data in DB
-} elsif ($op eq 'add_validate') {
+
+ if ( $av ) {
+ $template->param(
+ category_name => $av->category,
+ av => $av,
+ imagesets => C4::Koha::getImageSets( checked => $av->imageurl ),
+ );
+ } else {
+ $template->param(
+ category_name => $category,
+ imagesets => C4::Koha::getImageSets(),
+ );
+ }
+ $template->param(
+ branches_loop => \@branches_loop,
+ );
+
+} elsif ($op eq 'add') {
my $new_authorised_value = $input->param('authorised_value');
my $new_category = $input->param('category');
- my $imageurl = $input->param( 'imageurl' ) || '';
- $imageurl = '' if $imageurl =~ /removeImage/;
+ my $image = $input->param( 'image' ) || '';
+ my $imageurl =
+ $image eq 'removeImage' ? ''
+ : (
+ $image eq 'remoteImage' ? $input->param('remoteImage')
+ : $image
+ );
my $duplicate_entry = 0;
- my @branches = $input->param('branches');
+ my @branches = grep { $_ ne q{} } $input->multi_param('branches');
- if ( $id ) { # Update
- my $sth = $dbh->prepare( "SELECT category, authorised_value FROM authorised_values WHERE id = ? ");
- $sth->execute($id);
- my ($category, $authorised_value) = $sth->fetchrow_array();
- if ( $authorised_value ne $new_authorised_value ) {
- my $sth = $dbh->prepare_cached( "SELECT COUNT(*) FROM authorised_values " .
- "WHERE category = ? AND authorised_value = ? and id <> ? ");
- $sth->execute($new_category, $new_authorised_value, $id);
- ($duplicate_entry) = $sth->fetchrow_array();
- }
- unless ( $duplicate_entry ) {
- my $sth=$dbh->prepare( 'UPDATE authorised_values
- SET category = ?,
- authorised_value = ?,
- lib = ?,
- lib_opac = ?,
- imageurl = ?
- WHERE id=?' );
- my $lib = $input->param('lib');
- my $lib_opac = $input->param('lib_opac');
- undef $lib if ($lib eq ""); # to insert NULL instead of a blank string
- undef $lib_opac if ($lib_opac eq ""); # to insert NULL instead of a blank string
- $sth->execute($new_category, $new_authorised_value, $lib, $lib_opac, $imageurl, $id);
- if ( @branches ) {
- $sth = $dbh->prepare("DELETE FROM authorised_values_branches WHERE av_id = ?");
- $sth->execute( $id );
- $sth = $dbh->prepare(
- "INSERT INTO authorised_values_branches
- ( av_id, branchcode )
- VALUES ( ?, ? )"
- );
- for my $branchcode ( @branches ) {
- next if not $branchcode;
- $sth->execute($id, $branchcode);
- }
- }
- $sth->finish;
- print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=authorised_values.pl?searchfield=".$new_category."&offset=$offset\"></html>";
- exit;
+ if ( $new_category eq 'branches' or $new_category eq 'itemtypes' or $new_category eq 'cn_source' ) {
+ push @messages, {type => 'error', code => 'invalid_category_name' };
+ }
+ elsif ( $id ) { # Update
+ my $av = Koha::AuthorisedValues->new->find( $id );
+
+ $av->lib( scalar $input->param('lib') || undef );
+ $av->lib_opac( scalar $input->param('lib_opac') || undef );
+ $av->category( $new_category );
+ $av->authorised_value( $new_authorised_value );
+ $av->imageurl( $imageurl );
+ eval{
+ $av->store;
+ $av->replace_library_limits( \@branches );
+ };
+ if ( $@ ) {
+ push @messages, {type => 'error', code => 'error_on_update' };
+ } else {
+ push @messages, { type => 'message', code => 'success_on_update' };
}
}
else { # Insert
- my $sth = $dbh->prepare_cached( "SELECT COUNT(*) FROM authorised_values " .
- "WHERE category = ? AND authorised_value = ? ");
- $sth->execute($new_category, $new_authorised_value);
- ($duplicate_entry) = $sth->fetchrow_array();
- unless ( $duplicate_entry ) {
- my $sth=$dbh->prepare( 'INSERT INTO authorised_values
- ( category, authorised_value, lib, lib_opac, imageurl )
- values (?, ?, ?, ?, ?)' );
- my $lib = $input->param('lib');
- my $lib_opac = $input->param('lib_opac');
- undef $lib if ($lib eq ""); # to insert NULL instead of a blank string
- undef $lib_opac if ($lib_opac eq ""); # to insert NULL instead of a blank string
- $sth->execute( $new_category, $new_authorised_value, $lib, $lib_opac, $imageurl );
- $id = $dbh->{'mysql_insertid'};
- if ( @branches ) {
- $sth = $dbh->prepare(
- "INSERT INTO authorised_values_branches
- ( av_id, branchcode )
- VALUES ( ?, ? )"
- );
- for my $branchcode ( @branches ) {
- next if not $branchcode;
- $sth->execute($id, $branchcode);
+ eval {
+ my $av = Koha::AuthorisedValue->new(
+ {
+ category => $new_category,
+ authorised_value => $new_authorised_value,
+ lib => scalar $input->param('lib') || undef,
+ lib_opac => scalar $input->param('lib_opac') || undef,
+ imageurl => $imageurl,
}
- }
- print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=authorised_values.pl?searchfield=".$input->param('category')."&offset=$offset\"></html>";
- exit;
+ )->store;
+ $av->replace_library_limits( \@branches );
+ $av->store;
+ };
+
+ if ( $@ ) {
+ push @messages, {type => 'error', code => 'error_on_insert' };
+ } else {
+ push @messages, { type => 'message', code => 'success_on_insert' };
}
}
- if ( $duplicate_entry ) {
- $template->param(duplicate_category => $new_category,
- duplicate_value => $new_authorised_value,
- else => 1);
- default_form();
- }
-
-################## DELETE_CONFIRM ##################################
-# called by default form, used to confirm deletion of data in DB
-} elsif ($op eq 'delete_confirm') {
- my $sth=$dbh->prepare("select category,authorised_value,lib,lib_opac from authorised_values where id=?");
- $sth->execute($id);
- my $data=$sth->fetchrow_hashref;
- $id = $input->param('id') unless $id;
- $template->param(searchfield => $searchfield,
- Tlib => $data->{'lib'},
- Tlib_opac => $data->{'lib_opac'},
- Tvalue => $data->{'authorised_value'},
- id =>$id,
- );
- # END $OP eq DELETE_CONFIRM
-################## DELETE_CONFIRMED ##################################
-# called by delete_confirm, used to effectively confirm deletion of data in DB
-} elsif ($op eq 'delete_confirmed') {
- my $id = $input->param('id');
- my $sth=$dbh->prepare("delete from authorised_values where id=?");
- $sth->execute($id);
- print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=authorised_values.pl?searchfield=$searchfield&offset=$offset\"></html>";
- exit;
- # END $OP eq DELETE_CONFIRMED
-################## DEFAULT ##################################
-} else { # DEFAULT
- default_form();
-} #---- END $OP eq DEFAULT
-output_html_with_http_headers $input, $cookie, $template->output;
+ $op = 'list';
+ $searchfield = $new_category;
+} elsif ($op eq 'add_category' ) {
+ my $new_category = $input->param('category');
-exit 0;
+ my $already_exists = Koha::AuthorisedValueCategories->find(
+ {
+ category_name => $new_category,
+ }
+ );
-sub default_form {
- # build categories list
- my $sth = $dbh->prepare("select distinct category from authorised_values");
- $sth->execute;
- my @category_list;
- my %categories; # a hash, to check that some hardcoded categories exist.
- while ( my ($category) = $sth->fetchrow_array ) {
- push( @category_list, $category );
- $categories{$category} = 1;
+ if ( $already_exists ) {
+ if ( $new_category eq 'branches' or $new_category eq 'itemtypes' or $new_category eq 'cn_source' ) {
+ push @messages, {type => 'error', code => 'invalid_category_name' };
+ } else {
+ push @messages, {type => 'error', code => 'cat_already_exists' };
+ }
}
+ else { # Insert
+ my $av = Koha::AuthorisedValueCategory->new( {
+ category_name => $new_category,
+ } );
- # push koha system categories
- foreach (qw(Asort1 Asort2 Bsort1 Bsort2 SUGGEST DAMAGED LOST REPORT_GROUP REPORT_SUBGROUP)) {
- push @category_list, $_ unless $categories{$_};
- }
+ eval {
+ $av->store;
+ };
- #reorder the list
- @category_list = sort {$a cmp $b} @category_list;
- my $tab_list = CGI::scrolling_list(-name=>'searchfield',
- -id=>'searchfield',
- -values=> \@category_list,
- -default=>"",
- -size=>1,
- -multiple=>0,
- );
- if (!$searchfield) {
- $searchfield=$category_list[0];
- }
- my ($results) = AuthorizedValuesForCategory($searchfield);
- my $count = scalar(@$results);
- my @loop_data = ();
- # builds value list
- my $dbh = C4::Context->dbh;
- $sth = $dbh->prepare("SELECT b.branchcode, b.branchname FROM authorised_values_branches AS avb, branches AS b WHERE avb.branchcode = b.branchcode AND avb.av_id = ?");
- for (my $i=0; $i < $count; $i++){
- $sth->execute( $results->[$i]{id} );
- my @selected_branches;
- while ( my $branch = $sth->fetchrow_hashref ) {
- push @selected_branches, $branch;
+ if ( $@ ) {
+ push @messages, {type => 'error', code => 'error_on_insert_cat' };
+ } else {
+ push @messages, { type => 'message', code => 'success_on_insert_cat' };
+ $searchfield = $new_category;
}
- my %row_data; # get a fresh hash for the row data
- $row_data{category} = $results->[$i]{'category'};
- $row_data{authorised_value} = $results->[$i]{'authorised_value'};
- $row_data{lib} = $results->[$i]{'lib'};
- $row_data{lib_opac} = $results->[$i]{'lib_opac'};
- $row_data{imageurl} = getitemtypeimagelocation( 'intranet', $results->[$i]{'imageurl'} );
- $row_data{edit} = "$script_name?op=add_form&id=".$results->[$i]{'id'}."&offset=$offset";
- $row_data{delete} = "$script_name?op=delete_confirm&searchfield=$searchfield&id=".$results->[$i]{'id'}."&offset=$offset";
- $row_data{branches} = \@selected_branches;
- push(@loop_data, \%row_data);
- }
+ }
+
+ $op = 'list';
+} elsif ($op eq 'delete') {
+ my $av = Koha::AuthorisedValues->new->find( $id );
+ my $deleted = eval {$av->delete};
+ if ( $@ or not $deleted ) {
+ push @messages, {type => 'error', code => 'error_on_delete' };
+ } else {
+ push @messages, { type => 'message', code => 'success_on_delete' };
+ }
+
+ $op = 'list';
+} elsif ($op eq 'delete_category') {
+ my $category_name = $input->param('category_name');
+ my $avc = Koha::AuthorisedValueCategories->find( $category_name );
+ my $deleted = eval {$avc->delete};
+ if ( $@ or not $deleted ) {
+ push @messages, {type => 'error', code => 'error_on_delete_category' };
+ } else {
+ push @messages, { type => 'message', code => 'success_on_delete_category' };
+ }
- $template->param( loop => \@loop_data,
- tab_list => $tab_list,
- category => $searchfield );
+ $op = 'list';
}
+$template->param(
+ op => $op,
+ searchfield => $searchfield,
+ messages => \@messages,
+);
+
+if ( $op eq 'list' ) {
+ # build categories list
+ my @category_names = Koha::AuthorisedValueCategories->search(
+ {
+ category_name =>
+ { -not_in => [ '', 'branches', 'itemtypes', 'cn_source' ] }
+ },
+ { order_by => ['category_name'] }
+ )->get_column('category_name');
+
+ $searchfield ||= $category_names[0];
+
+ my @avs_by_category = Koha::AuthorisedValues->new->search( { category => $searchfield } )->as_list;
+ my @loop_data = ();
+ # builds value list
+ for my $av ( @avs_by_category ) {
+ my %row_data; # get a fresh hash for the row data
+ $row_data{category} = $av->category;
+ $row_data{authorised_value} = $av->authorised_value;
+ $row_data{lib} = $av->lib;
+ $row_data{lib_opac} = $av->lib_opac;
+ $row_data{image} = getitemtypeimagelocation( 'intranet', $av->imageurl );
+ $row_data{branches} = $av->library_limits ? $av->library_limits->as_list : [];
+ $row_data{id} = $av->id;
+ push(@loop_data, \%row_data);
+ }
+
+ $template->param(
+ loop => \@loop_data,
+ category => Koha::AuthorisedValueCategories->find($searchfield), # TODO Move this up and add a Koha::AVC->authorised_values method to replace call for avs_by_category
+ category_names => \@category_names,
+ );
+
+}
+output_html_with_http_headers $input, $cookie, $template->output;