Bug 9456: (follow-up) remove whitespaces and tab
[koha.git] / admin / z3950servers.pl
index b63807d..1dbd118 100755 (executable)
@@ -5,7 +5,7 @@
 # This software is placed under the gnu General Public License, v2 (http://www.gnu.org/licenses/gpl.html)
 
 # ALGO :
-# this script use an $op to know what to do.
+# this script use ano $op to know what to do.
 # if $op is empty or none of the above values,
 #      - the default screen is build (with all records, or filtered datas).
 #      - the   user can clic on add, modify or delete record.
 #      - we delete the record having primkey=$primkey
 
 use strict;
-use C4::Output;
+use warnings;
 use CGI;
-use C4::Search;
-use C4::Database;
 use C4::Context;
-use HTML::Template;
 use C4::Auth;
-use C4::Interface::CGI::Output;
+use C4::Output;
 
 sub StringSearch  {
-       my ($env,$searchstring,$type)=@_;
+       my ($searchstring,$type)=@_;
        my $dbh = C4::Context->dbh;
-       $searchstring=~ s/\'/\\\'/g;
-       my @data=split(' ',$searchstring);
-       my $count=@data;
-       my $query="Select host,port,db,userid,password,name,id,checked,rank,syntax from z3950servers where (name like \"$data[0]\%\") order by rank,name";
+    my @data = ('%');
+    my $count = 1;
+       if ( defined $searchstring ) {
+        $searchstring =~ s/\'/\\\'/g;
+        @data=split(' ',$searchstring);
+        $count=@data;
+    }
+    else {
+        $searchstring = '';
+    }
+
+    my $query    = "SELECT host,port,db,userid,password,name,id,checked,rank,syntax,encoding,timeout,recordtype";
+    $query      .= " FROM z3950servers";
+    if ( $searchstring ne '' ) { $query .= " WHERE (name like ?)" }
+    $query      .= " ORDER BY rank,name";
        my $sth=$dbh->prepare($query);
-       $sth->execute;
-       my @results;
-       my $cnt=0;
-       while (my $data=$sth->fetchrow_hashref) {
-           push(@results,$data);
-           $cnt ++;
-       }
-       #  $sth->execute;
-       $sth->finish;
-       $dbh->disconnect;
-       return ($cnt,\@results);
+
+    if ( $searchstring ne '' ) {
+        $sth->execute("$data[0]\%");
+    }
+    else {
+        $sth->execute;
+    }
+
+    my @results;
+    while (my $data=$sth->fetchrow_hashref) {
+        push(@results,$data);
+    }
+    $sth->finish;
+    return (scalar(@results),\@results);
 }
 
 my $input = new CGI;
 my $searchfield=$input->param('searchfield');
-my $reqsel="select host,port,db,userid,password,name,id,checked,rank,syntax from z3950servers where (name = '$searchfield') order by rank,name";
-my $reqdel="delete from z3950servers where name='$searchfield'";
-my $offset=$input->param('offset');
+my $offset=$input->param('offset') || 0;
 my $script_name="/cgi-bin/koha/admin/z3950servers.pl";
 
 my $pagesize=20;
-my $op = $input->param('op');
-$searchfield=~ s/\,//g;
+my $op = $input->param('op') || '';
 
 my ($template, $loggedinuser, $cookie) 
