Bug 14365

Summary: SQL Reports Last Edit Date column
Product: Koha Reporter: Cab Vinton <director>
Component: ReportsAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: cbrannon, jonathan.druart, katrin.fischer, kyle
Version: Main   
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: 18985    
Attachments: Bug 14365 - Populate never used saved_sql column last_run when execute_query is called
Bug 14365 - SQL Reports Last Edit Date column - Show in interface
Bug 14365 - Populate never used saved_sql column last_run when execute_query is called
Bug 14365 - SQL Reports Last Edit Date column - Show in interface
Bug 14365 - Change 'Last modified' to 'Last edit' for terseness
[SIGNED-OFF] Bug 14365 - Populate never used saved_sql column last_run when execute_query is called
[SIGNED-OFF] Bug 14365 - SQL Reports Last Edit Date column - Show in interface
[SIGNED-OFF] Bug 14365 - Change 'Last modified' to 'Last edit' for terseness
Bug 14365 - Populate never used saved_sql column last_run when execute_query is called
Bug 14365 - SQL Reports Last Edit Date column - Show in interface
Bug 14365 - Change 'Last modified' to 'Last edit' for terseness

Description Cab Vinton 2015-06-08 21:02:15 UTC
The Saved Reports page currently shows the Creation Date of Reports.

To my way of thinking, it would be much more helpful to have the Last Edit Date displayed & sortable.

When I'm working with Reports I am almost always working on the same few Reports within a relatively short time period & it would be great to have an easy way of retrieving these.

Along the same lines, this would also provide an easy way of identifying which Reports other librarians have been working on most recently.

Cheers!

Cab Vinton
Plaistow Public Library
Comment 1 Kyle M Hall 2017-02-07 15:58:59 UTC
Created attachment 59985 [details] [review]
Bug 14365 - Populate never used saved_sql column last_run when execute_query is called
Comment 2 Kyle M Hall 2017-02-07 15:59:07 UTC
Created attachment 59986 [details] [review]
Bug 14365 - SQL Reports Last Edit Date column - Show in interface
Comment 3 Kyle M Hall 2017-02-07 16:09:47 UTC
Created attachment 59988 [details] [review]
Bug 14365 - Populate never used saved_sql column last_run when execute_query is called
Comment 4 Kyle M Hall 2017-02-07 16:09:54 UTC
Created attachment 59989 [details] [review]
Bug 14365 - SQL Reports Last Edit Date column - Show in interface

Test Plan:
1) Apply these patches
2) Modify a saved sql report
3) Note the "Last modified" column is updated
4) Run the report
5) Note the "Last run" column is updated
Comment 5 Kyle M Hall 2017-03-01 11:26:21 UTC
Created attachment 60752 [details] [review]
Bug 14365 - Change 'Last modified' to 'Last edit' for terseness
Comment 6 Kyle M Hall 2017-03-14 13:04:18 UTC
Created attachment 61060 [details] [review]
[SIGNED-OFF] Bug 14365 - Populate never used saved_sql column last_run when execute_query is called

Signed-off-by: Cab Vinton <director@plaistowlibrary.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 7 Kyle M Hall 2017-03-14 13:04:25 UTC
Created attachment 61061 [details] [review]
[SIGNED-OFF] Bug 14365 - SQL Reports Last Edit Date column - Show in interface

Test Plan:
1) Apply these patches
2) Modify a saved sql report
3) Note the "Last modified" column is updated
4) Run the report
5) Note the "Last run" column is updated

Signed-off-by: Cab Vinton <director@plaistowlibrary.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 8 Kyle M Hall 2017-03-14 13:04:28 UTC
Created attachment 61062 [details] [review]
[SIGNED-OFF] Bug 14365 - Change 'Last modified' to 'Last edit' for terseness

Signed-off-by: Cab Vinton <director@plaistowlibrary.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 9 Jonathan Druart 2017-03-17 15:17:54 UTC
Not blocker but it would have been good to update the prototype of the subroutine to pass a hashref instead of the list of undef...
This new parameter is a bit awkward: if the id is passed, a DB field is updated. Not elegant but given the quality of the existing code it's hard to provide something better...

Passing QA
Comment 10 Jonathan Druart 2017-03-17 15:19:28 UTC
Created attachment 61220 [details] [review]
Bug 14365 - Populate never used saved_sql column last_run when execute_query is called

Signed-off-by: Cab Vinton <director@plaistowlibrary.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 11 Jonathan Druart 2017-03-17 15:19:33 UTC
Created attachment 61221 [details] [review]
Bug 14365 - SQL Reports Last Edit Date column - Show in interface

Test Plan:
1) Apply these patches
2) Modify a saved sql report
3) Note the "Last modified" column is updated
4) Run the report
5) Note the "Last run" column is updated

Signed-off-by: Cab Vinton <director@plaistowlibrary.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 12 Jonathan Druart 2017-03-17 15:19:38 UTC
Created attachment 61222 [details] [review]
Bug 14365 - Change 'Last modified' to 'Last edit' for terseness

Signed-off-by: Cab Vinton <director@plaistowlibrary.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 13 Kyle M Hall 2017-03-31 14:35:19 UTC
Pushed to master for 17.05!
Comment 14 Katrin Fischer 2017-04-02 17:43:35 UTC
This won't get ported back to 16.11.x as it is an enhancement.
Comment 15 Nick Clemens 2017-07-26 02:28:52 UTC
*** Bug 13047 has been marked as a duplicate of this bug. ***