Make internal list of wrapper private
This commit is contained in:
@@ -38,7 +38,7 @@ class BaseWrapperList(gui.display.Display):
|
||||
def __init__(self, graphFrame, parent):
|
||||
super().__init__(parent)
|
||||
self.graphFrame = graphFrame
|
||||
self._wrappers = []
|
||||
self.__wrappers = []
|
||||
|
||||
self.hoveredRow = None
|
||||
self.hoveredColumn = None
|
||||
@@ -51,7 +51,7 @@ class BaseWrapperList(gui.display.Display):
|
||||
@property
|
||||
def wrappers(self):
|
||||
# Sort fits first, then target profiles
|
||||
return sorted(self._wrappers, key=lambda w: not w.isFit)
|
||||
return sorted(self.__wrappers, key=lambda w: not w.isFit)
|
||||
|
||||
# UI-related stuff
|
||||
@property
|
||||
@@ -141,16 +141,16 @@ class BaseWrapperList(gui.display.Display):
|
||||
if row == -1:
|
||||
return None
|
||||
try:
|
||||
return self._wrappers[row]
|
||||
return self.__wrappers[row]
|
||||
except IndexError:
|
||||
return None
|
||||
|
||||
def removeWrappers(self, wrappers):
|
||||
wrappers = set(wrappers).intersection(self._wrappers)
|
||||
wrappers = set(wrappers).intersection(self.__wrappers)
|
||||
if not wrappers:
|
||||
return
|
||||
for wrapper in wrappers:
|
||||
self._wrappers.remove(wrapper)
|
||||
self.__wrappers.remove(wrapper)
|
||||
self.updateView()
|
||||
for wrapper in wrappers:
|
||||
if wrapper.isFit:
|
||||
@@ -169,16 +169,16 @@ class BaseWrapperList(gui.display.Display):
|
||||
return wrappers
|
||||
|
||||
def appendItem(self, item):
|
||||
self._wrappers.append(self.wrapperClass(item))
|
||||
self.__wrappers.append(self.wrapperClass(item))
|
||||
|
||||
def containsFitID(self, fitID):
|
||||
for wrapper in self._wrappers:
|
||||
for wrapper in self.__wrappers:
|
||||
if wrapper.isFit and wrapper.item.ID == fitID:
|
||||
return True
|
||||
return False
|
||||
|
||||
def containsProfileID(self, profileID):
|
||||
for wrapper in self._wrappers:
|
||||
for wrapper in self.__wrappers:
|
||||
if wrapper.isProfile and wrapper.item.ID == profileID:
|
||||
return True
|
||||
return False
|
||||
@@ -189,13 +189,13 @@ class BaseWrapperList(gui.display.Display):
|
||||
self.updateView()
|
||||
|
||||
def OnFitChanged(self, event):
|
||||
if set(event.fitIDs).intersection(w.item.ID for w in self._wrappers if w.isFit):
|
||||
if set(event.fitIDs).intersection(w.item.ID for w in self.__wrappers if w.isFit):
|
||||
self.updateView()
|
||||
|
||||
def OnFitRemoved(self, event):
|
||||
wrapper = next((w for w in self._wrappers if w.isFit and w.item.ID == event.fitID), None)
|
||||
wrapper = next((w for w in self.__wrappers if w.isFit and w.item.ID == event.fitID), None)
|
||||
if wrapper is not None:
|
||||
self._wrappers.remove(wrapper)
|
||||
self.__wrappers.remove(wrapper)
|
||||
self.updateView()
|
||||
|
||||
def OnProfileRenamed(self, event):
|
||||
@@ -207,9 +207,9 @@ class BaseWrapperList(gui.display.Display):
|
||||
self.updateView()
|
||||
|
||||
def OnProfileRemoved(self, event):
|
||||
wrapper = next((w for w in self._wrappers if w.isProfile and w.item.ID == event.profileID), None)
|
||||
wrapper = next((w for w in self.__wrappers if w.isProfile and w.item.ID == event.profileID), None)
|
||||
if wrapper is not None:
|
||||
self._wrappers.remove(wrapper)
|
||||
self.__wrappers.remove(wrapper)
|
||||
self.updateView()
|
||||
|
||||
# Context menu handlers
|
||||
@@ -223,7 +223,7 @@ class BaseWrapperList(gui.display.Display):
|
||||
self.graphFrame.draw()
|
||||
|
||||
def getExistingFitIDs(self):
|
||||
return [w.item.ID for w in self._wrappers if w.isFit]
|
||||
return [w.item.ID for w in self.__wrappers if w.isFit]
|
||||
|
||||
def addFitsByIDs(self, fitIDs):
|
||||
sFit = Fit.getInstance()
|
||||
|
||||
Reference in New Issue
Block a user