From 173cffdcfc8f225c7172b9ba01a2851896e5c81d Mon Sep 17 00:00:00 2001 From: Athanasius Date: Mon, 15 Mar 2021 17:06:26 +0000 Subject: [PATCH] Add extra logging to is_horizons() if things not as expected. * Log the type(s) if not dict. * Log if ships['shipyard_list'] isn't present, despite ships being a dict. --- plugins/eddn.py | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/plugins/eddn.py b/plugins/eddn.py index 6533f638..12f2b207 100644 --- a/plugins/eddn.py +++ b/plugins/eddn.py @@ -796,11 +796,23 @@ def is_horizons(economies: MAP_STR_ANY, modules: MAP_STR_ANY, ships: MAP_STR_ANY if isinstance(dict, economies): economies_colony = any(economy['name'] == 'Colony' for economy in economies.values()) + else: + logger.error(f'economies type is {type(economies)}') + if isinstance(dict, modules): modules_horizons = any(module.get('sku') == HORIZ_SKU for module in modules.values()) + else: + logger.error(f'modules type is {type(modules)}') + if isinstance(dict, ships): if ships.get('shipyard_list') is not None: ship_horizons = any(ship.get('sku') == HORIZ_SKU for ship in ships['shipyard_list'].values()) + else: + logger.debug('No ships["shipyard_list"] - Damaged station or FC ?') + + else: + logger.error(f'ships type is {type(ships)}') + return economies_colony or modules_horizons or ship_horizons