Closed (fixed)
Project:
Location
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Mar 2008 at 02:00 UTC
Updated:
18 Feb 2012 at 00:17 UTC
1) Submit address with "NOT LISTED" selected for province.
2) Return to form and look at the options list
"NOT LISTED" is not the default value.
When the user has previously selected "NOT LISTED", this default value should be displayed.
The function location_api2form() deletes $translated['province'] by not having an else statement for when $location['province'] == 'xx'.
In HEAD:
function location_api2form($location = array()) {
$translated = array();
foreach ($location as $key => $value) {
if ($key == 'province') {
if (strlen($value) && isset($location['country']) && strlen($location['country']) && $location['province'] != 'xx' && $location['country'] != 'xx') {
$translated['province'] = $location['country'] .'-'. $value;
}
}
else {
$translated[$key] = $value;
}
}
return $translated;
}
Comments
Comment #1
bdragon commentedErr, what's still calling that in HEAD?
I'm trying to remove that function....
Comment #2
raspberryman commentedHi bdragon - I just verified the function existed in HEAD, I did not test HEAD. Pardon my jump to conclusions :) My bad.
Comment #3
bdragon commentedCommented the functions out. ;)
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #5
cerup commentedWhere is this function? It still seems to be an issue where 'not listed' isn't saved for state/province field.
Comment #6
yesct commented@cerup please add more details about what the current problem is.
Please check to see if this is still a problem in the new release. If it is, reply with a new summary and set the status back to active.
Comment #7
ankur commentedClosing due to lack of follow up.