To test:
- Find/create a bib with at least one item
- From detail.pl, use the checkboxes in the holdings table to select one or more items
- click the "Modify Selected Items" link just above the holdings table to take your item(s) to batch modification
- Change something and click save
- Observe that the Return to the Record link points to a url without a biblionumber: "/cgi-bin/koha/catalogue/detail.pl?biblionumber="
- Apply patch
- Repeat
- The link now works!
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: David Nind <david@davidnind.com>
Bug 24423: Fix for deletion - cleanup accidental debug inclusiopm
To test:
Repeat prior test plan with 'Delete selected'
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
if (itemnumbers.length > 0) {
var url = '/cgi-bin/koha/tools/batchMod.pl?op=show&del=1';
url += '&itemnumber=' + itemnumbers.join('&itemnumber=');
+ url += '&biblionumber=[% biblionumber | uri %]';
url += '&src=CATALOGUING';
$('a.itemselection_action_delete').attr('href', url);
} else {
if (itemnumbers.length > 0) {
var url = '/cgi-bin/koha/tools/batchMod.pl?op=show';
url += '&itemnumber=' + itemnumbers.join('&itemnumber=');
+ url += '&biblionumber=[% biblionumber | uri %]';
url += '&src=CATALOGUING';
$('a.itemselection_action_modify').attr('href', url);
} else {
@notfounditemnumbers = grep { !exists $itemdata{$_} } @contentlist;
}
} else {
- if (defined $biblionumber){
+ if (defined $biblionumber && !@itemnumbers){
my @all_items = GetItemsInfo( $biblionumber );
foreach my $itm (@all_items) {
push @itemnumbers, $itm->{itemnumber};