Removed the util function - this would actually break older migrations if we added things that didn't exist in future iterations. Each migration should be self-contained
This commit is contained in:
@@ -5,7 +5,6 @@ Imports all item conversions since Migration 28 and runs them against module.bas
|
||||
forgotten about since it's been added.
|
||||
|
||||
"""
|
||||
from. utils import convert_modules
|
||||
from .upgrade36 import CONVERSIONS as u36
|
||||
from .upgrade37 import CONVERSIONS as u37
|
||||
from .upgrade38 import CONVERSIONS as u38
|
||||
@@ -13,4 +12,7 @@ from .upgrade39 import CONVERSIONS as u39
|
||||
|
||||
def upgrade(saveddata_engine):
|
||||
for conversions in [u36, u37, u38, u39]:
|
||||
convert_modules(saveddata_engine, conversions)
|
||||
for replacement_item, list in conversions.items():
|
||||
for retired_item in list:
|
||||
saveddata_engine.execute('UPDATE "modules" SET "baseItemID" = ? WHERE "baseItemID" = ?',
|
||||
(replacement_item, retired_item))
|
||||
|
||||
@@ -1,15 +0,0 @@
|
||||
def convert_modules(engine, conversions):
|
||||
'''Converts modules based on tiericide conversion mappings.
|
||||
|
||||
:param engine:
|
||||
:param conversions:
|
||||
:return:
|
||||
'''
|
||||
for replacement_item, list in conversions.items():
|
||||
for retired_item in list:
|
||||
engine.execute('UPDATE "modules" SET "itemID" = ? WHERE "itemID" = ?',
|
||||
(replacement_item, retired_item))
|
||||
engine.execute('UPDATE "modules" SET "baseItemID" = ? WHERE "baseItemID" = ?',
|
||||
(replacement_item, retired_item))
|
||||
engine.execute('UPDATE "cargo" SET "itemID" = ? WHERE "itemID" = ?',
|
||||
(replacement_item, retired_item))
|
||||
Reference in New Issue
Block a user