Bug 18643 - Remove dead code in reports/statistics 'Till reconciliation'
Summary: Remove dead code in reports/statistics 'Till reconciliation'
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Marc Véron
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks: 20510 20603
  Show dependency treegraph
 
Reported: 2017-05-20 09:41 UTC by Marc Véron
Modified: 2018-12-03 20:04 UTC (History)
2 users (show)

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


Attachments
Bug 18643: Remove dead code in reports/statistics 'Till reconciliation' (15.05 KB, patch)
2017-05-20 09:49 UTC, Marc Véron
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 18643: Remove dead code in reports/statistics 'Till reconciliation' (15.12 KB, patch)
2017-05-22 09:30 UTC, Josef Moravec
Details | Diff | Splinter Review
Bug 18643: Remove dead code in reports/statistics 'Till reconciliation' (15.20 KB, patch)
2017-06-02 09:03 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 Marc Véron 2017-05-20 09:41:10 UTC
File koha-tmpl/intranet-tmpl/prog/en/modules/reports/reports-home.tt contains a link to /cgi-bin/koha/reports/stats.screen.pl with label 'Till reconciliation' that is commented out since years. 
Remove this link and the related files:
cgi-bin/koha/reports/stats.screen.pl
koha-tmpl/intranet-tmpl/prog/en/modules/reports/stats_screen.tt
Comment 1 Marc Véron 2017-05-20 09:49:50 UTC
Created attachment 63585 [details] [review]
Bug 18643: Remove dead code in reports/statistics 'Till reconciliation'

File koha-tmpl/intranet-tmpl/prog/en/modules/reports/reports-home.tt contains a link to /cgi-bin/koha/reports/stats.screen.pl with label 'Till reconciliation' that is commented out since years.
Remove this link and the related files:
cgi-bin/koha/reports/stats.screen.pl
koha-tmpl/intranet-tmpl/prog/en/modules/reports/stats_screen.tt

To test:
- Apply patch
- Verify that Koha > Reports still display the same
- Verify that two files stats.screen.pl and stats_screen.tt are gone and thet they are not used
  anywhere in the Koha codebase
Comment 2 Josef Moravec 2017-05-22 09:30:11 UTC
Created attachment 63606 [details] [review]
[SIGNED-OFF] Bug 18643: Remove dead code in reports/statistics 'Till reconciliation'

File koha-tmpl/intranet-tmpl/prog/en/modules/reports/reports-home.tt contains a link to /cgi-bin/koha/reports/stats.screen.pl with label 'Till reconciliation' that is commented out since years.
Remove this link and the related files:
cgi-bin/koha/reports/stats.screen.pl
koha-tmpl/intranet-tmpl/prog/en/modules/reports/stats_screen.tt

To test:
- Apply patch
- Verify that Koha > Reports still display the same
- Verify that two files stats.screen.pl and stats_screen.tt are gone and thet they are not used
  anywhere in the Koha codebase

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 3 Marcel de Rooy 2017-06-02 09:03:44 UTC
Created attachment 63926 [details] [review]
Bug 18643: Remove dead code in reports/statistics 'Till reconciliation'

File koha-tmpl/intranet-tmpl/prog/en/modules/reports/reports-home.tt contains a link to /cgi-bin/koha/reports/stats.screen.pl with label 'Till reconciliation' that is commented out since years.
Remove this link and the related files:
cgi-bin/koha/reports/stats.screen.pl
koha-tmpl/intranet-tmpl/prog/en/modules/reports/stats_screen.tt

To test:
- Apply patch
- Verify that Koha > Reports still display the same
- Verify that two files stats.screen.pl and stats_screen.tt are gone and thet they are not used
  anywhere in the Koha codebase

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Jonathan Druart 2017-06-05 14:56:52 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 5 Fridolin Somers 2017-06-13 07:26:04 UTC
Enhancement not pushed to 17.05.x