X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=admin%2Fbiblio_framework.pl;h=d99351b18c0c2e603f496cfdcdf55e73d65218d8;hb=590cae04fd4cb0fc6fa309d6006fd14f48ea4a03;hp=f7a709135929377f6a6d81900642d78b725e8981;hpb=2c6765610856bd298cec045077a55296ae10abda;p=koha.git

diff --git a/admin/biblio_framework.pl b/admin/biblio_framework.pl
index f7a7091359..d99351b18c 100755
--- a/admin/biblio_framework.pl
+++ b/admin/biblio_framework.pl
@@ -1,145 +1,130 @@
 #!/usr/bin/perl
-# NOTE: 4-character tabs
-
-#written 20/02/2002 by paul.poulain@free.fr
-# This software is placed under the gnu General Public License, v2 (http://www.gnu.org/licenses/gpl.html)
 
 # Copyright 2000-2002 Katipo Communications
+# Copyright 2002 Paul Poulain
 #
 # 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 CGI;
+use Modern::Perl;
+use CGI qw ( -utf8 );
 use C4::Context;
 use C4::Auth;
 use C4::Output;
+use Koha::Biblios;
+use Koha::BiblioFramework;
+use Koha::BiblioFrameworks;
+use Koha::Caches;
 
-sub StringSearch  {
-	my $dbh = C4::Context->dbh;
-	my $sth=$dbh->prepare("Select * from biblio_framework where (frameworkcode like ?) order by frameworktext");
-	$sth->execute((shift || '') . '%');
-    return $sth->fetchall_arrayref({});
-}
+my $input         = new CGI;
+my $frameworkcode = $input->param('frameworkcode') || q||;
+my $op            = $input->param('op') || q|list|;
+my $cache         = Koha::Caches->get_instance();
+my @messages;
 
-my $input = new CGI;
-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.tt",
+        query           => $input,
+        type            => "intranet",
+        authnotrequired => 0,
+        flagsrequired   => { parameters => 'parameters_remaining_permissions' },
+        debug           => 1,
+    }
+);
 
