Fix misuse of format_plural() in multiple places [#103784]

I was just translating content_types.inc strings and found this rather "interesting" use of format_plural(). Using this function to translate 'is'/'are', 'node'/'nodes', and 'it'/'they' serves no purpose for translators. It does not allow for translation of these strings to quite a few languages, which have different rules for wording and placement then English, and it litters the string space with these single word translations without no use.

Remember that only complete strings can be format_plural()-ed, single words are a big no-no. Here is a patch to fix this. Since this string needs one more replacement, I needed to code strtr() into the patch, since there is no other API function to this. But this was simple enough, so I set it to RTBC.

Comment	File	Size	Author
#1	Drupal.fix.misuse.of.format_plural.is.system.module.patch.txt	1008 bytes	Gábor Hojtsy
	Drupal.fix.content_types.t.problem.patch.txt	1.45 KB	Gábor Hojtsy

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 17 December 2006 at 16:34

Title:

Fix serious t() issue in content_types.inc

» Fix misuse of format_plural() in multiple places

File	Size
Drupal.fix.misuse.of.format_plural.is.system.module.patch.txt	1008 bytes

Here is another patch which fixes a very similar error in system.module, so I renamed the issue to be more general. Misuse of format_plural() like this means no good for translators. Let me know if you are interested in a combined patch.