Configure: crash when id in patch.js is wrong
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- csharp, javascript
- Domaine
- desktop
Piste de recherche
Commencez dans thcrap_configure_v3/MainWindow.xaml.cs à la ligne référencée et suivez la manière dont l’id de patch_bootstrap est recherché dans knownPatches. Confirmez le comportement avec un id invalide provenant de patch.js ; c’est terminé lorsque l’outil de configuration signale une erreur compréhensible au lieu de planter lorsqu’aucun patch ne correspond.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- C++
- Étoiles
- 773
- Forks
- 51
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de thpatch/thcrap
-
Ending display bugs in 15.5Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 42/100
-
Broken custom.exeOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
-
Going to the 4th row of column in the achievements menu of touhou 19 UDoALG crashes the gameOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Toutes les issues de thpatch/thcrap
Issues similaires
-
feature request
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
gavinlouuu-kpt/mib-studio-qt#517 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
ws_bridge: stripping format=evr for matchmaker connections can concatenate the path and queryOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
EchoTools/nevr-runtime#116 ·
Les mainteneurs répondent en général sous 1 jour