I would like to reindent cataloging/addbiblio.tt: Remove tabs, correct indentation, add some comments to clarify the structure.
Created attachment 90786 [details] [review] Bug 23159: Reindent addbiblio.tt This patch corrects whitespace in the basic MARC editor template, addbiblio.tt: Tabs converted to spaces, indentation corrected. Some comments have been added in the markup to clarify the page structure. To test, apply the patch and open the basic MARC editor. Confirm that everything works correctly, including: - Showing/hiding subfields - Re-ordering subfields - Removing tags and subfields - Cloning tags and subfields - Plugin links Use the "-w" flag when using diff to examine the patch. The only changes should be places where single lines were broken up and where comments were added.
Comment on attachment 90786 [details] [review] Bug 23159: Reindent addbiblio.tt Review of attachment 90786 [details] [review]: ----------------------------------------------------------------- ::: koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt @@ +488,5 @@ > + [% ELSE %] > + <button type="submit" class="new" onclick="confirmnotdup('view'); return false;"><i class="fa fa-fw fa-save"></i> No, save as new record</button> > + [% END %] > + </form> > + </div> <!-- /.dialog.alert --> HTML comments and Template comments added make this harder to check. @@ +489,5 @@ > + <button type="submit" class="new" onclick="confirmnotdup('view'); return false;"><i class="fa fa-fw fa-save"></i> No, save as new record</button> > + [% END %] > + </form> > + </div> <!-- /.dialog.alert --> > + [% END # /IF duplicatebiblionumber %] Your insertion of comments while clarifying, would have been better in a separate commit, so it would be easier to check.
Created attachment 90789 [details] [review] Bug 23159: (followup) re-add comments" This reverts the previous followup, so that all patches can be pushed without only pushing the first one.
Created attachment 90790 [details] [review] Bug 23159: (followup) remove added comments
Created attachment 90791 [details] [review] Bug 23159: (followup) re-add comments" This reverts the previous followup, so that all patches can be pushed without only pushing the first one.
Created attachment 90792 [details] [review] Bug 23159: Reindent addbiblio.tt This patch corrects whitespace in the basic MARC editor template, addbiblio.tt: Tabs converted to spaces, indentation corrected. Some comments have been added in the markup to clarify the page structure. To test, apply the patch and open the basic MARC editor. Confirm that everything works correctly, including: - Showing/hiding subfields - Re-ordering subfields - Removing tags and subfields - Cloning tags and subfields - Plugin links Use the "-w" flag when using diff to examine the patch. The only changes should be places where single lines were broken up and where comments were added. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 90793 [details] [review] Bug 23159: (follow-up) remove added comments Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 90794 [details] [review] Bug 23159: (follow-up) re-add comments" This reverts the previous followup, so that all patches can be pushed without only pushing the first one. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
TEST PLAN (on a kohadevbox) --------- $ git checkout master $ git pull $ git checkout -b bug_23159 origin/master $ git bz apply 23159 -- do this interactively, and only apply the first 2 patches $ tr -d '[:space:]' < koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt > after.txt $ git checkout master $ tr -d '[:space:]' < koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt > before.txt $ diff before.txt after.txt -- no differences, which means the only differences are whitespace $ git checkout bug_23159 $ git bz apply 23159 -- apply the last patch $ restart_all Do the recommended testing in comment #1, though the "-w" won't help much, hence the provision of the remove and re-add comments patches.
Created attachment 90873 [details] [review] Bug 23159: Reindent addbiblio.tt This patch corrects whitespace in the basic MARC editor template, addbiblio.tt: Tabs converted to spaces, indentation corrected. Some comments have been added in the markup to clarify the page structure. To test, apply the patch and open the basic MARC editor. Confirm that everything works correctly, including: - Showing/hiding subfields - Re-ordering subfields - Removing tags and subfields - Cloning tags and subfields - Plugin links Use the "-w" flag when using diff to examine the patch. The only changes should be places where single lines were broken up and where comments were added. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 90874 [details] [review] Bug 23159: (follow-up) remove added comments Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 90875 [details] [review] Bug 23159: (follow-up) re-add comments" This reverts the previous followup, so that all patches can be pushed without only pushing the first one. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
(In reply to M. Tompsett from comment #2) > Your insertion of comments while clarifying, would have been better in a > separate commit, so it would be easier to check. Agree completely. Without the two patches of Mark, this patch would probably be sitting in the queue for much longer! @RM: Patch 2 and 3 do not need to be pushed (although they can), but prove that patch 1 can be pushed.
Nice work! Pushed to master for 19.11.00
Enhancement not pushed to 19.05.x
I choose to push to stable branch in order to ease next patches Pushed to 19.05.x for 19.05.03
i also wanted to have this in 18.11.x but I was unable to apply cleanly or rebase. no backport