Old border occasional wrong printings showing up

Aperta Adatta ai principianti
  • Qualcosa non va?
#9,238 8 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
72/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
java
Ambito
game-dev

Direzione di ricerca

Leggi forge-core/src/main/java/forge/item/PaperCardPredicates.java, in particolare printedInAnyEditions() intorno alle righe 67-92. Controlla l’esempio Badlands|30A rispetto a un’edizione 3ED consentita, quindi verifica che un oggetto carta proveniente da un’edizione non consentita non passi, mentre una stampa consentita passi. Il lavoro è completato quando allowedEditions filtra l’edizione effettiva della carta senza influire sui percorsi che non usano questo predicato.

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

Descrizione

Adventure keep

Dear devs, old border mod has an issue that can't be resolved in the config folder. It's with how allowedEditions is processed in

forge-core/src/main/java/forge/item/PaperCardPredicates.java (lines 67-92)

**Method: **printedInAnyEditions()

**Problem: **The predicate checks if the card name was printed in any allowed edition, then allows all printings of that card to pass - including printings from disallowed editions.

Example: Badlands|30A passes because Badlands exists in 3ED (allowed), but the actual card object is from 30A (not allowed).

**Fix: **We could change line 91 from:
return passes;
to:
return editions.contains(card.getEdition());

I think this fix shouldn't affect vanilla adventure because it doesn't use allowedEditions and so this predicate isn't called.

I've tried to solve it by inverting and using restrictedEditions instead, but then for some reason the PRM (Magic Online Promos) set forces its way in, even passed the restrictedEditions filter. And then there is a noticable lag when entering towns, maybe because the filter list is so long and I have to manually restrict a bunch more individual cards.

For old border and other planes mods, it would be preferential to use allowedEditions, because then we don't have to update config.json when a new set comes out.

Maybe there's a better solution I am not seeing.

Thanks for your help!

Image
Lingua principale
Java
Stelle
2.7k
Fork
1.1k
Merge medio
1g 8h
PR unite (30g)
344

Preparare l'ambiente

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.

Altre issue di Card-Forge/forge

Tutte le issue di Card-Forge/forge

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.