Provinces not getting saved to database [#79015]

After updating to the latest CVS version of location.module, I haven't been able to save the Province selection that I've made, even though every other value is saved properly. I looked in my MySQL database and the province simply isn't being updated. I looked through the most recent commits and couldn't find anything that would seem to affect this.

Just to note, the verification step seems to be working because if I try to use an American state with another country, it displays the proper error message. Nothing is being saved to the database however.

Can anyone else verify this problem in the current CVS? Any suggestions? Thanks!

Comment	File	Size	Author
#2	location.module_6.diff	810 bytes	werdnativ

Comments

Comment #1

solutiondrop commented 19 August 2006 at 07:30

David,
Yes, I can verify this. I was using 4.7.3 with the most recent Location module. For what it's worth I went back to version 1.62.2.1 (4.7), as that was the last known version to work on another site I'm devloping. The Province did save correctly using the older version, before the geocoding addition.
DS

Comment #2

werdnativ commented 22 August 2006 at 19:41

Status:

Active

» Needs review

Status	File	Size
new	location.module_6.diff	810 bytes

I've tried latest CVS (v 1.66 2006/08/12 13:23:20 ankur)
and the released 4.7 (v 1.62.2.4 2006/08/12 13:22:05 ankur)

Both of these commits have the bug. The fix is to remove a call to $node->location = location_form2api($node->location); on line 798 of location.module. A patch against the CVS version listed above is attached. (I hope this is the right diff format... I'm using diff -u)