Bug 24732

Summary: Make DumpTemplateVars more readable
Product: Koha Reporter: Nick Clemens <nick>
Component: Architecture, internals, and plumbingAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: 1joynelson, george, hayleypelham, lucas, mkstephens
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.05, 19.05.10
Attachments: Bug 24732: Set DumpTemplateVars Dumper options for Indent and Sort
Bug 24732: Set DumpTemplateVars Dumper options for Indent and Sort
Bug 24732: Set DumpTemplateVars Dumper options for Indent and Sort

Description Nick Clemens 2020-02-25 18:51:33 UTC
Currently we just use Dumper output directly, it would be helpful if we formatted things slightly
Comment 1 Nick Clemens 2020-02-25 18:54:08 UTC
Created attachment 99605 [details] [review]
Bug 24732: Set DumpTemplateVars Dumper options for Indent and Sort

To test:
1 - Enable sysprefs
    DumpTemplateVarsOpac
    DumpTemplateVarsIntranet
2 - View some pages on staff client and opac
3 - View the page source
4 - See a large mass of comments including the page vars at the top of the source
5 - Refresh page and note order of vars changes
6 - Apply patch
7 - View page source on some pages and note things are much easier to read
8 - Refresh page and note variable order does not change
Comment 2 ByWater Sandboxes 2020-02-27 04:12:13 UTC
Created attachment 99682 [details] [review]
Bug 24732: Set DumpTemplateVars Dumper options for Indent and Sort

To test:
1 - Enable sysprefs
    DumpTemplateVarsOpac
    DumpTemplateVarsIntranet
2 - View some pages on staff client and opac
3 - View the page source
4 - See a large mass of comments including the page vars at the top of the source
5 - Refresh page and note order of vars changes
6 - Apply patch
7 - View page source on some pages and note things are much easier to read
8 - Refresh page and note variable order does not change

Signed-off-by: George Williams <george@nekls.org>
Comment 3 Katrin Fischer 2020-02-29 09:15:44 UTC
Created attachment 99807 [details] [review]
Bug 24732: Set DumpTemplateVars Dumper options for Indent and Sort

To test:
1 - Enable sysprefs
    DumpTemplateVarsOpac
    DumpTemplateVarsIntranet
2 - View some pages on staff client and opac
3 - View the page source
4 - See a large mass of comments including the page vars at the top of the source
5 - Refresh page and note order of vars changes
6 - Apply patch
7 - View page source on some pages and note things are much easier to read
8 - Refresh page and note variable order does not change

Signed-off-by: George Williams <george@nekls.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Martin Renvoize 2020-03-02 10:43:17 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 5 Joy Nelson 2020-03-31 22:48:41 UTC
enhancement no backported to 19.11.x
Comment 6 Joy Nelson 2020-04-01 13:57:07 UTC
welp, i changed my mind.  Nick asked so nicely that I applied to this to the 19.11.x branch for 19.11.05.  :D
Comment 7 Lucas Gass 2020-04-08 21:01:12 UTC
backported to 19.05.x for 19.05.10
Comment 8 Hayley Pelham 2020-04-14 04:10:34 UTC
Enhancement will not be backported to 18.11.x series