EDDN: Only add missing StarPos if for current system

<https://github.com/EDCD/EDMarketConnector/issues/961>

It seems the game can report a Scan event *after* you've jumped to
another system.  So don't blindly set StarPos on Scan, check the
SystemAddress against where we think we are first.
This commit is contained in:
Athanasius committed 2021-04-01 08:56:35 +01:00
1 parent f9cc742de4
commit 96bb0959f5
1 file changed
+6
+6
View File
@@ -714,6 +714,12 @@ def journal_entry( # noqa: C901
logger.warning("this.coordinates is None, can't add StarPos")
return "this.coordinates is None, can't add StarPos"
# Gazelle[TD] reported seeing a lagged Scan event with incorrect
# augmented StarPos: <https://github.com/EDCD/EDMarketConnector/issues/961>
if this.systemaddress is None or this.systemaddress != entry['SystemAddress']:
logger.warning("event has no StarPos, but SystemAddress isn't current location")
return "Wrong System! Delayed Scan event?"
entry['StarPos'] = list(this.coordinates)
if 'SystemAddress' not in entry: