X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=virtualshelves%2Faddbybiblionumber.pl;h=a0d9fdd487ae5c042f70e367184c0060bc7b36e2;hb=03890c90ac41f66b2de04d0280e2e96a0d2e8be8;hp=b19e7da4f8a10870d5ecda65dec095580c123724;hpb=877da483e52db4fcc6eb44e0c5d1ffd163ae04a8;p=koha.git diff --git a/virtualshelves/addbybiblionumber.pl b/virtualshelves/addbybiblionumber.pl index b19e7da4f8..a0d9fdd487 100755 --- a/virtualshelves/addbybiblionumber.pl +++ b/virtualshelves/addbybiblionumber.pl @@ -16,9 +16,9 @@ # 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. =head1 NAME @@ -50,11 +50,16 @@ 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; @@ -62,16 +67,36 @@ use C4::VirtualShelves qw/:DEFAULT GetRecentShelves/; use C4::Circulation; use C4::Auth; -#use it only to debug ! -use CGI::Carp qw/fatalsToBrowser/; -use warnings; +# splits incoming biblionumber(s) to array and adds each to shelf. +sub AddBibliosToShelf { + my ($shelfnumber,@biblionumber)=@_; + + # 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; + +# 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 $newshelf = $query->param('newshelf'); my $category = $query->param('category'); -my $sortfield = $query->param('sortfield'); +my $sortfield = $query->param('sortfield'); +my $confirmed = $query->param('confirmed') || 0; + my ( $template, $loggedinuser, $cookie ) = get_template_and_user( { @@ -83,50 +108,120 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( } ); -$shelfnumber = AddShelf( $newvirtualshelf, $loggedinuser, $category, $sortfield ) - if $newvirtualshelf; +my @biblionumbers; +if ($biblionumbers) { + @biblionumbers = split '/', $biblionumbers; +} else { + @biblionumbers = (@biblionumber); +} +if (scalar(@biblionumber) == 1) { + @biblionumber = (split /\//,$biblionumber[0]); +} + +$shelfnumber = AddShelf( $newvirtualshelf, $loggedinuser, $category, $sortfield ) if $newvirtualshelf; if ( $shelfnumber || ( $shelfnumber == -1 ) ) { # the shelf already exist. - AddToShelfFromBiblio( $biblionumber, $shelfnumber ); - print -"Content-Type: text/html\n\n"; - exit; + + if ($confirmed == 1) { + AddBibliosToShelf($shelfnumber,@biblionumber); + print + "Content-Type: text/html\n\n"; + 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 ( $bibliocount, @biblios ) = GetBiblio($biblionumber); - - my $limit = 10; - my ($shelflist) = GetRecentShelves(1, $limit, $loggedinuser); + my $limit = 10; + my ($shelflist); my @shelvesloop; my %shelvesloop; - for my $shelf ( @{ $shelflist->[0] } ) { - push( @shelvesloop, $shelf->{shelfnumber} ); - $shelvesloop{$shelf->{shelfnumber}} = $shelf->{shelfname}; - } - # then open shelves... - my ($shelflist) = GetRecentShelves(3, $limit, undef); - for my $shelf ( @{ $shelflist->[0] } ) { - push( @shelvesloop, $shelf->{shelfnumber} ); - $shelvesloop{$shelf->{shelfnumber}} = $shelf->{shelfname}; - } - if(@shelvesloop gt 0){ - my $CGIvirtualshelves = CGI::scrolling_list( - -name => 'shelfnumber', - -values => \@shelvesloop, - -labels => \%shelvesloop, - -size => 1, - -tabindex => '', - -multiple => 0 - ); - $template->param( - CGIvirtualshelves => $CGIvirtualshelves, - ); - } + #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->[0] }) { + push(@shelvesloop, $shelf->{shelfnumber}); + $shelvesloop{$shelf->{shelfnumber}} = $shelf->{shelfname}; + } + } + + if(@shelvesloop gt 0){ + 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( - biblionumber => $biblionumber, - title => $biblios[0]->{'title'}, - author => $biblios[0]->{'author'}, + newshelf => $newshelf, + biblios=>\@biblios, + multiple => (scalar(@biblionumber) > 1), + total => scalar(@biblionumber), ); - output_html_with_http_headers $query, $cookie, $template->output; + 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;