Fix for #324 - do not reset itemID automatically upon removal due to way modules are swapped.

Author: blitzmann

eos/effectHandlerHelpers.py

@@ -109,7 +109,8 @@ def remove(self, thing):
         # We must flag it as modified, otherwise it not be removed from the database
         # @todo: flag_modified isn't in os x skel. need to rebuild to include
         #flag_modified(thing, "itemID")
-        thing.itemID = 0
+        if thing.isInvalid:  # see GH issue #324
+            thing.itemID = 0
         list.remove(self, thing)
 
 class HandledModuleList(HandledList):
@@ -191,6 +192,7 @@ def append(self, thing):
         oldObj = next((m for m in self if m.slot == thing.slot), None)
         if oldObj:
             logging.info("Slot %d occupied with %s, replacing with %s", thing.slot, oldObj.item.name, thing.item.name)
+            oldObj.itemID = 0  # hack to remove from DB. See GH issue #324
             self.remove(oldObj)
 
         HandledList.append(self, thing)