Skip to content

Match the tooltip guid to a group unit token when UnitTokenFromGUID returns nil - #394

Merged
Vladinator merged 1 commit into
RaiderIO:developfrom
KogasaPls:bugfix/tooltip-secret-unit-token
Sep 10, 2026
Merged

Vladinator merged 1 commit into
RaiderIO:developfrom
KogasaPls:bugfix/tooltip-secret-unit-token

Conversation

@KogasaPls

@KogasaPls KogasaPls commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Unit tooltips for a party or raid member resolve to no unit: the guid is readable, but UnitTokenFromGUID on it returns nil while the tooltip is being built, so OnTooltipSetUnit bails and no score is drawn. Tooltips anchored to target or focus are unaffected, since those tokens persist and resolve. This reaches any unit frame that shows group members, Blizzard's included. EllesmereUI already carries a Raider.IO-specific workaround in its raid frames that calls ShowProfile with a guid-matched token, but the fix belongs here so every frame gets it.

Use the guid directly when the unit token is secret or missing by comparing it to player, partyN, and raidN.

Tested with Grid2 raid frames on a 12.1.0 client: the score draws where it did not before. The nil token and readable UnitGUID results behind it were measured both in and out of instance combat.

@Vladinator

Copy link
Copy Markdown
Collaborator

I'll take a closer look soon.

I just tested a little during instance combat with default frames and noticed that the main GetTooltipUnit function doesn't get called when hovering party/raid frames during instance combat lockdown.

However, I do see that outside of combat the UnitTokenFromGUID can return nil when I expected it to always return a valid guid when not being restricted, so the suggested GetGroupUnitTokenFromGUID does help bridge that gap, as long it's outside of combat.

Did you also observe the same when you made the code?

…eturns nil

Unit tooltips for a party or raid member resolve to no unit token while the tooltip is being built, so the unit tooltip hook bailed and drew no score.
@KogasaPls
KogasaPls force-pushed the bugfix/tooltip-secret-unit-token branch from 7a44dd3 to 8032523 Compare September 9, 2026 22:46
@KogasaPls KogasaPls changed the title Match the tooltip guid to a group unit token when UnitTokenFromGUID is secret Match the tooltip guid to a group unit token when UnitTokenFromGUID returns nil Sep 9, 2026
@KogasaPls

Copy link
Copy Markdown
Contributor Author

I had tested Grid2 (not vanilla frames) in an instance while out of combat only.

I just tested in combat and noticed:

  • The UnitTokenFromGUID token is nil, not secret. The idea that they could be secret came from an EllesmereUI comment, I don't know if it's actually true so I've dropped it from the PR.
  • The token is nil for party frames in instances regardless of combat state.
  • UnitGUID("player") and UnitGUID("party1") both return plain guids in combat still so this method works in combat.

I've updated the PR slightly. The fix is the same.

@KogasaPls

KogasaPls commented Sep 9, 2026 via email

Copy link
Copy Markdown
Contributor Author

@Vladinator
Vladinator self-requested a review September 10, 2026 20:11
@Vladinator
Vladinator merged commit bf8ddf5 into RaiderIO:develop Sep 10, 2026
@Vladinator

Copy link
Copy Markdown
Collaborator

I merged it in as it worked for the default UI as well, at least there were no issues to hover people during raid.

I tested both with the combat tooltip option enabled and disabled just to ensure there were no issues with the default UI raid frames.

Thanks for the suggestion, I wasn't aware that the UnitTokenFromGUID was so stingy considering you can then resolve it like this - which should have been part of the API itself since this is possible. 🤷

@Vladinator

Copy link
Copy Markdown
Collaborator

I've heard reports where the recent change ends up with their tooltips sometimes having double profiles.

I suspect it might be related to the initial statement:

EllesmereUI already carries a Raider.IO-specific workaround in its raid frames that calls ShowProfile with a guid-matched token, but the fix belongs here so every frame gets it.

So I suspect I must let @EllesmereGaming know so they can adjust their code if this is the case. As it should no longer be needed to do this action.

@KogasaPls

KogasaPls commented Sep 15, 2026 via email

Copy link
Copy Markdown
Contributor Author

@KogasaPls

Copy link
Copy Markdown
Contributor Author

PR submitted: EllesmereGaming/EllesmereUI#2102

My mistake for not submitting the followup PR immediately after this was merged. Thank you for pointing it out.

EllesmereGaming pushed a commit to EllesmereGaming/EllesmereUI that referenced this pull request Sep 17, 2026
Raider.IO now matches group units by GUID upstream (RaiderIO/raiderio-addon#394), so manual profile drawing is no longer needed and caused duplicate tooltips.
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