Hi,
currently i18n taxonomy overrides the field definition from Taxonomy Term Reference fields and takes over all field hooks from taxonomy module. This architecture has some drawbacks.
Contrib does not know about this "hack" and code like $field['module'] == 'taxonomy' will fail.
Related Issues:
#1172116: Missing filter: taxonomy fields
Maybe there are some more issues in other modules ...
We could fix this issue and provide views integration for this faux taxonomy fields or refactor some code from i18n_taxonomy and switch back to original taxonomy fields. Either way i would work on this issue.
Cheers,
Florian
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 1356434-real-taxonomy-fields-3.patch | 9.33 KB | jose reyero |
| #5 | 1356434-real-taxonomy-fields-2.patch | 9.11 KB | jose reyero |
| #2 | 1356434-real-taxonomy-fields.patch | 9.25 KB | webflo |
Comments
Comment #1
webflo commentedComment #2
webflo commentedComment #3
jose reyero commentedI agree this is a much better approach, so giving a try to this patch.
(I think the main reason why we went for the current approach was not being able to handle the options callback for pre-existing fields. This seems to be fixed by the 'enable' trick now).
On a first test, getting these errors after applying the patch, still doing some more research:
Notice: Undefined index: taxonomy_term in i18n_taxonomy_field_formatter_view() (line 206 of /var/workspace/drupal7/sites/all/modules/i18n/i18n_taxonomy/i18n_taxonomy.module).
EntityMalformedException: Missing bundle property on entity of type taxonomy_term. in entity_extract_ids() (line 7409 of /var/workspace/drupal7/includes/common.inc).
Comment #4
jose reyero commentedOne more error, when disabling the module after applying this patch.
FieldUpdateForbiddenException: field_sql_storage cannot change the schema for an existing field with data. in field_sql_storage_field_update_forbid() (line 229 of /var/workspace/drupal7/modules/field/modules/field_sql_storage/field_sql_storage.module).
Comment #5
jose reyero commentedI think I've got it, see new patch.
It just adds back i18n_taxonomy_field_formatter_prepare_view()
@webflo,
It it works for you, just commit.
Comment #6
jose reyero commentedIt breaks some test (Synchronize translations) atm:
Fatal error: Call to undefined function i18n_taxonomy_field_prepare_translation() in /var/workspace/drupal7/sites/all/modules/i18n/i18n_sync/i18n_sync.module on line 317
Comment #7
jose reyero commentedFixed last issue and committed. This is the patch that got committed.