DFreds / dfreds-convenient-effects

A FoundryVTT module that adds easy to use toggleable active effects for any system.
MIT License
47 stars 39 forks source link

Vision/Detection Modes Incompatibility #259

Closed lacktheknack closed 1 year ago

lacktheknack commented 1 year ago

Bug: CE isn't working with Vision/Detection Modes 5E again.

Reproduction: When CE is active, using custom OR original effects such as Flying or Inaudible no longer blocks relevant detection (such as hearing or tremorsense) implemented in V/DM5E.

When CE is turned off, the original provided effects work as expected. A few days ago, both original and custom effects correctly named worked as expected even with CE turned on.

Posting this here due to history of CE causing issues with the two.

Foundry: 302 DND5E: 2.2.1 CE: 5.0.2 Vision/Detection Modes 5e: 1.4.0

dev7355608 commented 1 year ago

It works as expected for me. Could you provide step-by-step reproduction instructions?

lacktheknack commented 1 year ago

With CE on: Create a CE called "Inaudible" and make it a status effect, then log out and back in. You now have two Invisible status effects and two Inaudibles (assuming you're using CE's "Add" functionality).

Place two tokens. Make one invisible and inaudible using any combination of the status effects you like. Now switch to the other token, who can detect the invisible one through Hearing if it's close enough. Also tested with a Flying token (both VDM and CE variants) against a token with Tremorsense - the token was detected.

Turn off CE and restart the server. Now there's only the provided status effects, and they block the detection.

dev7355608 commented 1 year ago

All custom CE status effects seem to apply the status Convenient Effect: New Effect. This must be the bug here.

DFreds commented 1 year ago

Good find, will fix

DFreds commented 1 year ago

Fixed the issue with the status ID not being updated to the new name and will go out in the release. Let me know if this doesn't fix the issue

LukasPrism commented 1 year ago

I don't want to start a whole new issue for this but just wanted to alert that the new statuses aren't falling into alphabetical order, despite the settings in CE:

Screenshot_2023-07-05_at_4 32 27_PM

Many thanks!

dev7355608 commented 1 year ago

Vision 5e adds its status effects back after CE removes them. That's why they are not in alphabetical order. They would be if the status effects were CE effects. That would also allow the user to remove those status effects from the HUD if they don't want them. I could create a custom CE effects for Vision 5e's status effects, but the user is able to edit those and so might accidently break things. Maybe CE could add API for defining additional effects that show up in the list of conditions?

LukasPrism commented 1 year ago

You’re right about users perhaps breaking things but it would also allow some more flexibility, for example if a user wanted to add a TMFX fly effect to the flying status (I had to do that with TVA). You could perhaps make it a setting to enable this (off by default). That’s how Stealthy handles it, so users can customise how the Hidden or Spot statuses affect the tokens, or add rider effects.

Krigsmaskine commented 1 year ago

@dev7355608 It seems like the statusEffects are defined by DFred's startup based on the module DFred's setting statusEffectNames. If you set this setting to include the name of a DFred's custom effect, it will start showing up in statusEffects next reload. I've only tested with GUI made custom effects, but there seems to be functions for creating custom ones here