Configure: crash when id in patch.js is wrong
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- csharp, javascript
- Área
- desktop
Línea de trabajo
Comienza en thcrap_configure_v3/MainWindow.xaml.cs en la línea referenciada y sigue cómo se busca el id de patch_bootstrap en knownPatches. Confirma el comportamiento con un id no válido de patch.js; estará listo cuando la herramienta de configuración informe de un error legible en lugar de bloquearse cuando no coincida ningún parche.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When the id in patch.js is wrong, thcrap_configure_v3 crashes at this line: https://github.com/thpatch/thcrap/blob/ffc4277150946c8a33695b44cd272855e47dd9b6/thcrap_configure_v3/MainWindow.xaml.cs#LL105C15-L105C15
knownPatches contains the patch names from the various repo.js files, and the id variable comes from patch_bootstrap and contains the id from patch.js. That id isn't in knownPatches, so the function knownPatches.First() throws an exception because it didn't find anything.
Something is wrong and we don't need to make things work in this case, but a readable error message would be better than a crash.
- Lenguaje dominante
- C++
- Estrellas
- 773
- Forks
- 51
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de thpatch/thcrap
-
Ending display bugs in 15.5Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
-
Broken custom.exeAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
Going to the 4th row of column in the achievements menu of touhou 19 UDoALG crashes the gameAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de thpatch/thcrap
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
ws_bridge: stripping format=evr for matchmaker connections can concatenate the path and queryAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
EchoTools/nevr-runtime#116 ·
Los mantenedores suelen responder en 1 día
-
code-quality libc++
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
llvm/llvm-project#229284 ·
Los mantenedores suelen responder en 1 día
-
test-issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
llvm/offload-test-suite#1557 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día