Members.pm - dbh->{'mysql_insertid'} still needed, despite shortcomings.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

diff --git a/C4/Members.pm b/C4/Members.pm
index 9651b28..2a075f4 100644 (file)
--- a/C4/Members.pm
+++ b/C4/Members.pm
@@ -705,7 +705,8 @@ sub AddMember {
        #       print "Executing SQL: $query\n";
     $sth->execute;
     $sth->finish;
-    # $data{'borrowernumber'} = $dbh->{'mysql_insertid'};      # unneeded w/ autoincrement ?
+    $data{'borrowernumber'} = $dbh->{'mysql_insertid'};        # unneeded w/ autoincrement ?  
+       # mysql_insertid is probably bad.  not necessarily accurate and mysql-specific at best.
     
     &logaction(C4::Context->userenv->{'number'},"MEMBERS","CREATE",$data{'borrowernumber'},"") 
         if C4::Context->preference("BorrowersLog");