[Bug 19164] New: Allow MARC modification templates to be used in staged MARC imports
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Bug ID: 19164 Summary: Allow MARC modification templates to be used in staged MARC imports Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Bibliographic record staging/import Assignee: koha-bugs@lists.koha-community.org Reporter: J.P.Knight@lboro.ac.uk QA Contact: testopia@bugs.koha-community.org In Bug 18690 Katrin Fischer suggested that it would be useful to have MARC Modification Templates used in stage MARC imports in a similar way to that enhancement's inclusion in bulkmarcimport.pl -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |J.P.Knight@lboro.ac.uk Assignee|koha-bugs@lists.koha-commun |J.P.Knight@lboro.ac.uk |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #1 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Created attachment 66351 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66351&action=edit First cut at some changes to misc/stage_file.pl script to provide for MARC modification template support. Not ready for QA yet. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Woot! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I like using the name as it's easier to remember and harder to confuse. Also easier to set it up the same on multiple installations. Maybe we could just add a warning when a non-unique template name is used? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #4 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Yes, I'm just doing the same thing in bug 18690 now. If folk like what I do there, I'll pop the same code in here too. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #5 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Ah, just realised I was referencing the wrong Bug ID for the other bug - is actually 18389. Too many browser tabs open! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #6 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Created attachment 68239 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68239&action=edit Test MARC record This is a test MARC record (shamelessly stolen from Bug 19069) that can be used to test that MARC modification templates are applied using the batch misc/stage_file.pl script. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #7 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Created attachment 68240 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68240&action=edit Bug 19164 - Allow MARC modification templates to be used in staged MARC imports Patch checkes for duplicate MARC modificiation templates and if only one is found that matches the name given on the command line, submits the batch staging of the input files using that template. Test plan: -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff --- Comment #8 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Test plan: 1) In the staff web client got to "Home › Tools › MARC modification templates" and create a MARC Modification Template called "Test Bug 19164". Set the action to be "Copy field 999$l to 998$l using RegEx s/NASH/FLASH/ if 999$l matches NASH" 2) Download the test MARC record from this bug into a file called marc1.mrc into your koha dev user's home directory. 3) Run misc/stage_file.pl with the command line: misc/stage_file.pl --file ~/marc1.mrc --add-items --marcmodtemplate "Test Bug 19164" --item-action replace --comment "testing MARC mod templates on batch staging" 4) In the staff client, go to Home › Tools › Manage staged MARC records and check that the file is visible and in status "Staged" 5) Click on the filename to bring up details of the staged batch, and then click on "Basic Christianity. Stott, John R. W. (0802811892)". Note that there is a 998$l field with the value "FLASH" that has been created using the MARC Modification Template. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Hugo Agud <hagud@orex.es> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply CC| |hagud@orex.es --- Comment #9 from Hugo Agud <hagud@orex.es> --- Applying: First cut at some changes to misc/stage_file.pl script to provide for MARC modification template support. Not ready for QA yet. Using index info to reconstruct a base tree... M misc/stage_file.pl Falling back to patching base and 3-way merge... Auto-merging misc/stage_file.pl CONFLICT (content): Merge conflict in misc/stage_file.pl Failed to merge in the changes. Patch failed at 0001 First cut at some changes to misc/stage_file.pl script to provide for MARC modification template support. Not ready for QA yet. The copy of the patch that failed is found in: /home/vagrant/kohaclone/.git/rebase-apply/patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/First-cut-at-some-changes-to-miscstagefilepl-scrip-x5L2A_.patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #10 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Created attachment 68249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68249&action=edit Patch for misc/stage_file.pl script to provide for MARC modification template support. https://bugs.koha-community.org/show_bug.cgi?id=19164 Bug 19164 - Allow MARC modification templates to be used in staged MARC imports Patch checks for duplicate MARC modificiation templates and if only one is found that matches the name given on the command line, submits the batch staging of the input files using that template. Test plan: 1) In the staff web client got to "Home › Tools › MARC modification templates" and create a MARC Modification Template called "Test Bug 19164". Set the action to be "Copy field 999$l to 998$l using RegEx s/NASH/FLASH/ if 999$l matches NASH" 2) Download the test MARC record from this bug into a file called marc1.mrc into your koha dev user's home directory. 3) Run misc/stage_file.pl with the command line: misc/stage_file.pl --file ~/marc1.mrc --add-items --marcmodtemplate "Test Bug 19164" --item-action replace --comment "testing MARC mod templates on batch staging" 4) In the staff client, go to Home › Tools › Manage staged MARC records and check that the file is visible and in status "Staged" 5) Click on the filename to bring up details of the staged batch, and then click on "Basic Christianity. Stott, John R. W. (0802811892)". Note that there is a 998$l field with the value "FLASH" that has been created using the MARC Modification Template. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #66351|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68240|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff --- Comment #11 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Rebased the pair of patches (something must have changed in master since I did the first one that was already in my branch) into a single new patch and obsoleted the two original patches. Now applies cleanly from scratch in my kohadevbox using git bz apply. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Alex Buckley <alexbuckley@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |alexbuckley@catalyst.net.nz --- Comment #12 from Alex Buckley <alexbuckley@catalyst.net.nz> --- Created attachment 68341 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68341&action=edit Record not visible in staged file after running misc/stage-file.pl Hi Jon After running the misc/stage-file.pl with the parameters as set in your test plan I have found that instead of displaying the "Basic Christianity. Stott, John R. W. (0802811892)" record a record link 'null' is displayed (see screenshot) and it only contains MARC fields: 000, 001 003, 005 and 008 I copied the contents of your marc file across into my kohadevbox and pasted it into new file with vi and made sure the formatting matched your marc file so I am assuming that this issue may be in the misc/stage-file.pl script. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #13 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Comment on attachment 68249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68249 Patch for misc/stage_file.pl script to provide for MARC modification template support. Review of attachment 68249: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=19164&attachment=68249) ----------------------------------------------------------------- Hi Jon, failing QA because of Alex's comment - can you please take a look? Also spotted a tiny something, see below. ::: misc/stage_file.pl @@ +19,5 @@
# with this program; if not, write to the Free Software Foundation, Inc., # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+#use strict; +#use warnings;
You can delete the commented lines when using Modern:Perl; -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #14 from Jon Knight <J.P.Knight@lboro.ac.uk> --- (In reply to Alex Buckley from comment #12)
Created attachment 68341 [details] Record not visible in staged file after running misc/stage-file.pl
Hi Jon
After running the misc/stage-file.pl with the parameters as set in your test plan I have found that instead of displaying the "Basic Christianity. Stott, John R. W. (0802811892)" record a record link 'null' is displayed (see screenshot) and it only contains MARC fields: 000, 001 003, 005 and 008
I copied the contents of your marc file across into my kohadevbox and pasted it into new file with vi and made sure the formatting matched your marc file so I am assuming that this issue may be in the misc/stage-file.pl script.
Hmm, I've just tried on a fresh branch on an updated kohadevbox and I can't reproduce this. Could you add your copy of the marc file to this case so that I can see if vi has done anything odd to it? When I transferred it on my kohadevbox I just used vagrant's NFS share, so it never went near a text editor. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #15 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Created attachment 68425 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68425&action=edit Bug_19164: remove use strict/warn lines (were commented out anyway) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@univ-lyon3.fr -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #16 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Hello, just for information, we apply it on the top of a koha 16.11.12 and test it with our own marc file and our own model. We launch this command : stage_file.pl --encoding UTF-8 --file export_201710031044.mrc --add-items --marcmodtemplate "test" It works as expected, the modification model is apllied but there's several warnings : ERROR occurred in SetUTF8Flag Field 020 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 245 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 020 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 245 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 020 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 245 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. Use of uninitialized value in join or string at /usr/share/perl5/MARC/Field.pm line 696. ERROR occurred in SetUTF8Flag Field 020 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 245 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. Use of uninitialized value in join or string at /usr/share/perl5/MARC/Field.pm line 696. ERROR occurred in SetUTF8Flag Field 020 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. ERROR occurred in SetUTF8Flag Field 245 must have at least one subfield at /home/koha/src/C4/Charset.pm line 145. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |ASSIGNED --- Comment #17 from Jon Knight <J.P.Knight@lboro.ac.uk> --- OK, I'll take a look at this in a day or so. Assigned back to me. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #18 from Jon Knight <J.P.Knight@lboro.ac.uk> --- I don't suppose you could upload a copy of "export_201710031044.mrc" so that I can make sure that I'm playing with the same file. Also are you using MARC 21 or UNIMARC? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #19 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Hello, yes I will upload the file. We work in Unimarc but the file is in marc21. Actually, we want to use template for changing key fields from marc21 fields in unimarc fields. It's only temporary files for acquisitions. I think you're on something.. the warnings are perhaps only because our installation is in unimarc.. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #20 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Created attachment 70033 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=70033&action=edit marc_file -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #21 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Hello, if you have time to work on it before Marseille hackfest (begins on 12th March), I'm ok to test it on a master during the hackfest. Sonia -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #22 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Been doing some more playing with this. On my kohadevbox, applying this patch to the latest master, making a MARC modification rule of: Copy field 960$q to 998$q and then applying it to the export_201710031044.mrc file doesn't give me any errors on the command line, and appears to copy the 960$q to 998$q in the staged records as expected. Then I tried going to the 16.11.x branch in git and applying the patch on top of that. The patches apply OK, but the import fails (test1.mrc is my copy of the uploaded MARC file): vagrant@kohadevbox:kohaclone(16.11.x)$ git checkout -b bug_19164_16_11 Switched to a new branch 'bug_19164_16_11' vagrant@kohadevbox:kohaclone(bug_19164_16_11)$ git bz apply 19164 Bug 19164 - Allow MARC modification templates to be used in staged MARC imports 68249 - Patch for misc/stage_file.pl script to provide for MARC modification template support. 68425 - Bug_19164: remove use strict/warn lines (were commented out anyway) Apply? [(y)es, (n)o, (i)nteractive] y Applying: Patch for misc/stage_file.pl script to provide for MARC modification template support. Applying: Bug_19164: remove use strict/warn lines (were commented out anyway) vagrant@kohadevbox:kohaclone(bug_19164_16_11)$ misc/stage_file.pl --file ~/test.mrc --add-items --marcmodtemplate "Test Bug 19164" --item-action replace --comment "testing MARC mod templates on batch staging" ... staging MARC records -- please wait Not a HASH reference at /home/vagrant/kohaclone/C4/Biblio.pm line 2786. Hmm, so it looks like something is broken for using this patch with the latest 16.11.x. The question now is, as this is an enhancement (ie adding a new feature to Koha), do we worry about that too much if its working in the master train? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #23 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Hello, I have tried on a 16.11.16 and it works. I'm going to try it on a master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff --- Comment #24 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Great thank you - I'll change the status to needs sign off in that case! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Mackey Johnstone <mackeyfj@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Mackey Johnstone <mackeyfj@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68425|0 |1 is obsolete| | --- Comment #25 from Mackey Johnstone <mackeyfj@gmail.com> --- Created attachment 84167 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=84167&action=edit Bug_19164: remove use strict/warn lines (were commented out anyway) Signed-off-by: Mackey Johnstone <mackeyfj@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #26 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Comment on attachment 68249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68249 Patch for misc/stage_file.pl script to provide for MARC modification template support. Review of attachment 68249: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=19164&attachment=68249) ----------------------------------------------------------------- I don't believe this patch as a whole is signed off yet, but I've done an initial QA run on it anyway. Please see the code comments. ::: misc/stage_file.pl @@ +74,5 @@
+ if($this_template->{'name'} eq $marc_mod_template) { + $marc_mod_template_id = $this_template->{'template_id'}; + last; + } + if($this_template->{'name'} eq $marc_mod_template) {
This block can never be reached as the code stands... @@ +149,5 @@
+ BatchStageMarcRecords( + $record_type, $params->{encoding}, + $marc_records, $params->{input_file}, + $params->{'marc_mod_template_id'}, + $params->{batch_comment}, '',
Minor whitespace error introductions in the above four lines. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com QA Contact|testopia@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68249|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #84167|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #27 from Jon Knight <J.P.Knight@lboro.ac.uk> --- Created attachment 84982 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=84982&action=edit Bug 19164: Add MARC modification template support to misc/stage-file.pl. https://bugs.koha-community.org/show_bug.cgi?id=19164 Bug 19164 - Allow MARC modification templates to be used in staged MARC imports Patch checks for duplicate MARC modificiation templates and if only one is found that matches the name given on the command line, submits the batch staging of the input files using that template. Test plan: 1) In the staff web client got to "Home › Tools › MARC modification templates" and create a MARC Modification Template called "Test Bug 19164". Set the action to be "Copy field 999$l to 998$l using RegEx s/NASH/FLASH/ if 999$l matches NASH" 2) Download the test MARC record from this bug into a file called marc1.mrc into your koha dev user's home directory. 3) Run misc/stage_file.pl with the command line: misc/stage_file.pl --file ~/marc1.mrc --add-items --marcmodtemplate "Test Bug 19164" --item-action replace --comment "testing MARC mod templates on batch staging" 4) In the staff client, go to Home › Tools › Manage staged MARC records and check that the file is visible and in status "Staged" 5) Click on the filename to bring up details of the staged batch, and then click on "Basic Christianity. Stott, John R. W. (0802811892)". Note that there is a 998$l field with the value "FLASH" that has been created using the MARC Modification Template. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Jon Knight <J.P.Knight@lboro.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #28 from Jon Knight <J.P.Knight@lboro.ac.uk> --- I've rebased the patches so that the changes are all in one patch file now, and fixed the unreachable code. Run on my kohadevbox against master and it seems to work OK with the test marc record I uploaded originally. I also ran `qa -c 1 -v 2` on the new commit and it seems to pass the tests there. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #84982|0 |1 is obsolete| | --- Comment #29 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 84987 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=84987&action=edit Bug 19164: Add MARC modification template support to misc/stage-file.pl. https://bugs.koha-community.org/show_bug.cgi?id=19164 Bug 19164 - Allow MARC modification templates to be used in staged MARC imports Patch checks for duplicate MARC modificiation templates and if only one is found that matches the name given on the command line, submits the batch staging of the input files using that template. Test plan: 1) In the staff web client got to "Home › Tools › MARC modification templates" and create a MARC Modification Template called "Test Bug 19164". Set the action to be "Copy field 999$l to 998$l using RegEx s/NASH/FLASH/ if 999$l matches NASH" 2) Download the test MARC record from this bug into a file called marc1.mrc into your koha dev user's home directory. 3) Run misc/stage_file.pl with the command line: misc/stage_file.pl --file ~/marc1.mrc --add-items --marcmodtemplate "Test Bug 19164" --item-action replace --comment "testing MARC mod templates on batch staging" 4) In the staff client, go to Home › Tools › Manage staged MARC records and check that the file is visible and in status "Staged" 5) Click on the filename to bring up details of the staged batch, and then click on "Basic Christianity. Stott, John R. W. (0802811892)". Note that there is a 998$l field with the value "FLASH" that has been created using the MARC Modification Template. Signed-off-by: Mackey Johnstone <mackeyfj@gmail.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #30 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 84988 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=84988&action=edit Bug 19164: (QA follow-up) Minor whitespace cleanup -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #31 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- I've taken Mackey Johnstone as a signoff and added the line appropriately so I can move on to QA. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #32 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Works as expected, a nice simple but very helpful patch. Thanks for bearing with it so long Jon. Passing QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #33 from Nick Clemens <nick@bywatersolutions.com> --- Awesome work all! Pushed to master for 19.05 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to Master |RESOLVED --- Comment #34 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Enhancement, will not be backported to 18.11.x series. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement allows release notes| |applying MARC modification | |templates to batches of | |records during the | |import/staging process from | |the staff client -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 Simon Hohl <simon.hohl@dainst.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |simon.hohl@dainst.org --- Comment #35 from Simon Hohl <simon.hohl@dainst.org> --- Are there any plans to add this functionality to the web interface? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19164 --- Comment #36 from Jon Knight <J.P.Knight@lboro.ac.uk> --- (In reply to Simon Hohl from comment #35)
Are there any plans to add this functionality to the web interface?
Not by me. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org