[No] MAPINFO: EventHandlers -> an alias for AddEventHandlers

[m8f]

EventHandlers property clears all previously added event handlers. I see no reasonable use case for this behavior. If this is some kind of protection, it's easily broken by changing load order.

Up to this point I've seen two cases when this property breaks seemingly unrelated mods.

So, the suggestion: make it an alias for AddEventHandlers property. The cleaner solution would be to remove it, but it will break mods that use it.

[m8f]

This change is small enough, here is the PR.

[Graf Zahl]

I see no reasonable use case for this behavior. If this is some kind of protection, it's easily broken by changing load order.

That has been said countless times, until that one mod came along that in fact needed the feature...

[Rachael]

This gets a pretty strong [No] from me. I really don't see any reason for this. Regardless of the OP's intent, you simply cannot assume the mod's design. If this is really such an issue, people should be educated to use AddEventHandlers unless it's absolutely necessary.

[m8f]

Understood. No big deal, please tag accordingly.

[Rachael]

I'm going to wait for at least one other developer to respond, I do feel this still deserves more consideration, even though I've already stated my position on the issue, it's with what I feel is limited data at hand.

[Graf Zahl]

Case closed I'd say.

[Nash]

until that one mod came along that in fact needed the feature...

What mod is that?

[Graf Zahl]

the hypothetical mod that will inevitably be discovered - as it has been with every single change to the engine.