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

UnitInParty return true even when soloing #788

Open
Hemario opened this issue Jan 3, 2021 · 3 comments
Open

UnitInParty return true even when soloing #788

Hemario opened this issue Jan 3, 2021 · 3 comments

Comments

@Hemario
Copy link
Contributor

Hemario commented Jan 3, 2021

When soloing group content (Torghast, dungeons, ...) UnitInParty() returns true even though we are in fact alone in that group.
Perhaps a new condition UnitIsSolo() is in order where we check whether or not we are actually solo.

Ovale_TankScripts uses this condition to check if we can dump resources into damage instead of mitigation.

@johnnylam88
Copy link
Contributor

The implementation of UnitInParty() looks like it just forwards the call directly to the Wow API function of the same name and returns the result. What does UnitInParty("player") return in-game while solo?

@Hemario
Copy link
Contributor Author

Hemario commented Jan 3, 2021

Indeed @ultijlam it does return true in game while solo but in instanced content. It is a direct call to the WoW API.

That's why a UnitIsSolo() would probably make more sense. I'm considering just implementing that function in Ovale TankScripts because it's not really used anywhere in Ovale.

@johnnylam88
Copy link
Contributor

I think UnitInGroup() would be good to replace all incarnations of UnitIsSolo(), UnitInParty(), and UnitInRaid().

  • Is player solo? not player.UnitInGroup()
  • Is player in a group? player.UnitInGroup()
  • Is player in a party? player.UnitInGroup("party")
  • Is player in a raid? player.UnitInGroup("raid")

@Sidoine Sidoine added this to To do in Bugs via automation Jan 4, 2021
@Sidoine Sidoine added this to To Do in Features Jan 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Bugs
To do
Development

No branches or pull requests

3 participants