I plan to create a little patch to make it possible to place location information into a container provided by a cck fieldgroup.
I'm not yet sure whether the fieldgroup module or the location module is the better place to do this.

Any suggestions?

Comments

ray007’s picture

*Bump*

No suggestion for me on how to proceed? Nobody else interested in this?

ray007’s picture

Status: Active » Needs review
StatusFileSize
new3.99 KB

All right, the attached patch allow one to place location-data in a cck fieldgroup.
This is of course a per content-type setting.

Comments?

ray007’s picture

StatusFileSize
new4.07 KB

Little bug, little fix - had forgotten to reset parent in moved form elements, attached patch fixes that.

d0t101101’s picture

Awesome, thank you for your hard work on this. After searching for hours to find a way to do proximity searches on my CCK based forms using the location module, this is the only solution that fits the bill.

Ideally I would like to see this implemented as CCK fields rather then fieldgroups, so I could for instance easily hide the long/lat fields and prompt only for a postal code. Hope to see this patch or derivative work applied to the next release of the location module.

Best regards,
.

d0t101101’s picture

"easily hide the long/lat fields and prompt only for a postal code"

I spoke a little too soon - these long/lat fields were only visible as Admin, after logging with a different role they are hidden.

THANKS!
.

ray007’s picture

regarding #4:

In principle I agree with you that encapsulating it in a cck field would be the cleaner solution, but that would require either some code duplication with the location module or some refactoring of the code, while my fieldgroup hack is quite un-intrusive.

Since it works for me right now I'll probably not expand the effort to create a cck field for location data unless some sponsor shows up for that ;-)

ray007’s picture

New cck versions have (among other changes) a much higher weight on the fieldgroup module, so this patch doesn't work anymore.
Since I'm not sure what else will break when I start juggling around module-weight, I have move this functionality (in even more general form) to cck and the fieldgroup module: http://drupal.org/node/157899

Testers welcome ;-)

yesct’s picture

Status: Needs review » Needs work

It sounds like it might already be something that is in the latest version, as there are cck location fields. I'll mark it as needs work for now, and since it is so old, if no-one responds in 2 weeks, I'll mark it as closed.

ankur’s picture

Status: Needs work » Closed (won't fix)

Closing as this looks to be obsolete at this point.