Squashed commit of the following:
commit
105de81639cbac5084e4a5c099b19569043e69ff
Author: Nahuel ANGELINETTI <nahuel.angelinetti@biblibre.com>
Date: Tue Jul 13 11:58:01 2010 +0200
(bug #4931) fix forgottens input in buttons
the previous patch missed an hidden input in next buttons that break next page. This fix it.
commit
db00295a6b9d1d36fc888ba6a0558011fd6884ba
Author: Nahuel ANGELINETTI <nahuel.angelinetti@biblibre.com>
Date: Fri Jul 2 15:27:59 2010 +0200
(bug #4931) add the ability to choose home or holding branch in stocktaking
This add radio box in stocktaking to base it on home or holdingbranch
Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
Bug 4391 Followup: Adding back lost declaration of $branchcode
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
=cut
sub GetItemsForInventory {
- my ( $minlocation, $maxlocation,$location, $itemtype, $ignoreissued, $datelastseen, $branch, $offset, $size, $statushash ) = @_;
+ my ( $minlocation, $maxlocation,$location, $itemtype, $ignoreissued, $datelastseen, $branchcode, $branch, $offset, $size, $statushash ) = @_;
my $dbh = C4::Context->dbh;
my ( @bind_params, @where_strings );
push @where_strings, 'items.location = ?';
push @bind_params, $location;
}
-
- if ( $branch ) {
+
+ if ( $branchcode ) {
+ if($branch eq "homebranch"){
push @where_strings, 'items.homebranch = ?';
- push @bind_params, $branch;
+ }else{
+ push @where_strings, 'items.holdingbranch = ?';
+ }
+ push @bind_params, $branchcode;
}
if ( $itemtype ) {
<fieldset class="rows">
<legend>Select items you want to check</legend>
<ol><li>
+ <label for="branch">Branch</label>
+ <input type="radio" name="branch" value="homebranch">Home Branch</input>
+ <input type="radio" name="branch" value="holdingbranch">Holding Branch</input>
+ </li><li>
<label for="branchloop">Library</label><select id="branchloop" name="branchcode" style="width:12em;">
<option value="">All Locations</option>
<!-- TMPL_LOOP NAME="branchloop" -->
</form>
<!-- TMPL_IF NAME="offset" -->
<form method="post" action="/cgi-bin/koha/tools/inventory.pl">
+ <input type="hidden" name="branch" value="<!-- TMPL_VAR NAME="branch" -->" />
<input type="hidden" name="minlocation" value="<!-- TMPL_VAR NAME="minlocation" -->" />
<input type="hidden" name="maxlocation" value="<!-- TMPL_VAR NAME="maxlocation" -->" />
<input type="hidden" name="location" value="<!-- TMPL_VAR NAME="location" -->" />
<!-- /TMPL_IF -->
<!-- TMPL_IF NAME="nextoffset" -->
<form method="post">
+ <input type="hidden" name="branch" value="<!-- TMPL_VAR NAME="branch" -->" />
<input type="hidden" name="location" value="<!-- TMPL_VAR NAME="location" -->" />
<input type="hidden" name="branchcode" value="<!-- TMPL_VAR NAME="branchcode" -->" />
<input type="hidden" name="minlocation" value="<!-- TMPL_VAR NAME="minlocation" -->" />
my $pagesize = $input->param('pagesize');
$pagesize=50 unless $pagesize;
my $branchcode = $input->param('branchcode');
+my $branch = $input->param('branch');
my $op = $input->param('op');
my $res; #contains the results loop
# warn "uploadbarcodes : ".$uploadbarcodes;
location=>$location,
ignoreissued=>$ignoreissued,
branchcode=>$branchcode,
+ branch => $branch,
offset => $offset,
pagesize => $pagesize,
datelastseen => $datelastseen,
}
}
if ($markseen or $op) {
- $res = GetItemsForInventory($minlocation,$maxlocation,$location, $ignoreissued,$itemtype,$datelastseen,$branchcode,$offset,$pagesize,$staton);
+ $res = GetItemsForInventory( $minlocation, $maxlocation, $location, $ignoreissued, $itemtype, $datelastseen, $branchcode, $branch, $offset, $pagesize, $staton );
$template->param(loop =>$res,
nextoffset => ($offset+$pagesize),
prevoffset => ($offset?$offset-$pagesize:0),