[NPCMapLocations] Doesn't respect dynamic configuration changes.

[Yagisan]

I'm trying to support several NPCs that need more dynamic support.

Generally this involved disabling their map markers until some event or flag has been set, then enabling them.

According to the patch summary command, it is dynamically changing conditions, however NPCMapLocations only respects the very first match.

For example with NPC Juna ( https://www.nexusmods.com/stardewvalley/mods/8606 ) I have the following snippet

        // NPC Juna - https://www.nexusmods.com/stardewvalley/mods/8606
        // Juna should not be visible before her into event 86060001 and the flag PlayerChoseParents has not been set
        {
            "LogName": "Juna - Before event 86060001",
            "Action": "EditData",
            "Target": "Mods/Bouhm.NPCMapLocations/NPCs",
            "When": {
                "HasMod: |contains=elhrvy.NPCJuna": true,
                "HasFlag |contains=PlayerChoseParents": false,
                "HasSeenEvent |contains=86060001": false,
                "HasSeenEvent |contains=86060043": false

            },
            "Entries": {
                "Juna": {
                    "Exclude": true
                },
            }
        },
        // Juna should be visible after her into event 86060001 and the flag PlayerChoseParents has not been set
        {
            "LogName": "Juna - After event 86060001",
            "Action": "EditData",
            "Target": "Mods/Bouhm.NPCMapLocations/NPCs",
            "When": {
                "HasMod: |contains=elhrvy.NPCJuna": true,
                "HasFlag |contains=PlayerChoseParents": false,
                "HasSeenEvent |contains=86060001": true,
                "HasSeenEvent |contains=86060043": false

            },
            "Entries": {
                "Juna": {
                    "MarkerCropOffset": 1
                },
            }
        },

The initial condition ( Juna - Before event 86060001 ) is the only one that NPCMapLocations respects, and is the one that is written into the log file.

The expected behaviour was that NPCMaplocations would update it's configuration when the conditions changed.

If needed, the full source code is available at -> https://github.com/Yagisan/Stardew_Valley/tree/main/Custom%20NPCs%20for%20NPC%20Map%20Locations

Thank you for your assistance.