Don't trigger fitChanged if nothing changed, only call SetItem on text

changes

controller/fit.py

@@ -89,7 +89,7 @@ class Fit(object):
 
             return numSlots != len(fit.modules)
         else:
-            return False
+            return None
 
     def removeModule(self, fitID, position):
         fit = eos.db.getFit(fitID)

gui/builtinViewColumns/display.py

@@ -114,13 +114,13 @@ class Display(wx.ListCtrl):
                 oldImageId = colItem.GetImage()
                 newText = col.getText(st)
                 newImageId = col.getImageId(st)
-                if oldText != newText:
-                    colItem.SetText(newText)
 
-                if oldImageId != newImageId:
-                    colItem.SetImage(newImageId)
+                colItem.SetText(newText)
+                colItem.SetImage(newImageId)
+
+                if oldText != newText or oldImageId != newImageId:
+                    self.SetItem(colItem)
 
-                self.SetItem(colItem)
                 self.SetItemData(item, id)
 
         for i, col in enumerate(self.activeColumns):

gui/fittingView.py

@@ -59,8 +59,10 @@ class FittingView(d.Display):
         if fitID != None:
             cFit = controller.Fit.getInstance()
             populate = cFit.appendModule(fitID, itemID)
-            if populate: self.slotsChanged()
-            wx.PostEvent(self.mainFrame, FitChanged(fitID=fitID))
+            if populate:
+                self.slotsChanged()
+            if populate is not None:
+                wx.PostEvent(self.mainFrame, FitChanged(fitID=fitID))
 
     def removeItem(self, event):
         row, _ = self.HitTest(event.Position)