[PATCH] l10n_server_parse_po() not working properly with plurals

I also identified that the current l10n_server_parse_po() method is actually failing plurals *with context*.

I would like to add a test case update for that, but i found out that the current test cases are somewhat dependant from the potx ones. Since i need to modify the test module for strings, it would mean i need to update the potx test cases as well.

For example, i'd like to do the following:

Modify potx_test_7.module:

function potx_test_7_page() {
  t('This is a test string.');
  format_plural($count, '1 test string', '@count test strings');
  format_plural($count, '1 test string with context', '@count plural test strings with context', array(), array('context' => 'Plural context'));
  watchdog('test watchdog type', 'My watchdog message');

and then modify the sample po file from l10n_server and add:

msgctxt "Plural context"
msgid "1 test string with context"
msgid_plural "@count test strings with context"
msgstr[0] "1 test string with context - imported translation"
msgstr[1] "@count test strings with context - imported translation"

What do you think ?

I'm not very keen with having module *dependencies* regarding test cases. Should'nt we have a clean version of the test stuff for l10n_server only ?

I've decided to do the other way around, and propose new tests to the potx module to track the context + plurals issue.

See here : http://drupal.org/node/1039552

As soon as i'm getting some feedback, i'll update the test coverage for l10n_server as well.