Bug 15524: Set limit on maximum possible holds per patron by category

[koha.git] / admin / fieldmapping.pl

diff --git a/admin/fieldmapping.pl b/admin/fieldmapping.pl
index 05b5202..631e7d4 100755 (executable)
--- a/admin/fieldmapping.pl
+++ b/admin/fieldmapping.pl
@@ -1,5 +1,6 @@
 #!/usr/bin/perl
 # Copyright 2009 SARL BibLibre
+# Copyright 2017 Koha Development Team
 #
 # This file is part of Koha.
 #
@@ -16,69 +17,54 @@
 # 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 Modern::Perl;
 use CGI qw ( -utf8 );
 use C4::Auth;
 use C4::Biblio;
-use C4::Koha;
 use C4::Output;
 
-my $query = new CGI;
+use Koha::BiblioFrameworks;
+use Koha::FieldMappings;
 
-my $framework = $query->param('framework') || "";
+my $query = new CGI;
 
+my $frameworkcode = $query->param('framework') || "";
 my $field         = $query->param('fieldname');
 my $fieldcode     = $query->param('marcfield');
 my $subfieldcode  = $query->param('marcsubfield');
 my $op            = $query->param('op') || q{};
 my $id            = $query->param('id');
 
-my ($template, $loggedinuser, $cookie)
-    = get_template_and_user({template_name => "admin/fieldmapping.tt",
-                            query => $query,
-                            type => "intranet",
-                            authnotrequired => 0,
-                 flagsrequired => {parameters => 'parameters_remaining_permissions'},
-                            debug => 1,
-                            });
-
-# get framework list
-my $frameworks = getframeworks();
-my @frameworkloop;
-my $selected;
-my $frameworktext;
-foreach my $thisframeworkcode (keys %$frameworks) {
-        if ($thisframeworkcode eq $framework){
-                $selected = 1;
-                $frameworktext = $frameworks->{$thisframeworkcode}->{'frameworktext'};
-     } else {
-               $selected = 0;
-     }
-       my %row =(value => $thisframeworkcode,
-                               selected => $selected,
-                               frameworktext => $frameworks->{$thisframeworkcode}->{'frameworktext'},
-                       );
-       push @frameworkloop, \%row;
-}
-
-if($op eq "delete" and $id){
-    DeleteFieldMapping($id);
-    print $query->redirect("/cgi-bin/koha/admin/fieldmapping.pl?framework=".$framework);
-    exit;
-}
+my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+    {
+        template_name   => "admin/fieldmapping.tt",
+        query           => $query,
+        type            => "intranet",
+        authnotrequired => 0,
+        flagsrequired   => { parameters => 'parameters_remaining_permissions' },
+        debug           => 1,
+    }
+);
 
-# insert operation
-if($field and $fieldcode){
-    SetFieldMapping($framework, $field, $fieldcode, $subfieldcode);
+# FIXME Add exceptions
+if ( $op eq "delete" and $id ) {
+    Koha::FieldMappings->find($id)->delete;
+} elsif ( $field and $fieldcode ) {
+    my $params = { frameworkcode => $frameworkcode, field => $field, fieldcode => $fieldcode, subfieldcode => $subfieldcode };
+    my $exists = Koha::FieldMappings->search( $params )->count;;
+    unless ( $exists ) {
+        Koha::FieldMapping->new( $params )->store;
+    }
 }
 
-my $fieldloop = GetFieldMapping($framework);
+my $fields = Koha::FieldMappings->search({ frameworkcode => $frameworkcode });
 
-$template->param( frameworkloop => \@frameworkloop, 
-                  framework     => $framework,
-                  frameworktext => $frameworktext,
-                  fields        => $fieldloop,
-                );
+my $frameworks = Koha::BiblioFrameworks->search({}, { order_by => ['frameworktext'] });
+my $framework  = $frameworks->search( { frameworkcode => $frameworkcode } )->next;
+$template->param(
+    frameworks => $frameworks,
+    framework  => $framework,
+    fields     => $fields,
+);
 
 output_html_with_http_headers $query, $cookie, $template->output;