Closed (fixed)
Project:
Internationalization
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
11 May 2011 at 10:49 UTC
Updated:
13 May 2011 at 08:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
berdirOk, here is a first patch.
- The whole thing (adding and removing nodes to a translation set) should be working now.
- Also wrote some tests for it
- Fixed a bug in the tests base class and made it a bit more extensible (allow to provide additional admin permissions).
Comment #2
jose reyero commentedLooks great, you even added some test, thanks!
This is committed. There are a few minor issues with this page though that we can fix with next patches:
- Language neutral showing up?
- Language names for fields.
(changed issue status to active, instead of fixed, though patch is committed)
Comment #3
berdirOk, new patch to hide the language neutral row.
Comment #4
berdirUpdated patch with a better comment.
Comment #5
jose reyero commentedFixed, thanks.
Comment #6
lars toomre commentedA small nit when reviewing this issue for examples of tests... I think that the middle test below was meant to make sure that the english autocomplete suggestion was found. Am I wrong?
Comment #7
berdirActually, no. It was meant to make sure that the english autocomplete does *not* work when using the spanish autocomplete path.
Comment #8
lars toomre commentedOK... But then what is the purpose of the third test? It appears to be like the first of the three just using a different parameter.
Comment #9
berdirIt is. 1 and 2 test the same thing, just using two different languages. 2 tests that it does not work when using the wrong language.
Comment #10
lars toomre commentedThanks for the explanation... I am learning more about testing all the time and am now including an in-line comment in my own code above what here was 1), 2) and 3). It helps me remember in plain text what condition was being tested for.
Comment #11
jose reyero commentedI am seeing you get all languages for autocomplete (enabled and disabled ones) which is not consistent with other options (extended language option for nodes) and also not consistent with the list above.
See screenshot. Chinese and Portuguese are disabled languages.
Comment #12
berdirOk, disabled languages are now hidden, test has been updated to catch this.
Comment #13
berdirOk, disabled languages are now hidden, test has been updated to catch this.
Comment #14
jose reyero commentedCommitted, thanks.
It works great now!
Comment #15
jose reyero commentedKeeping our list clean.