In geocode.module, line 64 there is:

'field types' => array('postal', 'textfield'),

which I believe should be:

'field types' => array('postal', 'text'),

As far as I can see, CCK gives textfields a "text" type, rather than "textfield". In the original code, nothing ends up matching the required type, meaning nothing is put into the settings $form to allow you to choose the CCK text field(s) you want to geocode.

Jeff

Comments

becw’s picture

becw commented

I noticed this fix as well. +1

manski2’s picture

manski2 commented

this helped me too +1

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented

Can't roll a patch right now, but can confirm this as well.

allie micka’s picture

allie micka
she/her
commented
Status: Active » Fixed

This was fixed awhile ago and is available in the CVS version. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented

Created new issue:
#557096: Geocode 6.x-dev release not updated

gagarine’s picture

gagarine commented
Title: text or textfield type? » CCK textfield is now text
Status: Closed (fixed) » Active

Alpha2 became after your commit but it's still not fixed. I just take a look in the CVS and it sound like not fixed.

EDIT: sorry apparently the bug is NOT on Alpha2 (why I do with my files?) but I use drush to get the dev version and that's don't work certainly because #557096: Geocode 6.x-dev release not updated

gagarine’s picture

gagarine commented
Status: Active » Closed (fixed)