Skip to content

dwNeedItemFix MagicSkillMng.cpp #277

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

UTengine
Copy link
Contributor

@UTengine UTengine commented Apr 24, 2025

This is a fix to cast skills since the newer tbl's from version roughly 1070+ have a dwNeedItem 9 instead of the original dwNeedItem 0 skills are greyed out and fail to cast even if you equip weapons or class specific items. Also skills that require no items failed to cast with this update we can now cast skills and work further
image
image
image

💔 Thank you!

🚨 Checklist for this Pull Request

  • Provide a link to the ticket your pull request addresses or resolves. If there isn't one, create it.
  • Ensure you are making a pull request against the canary branch (left side). Start your branch from our canary.
  • Verify that your commit messages follow our style guidelines: start with a capitalized verb and limit the title to 72 characters; the description can be longer.
  • Ensure your code additions pass linting checks (run the format.ps1 script) and avoid irrelevant diffs.
  • Ensure your pull request contains isolated changes. Ideally, create separate PRs for unrelated changes.

This is a fix to cast skills since the newer tbl's from version roughly 1070+ have a dwNeedItem 9 instead of the original dwNeedItem 0 skills are greyed out and fail to cast even if you equip weapons or class specific items. Also skills that require no items failed to cast with this update we can now cast skills and work further
Skills were working for other classes but warrior was still having issues in page 1055/2055 1056/2056
@xGuTeK xGuTeK added the official_client_code_review Needs verification against the official client code using IDA/Ghidra label Apr 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
official_client_code_review Needs verification against the official client code using IDA/Ghidra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants