GTG

Merge lp:~sara.ribeiro/gtg/quickadd_tag_parser into lp:~gtg/gtg/old-trunk

Proposed by Sara Ribeiro on 2014-03-06
Status: Merged
Merged at revision: 1365
Proposed branch: lp:~sara.ribeiro/gtg/quickadd_tag_parser
Merge into: lp:~gtg/gtg/old-trunk
Diff against target: 38 lines (+5/-1)
3 files modified
CHANGELOG (+1/-0)
GTG/tools/tags.py (+1/-1)
tests/tools/test_tags.py (+3/-0)
To merge this branch: bzr merge lp:~sara.ribeiro/gtg/quickadd_tag_parser
Reviewer Review Type Date Requested Status
Izidor Matušov 2014-03-06 Needs Fixing on 2014-03-13
Review via email: mp+209584@code.launchpad.net

Description of the change

Changed the parser in the quick add entry to accept "&" inside a tag

To post a comment you must log in.
Izidor Matušov (izidor) wrote :

Can you please extend the test suite in tests/tools/test_tags.py ? ( You might need to merge trunk into your branch first)

review: Needs Fixing
1361. By Sara Ribeiro on 2014-03-13

Solve merge conflict

1362. By Sara Ribeiro on 2014-03-13

Extended test suit

Sara Ribeiro (sara.ribeiro) wrote :

Ok, I've done that and resubmitted!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CHANGELOG'
--- CHANGELOG 2014-03-11 03:14:22 +0000
+++ CHANGELOG 2014-03-14 05:36:47 +0000
@@ -22,6 +22,7 @@
22 * Fix for bug #1287259 : pep8ify the codebase, by Izidor Matušov22 * Fix for bug #1287259 : pep8ify the codebase, by Izidor Matušov
23 * Rework of test-suite, by Izidor Matušov23 * Rework of test-suite, by Izidor Matušov
24 * Fix for bug #1027604: Deeper integeration of userdoc, by Sara Ribeiro24 * Fix for bug #1027604: Deeper integeration of userdoc, by Sara Ribeiro
25 * Fix for bug #1288560 : Parsing error when creating a task (via quick add entry) that contains a tag with "&" in it, by Sara Ribeiro
2526
262013-11-24 Getting Things GNOME! 0.3.1272013-11-24 Getting Things GNOME! 0.3.1
27 * Fix for bug #1024473: Have 'Show Main Window' in notification area, by Antonio Roquentin28 * Fix for bug #1024473: Have 'Show Main Window' in notification area, by Antonio Roquentin
2829
=== modified file 'GTG/tools/tags.py'
--- GTG/tools/tags.py 2013-11-23 14:40:23 +0000
+++ GTG/tools/tags.py 2014-03-14 05:36:47 +0000
@@ -22,7 +22,7 @@
2222
23def extract_tags_from_text(text):23def extract_tags_from_text(text):
24 """ Given a string, returns a list of the @tags contained in that """24 """ Given a string, returns a list of the @tags contained in that """
25 return re.findall(r'(?:^|[\s])(@[\w\/\.\-\:]*\w)', text)25 return re.findall(r'(?:^|[\s])(@[\w\/\.\-\:\&]*\w)', text)
2626
2727
28def parse_tag_list(text):28def parse_tag_list(text):
2929
=== modified file 'tests/tools/test_tags.py'
--- tests/tools/test_tags.py 2014-03-10 03:37:32 +0000
+++ tests/tools/test_tags.py 2014-03-14 05:36:47 +0000
@@ -71,6 +71,9 @@
71 def ignore_colon_at_end(self):71 def ignore_colon_at_end(self):
72 self.assertTags("@:a:b:c:", ["@:a:b:c"])72 self.assertTags("@:a:b:c:", ["@:a:b:c"])
7373
74 def test_accepts_ampersand_in_tag(self):
75 self.assertTags("@home&work", ["@home&work"])
76
7477
75class TestParseTagList(TestCase):78class TestParseTagList(TestCase):
76 """ parse_tag_list """79 """ parse_tag_list """

Subscribers

People subscribed via source and target branches

to status/vote changes: