Closed (fixed)
Project:
Geocoder
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
19 Sep 2011 at 08:45 UTC
Updated:
11 Mar 2012 at 14:00 UTC
Jump to comment: Most recent file
This is actually a geocode issue, but unsure how to patch the git project there. (https://github.com/fillerwriter/geocode)
On an English only install, I found that geocode was perfectly coding address information supplied as a series of fields.
Once I installed another language, setting it as default but not allowing the content type to be translatable, I found no coding was being performed.
Tracked this to the geocode module, which sets the language based on the form_state variable. The fields, however, have language "und".
Would be interested in a more elegant version of the patch, and sorry for putting it on the wrong issue queue!
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | geocode-language-1283630-5.patch | 1.3 KB | gumanist |
| geocode-language-001.patch | 693 bytes | MichaelP |
Comments
Comment #1
michaelfavia commentedSubscribe.
Comment #2
michaelfavia commentedOk tested and confirmed the issue. The suggested fix/patch works but id like to know why we dont just ALWAYS pull the language from the $field key or even just use 'und' constant?
I have enabled both content translation AND entity translation but my $field associative key is ALWAYS 'und' regardless of the active locale and the content translation state of the node.. Anyone mind shedding a little light on what im missing here and why all this language logic is needed?
Comment #3
Letharion commentedThis patch seemingly has already been applied to the code?
However, that doesn't fix the issue in my case. Because source field has a language, while the geocoder field has LANGUAGE_NONE. Why that is I have yet to figure out. This however breaks the modules functionality in just the same way as above.
Unless I'm hitting a core bug here, this issue is not fully resolved yet.
Comment #4
Letharion commentedFurther research indicates that
1) Not all fields need to be translatable, which causes my problem.
2) Old versions of core created _all_ fields as translatable, while newer core does not. See #1164852: Inconsistencies in field language handling.
3) The version of this issue that I've hit, is only likely to occur when you have an old core install, and continue creating new fields after upgrading core. But clearly it's possible to have some fields translatable and some not, this is currently not properly supported.
Since "split language fields" this is likely a low priority issue, my suggestion is
1) Add "All fields on an entity must be in the same language" note to a KNOWN ISSUES.txt file
2) Close this issue fixed
3) Open a new feature request for "proper" field language handling.
Comment #5
gumanist commentedGot an issue with different language fields. Translation done using Entity translation module.
And there are three points I has when did this patch:
1. Translation form in entity_translation module has no 'language' value
2. Submission field items in submission form always have language
3. There is the only one language on field item in form
It is possible that I'm wrong with the second point, but seems not.
This patch works for me for both edit and translation forms.
Comment #6
phayes commentedI *think* this is fixed in the latest refactoring. Can you review and report back?
Comment #7
phayes commentedOkay, I'm sure enough this has been fixed to actually mark it as such. Although if it turns out this is not the case of please re-open.