[Koha-bugs] [Bug 5633] import borrower, add command line option

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5633

Paul Poulain <paul.poulain at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|PATCH-Sent (DO NOT USE)     |P2
             Status|Failed QA                   |ASSIGNED
            Version|rel_3_4                     |master
         AssignedTo|alex.arnaud at biblibre.com    |matthias.meusburger at biblibr
                   |                            |e.com

--- Comment #6 from Paul Poulain <paul.poulain at biblibre.com> 2012-01-19 14:56:16 UTC ---
Matthias, could you resubmit a patch for this feature BUT, it must not be an
enhancement in the tools/ directory, as we've decided not to have scripts that
can be run from commandline and http, for security reasons.
So please do a script in misc/migration_tools

As most of the code will be common to both scripts (tools & command-line), you
should put common subs into C4/Members.pm module where it's usefull

When reading the patch, seeing many lines like this:
-            $template->param('lastoverwritten'=>$borrower{'surname'}.' /
'.$borrowernumber);
+            $template->param('lastoverwritten'=>$borrower{'surname'}.' /
'.$borrowernumber) if (!$commandline);

make me feel the misc/migration_tools/import_borrowers.pl script will have less
lines than the tools one ;-)

-- 
Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are watching all bug changes.