Issue Details (XML | Word | Printable)

Key: FLM-306
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: Borys Musielak
Reporter: Borys Musielak
Votes: 0
Watchers: 0
Operations

Clone this issue
Create sub-task
If you were logged in you would be able to see more operations.
Filmaster

Notifications are sent in English only

Created: 08/Mar/10 07:02 PM   Updated: 24/Apr/10 11:55 PM
Component/s: Notifications
Affects Version/s: 1.0.2
Fix Version/s: 1.0.3

Time Tracking:
Not Specified

Issue Links:
Blocking
 


 Description  « Hide
After introducing delayed notifications using emit_notices, the e-mails are sent in English without applying the Polish translation for filmaster.pl.

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Borys Musielak added a comment - 08/Mar/10 10:35 PM
Fixed: http://bitbucket.org/filmaster/filmaster-test/changeset/9f8e95af87ce/

Adding this setting is required now to indicate the user Profile: NOTIFICATION_LANGUAGE_MODULE = 'core.profile'

We're using the user's LANG setting to determine the primary language for the user now (and as django-notifications requires it to be called "language" instead of LANG, a small hack using property and lambda was required)

Borys Musielak added a comment - 04/Apr/10 08:28 PM

Borys Musielak added a comment - 13/Apr/10 03:24 AM
OK, I'm officially stupid now.
Fixed, yet again: http://bitbucket.org/filmaster/filmaster-test/changeset/d103dba19e20/

Borys Musielak added a comment - 15/Apr/10 01:47 AM
Added one more fix: http://bitbucket.org/filmaster/filmaster-test/changeset/4e031d60b547/ - a quick and nasty one, for links. This needs to be resolved in a nicer way later.

Borys Musielak added a comment - 24/Apr/10 11:04 PM
This need to reimplemented:
- mark pending notifications with current language in database
- launch notification sending tasks for all localizations independently
- remove the code that messes with languages while sending the notification

Borys Musielak added a comment - 24/Apr/10 11:34 PM
Implemented as described.