gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 22478: (QA follow-up) Handle category in opac-shelves like a boolean 

The category parameter should be restricted to 1 or 2 (private/public). In
order to keep same behavior, no parameter means 1 (private).
Note: Adding the same line in intranet counterpart.

Test plan
[1] Check for category empty, '1a', '11' etc. And with script ;)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 3d06afa4cd)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit fd18249965)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Marcel de Rooy 2019-04-12 07:02:06 +00:00 committed by Fridolin Somers
parent e3f3f18b83
commit 524ba89732
2 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
opac/opac-shelves.pl
View File

@ -59,6 +59,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user({
my $op = $query->param('op') || 'list';
my $referer = $query->param('referer') || $op;
my $category = $query->param('category') || 1;
$category = 2 if $category ne "1";
my ( $shelf, $shelfnumber, @messages );
if ( $op eq 'add_form' ) {

1
virtualshelves/shelves.pl
View File

@ -50,6 +50,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
my $op = $query->param('op') || 'list';
my $referer = $query->param('referer') || $op;
my $category = $query->param('category') || 1;
$category = 2 if $category ne "1";
my ( $shelf, $shelfnumber, @messages );
if ( $op eq 'add_form' ) {