Bug 27308: Skip blank lines when inserting new
authorNick Clemens <nick@bywatersolutions.com>
Thu, 24 Dec 2020 13:35:28 +0000 (13:35 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Mon, 18 Jan 2021 14:33:57 +0000 (15:33 +0100)
commitd27b1018a96667c6fce8fc746e968e9a206858a0
tree73610c1b61f7404f08d635c7b7b8aa11baf6e8bd
parentd9b2c504639e3e336a0ef8a1f75c31452a556e90
Bug 27308: Skip blank lines when inserting new

The intent of the macro insert new function is to add a new field at the correct numerical place in the record. If you have a blank line while editing, however, the new line will be inserted at the blank line, rather than checking the rest of the record.

To test:
1 - Create a macro like
    new 690=‡aLOCAL SUBJECT
2 - Insert a blank line into your record somewhere before where the 690 should be inserted
3 - Run the macro
4 - Line is inserted before the blank line
5 - Apply patch
6 - Reload (ensure cache is cleared and JS files are reloaded)
7 - Remove the previously created field, but leave the blank
8 - Run the macro
9 - Field is created at correct position

Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Bug 27308: (QA follow-up) Remove commented line

I don't think we need to keep the original line as a comment.. this
patch simply cleans it up.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 3b0f86b53173d30da5dbb42710a7c9bf9edaa3bb)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
koha-tmpl/intranet-tmpl/lib/koha/cateditor/marc-editor.js