X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=virtualshelves%2Faddbybiblionumber.pl;h=7af17c8ef36efe74bb5e4e181a4273153fe42afc;hb=dfb2c774353ad609cb112f317feb67d74d30230d;hp=1ad57753d08313ce8df83ec4728f013bece21420;hpb=81c0d9e821bf619a6058d37497023f53555367b8;p=koha.git diff --git a/virtualshelves/addbybiblionumber.pl b/virtualshelves/addbybiblionumber.pl index 1ad57753d0..7af17c8ef3 100755 --- a/virtualshelves/addbybiblionumber.pl +++ b/virtualshelves/addbybiblionumber.pl @@ -16,11 +16,10 @@ # 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 @@ -51,27 +50,53 @@ 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::VirtualShelves; +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){ + AddToShelfFromBiblio($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 $confirmed = $query->param('confirmed') || 0; + my ( $template, $loggedinuser, $cookie ) = get_template_and_user( { @@ -83,98 +108,120 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( } ); -$shelfnumber = AddShelf( $newvirtualshelf, $loggedinuser, $category ) - 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 ($shelflist) = GetShelves( $loggedinuser, 3 ); + my $limit = 10; + my ($shelflist); my @shelvesloop; my %shelvesloop; - foreach my $element ( sort keys %$shelflist ) { - push( @shelvesloop, $element ); - $shelvesloop{$element} = $shelflist->{$element}->{'shelfname'}; - } - my $CGIvirtualshelves = CGI::scrolling_list( - -name => 'shelfnumber', - -values => \@shelvesloop, - -labels => \%shelvesloop, - -size => 1, - -tabindex => '', - -multiple => 0 - ); + #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'}, - CGIvirtualshelves => $CGIvirtualshelves, - intranetcolorstylesheet => - C4::Context->preference("intranetcolorstylesheet"), - intranetstylesheet => C4::Context->preference("intranetstylesheet"), - IntranetNav => C4::Context->preference("IntranetNav"), + 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 + ); + } + } - -# $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 -# - -# Local Variables: -# tab-width: 4 -# End: +output_html_with_http_headers $query, $cookie, $template->output;