Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Training lists duplicate classic puzzles because the training filter's set names don't match

Aperta Adatta ai principianti
#779 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

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

Direzione di ricerca

Inizia in PuzzleLibrary.lua, in invalidSetsForTraining, e confronta questi nomi con i nomi dei set integrati in default_data/puzzles/Puzzles.json (puzzle_set_name_classic_set_1–6 vs. puzzle_name_classic_set_1–6). Allinea il filtro (oppure usa una chiave più stabile del nome localizzato). Poi individua dove viene creata la lista di allenamento e rimuovi gli UUID duplicati, in modo che le copie da puzzles/stock (example).json non possano comparire accanto agli stessi puzzle integrati. Verifica aprendo l'allenamento e controllando che i puzzle classici non compaiano più due volte di seguito.

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

Descrizione

Training sometimes shows the same puzzle twice in a row: solving it immediately brings up an identical copy. In my training list, 58 puzzles appear twice.

Causes

  1. invalidSetsForTraining in PuzzleLibrary.lua is meant to keep the classic sets out of training, but it lists puzzle_name_classic_set_1–6. The built-in sets in default_data/puzzles/Puzzles.json are named puzzle_set_name_classic_set_1–6, so nothing is filtered.
  2. The old puzzles/stock (example).json contains the same puzzles as "Classic set 1"–"Classic set 6". UUIDs are computed from puzzle content, so these copies share UUIDs with the built-in puzzles and sort right next to them. cleanupDefaultPuzzles only deletes the file if it was last modified before 2025-09-14. Mine wasn't deleted: it contains recorded solutions, so the game has written to it since then.

Suggested fix

  • Correct the names in invalidSetsForTraining (or filter by something more stable than the localized set name).
  • Remove repeated UUIDs from the training list, so copied puzzle files can't add the same puzzle twice.

Found while working on #778.

Lingua principale
Lua
Stelle
30
Fork
20
Merge medio
6g 12h
PR unite (30g)
6

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 panel-attack/panel-game

Tutte le issue di panel-attack/panel-game

Issue simili

Altre issue su Lua

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.