From: Ian Walls <ian.walls@bywatersolutions.com>
Date: Wed, 20 Jul 2011 17:11:12 +0000 (-0400)
Subject: Bug 6521 Follow up: previous patch breaks autoMemberNum functionality
X-Git-Url: http://git.rot13.org/?a=commitdiff_plain;h=d119fbd91ceec78eb3a9caad209986cb4deeb4c2;p=koha.git

Bug 6521 Follow up: previous patch breaks autoMemberNum functionality

The previous patch broke the autoMemberNum feature by adding a line to fixup_cardnumber
that returned 'undef' for cardnumber in all usages of the subroutine (limited to members/memberentry.pl
and tools/import_borrowers.pl.  This prevented the automatic number from being calculated in all cases.

This patch removes the offending line.

Signed-off-by: Ian Walls <ian.walls@bywatersolutions.com>
Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
---

diff --git a/C4/Members.pm b/C4/Members.pm
index c6c6985129..497073596e 100644
--- a/C4/Members.pm
+++ b/C4/Members.pm
@@ -900,7 +900,6 @@ sub fixup_cardnumber ($) {
 
     #     if ($cardnumber !~ /\S/ && $autonumber_members) {
     ($autonumber_members) or return $cardnumber;
-    defined($cardnumber) or return $cardnumber;
     my $checkdigit = C4::Context->preference('checkdigit');
     my $dbh = C4::Context->dbh;
     if ( $checkdigit and $checkdigit eq 'katipo' ) {