X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=virtualshelves%2Faddbybiblionumber.pl;h=1dbffbbc350eba9a884320f708ac6fab5e2a0eb3;hb=5e94de956610072d3d37bbbde1c8d48920414118;hp=d565493d62bddb9bb4d79b05dd45183bcf4eddbf;hpb=47690dbcc440612a18ca2b870b3045e308915f5c;p=koha_fer diff --git a/virtualshelves/addbybiblionumber.pl b/virtualshelves/addbybiblionumber.pl index d565493d62..1dbffbbc35 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,9 +67,19 @@ 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; @@ -72,12 +87,16 @@ my $query = new CGI; 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( { @@ -93,35 +112,61 @@ my @biblionumbers; if ($biblionumbers) { @biblionumbers = split '/', $biblionumbers; } else { - @biblionumbers = ($biblionumber); + @biblionumbers = (@biblionumber); +} +if (scalar(@biblionumber) == 1) { + @biblionumber = (split /\//,$biblionumber[0]); } -$shelfnumber = AddShelf( $newvirtualshelf, $loggedinuser, $category, $sortfield ) - if $newvirtualshelf; +$shelfnumber = AddShelf( $newvirtualshelf, $loggedinuser, $category, $sortfield ) if $newvirtualshelf; if ( $shelfnumber || ( $shelfnumber == -1 ) ) { # the shelf already exist. - foreach my $biblionumber (@biblionumbers) { - AddToShelfFromBiblio( $biblionumber, $shelfnumber ); + + 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, + ); } - print -"Content-Type: text/html\n\n"; - exit; } else { # this shelf doesn't already exist. - 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}; + + #grab each type of shelf, open (type 3) should not be limited by user. + foreach my $shelftype (1,2,3) { + my ($shelflist) = GetRecentShelves($shelftype, undef, $shelftype == 3 ? undef : $loggedinuser); + for my $shelf (@{ $shelflist }) { + push(@shelvesloop, $shelf->{shelfnumber}); + $shelvesloop{$shelf->{shelfnumber}} = $shelf->{shelfname}; + } } - if(@shelvesloop gt 0){ + + if( @shelvesloop ){ my $CGIvirtualshelves = CGI::scrolling_list ( -name => 'shelfnumber', @@ -136,8 +181,23 @@ else { # this shelf doesn't already exist. CGIvirtualshelves => $CGIvirtualshelves, ); } - - unless ($biblionumbers) { + 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 @@ -163,5 +223,5 @@ else { # this shelf doesn't already exist. ); } - output_html_with_http_headers $query, $cookie, $template->output; } +output_html_with_http_headers $query, $cookie, $template->output;