[Bug 13372] New: Items lost report improvements
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Bug ID: 13372 Summary: Items lost report improvements Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Reports Assignee: gmcharlt@gmail.com Reporter: jonathan.druart@biblibre.com QA Contact: testopia@bugs.koha-community.org Add a new column to display the call number on the items lost report. On the way, DataTables and ColVis will be added on the table. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|gmcharlt@gmail.com |jonathan.druart@biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 --- Comment #1 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 34062 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34062&action=edit Bug 13372: Items lost report improvements This enhancement adds new column to display the call number on the items lost report. On the way, DataTables and ColVis are added on the table. Test plan: 1/ Go on the items lost report (reports/itemslost.pl) and verify that the call number column is correctly filled. 2/ On the way, verify that the 'Library' and 'Current location' now contain the name instead of the code. 3/ Go on the columns configuration in the admin module (admin/columns_settings.pl), play with the ColVis plugin and confirm that all works correctly. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 PTFS Europe Sandboxes <sandboxes@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |sandboxes@ptfs-europe.com --- Comment #2 from PTFS Europe Sandboxes <sandboxes@ptfs-europe.com> --- Patch tested with a sandbox, by David Roberts <david.roberts@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 PTFS Europe Sandboxes <sandboxes@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34062|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 --- Comment #3 from PTFS Europe Sandboxes <sandboxes@ptfs-europe.com> --- Created attachment 34163 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34163&action=edit Bug 13372: Items lost report improvements This enhancement adds new column to display the call number on the items lost report. On the way, DataTables and ColVis are added on the table. Test plan: 1/ Go on the items lost report (reports/itemslost.pl) and verify that the call number column is correctly filled. 2/ On the way, verify that the 'Library' and 'Current location' now contain the name instead of the code. 3/ Go on the columns configuration in the admin module (admin/columns_settings.pl), play with the ColVis plugin and confirm that all works correctly. Signed-off-by: David Roberts <david.roberts@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |kyle@bywatersolutions.com --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Small problem. When you choose the order by field, it's not used in the datatable. The data is sorted correctly ( via perl/sql ), but it should be sorted by the datatable instead. Because of this, the table sort arrow is not set correctly and the first resort takes two clicks instead of one. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 --- Comment #5 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 34404 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34404&action=edit Bug 13372: Sort is done by DataTables This patch removes the order by filter on the items lost report. The sort can now be done by DataTables. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #6 from Jonathan Druart <jonathan.druart@biblibre.com> --- The last patch let DataTables sort the table. It makes more sense to me. Let me know if you disagree. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #34163|0 |1 is obsolete| | Attachment #34404|0 |1 is obsolete| | --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 34913 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34913&action=edit [PASSED QA] Bug 13372: Items lost report improvements This enhancement adds new column to display the call number on the items lost report. On the way, DataTables and ColVis are added on the table. Test plan: 1/ Go on the items lost report (reports/itemslost.pl) and verify that the call number column is correctly filled. 2/ On the way, verify that the 'Library' and 'Current location' now contain the name instead of the code. 3/ Go on the columns configuration in the admin module (admin/columns_settings.pl), play with the ColVis plugin and confirm that all works correctly. Signed-off-by: David Roberts <david.roberts@ptfs-europe.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 --- Comment #8 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 34914 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=34914&action=edit [PASSED QA] Bug 13372: Sort is done by DataTables This patch removes the order by filter on the items lost report. The sort can now be done by DataTables. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Alex Arnaud <alex.arnaud@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |alex.arnaud@biblibre.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |10212 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com Status|Passed QA |Pushed to Master --- Comment #9 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patches pushed to master. Thanks Jonathan! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Theodoros Theodoropoulos <theod@lib.auth.gr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |theod@lib.auth.gr -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13372 Theodoros Theodoropoulos <theod@lib.auth.gr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|theod@lib.auth.gr | -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org