Bug 27586: Import patrons script has a confirm switch that doesn't nothing!
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 1 Feb 2021 18:22:58 +0000 (13:22 -0500)
committerAndrew Fuerste-Henry <andrew@bywatersolutions.com>
Wed, 17 Feb 2021 19:38:56 +0000 (19:38 +0000)
commitc0a7f53a62b6c90c87234288fc0cee910a7f6bd0
treec12cf596908dfa45b2ccc51275f50c51c8a86d0b
parentc01dd7a8fb3cf82226a9ca525f49ad0d12e05d68
Bug 27586: Import patrons script has a confirm switch that doesn't nothing!

If the script misc/import_patrons.pl is run without "--confirm" it reports
that it is "running in dry-run mode" but it is not!

Test Plan:
1) Import a CSV of patrons using import_patrons.pm *without* the --confirm parameter
2) Note that it does actually import the patrons
3) Apply this patch
4) Note that this time the patrons are not actually imported
5) Import again with --confirm
6) Note that the patrons were actually imported this time!

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit d3005a2a4c493c624ffd6204f6fccc83ea350361)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 19e0b91e12b520387938bb9c20bd605b757d3211)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Koha/Patrons/Import.pm
misc/import_patrons.pl