Bug 10132: Simplify code, call the method only once
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 19 Sep 2017 12:59:36 +0000 (09:59 -0300)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 19 Sep 2017 13:39:41 +0000 (10:39 -0300)
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
C4/AuthoritiesMarc.pm

index ac109de..160ff4a 100644 (file)
@@ -608,15 +608,17 @@ sub AddAuthority {
        if ($format eq "MARC21") {
         my $userenv = C4::Context->userenv;
         my $library;
+        my $marcorgcode = C4::Context->preference('MARCOrgCode');
         if ( $userenv && $userenv->{'branch'} ) {
             $library = Koha::Libraries->find( $userenv->{'branch'} );
+            $marcorgcode = $library->get_effective_marcorgcode;
         }
                if (!$record->leader) {
                        $record->leader($leader);
                }
                if (!$record->field('003')) {
                        $record->insert_fields_ordered(
-                MARC::Field->new('003', $library ? $library->get_effective_marcorgcode : C4::Context->preference('MARCOrgCode'))
+                MARC::Field->new('003', $marcorgcode),
                        );
                }
                my $date=POSIX::strftime("%y%m%d",localtime);
@@ -635,8 +637,8 @@ sub AddAuthority {
                if (!$record->field('040')) {
                 $record->insert_fields_ordered(
         MARC::Field->new('040','','',
-            'a' => $library ? $library->get_effective_marcorgcode : C4::Context->preference('MARCOrgCode'),
-            'c' => $library ? $library->get_effective_marcorgcode : C4::Context->preference('MARCOrgCode')
+            'a' => $marcorgcode,
+            'c' => $marcorgcode,
                                ) 
                        );
     }