Navigation Menu

Skip to content

Commit

Permalink
removed AddToShelfFromBiblio from C4::VirtualShelves
Browse files Browse the repository at this point in the history
Back in the day, lists could contain item records instead or in addition
to bibs, but that hasn't been the case for a while.  Therefore, removed
AddToShelfFromBiblio, which does exactly the same thing that AddToShelf does.

Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
  • Loading branch information
gmcharlt committed Oct 6, 2010
1 parent 58e2f6a commit 552a648
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 45 deletions.
50 changes: 7 additions & 43 deletions C4/VirtualShelves.pm
Expand Up @@ -40,7 +40,7 @@ BEGIN {
@ISA = qw(Exporter);
@EXPORT = qw(
&GetShelves &GetShelfContents &GetShelf
&AddToShelf &AddToShelfFromBiblio &AddShelf
&AddToShelf &AddShelf
&ModShelf
&ShelfPossibleAction
&DelFromShelf &DelShelf
Expand All @@ -67,7 +67,7 @@ C4::VirtualShelves - Functions for manipulating Koha virtual virtualshelves
This module provides functions for manipulating virtual virtualshelves,
including creating and deleting virtualshelves, and adding and removing
items to and from virtualshelves.
bibs to and from virtualshelves.
=head1 FUNCTIONS
Expand Down Expand Up @@ -237,7 +237,7 @@ sub GetShelf ($) {

=head2 GetShelfContents
$itemlist = &GetShelfContents($shelfnumber);
$biblist = &GetShelfContents($shelfnumber);
Looks up information about the contents of virtual virtualshelves number
C<$shelfnumber>. Sorted by a field in the biblio table. copyrightdate
Expand Down Expand Up @@ -331,8 +331,8 @@ sub AddShelf {
&AddToShelf($biblionumber, $shelfnumber);
Adds item number C<$biblionumber> to virtual virtualshelves number
C<$shelfnumber>, unless that item is already on that shelf.
Adds bib number C<$biblionumber> to virtual virtualshelves number
C<$shelfnumber>, unless that bib is already on that shelf.
=cut

Expand Down Expand Up @@ -364,42 +364,6 @@ sub AddToShelf {
$sth->execute( $shelfnumber );
}

=head2 AddToShelfFromBiblio
&AddToShelfFromBiblio($biblionumber, $shelfnumber)
this function allow to add a virtual into the shelf number $shelfnumber
from biblionumber.
=cut

sub AddToShelfFromBiblio {
my ( $biblionumber, $shelfnumber ) = @_;
return unless $biblionumber;
my $query = qq(
SELECT *
FROM virtualshelfcontents
WHERE shelfnumber=? AND biblionumber=?
);
my $sth = $dbh->prepare($query);
$sth->execute( $shelfnumber, $biblionumber );
unless ( $sth->rows ) {
my $query =qq(
INSERT INTO virtualshelfcontents
(shelfnumber, biblionumber, flags)
VALUES
(?, ?, 0)
);
$sth = $dbh->prepare($query);
$sth->execute( $shelfnumber, $biblionumber );
$query = qq(UPDATE virtualshelves
SET lastmodified = CURRENT_TIMESTAMP
WHERE shelfnumber = ?);
$sth = $dbh->prepare($query);
$sth->execute( $shelfnumber );
}
}

=head2 ModShelf
ModShelf($shelfnumber, $hashref)
Expand Down Expand Up @@ -486,8 +450,8 @@ sub ShelfPossibleAction {
&DelFromShelf( $biblionumber, $shelfnumber);
Removes item number C<$biblionumber> from virtual virtualshelves number
C<$shelfnumber>. If the item wasn't on that virtualshelves to begin with,
Removes bib number C<$biblionumber> from virtual virtualshelves number
C<$shelfnumber>. If the bib wasn't on that virtualshelves to begin with,
nothing happens.
=cut
Expand Down
2 changes: 1 addition & 1 deletion opac/opac-addbybiblionumber.pl
Expand Up @@ -42,7 +42,7 @@ sub AddBibliosToShelf {
@biblionumber = (split /\//,$biblionumber[0]);
}
for my $bib (@biblionumber){
AddToShelfFromBiblio($bib, $shelfnumber);
AddToShelf($bib, $shelfnumber);
}
}

Expand Down
2 changes: 1 addition & 1 deletion virtualshelves/addbybiblionumber.pl
Expand Up @@ -77,7 +77,7 @@ sub AddBibliosToShelf {
@biblionumber = (split /\//,$biblionumber[0]);
}
for my $bib (@biblionumber){
AddToShelfFromBiblio($bib, $shelfnumber);
AddToShelf($bib, $shelfnumber);
}
}

Expand Down

0 comments on commit 552a648

Please sign in to comment.