Attachment 123134 Details for Bug 28336 – Bug 28336: Add missing input for item_action

[patch] Bug 28336: Add missing input for item_action

Bug-28336-Add-missing-input-for-itemaction.patch (text/plain), 1.95 KB, created by Kyle M Hall (khall) on 2021-07-23 14:44:58 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2021-07-23 14:44:58 UTC

Size: 1.95 KB

patch

obsolete

>From 03109223f3706e589e63b07806bc58cfa25e5636 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 12 May 2021 17:00:57 +0000
>Subject: [PATCH] Bug 28336: Add missing input for item_action
>
>To test:
>1 - Stage a marc file of authorities
>2 - Manage the staged file
>3 - Attempt to change the matching rule
>4 - 500 Error, in intranet-error.log:
>[Wed May 12 16:53:03.046652 2021] [cgi:error] [pid 52072] [client 172.19.0.1:50580] AH01215: C4::ImportBatch::SetImportBatchItemAction(): DBI Exception: DBD::mysql::st execute failed: Column 'item_action' cannot be null [for Statement "UPDATE import_batches SET item_action = ? WHERE import_batch_id = ?" with ParamValues: 0=undef, 1="1"] at /kohadevbox/koha/tools/manage-marc-import.pl line 160: /kohadevbox/koha/tools/manage-marc-import.pl, referer: http://localhost:8081/cgi-bin/koha/tools/manage-marc-import.pl?import_batch_id=1
>5 - Apply patch
>6 - Reload page and change matching rule
>7 - Success!
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt   | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
>index 2a56b1a2d3..601d0a3dbd 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
>@@ -223,6 +223,8 @@
>                                             [% END %]
>                                         [% END %]
>                                     </li>
>+                                [% ELSE %]
>+                                    <input type ="hidden" name="item_action" value="ignore">
>                                 [% END %]
>                             </ol>
>                             [% IF ( can_commit ) %]
>-- 
>2.30.1 (Apple Git-130)

Actions: View | Diff | Splinter Review

Attachments on bug 28336: 120893 | 121613 | 123134