GTG

Merge lp:~antonio-roquentin-deactivatedaccount/gtg/bugfix-1013869 into lp:~gtg/gtg/old-trunk

Proposed by Roquentin
Status: Merged
Merged at revision: 1220
Proposed branch: lp:~antonio-roquentin-deactivatedaccount/gtg/bugfix-1013869
Merge into: lp:~gtg/gtg/old-trunk
Diff against target: 17 lines (+2/-0)
1 file modified
GTG/core/datastore.py (+2/-0)
To merge this branch: bzr merge lp:~antonio-roquentin-deactivatedaccount/gtg/bugfix-1013869
Reviewer Review Type Date Requested Status
Bertrand Rousseau (community) code, run Approve
Review via email: mp+116956@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Bertrand Rousseau (bertrand-rousseau) wrote :

Seems ok for me. Approve

review: Approve (code, run)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'GTG/core/datastore.py'
--- GTG/core/datastore.py 2012-07-23 12:04:01 +0000
+++ GTG/core/datastore.py 2012-07-26 20:28:21 +0000
@@ -152,12 +152,14 @@
152 self._add_new_tag(name, tag, search_filter,152 self._add_new_tag(name, tag, search_filter,
153 parameters, parent_id = CoreConfig.SEARCH_TAG)153 parameters, parent_id = CoreConfig.SEARCH_TAG)
154 Log.debug("*** view added %s ***" % name)154 Log.debug("*** view added %s ***" % name)
155 self.save_tagtree()
155 return tag156 return tag
156157
157 def remove_tag(self, name):158 def remove_tag(self, name):
158 """ Removes a tag from the tagtree """159 """ Removes a tag from the tagtree """
159 if self.__tagstore.has_node(name):160 if self.__tagstore.has_node(name):
160 self.__tagstore.del_node(name)161 self.__tagstore.del_node(name)
162 self.save_tagtree()
161 else:163 else:
162 raise IndexError("There is no tag %s" % name)164 raise IndexError("There is no tag %s" % name)
163165

Subscribers

People subscribed via source and target branches

to status/vote changes: