Moerill / fvtt-find-the-culprit

MIT License
8 stars 10 forks source link

Attempt a fix for error introduced in most recent update. #5

Closed Varriount closed 3 years ago

Varriount commented 3 years ago

The most recent update introduced a logic error - deactivationStep is used within the first step to initially deactivate all modules, and is passed the value of [] for chosenModules. This causes an erroneous short-circuit to renderFinalDialog with a non-existent array entry.

Moerill commented 3 years ago

Thanks for the PR, ye i made a mistake there...