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]: Paladin Spam Filter Additions #240

Open
1 task done
Retjesir opened this issue May 19, 2024 · 1 comment
Open
1 task done

[Bug]: Paladin Spam Filter Additions #240

Retjesir opened this issue May 19, 2024 · 1 comment

Comments

@Retjesir
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

WoW version

10.2.7.54762

xCT+ version

4.7.2

What happened?

Spam merger isn't picking up a few abilities for Paladin.

A few spells aren't rolling up into a total (ex. 163k x4)

I think the reason for this is the spellIDs in the Class Spells tab don't match the current spell ids.

Wake of Ashes says it is 204273 in XCT, my tooltip has it being 255937
Blade of Justice is not found in XCT, but should be 184575
Final Reckoning is not present in XCT, but it should be 343721
Divine Toll is not present in XCT, it should be 375576
Searing Light is not present in XCT, I think it is 404540

Even Templar's Verdict doesn't have the same spellID.
Screenshot 2024-05-18 213116
Screenshot 2024-05-18 214512

Related Errors

No response

@Cimplex
Copy link
Collaborator

Cimplex commented Jul 9, 2024

So it looks like you are giving me the tooltips for the talents themselves and not the triggered spell IDs.

The easiest way to get the actual spell IDs is to enable the Show Spell ID's option (I believe its in the Spell/Spam Merger section). Once you do that, trigger these spells again, on something like a target dummy and see which ID's you get.

I just mention this, because WoW does generally have two (or more) spell ID's depending on lots of things, such as having a talent that modifies the ability. If we look at the spell ID you gave me Wake of Ashes, it does mention that it is a talent and that it triggers several other spells.

xCT+ needs the actual combat log spell ID.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants