diff --git a/controller/fit.py b/controller/fit.py index a2fb55f59..0b8c3691c 100644 --- a/controller/fit.py +++ b/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) diff --git a/gui/builtinViewColumns/display.py b/gui/builtinViewColumns/display.py index 87a89b0a1..b2c14f07a 100644 --- a/gui/builtinViewColumns/display.py +++ b/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): diff --git a/gui/fittingView.py b/gui/fittingView.py index 363222641..5d77d4db9 100644 --- a/gui/fittingView.py +++ b/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)