Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#2408 closed defect (fixed)

[patch] trans_null doesn't have get_language_from_request

Reported by: James Bennett Owned by: hugo
Component: Translations Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

There are situations where one might want to have the USE_I18N setting False, but still take advantage of LocaleMiddleware to populate the Content-Language header of the response. Unfortunately, LocaleMiddleware wants to do get_language_from_request to figure out which translation to use, and trans_null in django.utils.translation doesn't provide a method by that name.

Adding a 'dummy' method which just spits back the default language code of the Django installation would fix this.

Change History (3)

by James Bennett, 19 years ago

Attachment: trans_null.diff​ added

Patch to fix this

comment:1 by James Bennett, 19 years ago

Summary: trans_null doesn't have get_language_from_request → [patch] trans_null doesn't have get_language_from_request

comment:2 by Adrian Holovaty, 19 years ago

Resolution: → fixed
Status: new → closed

(In [3458]) Fixed #2408 -- Added get_language_from_request() to trans_null.py. Thanks, ubernostrum

Note: See TracTickets for help on using tickets.
Back to Top