Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Disable Commands Effect not working #2787

Open
1 task done
MattyCanny opened this issue Sep 7, 2024 · 2 comments
Open
1 task done

[Bug] Disable Commands Effect not working #2787

MattyCanny opened this issue Sep 7, 2024 · 2 comments
Labels
Bug A defect in the app

Comments

@MattyCanny
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

One of my preset effects list was getting stuck on a "Disable Commands Effect" because the effect was unable to disable the commands with a given tag.

Expected Behavior

The effect should disable the commands with the tag

Steps To Reproduce

I do not know.

Version

5.63.2

What operating system are you using Firebot on?

Windows 11

Relevant log output

[2024-09-07 06:24:44.0631] - info: [v5.63.2] Websocket Overlay Connection from ::1
[2024-09-07 06:25:42.0712] - error: [v5.63.2] Unhandled promise rejection 
{
  message: "Cannot read properties of undefined (reading 'includes')",
  stack: "TypeError: Cannot read properties of undefined (reading 'includes')\n" +
    '    at C:\\Users\\Rando\\AppData\\Local\\firebot\\app-5.63.2\\resources\\app.asar\\build\\backend\\effects\\builtin\\toggle-command.js:111:56\n' +
    '    at Array.filter (<anonymous>)\n' +
    '    at Object.onTriggerEvent (C:\\Users\\Rando\\AppData\\Local\\firebot\\app-5.63.2\\resources\\app.asar\\build\\backend\\effects\\builtin\\toggle-command.js:111:33)\n' +
    '    at C:\\Users\\Rando\\AppData\\Local\\firebot\\app-5.63.2\\resources\\app.asar\\build\\backend\\common\\effect-runner.js:89:40\n' +
    '    at new Promise (<anonymous>)\n' +
    '    at triggerEffect (C:\\Users\\Rando\\AppData\\Local\\firebot\\app-5.63.2\\resources\\app.asar\\build\\backend\\common\\effect-runner.js:68:12)\n' +
    '    at C:\\Users\\Rando\\AppData\\Local\\firebot\\app-5.63.2\\resources\\app.asar\\build\\backend\\common\\effect-runner.js:147:40\n' +
    '    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)'
}

Contact Details (Optional)

No response

@MattyCanny MattyCanny added the Bug A defect in the app label Sep 7, 2024
@phroggster
Copy link
Collaborator

We traced this specific instance back to the sortTag. Fixed it by creating a new sortTag, adding all of the specific commands to it, then deleting the old sortTag. No clue how/where the sortTag got corrupted, no visual indication of a missing command guid nor corruption in the tags, but it did clear up after swapping it out for a new tag.

Even weirder, the disable commands by tag effect was in a preset effect list, and running that effect list asynchronously would work fine, while running it synchronously would cause this error.

@MattyCanny
Copy link
Author

MattyCanny commented Sep 16, 2024

This issue has occurred again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A defect in the app
Projects
Status: Todo
Development

No branches or pull requests

2 participants