Anonymous avatar Anonymous committed 730abfc

Fixes bug of having default argument for TimeRE's __init__ that caused the
LocaleTime instance to only be created once and thus not be recreated when
the locale changed.

Comments (0)

Files changed (1)

 class TimeRE(dict):
     """Handle conversion from format directives to regexes."""
 
-    def __init__(self, locale_time=LocaleTime()):
+    def __init__(self, locale_time=None):
         """Init inst with non-locale regexes and store LocaleTime object."""
         #XXX: Does 'Y' need to worry about having less or more than 4 digits?
         base = super(TimeRE, self)
             'y': r"(?P<y>\d\d)",
             'Y': r"(?P<Y>\d\d\d\d)"})
         base.__setitem__('W', base.__getitem__('U'))
-        self.locale_time = locale_time
+        if locale_time:
+            self.locale_time = locale_time
+        else:
+            self.locale_time = LocaleTime()
 
     def __getitem__(self, fetch):
         """Try to fetch regex; if it does not exist, construct it."""
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.