X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=admin%2Fauthorised_values.pl;h=d7872d649aa7e06eacfc4e9921c12084c5aa8d40;hb=9a168e7c9451e1960dadbecf83ba937851840310;hp=16f9fff4e47546be80c5c0b711a49658609009c0;hpb=4ed42893e8a11946707bfc574a768284926edca4;p=koha.git diff --git a/admin/authorised_values.pl b/admin/authorised_values.pl index 16f9fff4e4..d7872d649a 100755 --- a/admin/authorised_values.pl +++ b/admin/authorised_values.pl @@ -13,119 +13,159 @@ # 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., 59 Temple Place, -# Suite 330, Boston, MA 02111-1307 USA +# 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. use strict; +use warnings; + use CGI; use C4::Auth; use C4::Context; +use C4::Koha; use C4::Output; -use C4::Interface::CGI::Output; -use C4::Search; -use HTML::Template; -use C4::Context; -sub StringSearch { - my ($env,$searchstring,$type)=@_; - my $dbh = C4::Context->dbh; - $searchstring=~ s/\'/\\\'/g; - my @data=split(' ',$searchstring); - my $count=@data; - my $query="Select id,category,authorised_value,lib from authorised_values where (category like \"$data[0]%\") order by category,authorised_value"; - my $sth=$dbh->prepare($query); - $sth->execute; - my @results; - my $cnt=0; - while (my $data=$sth->fetchrow_hashref){ - push(@results,$data); - $cnt ++; - } - $sth->finish; - return ($cnt,\@results); +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({}); } my $input = new CGI; -my $searchfield=$input->param('searchfield'); -$searchfield=~ s/\,//g; -my $id = $input->param('id'); -my $reqsel="select category,authorised_value,lib from authorised_values where id='$id'"; -my $reqdel="delete from authorised_values where id='$id'"; -my $offset=$input->param('offset'); -my $script_name="/cgi-bin/koha/admin/authorised_values.pl"; +my $id = $input->param('id'); +my $op = $input->param('op') || ''; +my $offset = $input->param('offset') || 0; +my $searchfield = $input->param('searchfield'); +$searchfield = '' unless defined $searchfield; +$searchfield =~ s/\,//g; +my $script_name = "/cgi-bin/koha/admin/authorised_values.pl"; my $dbh = C4::Context->dbh; -my ($template, $borrowernumber, $cookie) - = get_template_and_user({template_name => "parameters/authorised_values.tmpl", - query => $input, - type => "intranet", - authnotrequired => 0, - flagsrequired => {parameters => 1}, - debug => 1, - }); -my $pagesize=20; -my $op = $input->param('op'); +my ($template, $borrowernumber, $cookie)= get_template_and_user({ + template_name => "admin/authorised_values.tmpl", + authnotrequired => 0, + flagsrequired => {parameters => 1}, + query => $input, + type => "intranet", + debug => 1, +}); -if ($op) { -$template->param(script_name => $script_name, - $op => 1); # we show only the TMPL_VAR names $op -} else { -$template->param(script_name => $script_name, - else => 1); # we show only the TMPL_VAR names $op -} +$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; if ($id) { - my $dbh = C4::Context->dbh; - my $sth=$dbh->prepare("select id,category,authorised_value,lib from authorised_values where id='$id'"); - $sth->execute; + 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->finish; } else { $data->{'category'} = $input->param('category'); } - if ($searchfield) { - $template->param(action => "Modify authorised value"); + if ($id) { + $template->param(action_modify => 1); + $template->param('heading_modify_authorized_value_p' => 1); } elsif ( ! $data->{'category'} ) { - $template->param(action => "Add new category"); - } else { - $template->param(action => "Add authorised value"); - } - $template->param(category => $data->{'category'}, - authorised_value => $data->{'authorised_value'}, - lib => $data->{'lib'}, - id => $data->{'id'} - ); - if ($data->{'category'}) { - $template->param(category => "$data->{'category'}"); + $template->param(action_add_category => 1); + $template->param('heading_add_new_category_p' => 1); } else { - $template->param(category => ""); + $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, + ); + ################## ADD_VALIDATE ################################## # called by add_form, used to insert/modify data in DB } elsif ($op eq 'add_validate') { - my $dbh = C4::Context->dbh; - my $sth=$dbh->prepare("replace authorised_values (id,category,authorised_value,lib) values (?,?,?,?)"); - my $lib = $input->param('lib'); - undef $lib if ($lib eq ""); # to insert NULL instead of a blank string + my $new_authorised_value = $input->param('authorised_value'); + my $new_category = $input->param('category'); + my $imageurl = $input->param( 'imageurl' ) || ''; + $imageurl = '' if $imageurl =~ /removeImage/; + my $duplicate_entry = 0; + + if ( $id ) { # Update + my $sth = $dbh->prepare( "SELECT category, authorised_value FROM authorised_values WHERE id='$id' "); + $sth->execute(); + 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 = '$new_category' AND authorised_value = '$new_authorised_value' and id<>$id"); + $sth->execute(); + ($duplicate_entry) = $sth->fetchrow_array(); + warn "**** duplicate_entry = $duplicate_entry"; + } + 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); + print "Content-Type: text/html\n\n"; + exit; + } + } + else { # Insert + my $sth = $dbh->prepare_cached( "SELECT COUNT(*) FROM authorised_values " . + "WHERE category = '$new_category' AND authorised_value = '$new_authorised_value' "); + $sth->execute(); + ($duplicate_entry) = $sth->fetchrow_array(); + unless ( $duplicate_entry ) { + my $sth=$dbh->prepare( 'INSERT INTO authorised_values + ( id, 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($id, $new_category, $new_authorised_value, $lib, $lib_opac, $imageurl ); + print "Content-Type: text/html\n\nparam('category')."&offset=$offset\">"; + exit; + } + } + if ( $duplicate_entry ) { + $template->param(duplicate_category => $new_category, + duplicate_value => $new_authorised_value, + else => 1); + default_form(); + } - $sth->execute($input->param('id'), $input->param('category'), $input->param('authorised_value'), $lib); - $sth->finish; - print "Content-Type: text/html\n\nparam('category')."\">"; - exit; ################## DELETE_CONFIRM ################################## # called by default form, used to confirm deletion of data in DB } elsif ($op eq 'delete_confirm') { - my $dbh = C4::Context->dbh; - my $sth=$dbh->prepare($reqsel); - $sth->execute; + my $sth=$dbh->prepare("select category,authorised_value,lib,lib_opac from authorised_values where id=?"); + $sth->execute($id); my $data=$sth->fetchrow_hashref; - $sth->finish; + $id = $input->param('id') unless $id; $template->param(searchfield => $searchfield, + Tlib => $data->{'lib'}, + Tlib_opac => $data->{'lib_opac'}, Tvalue => $data->{'authorised_value'}, id =>$id, ); @@ -134,25 +174,39 @@ if ($op eq 'add_form') { ################## DELETE_CONFIRMED ################################## # called by delete_confirm, used to effectively confirm deletion of data in DB } elsif ($op eq 'delete_confirmed') { - my $dbh = C4::Context->dbh; - my $sth=$dbh->prepare($reqdel); - $sth->execute; - $sth->finish; - print "Content-Type: text/html\n\n"; + 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"; exit; - # END $OP eq DELETE_CONFIRMED ################## DEFAULT ################################## } else { # DEFAULT + default_form(); +} #---- END $OP eq DEFAULT +output_html_with_http_headers $input, $cookie, $template->output; + +exit 0; + +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; } # push koha system categories + foreach (qw(Asort1 Asort2 Bsort1 Bsort2 SUGGEST DAMAGED LOST)) { + push @category_list, $_ unless $categories{$_}; + } + + #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, @@ -161,40 +215,24 @@ if ($op eq 'add_form') { if (!$searchfield) { $searchfield=$category_list[0]; } - my $env; - my ($count,$results)=StringSearch($env,$searchfield,'web'); - my $toggle="white"; + my ($results) = AuthorizedValuesForCategory($searchfield); + my $count = scalar(@$results); my @loop_data = (); # builds value list - for (my $i=$offset; $i < ($offset+$pagesize<$count?$offset+$pagesize:$count); $i++){ - if ($toggle eq 'white'){ - $toggle="#ffffcc"; - } else { - $toggle="white"; - } + for (my $i=0; $i < $count; $i++){ 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{edit} = "$script_name?op=add_form&id=".$results->[$i]{'id'}; - $row_data{delete} = "$script_name?op=delete_confirm&searchfield=$searchfield&id=".$results->[$i]{'id'}; + $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"; push(@loop_data, \%row_data); } - $template->param(loop => \@loop_data, - tab_list => $tab_list, - category => $searchfield); - - if ($offset>0) { - my $prevpage = $offset-$pagesize; - $template->param(previous => ""); - } - if ($offset+$pagesize<$count) { - my $nextpage =$offset+$pagesize; - $template->param(next => ""); - - - } -} #---- END $OP eq DEFAULT + $template->param( loop => \@loop_data, + tab_list => $tab_list, + category => $searchfield ); +} -output_html_with_http_headers $input, $cookie, $template->output;