hi,

the core taxonomy_autocomplete_validate simply does :

      if ($possibilities = taxonomy_term_load_multiple(array(), array('name' => trim($typed_term), 'vid' => array_keys($vocabularies)))) {
        $term = array_pop($possibilities);
      }

so it doesn't take care of term language at all.

i'll add a patch so that we have i18ntaxonomy_taxonomy_autocomplete_validate to take care of this.

CommentFileSizeAuthor
#11 i18n_taxonomy-handle_language_autocomplet_terms-2506981-10.patch3.39 KBjoseph.olstad
#5 i18n_taxonomy-handle_language_autocomplet_terms-2506981-5.patch3.39 KBizus
#3 i18ntaxonomy-handle_language_autocomplet_terms-2506981-3.patch3.42 KBizus
#1 i18ntaxonomy-handle_language_autocomplet_terms-2506981-1.patch3.42 KBizus
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

izus’s picture

izus CreditAttribution: izus commented
Status: Active » Needs review
FileSize
i18ntaxonomy-handle_language_autocomplet_terms-2506981-1.patch3.42 KB

here it is

Status: Needs review » Needs work

The last submitted patch, 1: i18ntaxonomy-handle_language_autocomplet_terms-2506981-1.patch, failed testing.

izus’s picture

izus CreditAttribution: izus commented
Status: Needs work » Needs review
FileSize
i18ntaxonomy-handle_language_autocomplet_terms-2506981-3.patch3.42 KB

Status: Needs review » Needs work

The last submitted patch, 3: i18ntaxonomy-handle_language_autocomplet_terms-2506981-3.patch, failed testing.

izus’s picture

izus CreditAttribution: izus commented
Status: Needs work » Needs review
FileSize
i18n_taxonomy-handle_language_autocomplet_terms-2506981-5.patch3.39 KB
izus’s picture

izus CreditAttribution: izus commented
Title: Autocomplete taxonomy terms should reference the terms of teh same language as the parent node » Autocomplete taxonomy terms should reference the terms of the same language as the parent node
ogggg’s picture

ogggg CreditAttribution: ogggg commented

#5 works for me

izus’s picture

izus CreditAttribution: izus commented

@ogggg can you then please set the status of the issue to "Reviewed & tested by the community", so that maintainers wil look at it
Thanks

ogggg’s picture

ogggg CreditAttribution: ogggg commented
Status: Needs review » Reviewed & tested by the community
Cyclodex’s picture

Cyclodex CreditAttribution: Cyclodex as a volunteer commented

I think this patch is basically for the case of Translate. Different terms will be allowed for each language and they can be translated.
Localize has a different issue, which I am investigating now.

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
FileSize
i18n_taxonomy-handle_language_autocomplet_terms-2506981-10.patch3.39 KB

due to testbot changes the original patch needs a reroll

rerolling original patch #5

PieterDC’s picture

PieterDC
Primary language Dutch
Location Ghent
CreditAttribution: PieterDC at Randstad Digital commented
Status: Reviewed & tested by the community » Needs work

My colleague Michel tested the patch and confirms it works.
I tested it as well.

As for the code of the patch itself: