Closed (outdated)
Project:
Location
Version:
6.x-3.1
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Mar 2011 at 02:08 UTC
Updated:
20 Jul 2016 at 11:47 UTC
Jump to comment: Most recent
In the User Location configuration configuration (admin/user/settings Collection settings) I set City and Country fields to be 'Required', but they are not marked as Required on the collection form, either when Edit user profile or collect during registration.
Comments
Comment #1
n00b0101 commentedExperienced the same thing... Changing
'#required' => ($a4 == 2),to
'#required' => ($a4 = 2),in location.module (function location_locationapi) seemed to work for me...
Comment #2
alexbk66- commentedYeah, that's not a fix, it's a hack.
So I guess the bug is that $a4 doesn't get passed to location_locationapi()?
HobbyBlob.com
Comment #3
tomdavidson commentedI have a different experience with the 'Required' setting being ignored. I have a large cck form that uses
CCK 6.x-2.9, Multistep 6.x-1.5, Clientside Validation 6.x-1.x-dev, and a few other contrib cck fields such as Date and Field Group.
One Location Field works as expected with the 'additional' field being set to 'allow' - the user is not required to enter data for the additional field and validation works fine.
Two other Location Fields do not work correctly with the 'additional' field set to 'allow' - the user is required to enter data for the additional field and validation fails for the entire Location:
I have deleted these fields and recreated but do not know where else to troubleshoot.
Comment #4
Loikkk commented@tomdavidson :
I have found how to go over this. Just Delete the field, turn off the location module and delete the location module and install it again.
I don't know why but everything work fine after this.
Give me a feedback
Comment #5
kourosh.a commentedany possible solution/fix yet?
Comment #6
gmario commentedIn case someone else needs this info.
You have to set as required the node's field (otherwise the call to location_normalize_settings function set all location fields as optional)
location 7.x-3.6
hope this help
a si biri
gmario
Comment #7
legolasboClosing old D6 issues as D6 is end of life