Bug 23221

Summary: Reindent tools/manage-marc-import.tt
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, lucas, mtompset
Version: Main   
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:
19.11.00,19.05.03
Bug Depends on:    
Bug Blocks: 23094    
Attachments: Bug 23221: Reindent tools/manage-marc-import.tt
Bug 23221: (follow-up) Add comments on markup structure
Bug 23221: Reindent tools/manage-marc-import.tt
Bug 23221: (follow-up) Add comments on markup structure
Bug 23221: Reindent tools/manage-marc-import.tt
Bug 23221: (follow-up) Add comments on markup structure
Bug 23221: Reindent tools/manage-marc-import.tt
Bug 23221: (follow-up) Add comments on markup structure
Bug 23221: Reindent tools/manage-marc-import.tt
Bug 23221: (follow-up) Add comments on markup structure

Description Owen Leonard 2019-06-26 15:18:54 UTC
I would like to reindent tools/manage-marc-import.tt: Correct indentation, add some comments to clarify the structure.
Comment 1 Owen Leonard 2019-06-26 17:59:15 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2019-06-26 17:59:18 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2019-06-26 18:39:33 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2019-06-26 18:39:36 UTC Comment hidden (obsolete)
Comment 5 Owen Leonard 2019-06-26 18:53:19 UTC
Created attachment 91032 [details] [review]
Bug 23221: Reindent tools/manage-marc-import.tt

This patch reformats the template for the Manage staged MARC records
page. It makes only whitespace changes. It should have no effect on the
behavior of the page.

In addition to re-indentation, some lines have been broken up. These
should be the only changes highlighted when using diff while ignoring
whitespace.

To test, apply the patch and view the Manage staged MARC records page.
The page should look correct and work correctly:

 - Paging through results
 - Viewing batches
 - Cleaning batches
 - Deleting batches
Comment 6 Owen Leonard 2019-06-26 18:53:21 UTC
Created attachment 91033 [details] [review]
Bug 23221: (follow-up) Add comments on markup structure

This patch adds a few comments to the markup in order to highlight the
structure of the page.
Comment 7 Mark Tompsett 2019-07-04 18:55:10 UTC
Comment on attachment 91033 [details] [review]
Bug 23221: (follow-up) Add comments on markup structure

Review of attachment 91033 [details] [review]:
-----------------------------------------------------------------

I don't understand the insertion of /'s in the comments.

::: koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
@@ +276,4 @@
>                                      </tr>
>                                  [% END %]
>                              </table>
> +                        [% END #/ IF did_commit %]

I would expect "# /" for consistency, but as a comment it doesn't really matter.
Comment 8 Mark Tompsett 2019-07-04 18:56:25 UTC
# really force master to be latest before.
git checkout master
git pull
git reset --hard origin/master

cat koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt | tr -d [:space:] > before.txt

git checkout -b bug_23221 origin/master
git bz apply 23221
# interactive and apply just the first patch
cat koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt | tr -d [:space:] > after.txt

diff before.txt after.txt
# should be no differences

# didn't use git diff -w origin/master, because too much scroll.
rm before.txt after.txt
Comment 9 Mark Tompsett 2019-07-04 18:57:02 UTC
Created attachment 91297 [details] [review]
Bug 23221: Reindent tools/manage-marc-import.tt

This patch reformats the template for the Manage staged MARC records
page. It makes only whitespace changes. It should have no effect on the
behavior of the page.

In addition to re-indentation, some lines have been broken up. These
should be the only changes highlighted when using diff while ignoring
whitespace.

To test, apply the patch and view the Manage staged MARC records page.
The page should look correct and work correctly:

 - Paging through results
 - Viewing batches
 - Cleaning batches
 - Deleting batches

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 10 Mark Tompsett 2019-07-04 18:57:05 UTC
Created attachment 91298 [details] [review]
Bug 23221: (follow-up) Add comments on markup structure

This patch adds a few comments to the markup in order to highlight the
structure of the page.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 11 Katrin Fischer 2019-07-06 07:47:05 UTC
Created attachment 91383 [details] [review]
Bug 23221: Reindent tools/manage-marc-import.tt

This patch reformats the template for the Manage staged MARC records
page. It makes only whitespace changes. It should have no effect on the
behavior of the page.

In addition to re-indentation, some lines have been broken up. These
should be the only changes highlighted when using diff while ignoring
whitespace.

To test, apply the patch and view the Manage staged MARC records page.
The page should look correct and work correctly:

 - Paging through results
 - Viewing batches
 - Cleaning batches
 - Deleting batches

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 12 Katrin Fischer 2019-07-06 07:47:09 UTC
Created attachment 91384 [details] [review]
Bug 23221: (follow-up) Add comments on markup structure

This patch adds a few comments to the markup in order to highlight the
structure of the page.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 13 Martin Renvoize 2019-07-15 11:22:16 UTC
Nice work!

Pushed to master for 19.11.00
Comment 14 Fridolin Somers 2019-08-06 13:09:41 UTC
Pushed to 19.05.x for 19.05.03
Comment 15 Lucas Gass 2019-08-06 19:47:47 UTC
backported to 18.11.x for 18.11.09