Old border occasional wrong printings showing up

Abierto Apto para principiantes
  • ¿Algo no está bien?
#9,238 8 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
72/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
java
Área
game-dev

Línea de trabajo

Lee forge-core/src/main/java/forge/item/PaperCardPredicates.java, especialmente printedInAnyEditions() alrededor de las líneas 67-92. Comprueba el ejemplo Badlands|30A frente a una edición 3ED permitida y verifica después que un objeto de carta de una edición no permitida no pase, mientras que una impresión permitida sí lo haga. Se considera terminado cuando allowedEditions filtra la edición real de la carta sin afectar a las rutas que no utilizan este predicado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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
Lenguaje dominante
Java
Estrellas
2.7k
Forks
1.1k
Merge medio
1 d 8 h
PR fusionados (30 d)
344

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Card-Forge/forge

Todos los issues de Card-Forge/forge

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.