View Single Post
12-04-12, 04:13 PM   #30
SDPhantom
A Pyroguard Emberseer
 
SDPhantom's Avatar
AddOn Author - Click to view addons
Join Date: Jul 2006
Posts: 2,326
I'm not sure if Tooltip:SetHyperlink() fires OnTooltipSetUnit on unit hyperlinks. Again, all this is pending results of a series of tests I need to do and unfortunately, I can't perform them at work right now. The "Neutral" handling is just a fall through in case it's unable to identify a specific faction. With lack of other options, I don't see any harm in letting it do that as long as it doesn't throw them into either of the Alliance or Horde category blindly. If you insist, you can have it pass nil,nil instead as that's the default response of UnitFactionGroup() for neutral NPCs.
__________________
WoWInterface AddOns
"All I want is a pretty girl, a decent meal, and the right to shoot lightning at fools."
-Anders (Dragon Age: Origins - Awakening)
  Reply With Quote