X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=authorities%2Fauthorities-home.pl;h=f67f5c9c40e17f68e36ab6f9353c15ffd9c1ddda;hb=cc748481d1f4ac02944d0f94624ea63e5b2a3b70;hp=c68001ae1b38217b934f4447088ce2ff1542ee86;hpb=b767f50c8f4d9216db2be82ed8c0b92e170d1db0;p=koha.git diff --git a/authorities/authorities-home.pl b/authorities/authorities-home.pl index c68001ae1b..f67f5c9c40 100755 --- a/authorities/authorities-home.pl +++ b/authorities/authorities-home.pl @@ -1,5 +1,4 @@ #!/usr/bin/perl -# WARNING: 4-character tab stops here # Copyright 2000-2002 Katipo Communications # @@ -14,248 +13,178 @@ # 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; -require Exporter; +use warnings; + use CGI; +use URI::Escape; use C4::Auth; use C4::Context; use C4::Auth; use C4::Output; -use C4::Interface::CGI::Output; use C4::AuthoritiesMarc; use C4::Acquisition; -use C4::Koha; # XXX subfield_is_koha_internal_p +use C4::Koha; # XXX subfield_is_koha_internal_p use C4::Biblio; -my $query=new CGI; -my $op = $query->param('op'); -my $authtypecode = $query->param('authtypecode'); -my $dbh = C4::Context->dbh; +my $query = new CGI; +my $dbh = C4::Context->dbh; +my $op = $query->param('op') || ''; +my $authtypecode = $query->param('authtypecode') || ''; +my $authid = $query->param('authid') || ''; -my $authid=$query->param('authid'); -my ($template, $loggedinuser, $cookie); +my ( $template, $loggedinuser, $cookie ); my $authtypes = getauthtypes; my @authtypesloop; -foreach my $thisauthtype (sort { $authtypes->{$a} <=> $authtypes->{$b} } keys %$authtypes) { - my $selected = 1 if $thisauthtype eq $authtypecode; - my %row =(value => $thisauthtype, - selected => $selected, - authtypetext => $authtypes->{$thisauthtype}{'authtypetext'}, - ); - push @authtypesloop, \%row; +foreach my $thisauthtype ( + sort { + $authtypes->{$a}{'authtypetext'} cmp $authtypes->{$b}{'authtypetext'} + } + keys %$authtypes + ) +{ + my %row = ( + value => $thisauthtype, + selected => $thisauthtype eq $authtypecode, + authtypetext => $authtypes->{$thisauthtype}{'authtypetext'}, + ); + push @authtypesloop, \%row; } -if ($op eq "do_search") { - my @marclist = $query->param('marclist'); - my @and_or = $query->param('and_or'); - my @excluding = $query->param('excluding'); - my @operator = $query->param('operator'); - my $orderby = $query->param('orderby'); - my @value = $query->param('value'); - - my $startfrom = $query->param('startfrom') || 1; - my $resultsperpage = $query->param('resultsperpage') || 19; +if ( $op eq "delete" ) { + ( $template, $loggedinuser, $cookie ) = get_template_and_user( + { + template_name => "authorities/authorities-home.tmpl", + query => $query, + type => 'intranet', + authnotrequired => 0, + flagsrequired => { catalogue => 1 }, + debug => 1, + } + ); + &DelAuthority( $authid, 1 ); - my ($results,$total) = SearchAuthorities( - \@marclist, - \@and_or, - \@excluding, - \@operator, - \@value, - ($startfrom - 1)*$resultsperpage, - $resultsperpage, - $authtypecode, + if ( $query->param('operator') ) { + # query contains search params so perform search + $op = "do_search"; + } + else { + $op = ''; + } +} +if ( $op eq "do_search" ) { + my $marclist = $query->param('marclist') || ''; + my $and_or = $query->param('and_or') || ''; + my $excluding = $query->param('excluding') || ''; + my $operator = $query->param('operator') || ''; + my $orderby = $query->param('orderby') || ''; + my $value = $query->param('value') || ''; + + my $startfrom = $query->param('startfrom') || 1; + my $resultsperpage = $query->param('resultsperpage') || 20; + + my ( $results, $total ) = SearchAuthorities( + [$marclist], [$and_or], + [$excluding], [$operator], + [$value], ( $startfrom - 1 ) * $resultsperpage, + $resultsperpage, $authtypecode, $orderby ); - ($template, $loggedinuser, $cookie) - = get_template_and_user({ - template_name => "authorities/searchresultlist.tmpl", - query => $query, - type => 'intranet', + ( $template, $loggedinuser, $cookie ) = get_template_and_user( + { + template_name => "authorities/searchresultlist.tmpl", + query => $query, + type => 'intranet', authnotrequired => 0, - flagsrequired => {catalogue => 1}, - debug => 1, - }); + flagsrequired => { catalogue => 1 }, + debug => 1, + } + ); - my @field_data = (); + $template->param( + marclist => $marclist, + and_or => $and_or, + excluding => $excluding, + operator => $operator, + orderby => $orderby, + value => $value, + authtypecode => $authtypecode, + startfrom => $startfrom, + resultsperpage => $resultsperpage, + ); - # we must get parameters once again. Because if there is a mainentry, it - # has been replaced by something else during the search, thus the links - # next/previous would not work anymore - my @marclist_ini = $query->param('marclist'); - for(my $i = 0 ; $i <= $#marclist ; $i++) - { - push @field_data, { term => "marclist" , val=>$marclist_ini[$i] }; - push @field_data, { term => "and_or" , val=>$and_or[$i] }; - push @field_data, { term => "excluding" , val=>$excluding[$i] }; - push @field_data, { term => "operator" , val=>$operator[$i] }; - push @field_data, { term => "value" , val=>$value[$i] }; - } + # we must get parameters once again. Because if there is a mainentry, it + # has been replaced by something else during the search, thus the links + # next/previous would not work anymore # construction of the url of each page - my $base_url = - 'authorities-home.pl?' - .join( - '&', - map { $_->{term}.'='.$_->{val} } @field_data - ) - .'&' - .join( - '&', - map { $_->{term}.'='.$_->{val} } ( - {term => 'resultsperpage', val => $resultsperpage}, - {term => 'type' , val => 'intranet'}, - {term => 'op' , val => 'do_search'}, - {term => 'authtypecode' , val => $authtypecode}, - {term => 'orderby' , val => $orderby}, - ) - ) - ; - - my $from = ($startfrom - 1) * $resultsperpage + 1; - my $to; - - if ($total < $startfrom * $resultsperpage) { - $to = $total; - } + my $value_url = uri_escape($value); + my $base_url = "authorities-home.pl?" + ."marclist=$marclist" + ."&and_or=$and_or" + ."&excluding=$excluding" + ."&operator=$operator" + ."&value=$value_url" + ."&resultsperpage=$resultsperpage" + ."&type=intranet" + ."&op=do_search" + ."&authtypecode=$authtypecode" + ."&orderby=$orderby"; + + my $from = ( $startfrom - 1 ) * $resultsperpage + 1; + my $to; + if ( !defined $total ) { + $total = 0; + } + + if ( $total < $startfrom * $resultsperpage ) { + $to = $total; + } else { - $to = $startfrom * $resultsperpage; - } + $to = $startfrom * $resultsperpage; + } - $template->param(result => $results) if $results; + $template->param( result => $results ) if $results; - $template->param( + $template->param( pagination_bar => pagination_bar( - $base_url, - int($total/$resultsperpage)+1, - $startfrom, - 'startfrom' - ), - total=>$total, - from=>$from, - to=>$to, + $base_url, int( $total / $resultsperpage ) + 1, + $startfrom, 'startfrom' + ), + total => $total, + from => $from, + to => $to, isEDITORS => $authtypecode eq 'EDITORS', ); -} elsif ($op eq "delete") { - - &DelAuthority($authid, 1); - - ($template, $loggedinuser, $cookie) - = get_template_and_user({template_name => "authorities/authorities-home.tmpl", - query => $query, - type => 'intranet', - authnotrequired => 0, - flagsrequired => {catalogue => 1}, - debug => 1, - }); -# $template->param("statements" => \@statements, -# "nbstatements" => $nbstatements); } -elsif ($op eq "AddStatement") { - - ($template, $loggedinuser, $cookie) - = get_template_and_user({template_name => "authorities/authorities-home.tmpl", - query => $query, - type => 'intranet', - authnotrequired => 0, - flagsrequired => {catalogue => 1}, - debug => 1, - }); - - # Gets the entered information - my @marcfields = $query->param('marclist'); - my @and_or = $query->param('and_or'); - my @excluding = $query->param('excluding'); - my @operator = $query->param('operator'); - my @value = $query->param('value'); - - my @statements = (); - - # List of the marc tags to display - my $marcarray = create_marclist(); - - my $nbstatements = $query->param('nbstatements'); - $nbstatements = 1 if(!defined $nbstatements); - - for(my $i = 0 ; $i < $nbstatements ; $i++) - { - my %fields = (); - - # Recreates the old scrolling lists with the previously selected values - my $marclist = create_scrolling_list({name=>"marclist", - values=> $marcarray, - size=> 1, - -tabindex=>'', - default=>$marcfields[$i], - onChange => "sql_update()"} - ); - - $fields{'marclist'} = $marclist; - $fields{'first'} = 1 if($i == 0); - - # Restores the and/or parameters (no need to test the 'and' for activation because it's the default value) - $fields{'or'} = 1 if($and_or[$i] eq "or"); - - #Restores the "not" parameters - $fields{'not'} = 1 if($excluding[$i]); - - #Restores the operators (most common operators first); - if($operator[$i] eq "=") { $fields{'eq'} = 1; } - elsif($operator[$i] eq "contains") { $fields{'contains'} = 1; } - elsif($operator[$i] eq "start") { $fields{'start'} = 1; } - elsif($operator[$i] eq ">") { $fields{'gt'} = 1; } #greater than - elsif($operator[$i] eq ">=") { $fields{'ge'} = 1; } #greater or equal - elsif($operator[$i] eq "<") { $fields{'lt'} = 1; } #lower than - elsif($operator[$i] eq "<=") { $fields{'le'} = 1; } #lower or equal - - #Restores the value - $fields{'value'} = $value[$i]; - - push @statements, \%fields; - } - $nbstatements++; - - # The new scrolling list - my $marclist = create_scrolling_list({name=>"marclist", - values=> $marcarray, - size=>1, - -tabindex=>'', - onChange => "sql_update()"}); - push @statements, {"marclist" => $marclist }; - - $template->param("statements" => \@statements, - "nbstatements" => $nbstatements); +if ( $op eq '' ) { + ( $template, $loggedinuser, $cookie ) = get_template_and_user( + { + template_name => "authorities/authorities-home.tmpl", + query => $query, + type => 'intranet', + authnotrequired => 0, + flagsrequired => { catalogue => 1 }, + debug => 1, + } + ); } -else { - ($template, $loggedinuser, $cookie) - = get_template_and_user({template_name => "authorities/authorities-home.tmpl", - query => $query, - type => 'intranet', - authnotrequired => 0, - flagsrequired => {catalogue => 1}, - debug => 1, - }); -} +$template->param( + authtypesloop => \@authtypesloop, + op => $op, +); -$template->param(authtypesloop => \@authtypesloop, - intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"), - intranetstylesheet => C4::Context->preference("intranetstylesheet"), - IntranetNav => C4::Context->preference("IntranetNav"), - ); +$template->{VARS}->{marcflavour} = C4::Context->preference("marcflavour"); # Print the page output_html_with_http_headers $query, $cookie, $template->output; - -# Local Variables: -# tab-width: 4 -# End: