From 817e99a05d81cf4d8cf57a6cad60fe4bb0aa80d0 Mon Sep 17 00:00:00 2001
From: DarkPhoenix <phoenix@mail.ru>
Date: Sat, 10 Aug 2019 02:42:19 +0300
Subject: [PATCH] Change the way we close character editor

---
 gui/characterEditor.py | 29 ++++++++++-------------------
 1 file changed, 10 insertions(+), 19 deletions(-)

diff --git a/gui/characterEditor.py b/gui/characterEditor.py
index 7b272d97f..0f1f8b10c 100644
--- a/gui/characterEditor.py
+++ b/gui/characterEditor.py
@@ -209,7 +209,7 @@ class CharacterEditor(wx.Frame):
 
         self.Centre(wx.BOTH)
 
-        self.Bind(wx.EVT_CLOSE, self.closeEvent)
+        self.Bind(wx.EVT_CLOSE, self.OnClose)
         self.Bind(wx.EVT_CHAR_HOOK, self.kbEvent)
         self.Bind(GE.CHAR_LIST_UPDATED, self.refreshCharacterList)
         self.entityEditor.Bind(wx.EVT_CHOICE, self.charChanged)
@@ -235,9 +235,7 @@ class CharacterEditor(wx.Frame):
             event.Skip()
 
     def editingFinished(self, event):
-        # del self.disableWin
-        wx.PostEvent(self.mainFrame, GE.CharListUpdated())
-        self.Destroy()
+        self.Close()
 
     def saveChar(self, event):
         sChr = Character.getInstance()
@@ -260,16 +258,18 @@ class CharacterEditor(wx.Frame):
         keycode = event.GetKeyCode()
         mstate = wx.GetMouseState()
         if keycode == wx.WXK_ESCAPE and mstate.GetModifiers() == wx.MOD_NONE:
-            self.closeWindow()
+            self.Close()
             return
         event.Skip()
 
-    def closeEvent(self, event):
-        self.closeWindow()
-
-    def closeWindow(self):
+    def OnClose(self, event):
         wx.PostEvent(self.mainFrame, GE.CharListUpdated())
-        self.Destroy()
+        sFit = Fit.getInstance()
+        fitID = self.mainFrame.getActiveFit()
+        if fitID is not None:
+            sFit.clearFit(fitID)
+            wx.PostEvent(self.mainFrame, GE.FitChanged(fitIDs=(fitID,)))
+        event.Skip()
 
     def restrict(self):
         self.entityEditor.btnRename.Enable(False)
@@ -291,15 +291,6 @@ class CharacterEditor(wx.Frame):
         if event is not None:
             event.Skip()
 
-    def Destroy(self):
-        sFit = Fit.getInstance()
-        fitID = self.mainFrame.getActiveFit()
-        if fitID is not None:
-            sFit.clearFit(fitID)
-            wx.PostEvent(self.mainFrame, GE.FitChanged(fitIDs=(fitID,)))
-
-        wx.Frame.Destroy(self)
-
     @staticmethod
     def SaveCharacterAs(parent, charID):
         sChar = Character.getInstance()