dave/pyfa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 5 Wiki Activity

Use getattr directly, insead of doing a check then setting it (yay for defaults)

Browse Source
This commit is contained in:
Ebag333
2017-02-10 06:44:40 -08:00
parent 514f927a56
commit f610f525ba
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gui/builtinViews/fittingView.py
View File

@@ -406,13 +406,10 @@ class FittingView(d.Display):
if mod1.slot != mod2.slot:
return
if not getattr(mod2, "modPosition"):
self.mods.modPosition = mod2.modPosition = dstRow
if clone and mod2.isEmpty:
sFit.cloneModule(self.mainFrame.getActiveFit(), srcIdx, mod2.modPosition)
sFit.cloneModule(self.mainFrame.getActiveFit(), srcIdx, getattr(mod2, "modPosition", dstRow))
else:
sFit.swapModules(self.mainFrame.getActiveFit(), srcIdx, mod2.modPosition)
sFit.swapModules(self.mainFrame.getActiveFit(), srcIdx, getattr(mod2, "modPosition", dstRow))
wx.PostEvent(self.mainFrame, GE.FitChanged(fitID=self.mainFrame.getActiveFit()))