Bug 10572: Add phone to message_transport_types table for new installs
[koha.git] / admin / biblio_framework.pl
index 97d364b..583bc50 100755 (executable)
 # 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::Context;
 use C4::Auth;
 use C4::Output;
 
-
 sub StringSearch  {
-       my ($searchstring,$type)=@_;
        my $dbh = C4::Context->dbh;
-       $searchstring=~ s/\'/\\\'/g;
-       my @data=split(' ',$searchstring);
-       my $count=@data;
        my $sth=$dbh->prepare("Select * from biblio_framework where (frameworkcode like ?) order by frameworktext");
-       $sth->execute("$data[0]%");
-       my @results;
-       while (my $data=$sth->fetchrow_hashref){
-       push(@results,$data);
-       }
-       #  $sth->execute;
-       $sth->finish;
-       return (scalar(@results),\@results);
+       $sth->execute((shift || '') . '%');
+    return $sth->fetchall_arrayref({});
 }
 
 my $input = new CGI;
-my $searchfield=$input->param('frameworkcode');
-my $offset=$input->param('offset');
-my $script_name="/cgi-bin/koha/admin/biblio_framework.pl";
-my $frameworkcode=$input->param('frameworkcode');
-my $pagesize=20;
-my $op = $input->param('op');
-$searchfield=~ s/\,//g;
+my $script_name   = "/cgi-bin/koha/admin/biblio_framework.pl";
+my $frameworkcode = $input->param('frameworkcode') || '';
+my $offset        = $input->param('offset') || 0;
+my $op            = $input->param('op') || '';
+my $pagesize      = 20;
+
 my ($template, $borrowernumber, $cookie)
     = get_template_and_user({template_name => "admin/biblio_framework.tmpl",
                             query => $input,
                             type => "intranet",
                             authnotrequired => 0,
-                            flagsrequired => {parameters => 1},
+                 flagsrequired => {parameters => 'parameters_remaining_permissions'},
                             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);
+$template->param(($op||'else') => 1);
 
+my $dbh = C4::Context->dbh;
 ################## ADD_FORM ##################################
 # called by default. Used to create form to add or  modify a record
 if ($op eq 'add_form') {
@@ -80,82 +62,73 @@ if ($op eq 'add_form') {
        #---- if primkey exists, it's a modify action, so read values to modify...
        my $data;
        if ($frameworkcode) {
-               my $dbh = C4::Context->dbh;
                my $sth=$dbh->prepare("select * from biblio_framework where frameworkcode=?");
                $sth->execute($frameworkcode);
                $data=$sth->fetchrow_hashref;
-               $sth->finish;
        }
-       $template->param(frameworkcode => $frameworkcode,
-                                                       frameworktext => $data->{'frameworktext'},
-                                                       );
-;
+       $template->param(
+        frameworkcode => $frameworkcode,
+        frameworktext => $data->{'frameworktext'},
+    );
                                                                                                        # END $OP eq ADD_FORM
 ################## 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 biblio_framework (frameworkcode,frameworktext) values (?,?)");
-       $sth->execute($input->param('frameworkcode'),$input->param('frameworktext'));
-       $sth->finish;
-       print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=biblio_framework.pl\"></html>";
+       if($input->param('frameworktext') and $input->param('frameworkcode')){
+        if ($input->param('modif')) {
+            my $sth=$dbh->prepare("UPDATE biblio_framework SET frameworktext=? WHERE frameworkcode=?");
+            $sth->execute($input->param('frameworktext'),$input->param('frameworkcode'));
+        } else {
+            my $sth=$dbh->prepare("INSERT into biblio_framework (frameworkcode,frameworktext) values (?,?)");
+            $sth->execute($input->param('frameworkcode'),$input->param('frameworktext'));
+        }
+       }
+       print $input->redirect($script_name);   # FIXME: unnecessary redirect
        exit;
                                                                                                        # END $OP eq ADD_VALIDATE
 ################## DELETE_CONFIRM ##################################
 # called by default form, used to confirm deletion of data in DB
 } elsif ($op eq 'delete_confirm') {
-       #start the page and read in includes
-       my $dbh = C4::Context->dbh;
-
        # Check both categoryitem and biblioitems, see Bug 199
-       my $total = 0;
-       for my $table ('marc_tag_structure') {
-          my $sth=$dbh->prepare("select count(*) as total from $table where frameworkcode=?");
-          $sth->execute($frameworkcode);
-          $total += $sth->fetchrow_hashref->{total};
-          $sth->finish;
-       }
+    my $sth = $dbh->prepare("select count(*) as total from biblio where frameworkcode=?");
+    $sth->execute($frameworkcode);
+    my $total = $sth->fetchrow_hashref->{total};
 
-       my $sth=$dbh->prepare("select * from biblio_framework where frameworkcode=?");
+       $sth = $dbh->prepare("select * from biblio_framework where frameworkcode=?");
        $sth->execute($frameworkcode);
-       my $data=$sth->fetchrow_hashref;
-       $sth->finish;
+       my $data = $sth->fetchrow_hashref;
 
-       $template->param(frameworkcode => $frameworkcode,
-                                                       frameworktext => $data->{'frameworktext'},
-                                                       total => $total);
+       $template->param(
+        frameworkcode => $frameworkcode,
+        frameworktext => $data->{'frameworktext'},
+        total => $total
+    );
                                                                                                        # 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') {
-       #start the page and read in includes
-       my $dbh = C4::Context->dbh;
-       my $frameworkcode=uc($input->param('frameworkcode'));
-       my $sth=$dbh->prepare("delete from marc_tag_structure where frameworkcode=?");
-       $sth->execute($frameworkcode);
-       $sth=$dbh->prepare("delete from marc_subfield_structure where frameworkcode=?");
-       $sth->execute($frameworkcode);
-       $sth=$dbh->prepare("delete from biblio_framework where frameworkcode=?");
-       $sth->execute($frameworkcode);
-       $sth->finish;
-       print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=biblio_framework.pl\"></html>";
+    if ($frameworkcode) { 
+               my $sth=$dbh->prepare("delete from marc_tag_structure where frameworkcode=?");
+               $sth->execute($frameworkcode);
+               $sth=$dbh->prepare("delete from marc_subfield_structure where frameworkcode=?");
+               $sth->execute($frameworkcode);
+               $sth=$dbh->prepare("delete from biblio_framework where frameworkcode=?");
+               $sth->execute($frameworkcode);
+       }
+       print $input->redirect($script_name);   # FIXME: unnecessary redirect
        exit;
                                                                                                        # END $OP eq DELETE_CONFIRMED
 ################## DEFAULT ##################################
 } else { # DEFAULT
-       my ($count,$results)=StringSearch($searchfield,'web');
-       my $toggle="white";
+       my $results = StringSearch($frameworkcode);
+    my $count = scalar(@$results);
        my @loop_data;
        for (my $i=$offset; $i < ($offset+$pagesize<$count?$offset+$pagesize:$count); $i++){
-               my %row_data;
-               if ($toggle eq 'white'){
-                       $row_data{toggle}="#ffffcc";
-               } else {
-                       $row_data{toggle}="white";
-               }
-               $row_data{frameworkcode} = $results->[$i]{'frameworkcode'};
-               $row_data{frameworktext} = $results->[$i]{'frameworktext'};
-               push(@loop_data, \%row_data);
+               push @loop_data, {
+            frameworkcode => $results->[$i]{'frameworkcode'},
+            frameworktext => $results->[$i]{'frameworktext'},
+        };
        }
        $template->param(loop => \@loop_data);
        if ($offset>0) {
@@ -167,12 +140,6 @@ if ($op eq 'add_form') {
                $template->param(next => "$script_name?offset=".$nextpage);
        }
 } #---- END $OP eq DEFAULT
-$template->param(intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
-               intranetstylesheet => C4::Context->preference("intranetstylesheet"),
-               IntranetNav => C4::Context->preference("IntranetNav"),
-               );
+
 output_html_with_http_headers $input, $cookie, $template->output;
 
-# Local Variables:
-# tab-width: 4
-# End: