Bug 20663

Summary: Dead report code for "Create Compound Report" since prior to 3.0.x
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: ReportsAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: f.demians, jonathan.druart, katrin.fischer, m.de.rooy, martin.renvoize
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20663: Remove dead code related to compound reports
Bug 20663: Add some POD, so koha test tools don't complain
Bug 20663: Remove dead code related to compound reports
Bug 20663: Add some POD, so koha test tools don't complain
Bug 20663: Remove dead code related to compound reports
Bug 20663: Add some POD, so koha test tools don't complain

Description Mark Tompsett 2018-04-26 04:06:03 UTC
git grep "Create Compound Report"
only generates one matching line in guided reports. It makes no sense to keep this dead code and the code which it would trigger.
Comment 1 Mark Tompsett 2018-04-26 04:34:23 UTC
Created attachment 74871 [details] [review]
Bug 20663: Remove dead code related to compound reports

TEST PLAN
---------
git grep "Create Compound Report"
-- only one file
git grep compound | grep 1
-- this is the only setting of the compound tt variable
less koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
-- There is an TT IF statement for compound.
-- In that statement it would trigger 'Save Compound'
git grep save_compound
-- only the template and the guided report perl
git grep create_compound
-- only triggered by the save code in the guided report perl
-- in the export for the C4/Reports/Guided.pm
git grep run_compound
-- left over in export

apply the patch
look around and see the pieces are cleaned up.

run koha qa test tools
Comment 2 Mark Tompsett 2018-04-26 04:34:25 UTC
Created attachment 74872 [details] [review]
Bug 20663: Add some POD, so koha test tools don't complain
Comment 3 Katrin Fischer 2018-04-29 22:31:01 UTC
Created attachment 74925 [details] [review]
Bug 20663: Remove dead code related to compound reports

TEST PLAN
---------
git grep "Create Compound Report"
-- only one file
git grep compound | grep 1
-- this is the only setting of the compound tt variable
less koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
-- There is an TT IF statement for compound.
-- In that statement it would trigger 'Save Compound'
git grep save_compound
-- only the template and the guided report perl
git grep create_compound
-- only triggered by the save code in the guided report perl
-- in the export for the C4/Reports/Guided.pm
git grep run_compound
-- left over in export

apply the patch
look around and see the pieces are cleaned up.

run koha qa test tools

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Katrin Fischer 2018-04-29 22:31:04 UTC
Created attachment 74926 [details] [review]
Bug 20663: Add some POD, so koha test tools don't complain

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Frédéric Demians 2018-04-30 16:51:21 UTC
Isn't this patch supposed to have the Signed-off status?
Comment 6 Katrin Fischer 2018-04-30 16:53:47 UTC
Correct, thx Frederic.
Comment 7 Marcel de Rooy 2018-05-04 09:36:28 UTC
Created attachment 75055 [details] [review]
Bug 20663: Remove dead code related to compound reports

TEST PLAN
---------
git grep "Create Compound Report"
-- only one file
git grep compound | grep 1
-- this is the only setting of the compound tt variable
less koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
-- There is an TT IF statement for compound.
-- In that statement it would trigger 'Save Compound'
git grep save_compound
-- only the template and the guided report perl
git grep create_compound
-- only triggered by the save code in the guided report perl
-- in the export for the C4/Reports/Guided.pm
git grep run_compound
-- left over in export

apply the patch
look around and see the pieces are cleaned up.

run koha qa test tools

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 8 Marcel de Rooy 2018-05-04 09:36:32 UTC
Created attachment 75056 [details] [review]
Bug 20663: Add some POD, so koha test tools don't complain

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 9 Jonathan Druart 2018-05-04 12:49:45 UTC
Dead since
  commit 68ce64a793b023164ac19a56602747945a1bdca1
  Date:   Mon Oct 29 13:19:58 2007 -0500
    Bugfixing the template, and removing link to compound reports
Comment 10 Jonathan Druart 2018-05-04 12:50:35 UTC
Pushed to master for 18.05, thanks to everybody involved!