From 0f2fa62323e21fcf5ed2b761dea214bc5fd93a6d Mon Sep 17 00:00:00 2001 From: Reece Williams Date: Thu, 26 Oct 2023 21:30:29 -0500 Subject: [PATCH] only migrate packetforward subspace --- .../testing/simapp/upgrades/upgrades.go | 21 +++++-------------- 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/middleware/packet-forward-middleware/testing/simapp/upgrades/upgrades.go b/middleware/packet-forward-middleware/testing/simapp/upgrades/upgrades.go index 5d803dd6..e32dcdff 100644 --- a/middleware/packet-forward-middleware/testing/simapp/upgrades/upgrades.go +++ b/middleware/packet-forward-middleware/testing/simapp/upgrades/upgrades.go @@ -38,24 +38,13 @@ func CreateV2UpgradeHandler( ) upgradetypes.UpgradeHandler { return func(ctx sdk.Context, _ upgradetypes.Plan, vm module.VersionMap) (module.VersionMap, error) { // NOTE: If you already migrated the previous module, you ONLY need to migrate packetforward case now. - for _, subspace := range paramskeeper.GetSubspaces() { - subspace := subspace - - var keyTable paramstypes.KeyTable - // switch subspace.Name() { - // case packetforwardtypes.ModuleName: - // keyTable = packetforwardtypes.ParamKeyTable() - // } - - if subspace.Name() == packetforwardtypes.ModuleName { - keyTable = packetforwardtypes.ParamKeyTable() - } - - if !subspace.HasKeyTable() { - subspace.WithKeyTable(keyTable) - } + p, ok := paramskeeper.GetSubspace(packetforwardtypes.ModuleName) + if !ok { + panic("paramskeeper does not have packetforward subspace") } + p.WithKeyTable(packetforwardtypes.ParamKeyTable()) + // Migrate Tendermint consensus parameters from x/params module to a deprecated x/consensus module. // The old params module is required to still be imported in your app.go in order to handle this migration. baseAppLegacySS := paramskeeper.Subspace(baseapp.Paramspace).WithKeyTable(paramstypes.ConsensusParamsKeyTable())