Skip to content

Fix: Support Classic and Forever Clients - #5

Merged
silverfoxy merged 2 commits into
silverfoxy:masterfrom
thijsdaniels:fix/classic-specialization
Sep 30, 2026
Merged

silverfoxy merged 2 commits into
silverfoxy:masterfrom
thijsdaniels:fix/classic-specialization

Conversation

@thijsdaniels

Copy link
Copy Markdown
Contributor

Summary

In Classic Era and the new WoW Forever Beta client (patch 1.60.1), GetSpecialization() is not available and returns nil. This caused DecktationContext.lua to throw a Lua error on player login and specialization changes.

Changes

  • Added a GetPlayerSpec() fallback helper that inspects talent tabs using GetNumTalentTabs() and GetTalentTabInfo() when GetSpecialization is nil.
  • Fixed duplicate EventFrame:SetScript("OnEvent", ...) calls that were overwriting event handler registrations.
  • Added Classic Era / WoW Forever interface numbers (11506, 16001, 160000) to DecktationContext.toc.
  • Updated CHANGELOG.md to version 1.1.1.

Testing

  • Verified on WoW Forever Beta (1.60.1 / _classic_beta_): no Lua errors on login or spec changes, and context dump correctly captures player class and talent specialization.

@thijsdaniels thijsdaniels changed the title fix(addon): support Classic Era and WoW Forever clients without GetSpecialization Fix: Support Classic and Forever Clients Sep 27, 2026
@silverfoxy
silverfoxy merged commit 240226b into silverfoxy:master Sep 30, 2026
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

Successfully merging this pull request may close these issues.

2 participants