Fehler #2937
geschlossen
Healprobleme in Vehicels
Beschreibung
Durch einen Fix der beinhaltete hat das leute, die sinch in fahzeugen befinden nicht mehr angegriffen werden können. Wurde ausversehen durch einen fehler im core auch der erhaltene Heal deaktiviert.
Zugehörige Tickets
Von karn vor etwa 12 Jahren aktualisiert
Von DoesntMatter vor etwa 12 Jahren aktualisiert
- Status wurde von Bestätigt zu Testbereit geändert
- Zugewiesen an
DoesntMatterwurde gelöscht - % erledigt wurde von 0 zu 100 geändert
Generell ist das eine sehr merkwürdige Sache..
Die Passagiere der Vehicles haben lediglich eine Flag bekommen, die sie "nicht angreifbar" machen sollte.
Allerdings ist der tatsächliche Effekt viel umfangreicher.
Das Flag UNIT_FLAG_NON_ATTACKABLE sorgt nämlich dafür, dass jegliche Art von Spell keinen Effekt auf dem Ziel zeigt.
Für mich persönlich würde es aber nur bei negativen Spells einen Sinn machen, da es hier schließlich um ein "Angreifen" geht.
Daher habe ich jetzt bei dem Flag einen zusätzlichen Check auf negative Spells hinzugefügt:
https://github.com/rising-gods/Core/commit/054a1fb2a5917133c81ab61ca8041bd427642593
Was ich hier für etwas problematisch halte, ist das Testen an sich.
Nach meinen Tests ist es damit wohl möglich, die Spieler in Vehicles zu heilen, aber keinen Schaden zuzufügen.
Ob und inwiefern hierdurch aber andere Dinge betroffen sind, kann ich ehrlich gesagt nicht wirklich abschätzen.
Wahrscheinlich macht es Sinn den Patch für ein paar Tage/Wochen auf dem Test-Server zu behalten.
Falls dort keine Bugs auffallen, kann man das dann auf die Live-Server packen.
Von karn vor etwa 12 Jahren aktualisiert
- Status wurde von Testbereit zu Pending geändert
Damit kann der fix zu http://redmine.rising-gods.de/issues/2925 wieder aktiv gemacht werden
Von Lareine vor etwa 12 Jahren aktualisiert
- Status wurde von Pending zu Testbereit geändert
- Zugewiesen an wurde auf karn gesetzt
- Priorität wurde von Sofort zu Normal geändert
Bitte die entsprechenden Mechaniken bei Kologarn und Ignis nochmal testen, oder hast du das bereits getan?
Von marmil vor etwa 12 Jahren aktualisiert
Heal bei Kologarn und Ignis war eben möglich. Habe ich mit Karn getestet.
Von marmil vor etwa 12 Jahren aktualisiert
- Status wurde von Testbereit zu Pending geändert
Von Cyberfreak vor etwa 12 Jahren aktualisiert
Problem bei Ignis weiterhin existent, wir hatten eben 2 Versuche bei dem der Spieler im Schlacketopf nicht im Sichtfeld wären, obwohl sie es definitiv waren.
Von
Von karn vor mehr als 10 Jahren aktualisiert
- Status wurde von Bestätigt zu Live geändert
- Zugewiesen an wurde auf Arishi gesetzt
Nicht in sichweite ungleich nicht heilbar
Bitte neues ticket
Von Anteloa vor mehr als 10 Jahren aktualisiert
- Status wurde von Live zu Bestätigt geändert
- Zugewiesen an
Arishiwurde gelöscht
Problem besteht weiterhin, jeder der bei Yogg ein Opfer der Würgetentakel heilen will, wird feststellen, dass das nicht funktioniert und das kann dort nicht an LOS-Problemen liegen
Von Anteloa vor mehr als 10 Jahren aktualisiert
- Status wurde von Bestätigt zu Neu geändert
- Zugewiesen an wurde auf Anteloa gesetzt
Ich schau mal nochmal drüber, aufm testserver gehts, auf dem Live bin ich mir aber ziemlich sicher nicht...
Von Anteloa vor mehr als 10 Jahren aktualisiert
- Status wurde von Neu zu Bestätigt geändert
- Zugewiesen an
Anteloawurde gelöscht
Von Rushor vor mehr als 9 Jahren aktualisiert
- Dupliziert durch Fehler #5529: [Ulduar] Trash - Arachnopodzerstörer nicht heilbar wurde hinzugefügt
Von Rushor vor fast 8 Jahren aktualisiert
https://github.com/TrinityCore/TrinityCore/issues/13944
The reason is fairly simple, it is due to _IsValidAttackTarget never being checked for TARGET_CHECK_ENTRY in
bool WorldObjectSpellTargetCheck::operator()(WorldObject* target)
But adding a check for it is not as simple, as it will end up screwing up a lot of spells which are supposed to hit untargettable targets.
Just one addition: using _IsValidAttackTarget is a bad idea (the place is correct however). I tried it and it broke other things. So the check should probably be slightly different (even if the location should be the same).
Von Daejiv vor mehr als 7 Jahren aktualisiert
- Kategorie wurde von Kritisch zu Mechanik geändert
- Priorität wurde von Normal zu Hoch geändert
- Zielversion wurde von Kritische Bugs zu Welt geändert
Von SpeedCore vor fast 7 Jahren aktualisiert
- Status wurde von Bestätigt zu Neu geändert
- Zugewiesen an
SpeedCorewurde gelöscht
Da uralt - bite nochmals überprüfen
Von Sockentester vor fast 7 Jahren aktualisiert
- Status wurde von Neu zu Abgewiesen geändert
Abgewiesen weil wtf Ticket
=> Bei erneutem Auftreten bitte neues Ticket