Bug 19669

Summary: Remove deprecated checkouts by patron category report
Product: Koha Reporter: Owen Leonard <oleonard>
Component: ReportsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Kyle M Hall <kyle>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, kyle, lucas, nick, victor
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3431
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20092
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19669: Remove deprecated checkouts by patron category report
Bug 19669: Remove deprecated checkouts by patron category report
Bug 19669: Remove deprecated checkouts by patron category report
Bug 19669 [QA Followup] - Remove itemstypes.plugin

Description Owen Leonard 2017-11-21 13:49:57 UTC
Koha has a report which isn't linked to from anywhere. A comment on Bug 840 says that the report is deprecated. I propose that thirteen years is enough of a wait and we should probably go ahead and remove it.
Comment 1 Owen Leonard 2017-11-21 13:57:58 UTC
Created attachment 69246 [details] [review]
Bug 19669: Remove deprecated checkouts by patron category report

The checkouts by patron category report has been hidden and considered
deprecated since 2004. This patch removes it.

To test, apply the patch and confirm that nothing at all seems to have
changed and everything feels fine.
Comment 2 Dilan Johnpullé 2017-11-28 00:10:35 UTC
Created attachment 69406 [details] [review]
Bug 19669: Remove deprecated checkouts by patron category report

Patch applies. Reports and general OPAC and staff client functionality is intact.
Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>
Comment 3 Owen Leonard 2017-11-29 12:54:01 UTC
*** Bug 19668 has been marked as a duplicate of this bug. ***
Comment 4 Owen Leonard 2017-11-29 12:54:04 UTC
*** Bug 19667 has been marked as a duplicate of this bug. ***
Comment 5 Owen Leonard 2017-11-29 12:54:07 UTC
*** Bug 19666 has been marked as a duplicate of this bug. ***
Comment 6 Owen Leonard 2017-11-29 12:54:11 UTC
*** Bug 19665 has been marked as a duplicate of this bug. ***
Comment 7 Jonathan Druart 2017-12-19 15:44:54 UTC
What about the other .plugin file (reports/itemtypes.plugin), see bug 3431 comment 2.
And then remove reports/manager.pl?
Is this plugin system used?
Comment 8 Kyle M Hall 2017-12-21 14:17:21 UTC
Created attachment 70029 [details] [review]
Bug 19669: Remove deprecated checkouts by patron category report

Patch applies. Reports and general OPAC and staff client functionality is intact.
Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 9 Kyle M Hall 2017-12-21 14:17:28 UTC
Created attachment 70030 [details] [review]
Bug 19669 [QA Followup] - Remove itemstypes.plugin

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 10 Jonathan Druart 2017-12-21 16:08:58 UTC
(In reply to Kyle M Hall from comment #9)
> Created attachment 70030 [details] [review] [review]
> Bug 19669 [QA Followup] - Remove itemstypes.plugin
> 
> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Kyle, Are you sure about that?
If yes we should remove it on a specific bug report, as well as the reports/manager.pl stuffs.
Comment 11 Jonathan Druart 2017-12-21 16:24:15 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 12 Nick Clemens 2018-01-15 13:14:23 UTC
Awesome work all! Pushed to stable for 17.11.02
Comment 13 Fridolin Somers 2018-01-15 14:46:34 UTC
Pushed to 17.05.x for v17.05.08
Comment 14 Victor Grousset/tuxayo 2018-01-26 09:26:50 UTC
Link for reports/itemtypes.plugin is stil in
staff:/cgi-bin/koha/reports/reports-home.pl
with the name "Catalog by itemtype"

New bug for this or do we use the current one?
Comment 15 Fridolin Somers 2018-01-26 11:32:07 UTC
(In reply to Victor Grousset/tuxayo from comment #14)
> Link for reports/itemtypes.plugin is stil in
> staff:/cgi-bin/koha/reports/reports-home.pl
> with the name "Catalog by itemtype"
> 
> New bug for this or do we use the current one?

I'd say new bug since this one has been released.
Comment 16 Jonathan Druart 2018-01-30 15:27:07 UTC
(In reply to Jonathan Druart from comment #10)
> (In reply to Kyle M Hall from comment #9)
> > Created attachment 70030 [details] [review] [review] [review]
> > Bug 19669 [QA Followup] - Remove itemstypes.plugin
> > 
> > Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
> 
> Kyle, Are you sure about that?
> If yes we should remove it on a specific bug report, as well as the
> reports/manager.pl stuffs.

Oops, I forgot to remove this commit before the push, I am going to revert it.
Comment 17 Nick Clemens 2018-02-02 17:03:49 UTC
(In reply to Jonathan Druart from comment #16)
> (In reply to Jonathan Druart from comment #10)
> > (In reply to Kyle M Hall from comment #9)
> > > Created attachment 70030 [details] [review] [review] [review] [review]
> > > Bug 19669 [QA Followup] - Remove itemstypes.plugin
> > > 
> > > Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
> > 
> > Kyle, Are you sure about that?
> > If yes we should remove it on a specific bug report, as well as the
> > reports/manager.pl stuffs.
> 
> Oops, I forgot to remove this commit before the push, I am going to revert
> it.

Reverted last patch for stable for 17.11.03
Comment 18 Fridolin Somers 2018-02-05 10:25:56 UTC
(In reply to Nick Clemens from comment #17)
> 
> Reverted last patch for stable for 17.11.03

Same for 17.05 then, reverted in v17.05.09
Comment 19 Victor Grousset/tuxayo 2018-02-07 08:23:22 UTC
Just for anyone else who might have also a doubt: it's indeed reverted in master (commit 7cb86656956131ed81cc74069a765a37664edd8e)
Comment 20 Nick Clemens 2018-03-12 14:59:35 UTC
*** Bug 20368 has been marked as a duplicate of this bug. ***