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

Paul Poulain <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|PATCH-Sent (DO NOT USE)     |P2
             Status|Failed QA                   |ASSIGNED
            Version|rel_3_4                     |master
         AssignedTo|[email protected]    |matthias.meusburger@biblibr
                   |                            |e.com

--- Comment #6 from Paul Poulain <[email protected]> 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.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to