Permalink
Browse files

Bug 7788: [SIGNED-OFF] Followup: GetShelf call in addbybiblio script …

…corrected

Patch resolves incorrect call to GetShelf in opac and staff. It was and is harmless, because the singlecategory variable was not really used (the category$x param was ignored in the template). Just a cleanup operation.

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
  • Loading branch information...
1 parent 644c812 commit f9c8822766b380b00425882dafa29f17a320d82c @mderooy mderooy committed with ranginui Mar 22, 2012
Showing with 2 additions and 4 deletions.
  1. +1 −2 opac/opac-addbybiblionumber.pl
  2. +1 −2 virtualshelves/addbybiblionumber.pl
@@ -115,12 +115,11 @@ sub HandleShelfNumber {
sub HandleSelectedShelf {
if($authorized= ShelfPossibleAction( $loggedinuser, $selectedshelf, 'add')){
#adding to specific shelf
- my ($singleshelf, $singleshelfname, $singlecategory)= GetShelf($query->param('selectedshelf'));
+ my ($singleshelf, $singleshelfname)= GetShelf($query->param('selectedshelf'));
$template->param(
singleshelf => 1,
shelfnumber => $singleshelf,
shelfname => $singleshelfname,
- "category$singlecategory" => 1
);
}
}
@@ -161,12 +161,11 @@ sub HandleShelfNumber {
sub HandleSelectedShelf {
if($authorized= ShelfPossibleAction( $loggedinuser, $shelfnumber, 'add')){
#confirm adding to specific shelf
- my ($singleshelf, $singleshelfname, $singlecategory)= GetShelf($shelfnumber);
+ my ($singleshelf, $singleshelfname)= GetShelf($shelfnumber);
$template->param(
singleshelf => 1,
shelfnumber => $singleshelf,
shelfname => $singleshelfname,
- "category$singlecategory" => 1
);
}
else {

0 comments on commit f9c8822

Please sign in to comment.