zope.pagetemplate 3.4.1 contains obviously wrong code

[ChrisW]

Here's the chunk of code from talinterpreter.py:

    def translate(self, msgid, default=None, i18ndict=None,
                  obj=None, domain=None):
        if default is None:
            default = getattr(msgid, 'default', unicode(msgid))
        if i18ndict is None:
            i18ndict = {}
        if domain is None:
            domain = getattr(msgid, 'domain', self.i18nContext.domain)
        if obj:
            i18ndict.update(obj)
        if not self.i18nInterpolate:
            return msgid
        # TODO: We need to pass in one of context or target_language
        return self.engine.translate(msgid, self.i18nContext.domain,
                                     i18ndict, default=default)

Any domain passed to the translate call or obtained from the msgid is just plain ignored and the one from self.i18nContext is always used.

[Tres Seaver (tseaver)]

Wrong project: this is at zope.tal bug.

[Colin Watson (cjwatson)]

The zope.tal project on Launchpad has been archived at the request of the Zope developers (see https://answers.launchpad.net/launchpad/+question/683589 and https://answers.launchpad.net/launchpad/+question/685285). If this bug is still relevant, please refile it at https://github.com/zopefoundation/zope.tal.