[Koha-bugs] [Bug 12598] New misc/import_borrowers.pl command line tool

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Feb 16 14:45:16 CET 2018


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598

Kyle M Hall <kyle at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #66575|0                           |1
        is obsolete|                            |
  Attachment #66576|0                           |1
        is obsolete|                            |
  Attachment #66577|0                           |1
        is obsolete|                            |
  Attachment #66578|0                           |1
        is obsolete|                            |
  Attachment #66579|0                           |1
        is obsolete|                            |
  Attachment #66580|0                           |1
        is obsolete|                            |
  Attachment #67276|0                           |1
        is obsolete|                            |
  Attachment #67277|0                           |1
        is obsolete|                            |
  Attachment #70265|0                           |1
        is obsolete|                            |
  Attachment #70266|0                           |1
        is obsolete|                            |

--- Comment #169 from Kyle M Hall <kyle at bywatersolutions.com> ---
Created attachment 71755
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=71755&action=edit
Bug 12598: New misc/import_borrowers.pl command line tool

Test Plan:
1) Apply this patch
2) Test importing patrons from command line,
   options are availble with --help.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>

Tested with minimal csv
(cardnumber,surname,firstname,categorycode,branchcode,password,userid)

Overwrite does not change category or branch.
Patrons are loaded, userid & password works

Updated license to GPLv3
No other koha-qa errors.

Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Tidy import_borrowers.pl

Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Move importing code to a subroutine

Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Update command line script to use patron import subroutine

Signed-off-by: Benjamin Rokseth <benjamin.rokseth at kul.oslo.kommune.no>

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 [QA Followup]

* Fix copyright on import_borrowers.pl
* Changes -c --csv to -f --file
* Adds -c --confirm option
* Renames misc/import_borrowers.pl to misc/import_patrons.pl
* Restore userid matchpoint option

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Fix merge to master. Backport 3 updates from latest
import_borrowers.pl

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Started regression tests. Fix missing C4::Members::Attributes
package

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - More refactoring and regression tests in Koha::Patrons::Import

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Creating objects in misc/import_patrons.pl and
tools/import_borrowers.pl

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Refactoring Koha::Patrons::Import includes bug fixed for critical
date types and header column parsing

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598 - Rebase + backport of 16426 plus fixing 16426

Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

Bug 12598: catch warnings raised by import_patrons in tests

Signed-off-by: Colin Campbell <colin.campbell at ptfs-europe.com>

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list