Bug 6050 Make calls to GetItemsInfo consistent
[koha.git] / virtualshelves / addbybiblionumber.pl
index 112ecec..f1ac674 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.
 
-# $Id$
 
 =head1 NAME
 
-    addvirtualbybiblionumber.pl
+    addbybiblionumber.pl
 
 =head1 DESCRIPTION
 
 
     if this script has to add a shelf, it add one with this category.
 
+=item newshelf
+
+    if this parameter exists, then we create a new shelf
+
 =back
 
 =cut
 
 use strict;
+#use warnings; FIXME - Bug 2505
 use C4::Biblio;
 use CGI;
 use C4::Output;
-use C4::BookShelves;
+use C4::VirtualShelves qw/:DEFAULT GetRecentShelves/;
 use C4::Circulation;
 use C4::Auth;
 
+# splits incoming biblionumber(s) to array and adds each to shelf.
+sub AddBibliosToShelf {
+    my ($shelfnumber,@biblionumber)=@_;
 
-#use it only to debug !
-use CGI::Carp qw/fatalsToBrowser/;
-use warnings;
+    # multiple bibs might come in as '/' delimited string (from where, i don't see), or as array.
+    # (Note : they come in as '/' when added from the cart)
+    if (scalar(@biblionumber) == 1) {
+        @biblionumber = (split /\//,$biblionumber[0]);
+    }
+    for my $bib (@biblionumber){
+        AddToShelf($bib, $shelfnumber);
+    }
+}
 
-my $query = new CGI;
-my $biblionumber = $query->param('biblionumber');
-my $shelfnumber = $query->param('shelfnumber');
+my $query           = new CGI;
+
+# If set, then single item case.
+my $biblionumber    = $query->param('biblionumber');
+
+# If set, then multiple item case.
+my @biblionumber   = $query->param('biblionumber');
+my $biblionumbers   = $query->param('biblionumbers');
+
+my $shelfnumber     = $query->param('shelfnumber');
 my $newvirtualshelf = $query->param('newvirtualshelf');
-my $category = $query->param('category');
-
-my ($template, $loggedinuser, $cookie)
-= get_template_and_user({template_name => "virtualshelves/addvirtualbybiblionumber.tmpl",
-                            query => $query,
-                            type => "intranet",
-                            authnotrequired => 0,
-                            flagsrequired => {catalogue => 1},
-                        });
-
-$shelfnumber = AddShelf($newvirtualshelf,$loggedinuser,$category) if $newvirtualshelf;
-
-if ($shelfnumber || ($shelfnumber == -1)) { # the shelf already exist.
-    &AddToShelfFromBiblio($biblionumber, $shelfnumber);
-    print "Content-Type: text/html\n\n<html><body onload=\"window.close()\"></body></html>";
-    exit;
-} else {    # this shelf doesn't already exist.
-    my  ( $bibliocount, @biblios )  = GetBiblio($biblionumber);
-
-    my ($shelflist) = GetShelves($loggedinuser,3);
-    my @shelvesloop;
-    my %shelvesloop;
-    foreach my $element (sort keys %$shelflist) {
-        push (@shelvesloop, $element);
-        $shelvesloop{$element} = $shelflist->{$element}->{'shelfname'};
+my $newshelf        = $query->param('newshelf');
+my $category        = $query->param('category');
+my $sortfield      = $query->param('sortfield');
+my $confirmed       = $query->param('confirmed') || 0;
+
+
+my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+    {
+        template_name   => "virtualshelves/addbybiblionumber.tmpl",
+        query           => $query,
+        type            => "intranet",
+        authnotrequired => 0,
+        flagsrequired   => { catalogue => 1 },
     }
+);
 
-    my $CGIvirtualshelves=CGI::scrolling_list(
-                -name     => 'shelfnumber',
-                -values   => \@shelvesloop,
-                -labels   => \%shelvesloop,
-                -size     => 1,
-                -tabindex=>'',
-                -multiple => 0 );
+my @biblionumbers;
+if ($biblionumbers) {
+    @biblionumbers = split '/', $biblionumbers;
+} else {
+    @biblionumbers = (@biblionumber);
+}
+if (scalar(@biblionumber) == 1) {
+        @biblionumber = (split /\//,$biblionumber[0]);
+}
 
-    $template->param(
-                biblionumber => $biblionumber,
-                title => $biblios[0]->{'title'},
-                author => $biblios[0]->{'author'},
-                CGIvirtualshelves => $CGIvirtualshelves,
-                intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
-                intranetstylesheet => C4::Context->preference("intranetstylesheet"),
-                IntranetNav => C4::Context->preference("IntranetNav"),
-                );
-
-    output_html_with_http_headers $query, $cookie, $template->output;
+$shelfnumber = AddShelf( $newvirtualshelf, $loggedinuser, $category, $sortfield ) if $newvirtualshelf;
+if ( $shelfnumber || ( $shelfnumber == -1 ) ) {    # the shelf already exist.
+
+    if ($confirmed == 1) {
+       AddBibliosToShelf($shelfnumber,@biblionumber);
+       print
+    "Content-Type: text/html\n\n<html><body onload=\"window.opener.location.reload(true);window.close()\"></body></html>";
+       exit;
+    } else {
+       my ( $singleshelf, $singleshelfname, $singlecategory ) = GetShelf( $query->param('shelfnumber') );
+       my @biblios;
+        for my $bib (@biblionumber) {
+           my $data = GetBiblioData( $bib );
+            push(@biblios,
+                        { biblionumber => $bib,
+                          title        => $data->{'title'},
+                          author       => $data->{'author'},
+                        } );
+        }
+
+               $template->param
+        (
+         biblionumber => \@biblionumber,
+         biblios      => \@biblios,
+         multiple     => (scalar(@biblionumber) > 1),
+         singleshelf  => 1,
+         shelfname    => $singleshelfname,
+         shelfnumber  => $singleshelf,
+         total        => scalar(@biblionumber),
+         confirm      => 1,
+        );
+    }
 }
+else {    # this shelf doesn't already exist.
+    my $limit = 10;
+    my ($shelflist);
+    my @shelvesloop;
+    my %shelvesloop;
 
-# $Log$
-# Revision 1.8  2007/04/24 13:54:29  hdl
-# functions that were in C4::Interface::CGI::Output are now in C4::Output.
-# So this implies quite a change for files.
-# Sorry about conflicts which will be caused.
-# directory Interface::CGI should now be dropped.
-# I noticed that many scripts (reports ones, but also some circ/stats.pl or opac-topissues) still use Date::Manip.
-#
-# Revision 1.7  2007/04/04 16:46:22  tipaul
-# HUGE COMMIT : code cleaning circulation.
-#
-# some stuff to do, i'll write a mail on koha-devel NOW !
-#
-# Revision 1.6  2007/03/09 14:32:26  tipaul
-# rel_3_0 moved to HEAD
-#
-# Revision 1.4.2.6  2006/12/18 16:35:17  toins
-# removing use HTML::Template from *.pl.
-#
-# Revision 1.4.2.5  2006/12/05 11:35:29  toins
-# Biblio.pm cleaned.
-# additionalauthors, bibliosubject, bibliosubtitle tables are now unused.
-# Some functions renamed according to the coding guidelines.
-#
-# Revision 1.4.2.4  2006/11/30 18:23:51  toins
-# theses scripts don't need to use C4::Search.
-#
-# Revision 1.4.2.3  2006/10/30 09:48:19  tipaul
-# samll bugfix to create a virtualshelf correctly
-#
-# Revision 1.4.2.2  2006/08/30 16:13:54  toins
-# correct an error in the "if condition".
-#
-# Revision 1.4.2.1  2006/08/30 15:59:14  toins
-# Code cleaned according to coding guide lines.
-#
-# Revision 1.4  2006/07/04 14:36:51  toins
-# Head & rel_2_2 merged
-#
-# Revision 1.3.2.4  2006/06/20 16:21:42  oleonard
-# Adding "tabindex=''" to CGI:scrolling_lists to prevent incorrect tabbing. See Bug 1098
-#
-# Revision 1.3.2.3  2006/02/05 21:59:21  kados
-# Adds script support for IntranetNav ... see mail to koha-devel for
-# details
-#
-# Revision 1.3.2.2  2006/02/05 21:45:25  kados
-# Adds support for intranetstylesheet system pref in Koha scripts
-#
+    #grab each type of shelf, open (type 3) should not be limited by user.
+    foreach my $shelftype (1,2,3) {
+           my ($shelflist) = GetRecentShelves($shelftype, $limit, $shelftype == 3 ? undef : $loggedinuser);
+           for my $shelf (@{ $shelflist }) {
+                   push(@shelvesloop, $shelf->{shelfnumber});
+                   $shelvesloop{$shelf->{shelfnumber}} = $shelf->{shelfname};
+           }
+    }
 
-# Local Variables:
-# tab-width: 4
-# End:
+    if( @shelvesloop ){
+        my $CGIvirtualshelves = CGI::scrolling_list
+          (
+           -name     => 'shelfnumber',
+           -values   => \@shelvesloop,
+           -labels   => \%shelvesloop,
+           -size     => 1,
+           -tabindex => '',
+           -multiple => 0
+          );
+        $template->param
+          (
+           CGIvirtualshelves => $CGIvirtualshelves,
+          );
+    }
+       my @biblios;
+        for my $bib (@biblionumber) {
+           my $data = GetBiblioData( $bib );
+            push(@biblios,
+                        { biblionumber => $bib,
+                          title        => $data->{'title'},
+                          author       => $data->{'author'},
+                        } );
+        }
+    $template->param(
+           newshelf     => $newshelf,
+          biblios=>\@biblios,
+           multiple     => (scalar(@biblionumber) > 1),
+           total        => scalar(@biblionumber),
+    );
+
+    unless (@biblionumbers) {
+        my ( $bibliocount, @biblios ) = GetBiblio($biblionumber);
+    
+        $template->param
+          (
+           biblionumber      => $biblionumber,
+           title             => $biblios[0]->{'title'},
+           author            => $biblios[0]->{'author'},
+          );
+    } else {
+        my @biblioloop = ();
+        foreach my $biblionumber (@biblionumbers) {
+            my ( $bibliocount, @biblios ) = GetBiblio($biblionumber);
+            my %biblioiter = (
+                              title=>$biblios[0]->{'title'},
+                              author=>$biblios[0]->{'author'}
+                             );
+            push @biblioloop, \%biblioiter;
+        }
+        $template->param
+          (
+           biblioloop => \@biblioloop,
+           biblionumbers => $biblionumbers
+          );
+    }
+    
+}
+output_html_with_http_headers $query, $cookie, $template->output;