-    = get_template_and_user({template_name => "parameters/z3950servers.tmpl",
+    = get_template_and_user({template_name => "admin/z3950servers.tmpl",
                                query => $input,
                                type => "intranet",
                                authnotrequired => 0,
-                               flagsrequired => {parameters => 1},
+                flagsrequired => {parameters => 'parameters_remaining_permissions'},
                                debug => 1,
                                });
 
@@ -77,79 +85,94 @@ $template->param(script_name => $script_name,
 
 ################## ADD_FORM ##################################
 # called by default. Used to create form to add or  modify a record
-if ($op eq 'add_form') {
-       $template->param(add_form => 1);
-       #---- if primkey exists, it's a modify action, so read values to modify...
-       my $data;
-       if ($searchfield) {
-               my $dbh = C4::Context->dbh;
-               my $sth=$dbh->prepare("select host,port,db,userid,password,name,id,checked,rank,syntax from z3950servers where (name = '$searchfield') order by rank,name");
-               $sth->execute;
-               $data=$sth->fetchrow_hashref;
-               $sth->finish;
-       }
-       
-       $template->param(host => $data->{'host'},
-                        port => $data->{'port'},
-                        db   => $data->{'db'},
-                        userid => $data->{'userid'},
-                        password => $data->{'password'},
-                        checked => $data->{'checked'},
-                        rank => $data->{'rank'});
-                                                                                                       # END $OP eq ADD_FORM
+if ( $op eq 'add_form' ) {
+    $template->param( add_form => 1 );
+
+    #---- if primkey exists, it's a modify action, so read values to modify...
+    my $data;
+    if ($searchfield) {
+        my $dbh = C4::Context->dbh;
+        my $sth = $dbh->prepare(
+"select host,port,db,userid,password,name,id,checked,rank,syntax,encoding,timeout,recordtype from z3950servers where (name = ?) order by rank,name"
+        );
+        $sth->execute($searchfield);
+        $data = $sth->fetchrow_hashref;
+        $sth->finish;
+    }
+    $template->param( $_ => $data->{$_} )
+      for (qw( host port db userid password checked rank timeout encoding ));
+    $template->param( $_ . $data->{$_} => 1 ) for (qw( syntax recordtype ));
+
+    # END $OP eq ADD_FORM
 ################## ADD_VALIDATE ##################################
-# called by add_form, used to insert/modify data in DB
+    # called by add_form, used to insert/modify data in DB
 } elsif ($op eq 'add_validate') {
-       $template->param(add_validate => 1);
        my $dbh=C4::Context->dbh;
        my $sth=$dbh->prepare("select * from z3950servers where name=?");
        $sth->execute($input->param('searchfield'));
+       my $checked = $input->param('checked') ? 1 : 0;
        if ($sth->rows) {
-               $sth=$dbh->prepare("update z3950servers set host=?, port=?, db=?, userid=?, password=?, name=?, checked=?, rank=?,syntax=? where name=?");
+        $template->param(confirm_update => 1);
+             $sth=$dbh->prepare("update z3950servers set host=?, port=?, db=?, userid=?, password=?, name=?, checked=?, rank=?,syntax=?,encoding=?,timeout=?,recordtype=? where name=?");
                $sth->execute($input->param('host'),
                      $input->param('port'),
                      $input->param('db'),
                      $input->param('userid'),
                      $input->param('password'),
                      $input->param('searchfield'),
-                     $input->param('checked'),
+                     $checked,
                      $input->param('rank'),
-                        $input->param('syntax'),
+                         $input->param('syntax'),
+              $input->param('encoding'),
+              $input->param('timeout'),
+              $input->param('recordtype'),
                      $input->param('searchfield'),
                      );
-       } else {
-               $sth=$dbh->prepare("insert into z3950servers (host,port,db,userid,password,name,checked,rank,syntax) values (?, ?, ?, ?, ?, ?, ?, ?,?)");
-               $sth->execute($input->param('host'),
-                     $input->param('port'),
-                     $input->param('db'),
-                     $input->param('userid'),
-                     $input->param('password'),
-                     $input->param('searchfield'),
-                     $input->param('checked'),
-                     $input->param('rank'),
-                        $input->param('syntax'),
-                     );
-       }
-       $sth->finish;
-                                                                                                       # END $OP eq ADD_VALIDATE
+       } 
+       else {
+        $template->param(confirm_add => 1);
+               $sth=$dbh->prepare(
+                 "INSERT INTO z3950servers " .
+              "(host,port,db,userid,password,name,checked,rank,syntax,encoding,timeout,recordtype) " .
+               "VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)" );
+        $sth->execute(
+            $input->param('host'),     $input->param('port'),
+            $input->param('db'),       $input->param('userid'),
+            $input->param('password'), $input->param('searchfield'),
+            $checked,                  $input->param('rank'),
+            $input->param('syntax'),   $input->param('encoding'),
+            $input->param('timeout'),  $input->param('recordtype')
+        );
+    }
+    $sth->finish;
+
+    # END $OP eq ADD_VALIDATE
 ################## DELETE_CONFIRM ##################################
 # called by default form, used to confirm deletion of data in DB
 } elsif ($op eq 'delete_confirm') {
-       $template->param(delete_confirm => 1);
-       my $dbh = C4::Context->dbh;
-
-       my $sth2=$dbh->prepare($reqsel);
-       $sth2->execute;
-       my $data=$sth2->fetchrow_hashref;
-       $sth2->finish;
-
-        $template->param(host => $data->{'host'},
-                         port => $data->{'port'},
-                         db   => $data->{'db'},
-                         userid => $data->{'userid'},
-                         password => $data->{'password'},
-                         checked => $data->{'checked'},
-                         rank => $data->{'rank'});
+    $template->param( delete_confirm => 1 );
+    my $dbh = C4::Context->dbh;
+
+    my $sth2 = $dbh->prepare(
+"select host,port,db,userid,password,name,id,checked,rank,syntax,encoding,timeout,recordtype from z3950servers where (name = ?) order by rank,name"
+    );
+    $sth2->execute($searchfield);
+    my $data = $sth2->fetchrow_hashref;
+    $sth2->finish;
+
+    $template->param(
+        host       => $data->{'host'},
+        port       => $data->{'port'},
+        db         => $data->{'db'},
+        userid     => $data->{'userid'},
+        password   => $data->{'password'},
+        checked    => $data->{'checked'},
+        rank       => $data->{'rank'},
+        syntax     => $data->{'syntax'},
+        timeout    => $data->{'timeout'},
+        recordtype => $data->{'recordtype'},
+        encoding   => $data->{'encoding'}
+    );
 
                                                                                                        # END $OP eq DELETE_CONFIRM
 ################## DELETE_CONFIRMED ##################################
@@ -157,20 +180,17 @@ if ($op eq 'add_form') {
 } elsif ($op eq 'delete_confirmed') {
        $template->param(delete_confirmed => 1);
        my $dbh=C4::Context->dbh;
-       my $sth=$dbh->prepare($reqdel);
-       $sth->execute;
+       my $sth=$dbh->prepare("delete from z3950servers where name=?");
+       $sth->execute($searchfield);
        $sth->finish;
                                                                                                        # END $OP eq DELETE_CONFIRMED
 ################## DEFAULT ##################################
 } else { # DEFAULT
        $template->param(else => 1);
-
-       my $env;
-       my ($count,$results)=StringSearch($env,$searchfield,'web');
+       my ($count,$results)=StringSearch($searchfield,'web');
        my @loop;
-       my $toggle = 'white';
+
        for (my $i=$offset; $i < ($offset+$pagesize<$count?$offset+$pagesize:$count); $i++){
-                       
                my $urlsearchfield=$results->[$i]{name};
                $urlsearchfield=~s/ /%20/g;
                my %row = ( name => $results->[$i]{'name'},
@@ -182,18 +202,11 @@ if ($op eq 'add_form') {
                        checked => $results->[$i]{'checked'},
                        rank => $results->[$i]{'rank'},
                        syntax => $results->[$i]{'syntax'},
-                       toggle => $toggle);
+                       encoding => $results->[$i]{'encoding'},
+      timeout => $results->[$i]{'timeout'},
+            recordtype => $results->[$i]{'recordtype'});
                push @loop, \%row;
 
-                if ( $toggle eq 'white' )
-                {
-                        $toggle = '#ffffcc';
-                }
-                else
-                {
-                        $toggle = 'white';
-                }
-
        }
        $template->param(loop => \@loop);
        if ($offset>0) {
@@ -205,5 +218,4 @@ if ($op eq 'add_form') {
                                 nextpage => $offset+$pagesize);
        }
 } #---- END $OP eq DEFAULT
-
 output_html_with_http_headers $input, $cookie, $template->output;