TrinityCore / TrinityCore/TrinityCore

[3.3.5a] Instance bind incorrect check

Aperta
#20,568 1 commento 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Branch-3.3.5a Branch-master Comp-Core Sub-Instances
Lingua principale
C++
Stelle
10.8k
Fork
6.4k
Merge medio
3g 16m
PR unite (30g)
6

Descrizione

Hello,
I was inspecting some code for SPELL_EFFECT_SUMMON_PLAYER for inspiration when I stumbled upon the following weird logic:

https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/game/Spells/Spell.cpp#L5495

On this line we check for IsRaid for an instance bind. As a matter of fact not only raids have instance binds but also Heroic instances do as well (for 24h). Is this logic missing on purpose from this piece of code? If yes please disregard my note and close this issue.

TC hash/commit:
607034064f04b33f5de6e166a87c8e1c8a2fe953

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia da src/server/game/Spells/Spell.cpp intorno alla riga 5495, usando come contesto il commit 3.3.5 indicato. Segui il controllo del vincolo dell’istanza e verifica come vengono rappresentate le istanze eroiche rispetto ai raid. Il lavoro è completato quando si è determinato se il controllo è intenzionale e, in caso contrario, lo si è aggiornato affinché i tipi di istanza rilevanti siano gestiti correttamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
game-dev
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.