If the field widgets are deeper in the form structure and not at the root of the form, the geocoding fails, manily because it assumes that the geocoder_field is at the root of the form, but I think should only assume that the geocoder_field is in the same container as the geofield widget.

CommentFileSizeAuthor
#7 geofield-handle_nested_form_structures-1991770-7.patch2.52 KBjasonawant
#6 geofield-1991770-6-nested-form-structures.patch2.49 KBsolotandem
#4 geofield-handle_nested_form_structures-1991770-3.patch2.56 KBryan.ryan
#2 geofield-handle_nested_form_structures-1991770-2.patch2.34 KBGaborTorok

Comments

GaborTorok’s picture

GaborTorok commented
Title: Respect #parents for geocoder_field » Geocode from another field doesn't handle nested form structures

Additionally openlayers_widget_geocode() uses #parents instead of #array_parents for retrieving elements from the form structure, but #parents defines the path in form_state['values'] and #array_parents defines the path in the form structure.

GaborTorok’s picture

GaborTorok commented
Status: Active » Needs review
StatusFileSize
new geofield-handle_nested_form_structures-1991770-2.patch2.34 KB

This patch solves the problems I described and now the geocoding works with my form that has the geofield deep in the form structure with #parents != #array_parents.

GaborTorok’s picture

GaborTorok commented
Issue summary: View changes

Made the description more accurate

ryan.ryan’s picture

ryan.ryan commented
Version: 7.x-2.x-dev » 7.x-2.3
Issue summary: View changes

Rerolling this patch against the latest stable version of geofield.

ryan.ryan’s picture

ryan.ryan commented
StatusFileSize
new geofield-handle_nested_form_structures-1991770-3.patch2.56 KB

Adding the patch :)

solotandem’s picture

solotandem commented

@ryanissamson, I came up with essentially the same patch (before finding this issue..lol), but did not include the statements that set $field_value[LANGUAGE_NONE] instead of $field_value[$langcode] only to be followed up with this:

$field_value[$langcode] = $field_value[LANGUAGE_NONE];

Is there some reason you suggest these extra statements?

Otherwise, the patch is RTBC.

solotandem’s picture

solotandem commented
Assigned: GaborTorok » Unassigned
StatusFileSize
new geofield-1991770-6-nested-form-structures.patch2.49 KB

Attached patch removes the extraneous statements from previous patch.

jasonawant’s picture

jasonawant
he/him
Primary language English
Location New Orleans, USA
commented
StatusFileSize
new geofield-handle_nested_form_structures-1991770-7.patch2.52 KB

Hi,

The additional step sets the field value using the form elements language key, in case the field has been translated. I've removed this line $field_value[$langcode] = $field_value[LANGUAGE_NONE];, which unsets the returned value when using this on the node add form. At this point, the form element language is undefined.

Despite lines 305 and 306 in geocoder.widget.inc, the field can be translated.

// Set the values - geofields do not support languages
return array(LANGUAGE_NONE => $values);

New patch attached. Jason