Bug 5433

Summary: Separate state field for cities
Product: Koha Reporter: Owen Leonard <oleonard>
Component: System AdministrationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: enhancement    
Priority: PATCH-Sent (DO NOT USE) CC: chris, cmurdock, gmcharlt, magnus, nengard, paul.poulain
Version: 3.4   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/admin/cities.pl
Change sponsored?: --- Patch complexity: ---
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed fix
proposed patch (passed QA)
proposed patch follow-up (passed QA)
proposed patch follow-up 2 version bumping (passed QA)

Description Owen Leonard 2010-11-24 16:04:05 UTC
If the proposed changes for Bug 5422 are accepted, the same change should be
made to the way cities are stored. The JavaScript in the template for members/memberentry.pl for automatically filling the city and zip fields will have to be modified to do the state as well.
Comment 1 Cindy Murdock Ames 2011-06-07 15:50:16 UTC
It looks like the proposed changes for 5422 have been accepted--in 3.4 there is now a State field in the Add/Edit Patron form.  However, the JavaScript in the form still puts the value for the State in the City field.
Comment 2 Owen Leonard 2011-07-22 15:15:00 UTC
Currently if you add a city, you must also add a zip code. There is JavaScript validation which asks you to fill in both. Should state also be required? I would answer yes for US libraries. How about elsewhere?
Comment 3 Magnus Enger 2011-07-22 15:46:41 UTC
"No" for Norwegian libraries. We only use city and zip, not state.
Comment 4 Owen Leonard 2011-07-23 13:57:57 UTC Comment hidden (obsolete)
Comment 5 Paul Poulain 2011-08-03 14:45:02 UTC
Created attachment 4825 [details] [review]
proposed patch (passed QA)
Comment 6 Paul Poulain 2011-08-03 14:46:49 UTC
Created attachment 4826 [details] [review]
proposed patch follow-up (passed QA)

QA comment

The patch did not include the kohastructure.sql to reflect the updatedatabase. Fixing this mistake

Otherwise, no specific comment: easy to test, cleanly written, do what it was supposed to do, no side effect detected
Comment 7 Paul Poulain 2011-08-03 14:47:27 UTC
Created attachment 4827 [details] [review]
proposed patch follow-up 2 version bumping (passed QA)
Comment 8 Chris Cormack 2011-08-04 08:59:44 UTC
Pushed, please test