Fix issue with adding multiples of the same skill to character via ch…

…aracter import (#563)
pyfa-org · Apr 30, 2016 · fe9139a · fe9139a
1 parent 87a354a
commit fe9139a
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/eos/saveddata/character.py b/eos/saveddata/character.py
@@ -142,6 +142,14 @@ def skills(self):
         return self.__skills
 
     def addSkill(self, skill):
+        if skill.itemID in self.__skillIdMap:
+            oldSkill = self.__skillIdMap[skill.itemID]
+            if skill.level > oldSkill.level:
+                # if new skill is higher, remove old skill (new skill will still append)
+                self.__skills.remove(oldSkill)
+            else:
+                return
+
         self.__skills.append(skill)
         self.__skillIdMap[skill.itemID] = skill