Bug 6446

Summary: batch item deletion interface problems
Product: Koha Reporter: Christopher Chagal <chagal>
Component: ToolsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chagal, chris, magnus, nengard
Version: 3.4   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 5917    
Attachments: Proposed fix

Description Christopher Chagal 2011-06-01 21:42:19 UTC
I'm testing the latest pull from origin/3.4.x. The batch item deletion interface has some issues. One is just a display issue: the field data is shifted in the display so that "Title" is in the place where the check box should be and "Withdrawn status" is where "Title" should be, etc. In addition, all of the "Show/hide columns:" checkboxes control the "Title" field.
The more pressing issue is that the checkboxes that allow you to select items to be deleted do not show up.

I've tried it in Firefox 3.6 & Firefox 4.0 & IE 8 & Chrome, on Linux (Ubuntu 10.4) & Windows (XP). Same problem in all environments.
Comment 1 Owen Leonard 2011-06-03 12:18:26 UTC
Created attachment 4353 [details] [review]
Proposed fix

Scoping error in T:T variable caused incorrect column display
in batch delete and batch edit interfaces.

After applying this patch header cells should align correctly
with content and checkbox column should appear.
Comment 2 Christopher Chagal 2011-06-03 14:51:23 UTC
That did it.  Thanks, Owen.
Comment 3 Owen Leonard 2011-06-03 14:52:41 UTC
Bugs should not be marked as fixed until the patch passes QA and is pushed.
Comment 4 Christopher Chagal 2011-06-03 14:55:17 UTC
Oops. Sorry about that.
Comment 5 Chris Cormack 2011-06-05 06:51:03 UTC
Simple TT fix, pushed
Comment 6 Magnus Enger 2011-07-16 08:38:59 UTC
Typed a few barcodes into the "Barcode list (one barcode per line)" field and clicked on "Continue". The table looks ok now.