Bugzilla – Attachment 116453 Details for
Bug 27586
Import patrons script has a confirm switch that doesn't do anything
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 27586: Rollback should be before return statement
Bug-27586-Rollback-should-be-before-return-stateme.patch (text/plain), 953 bytes, created by
Katrin Fischer
on 2021-02-07 11:01:08 UTC
(
hide
)
Description:
Bug 27586: Rollback should be before return statement
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2021-02-07 11:01:08 UTC
Size:
953 bytes
patch
obsolete
>From 089307ee369f3b76b47bb89d36e07d1a4c5ba827 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Fri, 5 Feb 2021 06:15:06 -0500 >Subject: [PATCH] Bug 27586: Rollback should be before return statement > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > Koha/Patrons/Import.pm | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/Koha/Patrons/Import.pm b/Koha/Patrons/Import.pm >index 6f5646a416..03e3b0ad85 100644 >--- a/Koha/Patrons/Import.pm >+++ b/Koha/Patrons/Import.pm >@@ -420,6 +420,8 @@ sub import_patrons { > } > } > >+ $schema->storage->txn_rollback if $dry_run; >+ > return { > feedback => \@feedback, > errors => \@errors, >@@ -429,8 +431,6 @@ sub import_patrons { > invalid => $invalid, > imported_borrowers => \@imported_borrowers, > }; >- >- $schema->storage->txn_rollback if $dry_run; > } > > =head2 prepare_columns >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 27586
:
116179
|
116338
|
116390
|
116452
| 116453