Bug 14544: Get rid of C4::VirtualShelves and C4::VirtualShelves::Page
[koha.git] / opac / opac-addbybiblionumber.pl
index 4c730d1..b8ad330 100755 (executable)
@@ -24,7 +24,6 @@ use warnings;
 
 use CGI qw ( -utf8 );
 use C4::Biblio;
-use C4::VirtualShelves qw/:DEFAULT/;
 use C4::Output;
 use C4::Auth;
 
@@ -41,6 +40,11 @@ our $authorized          = 1;
 our $errcode           = 0;
 our @biblios;
 
+
+if (scalar(@biblionumber) == 1) {
+    @biblionumber = (split /\//,$biblionumber[0]);
+}
+
 our ( $template, $loggedinuser, $cookie ) = get_template_and_user(
     {
         template_name   => "opac-addbybiblionumber.tt",
@@ -72,61 +76,64 @@ else {
 }
 #end
 
-sub AddBibliosToShelf {
-    #splits incoming biblionumber(s) to array and adds each to shelf.
-    my ($shelfnumber,@biblionumber)=@_;
+sub HandleNewVirtualShelf {
+    if ( $loggedinuser > 0 and
+        (
+            $category == 1
+                or $category == 2 and $loggedinuser>0 && C4::Context->preference('OpacAllowPublicListCreation')
+        )
+    ) {
+        my $shelf = eval {
+            Koha::Virtualshelf->new(
+                {
+                    shelfname => $newvirtualshelf,
+                    category => $category,
+                    owner => $loggedinuser,
+                }
+            )->store;
+        };
+        if ( $@ or not $shelf ) {
+            $authorized = 0;
+            $errcode = 1;
+            return;
+        }
 
-    #multiple bibs might come in as '/' delimited string (from where, i don't see), or as array.
-    if (scalar(@biblionumber) == 1) {
-        @biblionumber = (split /\//,$biblionumber[0]);
-    }
-    for my $bib (@biblionumber) {
-        AddToShelf($bib, $shelfnumber, $loggedinuser);
-    }
-}
+        for my $bib (@biblionumber) {
+            $shelf->add_biblio( $bib, $loggedinuser );
+        }
 
-sub HandleNewVirtualShelf {
-    if($authorized= ShelfPossibleAction($loggedinuser, undef, $category==1? 'new_private': 'new_public')) {
-    my $shelf = eval {
-        Koha::Virtualshelf->new(
-            {
-                shelfname => $newvirtualshelf,
-                category => $category,
-                owner => $loggedinuser,
-            }
-        );
-    };
-    if ( $@ or not $shelf ) {
-        $authorized=0;
-        $errcode=1;
-        return;
-    }
-    AddBibliosToShelf($shelfnumber, @biblionumber);
-    #Reload the page where you came from
-    print $query->header;
-    print "<html><meta http-equiv=\"refresh\" content=\"0\" /><body onload=\"window.opener.location.reload(true);self.close();\"></body></html>";
+        #Reload the page where you came from
+        print $query->header;
+        print "<html><meta http-equiv=\"refresh\" content=\"0\" /><body onload=\"window.opener.location.reload(true);self.close();\"></body></html>";
     }
 }
 
 sub HandleShelfNumber {
-    if($authorized= ShelfPossibleAction($loggedinuser, $shelfnumber, 'add')) {
-    AddBibliosToShelf($shelfnumber,@biblionumber);
-    #Close this page and return
-    print $query->header;
-    print "<html><meta http-equiv=\"refresh\" content=\"0\" /><body onload=\"self.close();\"></body></html>";
+    my $shelfnumber = $query->param('shelfnumber');
+    my $shelf = Koha::Virtualshelves->find( $shelfnumber );
+    if ( $shelf->can_biblios_be_added( $loggedinuser ) ) {
+        for my $bib (@biblionumber) {
+            $shelf->add_biblio( $bib, $loggedinuser );
+        }
+        #Close this page and return
+        print $query->header;
+        print "<html><meta http-equiv=\"refresh\" content=\"0\" /><body onload=\"self.close();\"></body></html>";
+    } else {
+        # TODO
     }
 }
 
 sub HandleSelectedShelf {
-    if($authorized= ShelfPossibleAction( $loggedinuser, $selectedshelf, 'add')){
-        #adding to specific shelf
-        my $shelfnumber = $query->param('selectedshelf');
-        my $shelf = Koha::Virtualshelves->find( $shelfnumber );
+    my $shelfnumber = $query->param('selectedshelf');
+    my $shelf = Koha::Virtualshelves->find( $shelfnumber );
+    if ( $shelf->can_biblios_be_added( $loggedinuser ) ) {
         $template->param(
             singleshelf               => 1,
             shelfnumber               => $shelf->shelfnumber,
             shelfname                 => $shelf->shelfname,
         );
+    } else {
+        # TODO
     }
 }
 
@@ -170,17 +177,13 @@ sub HandleSelect {
 }
 
 sub LoadBib {
-    #see comment in AddBibliosToShelf
-    if (scalar(@biblionumber) == 1) {
-        @biblionumber = (split /\//,$biblionumber[0]);
-    }
     for my $bib (@biblionumber) {
         my $data = GetBiblioData( $bib );
-    push(@biblios,
-        { biblionumber => $bib,
-          title        => $data->{'title'},
-          author       => $data->{'author'},
-    } );
+        push(@biblios,
+            { biblionumber => $bib,
+              title        => $data->{'title'},
+              author       => $data->{'author'},
+        } );
     }
     $template->param(
         multiple => (scalar(@biblios) > 1),