Merge pull request #834 from BentleyJOakes/fix_crash

Properly disconnect add person connection.
olivierkes · Feb 22, 2021 · 4e6fa11 · 4e6fa11
2 parents 3effe5c + 7e9fbf2
commit 4e6fa11
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/manuskript/mainWindow.py b/manuskript/mainWindow.py
@@ -952,8 +952,8 @@ def makeConnections(self):
         # Characters
         self.lstCharacters.setCharactersModel(self.mdlCharacter)
         self.tblPersoInfos.setModel(self.mdlCharacter)
-        self.btnAddPerso.clicked.connect(self.lstCharacters.addCharacter, F.AUC)
         try:
+            self.btnAddPerso.clicked.connect(self.lstCharacters.addCharacter, F.AUC)
             self.btnRmPerso.clicked.connect(self.lstCharacters.removeCharacter, F.AUC)
 
             self.btnPersoColor.clicked.connect(self.lstCharacters.choseCharacterColor, F.AUC)
@@ -1110,7 +1110,7 @@ def breakConnections(self):
         # Break connections for UI elements that were connected in makeConnections()
 
         # Characters
-        self.disconnectAll(self.btnAddPerso.clicked, self.mdlCharacter.addCharacter)
+        self.disconnectAll(self.btnAddPerso.clicked, self.lstCharacters.addCharacter)
         self.disconnectAll(self.btnRmPerso.clicked, self.lstCharacters.removeCharacter)
 
         self.disconnectAll(self.btnPersoColor.clicked, self.lstCharacters.choseCharacterColor)