Bug 23399

Summary: Reindent returns.tt
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: RESOLVED FIXED QA Contact: Martin Renvoize <martin.renvoize>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, martin.renvoize
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Who signed the patch off: Text to go in the release notes:
Version(s) released in:
19.11.00
Bug Depends on: 23328, 23286    
Bug Blocks: 23039, 23434    
Attachments: Bug 23399: Reindent returns.tt
Bug 23399: (follow-up) Markup comments and corrections
Bug 23399: Reindent returns.tt
Bug 23399: (follow-up) Markup comments and corrections
Bug 23399: Reindent returns.tt
Bug 23399: (follow-up) Markup comments and corrections

Description Owen Leonard 2019-07-30 13:11:40 UTC
I would like to reindent circulation/returns.tt: Remove tabs, correct indentation, add some comments to clarify the structure.
Comment 1 Owen Leonard 2019-07-30 13:58:11 UTC
Created attachment 91877 [details] [review]
Bug 23399: Reindent returns.tt

This patch reindents circulation/returns.tt to use four spaces,
following coding guidelines.

To test, apply the patch and use your preferred method for checking the
differences between files while ignoring whitespace. I use diff with the
"-w" flag, but each must take their own journey.

Test the checkin process to confirm that everything still works
correctly.
Comment 2 Owen Leonard 2019-07-30 13:58:13 UTC
Created attachment 91878 [details] [review]
Bug 23399: (follow-up) Markup comments and corrections

This patch adds comments to the template to highlight the markup
structure. It also corrects some minor markup errors.

This patch should have no effect on the interface or functionality.
Comment 3 Kyle M Hall 2019-08-01 17:08:00 UTC
Created attachment 91910 [details] [review]
Bug 23399: Reindent returns.tt

This patch reindents circulation/returns.tt to use four spaces,
following coding guidelines.

To test, apply the patch and use your preferred method for checking the
differences between files while ignoring whitespace. I use diff with the
"-w" flag, but each must take their own journey.

Test the checkin process to confirm that everything still works
correctly.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 4 Kyle M Hall 2019-08-01 17:08:11 UTC
Created attachment 91911 [details] [review]
Bug 23399: (follow-up) Markup comments and corrections

This patch adds comments to the template to highlight the markup
structure. It also corrects some minor markup errors.

This patch should have no effect on the interface or functionality.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Martin Renvoize 2019-08-08 11:59:35 UTC
Created attachment 92070 [details] [review]
Bug 23399: Reindent returns.tt

This patch reindents circulation/returns.tt to use four spaces,
following coding guidelines.

To test, apply the patch and use your preferred method for checking the
differences between files while ignoring whitespace. I use diff with the
"-w" flag, but each must take their own journey.

Test the checkin process to confirm that everything still works
correctly.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize 2019-08-08 11:59:38 UTC
Created attachment 92071 [details] [review]
Bug 23399: (follow-up) Markup comments and corrections

This patch adds comments to the template to highlight the markup
structure. It also corrects some minor markup errors.

This patch should have no effect on the interface or functionality.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 7 Martin Renvoize 2019-08-08 12:00:05 UTC
Always good to make the indenting a bit more sane.. thanks Owen!

Passing QA
Comment 8 Martin Renvoize 2019-08-08 12:07:08 UTC
Nice work!

Pushed to master for 19.11.00
Comment 9 Fridolin SOMERS 2019-08-09 14:08:27 UTC
Enhancement not pushed to 19.05.x