i18n_taxonomy is specified in i18nviews.info as dependency. I propose we remove this dependency and check in hook_views_data_alter, hook_field_views_data_alter and hook_views_plugins which module are enabled.

CommentFileSizeAuthor
#2 remove-dependency-to-i18n_taxonomy--1257994-1-D7.patch2.68 KBwebflo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker commented

fine fine.

webflo’s picture

webflo CreditAttribution: webflo commented
FileSize
remove-dependency-to-i18n_taxonomy--1257994-1-D7.patch2.68 KB

Patch attached.

thePanz’s picture

thePanz CreditAttribution: thePanz commented

I think that the function named "i18nviews_views_data_alter__i18n_taxonomy" should be renamed deleting the double "__" in the function name:

function i18nviews_views_data_alter_i18n_taxonomy(&$data)

with only one underscore between "alter" and "i18n"

Cheers

thePanz’s picture

thePanz CreditAttribution: thePanz commented

BTW I tested the "non-taxonomy" translation features and it works fine with this patch.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker commented
Status: Needs review » Fixed

Committed and pushed. Removed the "__" name, switche to single underline as this is more common afaik.
Attributed to webflo.

Status: Fixed » Closed (fixed)

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