-my ($template, $borrowernumber, $cookie)
-    = get_template_and_user({template_name => "admin/biblio_framework.tmpl",
-			     query => $input,
-			     type => "intranet",
-			     authnotrequired => 0,
-			     flagsrequired => {parameters => 1},
-			     debug => 1,
-			     });
+my $dbh = C4::Context->dbh;
+if ( $op eq 'add_form' ) {
+    my $framework;
+    if ($frameworkcode) {
+        $framework = Koha::BiblioFrameworks->find($frameworkcode);
+    }
+    $template->param( framework => $framework );
+} elsif ( $op eq 'add_validate' ) {
+    my $frameworkcode = $input->param('frameworkcode');
+    my $frameworktext = $input->param('frameworktext');
+    my $is_a_modif    = $input->param('is_a_modif');
 
-$template->param( script_name  => $script_name);
-$template->param(($op||'else') => 1);
+    if ($is_a_modif) {
+        my $framework = Koha::BiblioFrameworks->find($frameworkcode);
+        $framework->frameworktext($frameworktext);
+        eval { $framework->store; };
+        if ($@) {
+            push @messages, { type => 'error', code => 'error_on_update' };
+        } else {
+            push @messages, { type => 'message', code => 'success_on_update' };
+        }
+    } else {
+        my $framework = Koha::BiblioFramework->new(
+            {   frameworkcode => $frameworkcode,
+                frameworktext => $frameworktext,
+            }
+        );
+        eval { $framework->store; };
+        if ($@) {
+            push @messages, { type => 'error', code => 'error_on_insert' };
+        } else {
+            push @messages, { type => 'message', code => 'success_on_insert' };
+        }
+    }
+    $cache->clear_from_cache("MarcStructure-0-$frameworkcode");
+    $cache->clear_from_cache("MarcStructure-1-$frameworkcode");
+    $cache->clear_from_cache("default_value_for_mod_marc-");
+    $cache->clear_from_cache("MarcSubfieldStructure-$frameworkcode");
+    $op = 'list';
+} elsif ( $op eq 'delete_confirm' ) {
+    my $framework = Koha::BiblioFrameworks->find($frameworkcode);
+    my $count = Koha::Biblios->search( { frameworkcode => $frameworkcode, } )->count;
 
-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') {
-	#start the page and read in includes
-	#---- if primkey exists, it's a modify action, so read values to modify...
-	my $data;
-	if ($frameworkcode) {
-		my $sth=$dbh->prepare("select * from biblio_framework where frameworkcode=?");
-		$sth->execute($frameworkcode);
-		$data=$sth->fetchrow_hashref;
-	}
-	$template->param(
-        frameworkcode => $frameworkcode,
-        frameworktext => $data->{'frameworktext'},
+    $template->param(
+        framework                  => $framework,
+        biblios_use_this_framework => $count,
     );
-													# 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;
-	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'));
+} elsif ( $op eq 'delete_confirmed' ) {
+    my $framework = Koha::BiblioFrameworks->find($frameworkcode);
+    my $deleted = eval { $framework->delete; };
+
+    if ( $@ or not $deleted ) {
+        push @messages, { type => 'error', code => 'error_on_delete' };
+    } else {
+        eval {
+            my $dbh = C4::Context->dbh;
+            $dbh->do( q|DELETE FROM marc_tag_structure WHERE frameworkcode=?|,      undef, $frameworkcode );
+            $dbh->do( q|DELETE FROM marc_subfield_structure WHERE frameworkcode=?|, undef, $frameworkcode );
+        };
+        if ($@) {
+            push @messages, { type => 'error', code => 'error_on_delete_fk' };
         } else {
-            my $sth=$dbh->prepare("INSERT into biblio_framework (frameworkcode,frameworktext) values (?,?)");
-            $sth->execute($input->param('frameworkcode'),$input->param('frameworktext'));
+            push @messages, { type => 'message', code => 'success_on_delete' };
         }
-	}
-	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') {
-	# Check both categoryitem and biblioitems, see Bug 199
-    my $sth = $dbh->prepare("select count(*) as total from marc_tag_structure where frameworkcode=?");
-    $sth->execute($frameworkcode);
-    my $total = $sth->fetchrow_hashref->{total};
+    }
+    $cache->clear_from_cache("MarcStructure-0-$frameworkcode");
+    $cache->clear_from_cache("MarcStructure-1-$frameworkcode");
+    $cache->clear_from_cache("default_value_for_mod_marc-");
+    $cache->clear_from_cache("MarcSubfieldStructure-$frameworkcode");
+    $op = 'list';
+}
 
-	$sth = $dbh->prepare("select * from biblio_framework where frameworkcode=?");
-	$sth->execute($frameworkcode);
-	my $data = $sth->fetchrow_hashref;
+if ( $op eq 'list' ) {
+    my $frameworks = Koha::BiblioFrameworks->search( {}, { order_by => ['frameworktext'], } );
+    $template->param( frameworks => $frameworks, );
+}
 
-	$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') {
-    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 $results = StringSearch($frameworkcode);
-    my $count = scalar(@$results);
-	my @loop_data;
-	for (my $i=$offset; $i < ($offset+$pagesize<$count?$offset+$pagesize:$count); $i++){
-		push @loop_data, {
-            frameworkcode => $results->[$i]{'frameworkcode'},
-            frameworktext => $results->[$i]{'frameworktext'},
-        };
-	}
-	$template->param(loop => \@loop_data);
-	if ($offset>0) {
-		my $prevpage = $offset-$pagesize;
-		$template->param(previous => "$script_name?offset=".$prevpage);
-	}
-	if ($offset+$pagesize<$count) {
-		my $nextpage =$offset+$pagesize;
-		$template->param(next => "$script_name?offset=".$nextpage);
-	}
-} #---- END $OP eq DEFAULT
+$template->param(
+    messages => \@messages,
+    op       => $op,
+);
 
 output_html_with_http_headers $input, $cookie, $template->output;