Browse Source

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>
3.10.x
Marcel de Rooy 12 years ago
committed by Paul Poulain
parent
commit
3945ecf9c8
  1. 3
      opac/opac-addbybiblionumber.pl
  2. 3
      virtualshelves/addbybiblionumber.pl

3
opac/opac-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
);
}
}

3
virtualshelves/addbybiblionumber.pl

@ -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 {

Loading…
Cancel
Save