opet

commit d4bb323943049a0ac6f5fd7dfa8d2ab04f679799
parent 4a4602aacd147d8638d8c4438209fc4c0e10eaca
Author: Erik Letson <hmagellan@hmagellan.com>
Date:   Sat,  9 Jan 2021 20:18:18 -0600

fixed sort fail regression

Diffstat:
M
src/opet.py
 | 
3
++-
M
src/wxcustom.py
 | 
2
++

2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/opet.py b/src/opet.py
@@ -172,7 +172,7 @@ class OPETFrame(wx.Frame):
                 self.esp_list_ctrl.CheckItem(fi, not self.esp_list_ctrl.IsItemChecked(fi))
         elif col == 1:
             if fi != -1:
-                self.last_order_value = int(self.esp_list_ctrl.GetItem(fi, 1).GetText())
+                self.esp_list_ctrl.last_order_value = int(self.esp_list_ctrl.GetItem(fi, 1).GetText())
             event.Skip()
         elif col == 2:
             event.Veto()
@@ -187,6 +187,7 @@ class OPETFrame(wx.Frame):
             if lbint < 0:
                 lbint = 0
             if self.esp_list_ctrl.last_order_value != None and lbint != self.esp_list_ctrl.last_order_value:
+                print("here")
                 self.esp_list_ctrl.SetItem(event.GetIndex(), event.GetColumn(), str(lbint))
                 # TODO: This is a bit fragile and relies on file names...
                 entry = self.esp_list_ctrl.GetItem(event.GetIndex(), col = 2).GetText()
diff --git a/src/wxcustom.py b/src/wxcustom.py
@@ -30,6 +30,8 @@ class ESPListCtrl(wx.ListCtrl, listctrl.TextEditMixin, listctrl.ListCtrlAutoWidt
         self.EnableCheckBoxes(True)
 
         # Dynamics
+        # TODO: Maybe this should be in the parent object instead???
+        #       Depends on how to solve the problem of where to bind...
         self.last_order_value = None
 
         # Columns