Edgewall Software
Modify

Opened 15 years ago

Closed 15 years ago

#764 closed defect (fixed)

mod_python does not call locale.setlocale(locale.LC_ALL, '')

Reported by: anonymous Owned by: Christopher Lenz
Priority: normal Milestone: 0.8
Component: web frontend/mod_python Version: devel
Severity: normal Keywords: locale
Cc: Branch:
Release Notes:
API Changes:

Description

I've included a patch (it's only two lines, but this is the first time I've ever submitted a patch, so I'm all keen). I've no idea if this violates the thread safety that apparently needs to be followed with this call…

It's a patch against revision 921.


Index: trac/ModPythonHandler.py
===================================================================
--- trac/ModPythonHandler.py    (revision 921)
+++ trac/ModPythonHandler.py    (working copy)
@@ -19,6 +19,10 @@
 #
 # Author: Christopher Lenz <cmlenz@gmx.de>
 
+# Not sure if this is safe, but...
+import locale
+locale.setlocale(locale.LC_ALL, '')
+
 import os
 import re, threading
 import auth, core, Environment, Href

Attachments (0)

Change History (2)

comment:1 by Christopher Lenz, 15 years ago

Keywords: locale added
Milestone: 0.8
Status: newassigned

comment:2 by Christopher Lenz, 15 years ago

Resolution: fixed
Status: assignedclosed

Should be fixed in [924].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christopher Lenz.
The resolution will be deleted. Next status will be 'reopened'.
to as closed The owner will be changed from Christopher Lenz to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.