bpo-18319: gettext() can retrieve a message even if a plural form exists

[gbassiere]

Let's say you have this in your code:

gettext('egg') 

and elsewhere this :

ngettext('egg', 'eggs', n) 

GNU gettext tools extracts these as the same unique message and sensibly creates a pluralized entry in the PO file.

In the Python gettext module, when the MO file is parsed, a singular 'egg' will be stored as:

catalog['egg'] 

whereas a pluralized 'egg' is stored as:

catalog[('egg', 0)] catalog[('eggs', 1)] 

As a consequence, gettext('egg') fail to retrieve translations and you have to resort to ngettext('egg', 'eggs', 1) which is odd.

My suggestion is simply to modify gettext() so that is looks for catalog[('egg', 0)] if catalog['egg'] didn't match.

https://bugs.python.org/issue18319

[the-knights-who-say-ni]

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@gbassiere

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

[remilapeyre]

Suggested change

	tmsg=self._catalog[(message, self.plural(1))]
	returnself._catalog[(message, self.plural(1))]

[taleinat]

I suggest simply adding the following before the existing if tmsg is missing: ... block:

iftmsgismissing: tmsg=self._catalog.get((message, self.plural(1)), missing)

[ThiefMaster]

Just noticed this annoying bug as well (via babel). It's impossible to work around without starting to mess around with message contexts (and I should NOT need to add garbage to my code (and pot files) because of this), so it would be great if this fix could get merged..

[akulakov]

Should this logic be also added to pgettext() ?

[miss-islington]

Thanks @gbassiere for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

[miss-islington]

Thanks @gbassiere for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

[bedevere-bot]

GH-107107 is a backport of this pull request to the 3.11 branch.

[bedevere-bot]

GH-107108 is a backport of this pull request to the 3.12 branch.