[Pkg-bazaar-commits] ./bzr-gtk/unstable r393: Merged fix of revision tagging.

[Daniel Schierbeck]

------------------------------------------------------------
revno: 393
committer: Daniel Schierbeck <daniel.schierbeck at gmail.com>
branch nick: menubar
timestamp: Fri 2007-11-09 19:21:13 +0100
message:
  Merged fix of revision tagging.
modified:
  viz/branchwin.py
    ------------------------------------------------------------
    revno: 392.1.1
    committer: Daniel Schierbeck <daniel.schierbeck at gmail.com>
    branch nick: viz-tags
    timestamp: Fri 2007-11-09 19:20:24 +0100
    message:
      Made it possible to tag revisions again.
    modified:
      viz/branchwin.py
-------------- next part --------------
=== modified file 'viz/branchwin.py'
--- a/viz/branchwin.py	2007-11-09 17:20:07 +0000
+++ b/viz/branchwin.py	2007-11-09 18:20:24 +0000
@@ -395,19 +395,26 @@
         gobject.idle_add(self.set_revision, revision.revision_id)
 
     def _tag_revision_cb(self, w):
-        dialog = AddTagDialog(self.branch.repository, self.treeview.get_revision().revision_id, self.branch)
-        response = dialog.run()
-        if response != gtk.RESPONSE_NONE:
-            dialog.hide()
-        
-            if response == gtk.RESPONSE_OK:
-                try:
-                    self.branch.lock_write()
-                    self.branch.tags.set_tag(dialog.tagname, dialog._revid)
-                finally:
-                    self.branch.unlock()
+        try:
+            self.treeview.set_sensitive(False)
+            self.branch.unlock()
+            dialog = AddTagDialog(self.branch.repository, self.treeview.get_revision().revision_id, self.branch)
+            response = dialog.run()
+            if response != gtk.RESPONSE_NONE:
+                dialog.hide()
             
-            dialog.destroy()
+                if response == gtk.RESPONSE_OK:
+                    try:
+                        self.branch.lock_write()
+                        self.branch.tags.set_tag(dialog.tagname, dialog._revid)
+                    finally:
+                        self.branch.unlock()
+                
+                dialog.destroy()
+
+        finally:
+            self.branch.lock_read()
+            self.treeview.set_sensitive(True)
 
     def _col_visibility_changed(self, col, property):
         self.config.set_user_option(property + '-column-visible', col.get_active())