Bug 5210 - Function of back button on batch modification result page unclear
Summary: Function of back button on batch modification result page unclear
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: rel_3_2
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Owen Leonard
QA Contact: Bugs List
URL: /cgi-bin/koha/tools/batchMod.pl
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-09 13:22 UTC by Katrin Fischer
Modified: 2010-11-23 10:30 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Proposed fix (1.23 KB, patch)
2010-09-09 14:10 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2010-09-09 13:22:58 UTC
Steps:

- Tools > Batch item modification
- Scan barcodes or use file
- Make a change, for example change the holding library
- Result page with item table

At the bottom is a back button.

If you click on the button you can see the old table without the changes, but the changes are not undone.

You can't make another change to the result list. Nothing happens when you click on save.

Cancel there will bring you back to the first screen with your scanned in barcodes / barcode file. 

There is no way to get back to the result list after using the back button.

What is the function of it?
Comment 1 Owen Leonard 2010-09-09 14:10:03 UTC
Created attachment 2610 [details] [review]
Proposed fix

I can't see any good use for a back button here, so I'm taking it out.
    
An improvement might be to build in a method to resubmit the same batch of barcodes/item numbers again so that you could make additional changes to the same set.
Comment 2 Katrin Fischer 2010-09-10 06:58:45 UTC
Tested against HEAD, signed-off and resent.
Comment 3 Katrin Fischer 2010-11-12 20:58:26 UTC
Tested again after rebase, signed-off and resent.
Comment 4 Katrin Fischer 2010-11-14 22:37:50 UTC
Tested with HEAD, back button is gone.
Comment 5 Chris Nighswonger 2010-11-18 03:15:10 UTC
This bug has a patch presently committed to HEAD and 3.2.x.

Please take the appropriate action to update the status of this bug.

Remember, bugs which have been resolved should be tested and marked closed, preferably by the original reporter.
Comment 6 Katrin Fischer 2010-11-23 10:30:16 UTC
Successfully tested in 3.2 too.