ironmonk88 / monks-active-tiles

GNU General Public License v3.0
49 stars 30 forks source link

[BUG] dfreds integration is no longer working. #831

Open kwilli96 opened 3 weeks ago

kwilli96 commented 3 weeks ago

Describe the bug I tried to just create a fix in github and push it to you but there is an issue with the merge. I think it had to do with new lines but it doesn't matter. Essentially you were getting dfreds effects using game.dfreds.effects.all and game.dfreds.effectInterface.findEffectByName but they seem to no longer exist or at least don't exist in my system.

Here are the changes that I made to get it to work:

File: Action.js

Line: 8943

const foundEffect = game.dfreds.effectInterface.findEffect({effectName:action.data.effect});

Lines: 8983-8987

list: () => {
      return game.dfreds.effectInterface.findEffects().map((effect) => {
            return i18n(effect.name);
       }).sort((a, b) => a.localeCompare(b));
},

Lines: 8917-8921

list: () => {
      return game.dfreds.effectInterface.findEffects().map((effect) => {
            return i18n(effect.name);
       }).sort((a, b) => a.localeCompare(b));
},

To Reproduce Steps to reproduce the behavior:

  1. Set an active tile action to convenient effects
  2. Nothing shows up.

Please complete as much of the following information as possible:

phre3d commented 3 weeks ago

Not sure if its the same issue but i just tried it with DFCE and there's no box to select the specific effect (see attached) DF-CE 1

kwilli96 commented 3 weeks ago

Not sure if its the same issue but i just tried it with DFCE and there's no box to select the specific effect (see attached)

@phre3d Yeah that is the issue that I found. If you are wanting to fix it you can make the changes to this file: %LOCALAPPDATA%\FoundryVTT\Data\modules\monks-active-tiles\actions.js

phre3d commented 3 weeks ago

Thanks, I'll try it tomorrow!