Bug 8109 - DataTables in Acquisision module: basket.tt
Summary: DataTables in Acquisision module: basket.tt
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: 3.10
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jonathan Druart
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks: 5345
  Show dependency treegraph
 
Reported: 2012-05-16 13:30 UTC by Jonathan Druart
Modified: 2013-12-05 20:01 UTC (History)
2 users (show)

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


Attachments
basket.tt (2.87 KB, patch)
2012-05-16 13:31 UTC, Jonathan Druart
Details | Diff | Splinter Review
[PASSED_QA] basket.tt (2.60 KB, patch)
2012-05-24 12:14 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 5345: Followup DataTables integration in acquisition module [4] (1.42 KB, patch)
2012-05-24 12:22 UTC, Jonathan Druart
Details | Diff | Splinter Review
[PASSED_QA] Followup (1.52 KB, patch)
2012-05-24 12:27 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2012-05-16 13:30:16 UTC
DataTables in Acquisision module: basket.tt
Comment 1 Jonathan Druart 2012-05-16 13:31:29 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2012-05-16 13:31:58 UTC
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Columns for 'Modify' and 'Delete' are sortable, which doesn't hurt but could be removed.
Comment 3 Marcel de Rooy 2012-05-24 12:04:32 UTC
Please clarify why I find the function confirm_reopen twice in basket.tt ?
Changing status to reflect need for clarification.
Comment 4 Jonathan Druart 2012-05-24 12:14:49 UTC
Created attachment 9756 [details] [review]
[PASSED_QA] basket.tt

hum... I don't know :)
This patch remove these lines.
Comment 5 Jonathan Druart 2012-05-24 12:22:02 UTC Comment hidden (obsolete)
Comment 6 Marcel de Rooy 2012-05-24 12:23:30 UTC
Thx for your quick correction.
Passed QA
Comment 7 Marcel de Rooy 2012-05-24 12:27:27 UTC
Created attachment 9758 [details] [review]
[PASSED_QA] Followup