RG Live WotLK 3.3.5a

hier auch nochmal https://www.youtube.com/watch?v=4ui6mf3lHSU&feature=youtu.be&t=7m

// always crit against a sitting target (except 0 crit chance)
if (victim->GetTypeId() == TYPEID_PLAYER && crit_chance > 0 && !victim->IsStandState()) {
TC_LOG_DEBUG("entities.unit", "RollMeleeOutcomeAgainst: CRIT (sitting victim)");
return MELEE_HIT_CRIT;
}

hm gegen spieler sollte es schon funktionieren - hast du es nur an npcs getestet?

Ich habe es als Tank an nem Raid Boss bemerkt.
Wipe angesagt, hingesetzt zum umcritten lassen.

Spieler spezifisch hatte ich nicht getestet, das ist richtig.

Edit: Gerade nochmal n Duel gegen jemanden probiert: auch Melee Hits von Spielern sind nicht 100% Critical.

da fällt mir - wollen wir das wirklich für kreaturen hinzufügen?

ganz oben im spellcritcheck wird ausdrücklich gesagt, dass kreaturen nicht citten sollten

Spells sollten nie critten können von Mobs, es geht nur um non-spell attacks. Melee und Ranged also, sowohl Abilities als auch Auto Attacks. Und die sollten auch von Kreaturen immer critten, wenn das Ziel nicht steht.

Ist auch Blizzlike, hat man zum schneller auswipen immer benutzt. Tank setzt sich hin und lässt sich umcritten.

E:
Ich vermute, dass darum der > 0% Crit Chance Check da drin ist.

Vorschlag: Mögliches Conditionrace bei der IsStandState()-Funktion?
Der Code an sich sieht ja iO aus, da man durch Attacks aber aufsteht, gibt die Funktion vllt einfach immer true zurück?

Als Idee.

Gibts da noch Details, woran es lag?
Wenn ich da recht nah dran war, würde mich das dann schon interessieren. :)

in funktion Unit::AttackerStateUpdate wurde CombatStart(victim); immer VOR DealMeleeDamage(&damageInfo, true); aufgerufen, demnach konnte der spellcrit wert nie richtig gecheckt werden

und bewerb dich mal als dev oder sowas, fix bissel was etc pp

Ticket wieder auf bestätigt? :>

wegen dem Teil hier:

und bewerb dich mal als dev oder sowas, fix bissel was etc pp

Als Dev wäre ich eher unnütz, da meine CPP-Kenntnisse zu rudimentär dafür sind.

Dürfte immernoch was fehlen.

und was?

Habe gerade nochmal kurz getestet und Melee-Abilities von NPCs (Spieler nicht getestet) sind nicht 100% Critical und soweit ich das hier verfolgt habe, fehlen auch noch n paar Waffentypen, von denen Dagger am relevantesten sein sollten (Rogues und so)

Dire Bear Form vom Druid geht auch nicht, Cat Form komischerweise schon (zumindest waren bei 10 swings in Cat Form alle Critical)

Edit nochmal: Dagger gehen, Spieler Abilities nicht

Bedeutung Live:

Der Bug wurde erfolgreich behoben und der Fix dazu befindet sich auf dem Spielserver. Sollte der Fehler weiterhin bestehen sollte ein neues Ticket ( https://redmine.rising-gods.de/projects/live/issues/new ) erstellt werden. Anmerkungen in dem alten abgewiesenen oder live genommenen Ticket werden ignoriert und gelöscht.

https://github.com/robinsch/TBCPvP/commit/7875bd5873ad16afe992f5cb9f111c2572f4137e