Modifying a vocabulary drop terms language settings.

I use Drupal 4.6.1 with PHP 4.3.11 and the latest i18n module (4.6.0, 25/04/2005 - 02:15).

To reproduce this issue:

1. Create a vocabulary without any language (vocabulary language: none)
2. Create some terms with various languages.
3. Link these terms as translations of each other.
4. Modify the vocabulary (you do not need to change anything but the vocabulary language should still be set on none).

=> Translation relationships and terms language settings vanish.

It seems that terms language settings are not independant of the vocabulary language setting even when it is set to none.

CommentFileSizeAuthor
#3 i18n.inc_0.patch464 bytesPierreM-1

Comments

PierreM-1’s picture

I will add that if I set the language of every term to its previous value, translation relationships between terms reappear.

So I believe they are not lost, only term languages are reset to "none".

jose reyero’s picture

Status: Active » Closed (won't fix)

If you read the text below the vocabulary language field, it says:

"This language will be set for all terms in this vocabulary"

This is the intended behaviour. You can have language for vocabulary or for terms but not both and I think this makes sense.

PierreM-1’s picture

Assigned: Unassigned » PierreM-1
Status: Closed (won't fix) » Needs review
StatusFileSize
new464 bytes

If you read the text below the vocabulary language field, it says:

"This language will be set for all terms in this vocabulary"

This is the intended behaviour. You can have language for vocabulary or for terms but not both and I think this makes sense.

I agree Jose, if you have set specific term languages and if you change your mind and set a global vocabulary language then term languages should be overwritten (as they inherit the language from the vocabulary), but if the vocabulary language is still set to none (= "you can set languages freely for terms"), there is no reason that term languages are changed.

I think this is a confusing behavior.

I suggest a patch here, it is very simple (one small addition to a test).

Please, tell me what you think :)

Thank you.

jose reyero’s picture

Status: Needs review » Reviewed & tested by the community

Yes, it makes sense and the patch looks ok

jose reyero’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)