[Koha-bugs] [Bug 1836] Adding patrons fails when using cites/zipcode feature

[bugzilla-daemon at pippin.metavore.com]

http://bugs.koha.org/cgi-bin/bugzilla/show_bug.cgi?id=1836


mason.james at liblime.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Adding patrons fails        |Adding patrons fails when
                   |unexpectedly, due to empty  |using cites/zipcode feature
                   |input fields                |




------- Comment #2 from mason.james at liblime.com  2008-02-14 00:00 -------
(In reply to comment #0)
> looks like a few columns in 'borrowers' are set to not allow null values 

OK , after looking at this again it seems to be a bigger weirder issue

there seems to be a mismatch btw memberentrygen.tmpl and memberentry.pl

specifically with these 3 variables: $select_city, $zipcode, $city

$zipcode and $city are not longer being used by memberentry.pl anymore?
and $select_city looks like this '777|city1'   ?

from the template...
   $("#select_city").change(function(){
               var myRegEx=new RegExp(/(.*)\|(.*)/);     
               document.form.select_city.value.match(myRegEx);
               document.form.zipcode.value=RegExp.$1;
               document.form.city.value=RegExp.$2;           


yet, in the script, the call to getzipnamecity() is always going to fail...

  if ($select_city){
    my ($borrower_city,$borrower_zipcode)=&getzipnamecity($select_city);
    $newdata{'city'}= $borrower_city;
    $newdata{'zipcode'}=$borrower_zipcode;
    }   

why ae we doing (and failing) a getzipnamecity() lookup when we already have
the city and zipcode?




------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.