[Koha-bugs] [Bug 15840] Fix 'import borrowers' tool to not die when importing a file with blank 'userid' values

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

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

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jonathan.druart at bugs.koha-c
                   |                            |ommunity.org
           See Also|                            |https://bugs.koha-community
                   |                            |.org/bugzilla3/show_bug.cgi
                   |                            |?id=15842

--- Comment #5 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
I don't reproduce, because the import patron tool is broken at the moment, by
bug 15842.
But looking at the code, I don't understand how it could happen, there is:

 733     $data{'userid'} = Generate_Userid( $data{'borrowernumber'},
$data{'firstname'}, $data{'surname'} )
 734       if ( $data{'userid'} eq '' || !Check_Userid( $data{'userid'} ) );

in C4::Members::AddMember which should avoid that.

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