← Back to team overview

gtg team mailing list archive

[Merge] lp:~jonathan-barnoud/gtg/Bug_615519_in_quick-add into lp:gtg

 

Pititjo has proposed merging lp:~jonathan-barnoud/gtg/Bug_615519_in_quick-add into lp:gtg.

Requested reviews:
  Gtg developers (gtg)
Related bugs:
  #615519 Accents issue in @tag syntax in quickadd entry
  https://bugs.launchpad.net/bugs/615519


Fix bug #615519 about accents in @tag syntax in the quick-add entry.
-- 
https://code.launchpad.net/~jonathan-barnoud/gtg/Bug_615519_in_quick-add/+merge/32231
Your team Gtg developers is requested to review the proposed merge of lp:~jonathan-barnoud/gtg/Bug_615519_in_quick-add into lp:gtg.
=== modified file 'GTG/gtk/browser/browser.py'
--- GTG/gtk/browser/browser.py	2010-08-04 00:30:22 +0000
+++ GTG/gtk/browser/browser.py	2010-08-10 17:35:05 +0000
@@ -910,7 +910,7 @@
             self.priv["collapsed_tids"].append(tid)
 
     def on_quickadd_activate(self, widget):
-        text = self.quickadd_entry.get_text()
+        text = unicode(self.quickadd_entry.get_text())
         due_date = no_date
         defer_date = no_date
         if text:
@@ -920,7 +920,7 @@
             # a non-capturing group, so it must not be returned
             # to findall. http://www.amk.ca/python/howto/regex/regex.html
             # ~~~~Invernizzi
-            for match in re.findall(r'(?:^|[\s])(@\w+)', text):
+            for match in re.findall(r'(?:^|[\s])(@\w+)', text, re.UNICODE):
                 tags.append(GTG.core.tagstore.Tag(match, self.req))
                 # Remove the @
                 #text =text.replace(match,match[1:],1)


Follow ups