Patch #1594554: Always close a tkSimpleDialog on ok(), even

if an exception occurs.

Patch #1594554: Always close a tkSimpleDialog on ok(), even
if an exception occurs.
e350c840 · Martin v. Löwis · ce9212f0 · e350c840 · e350c840
Commit e350c840 authored Nov 18, 2006 by Martin v. Löwis
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

Lib/lib-tk/tkSimpleDialog.py Lib/lib-tk/tkSimpleDialog.py +4 -3

Misc/NEWS Misc/NEWS +3 -0

No files found.
--- a/Lib/lib-tk/tkSimpleDialog.py
+++ b/Lib/lib-tk/tkSimpleDialog.py
@@ -129,9 +129,10 @@ class Dialog(Toplevel):
        self.withdraw()
        self.update_idletasks()

-        self.apply()
-
-        self.cancel()
+        try:
+            self.apply()
+        finally:
+            self.cancel()

    def cancel(self, event=None):


--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -103,6 +103,9 @@ Extension Modules
 Library
 -------

+- Patch #1594554: Always close a tkSimpleDialog on ok(), even
+  if an exception occurs.
+
 - Patch #1538878: Don't make tkSimpleDialog dialogs transient if
  the parent window is withdrawn.