Boundary condition checks and arguments check is missing
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 38/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript, react
- Domaine
- frontend
Piste de recherche
Commencez par reproduire l'appel avec l'argument manquant contre index.js, en utilisant l'exemple de l'issue où l'argument de la fonction est omis. Vérifiez la gestion existante des arguments et vérifiez que les entrées invalides produisent l'avertissement signalé sans le string.replace TypeError ; l'issue ne nomme aucun fichier de test.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hi, I found some test cases where the function is failing:
reactStringReplace('Apt 111, phone number 555-5555-555.', (match, i) => (
console.log( <span key=${i} style={{ color: 'red' }}>${match})
));
so what I did I removed the second parameter and checked, so function is passed as matched and fn is passed as undefined. And got the below error:
F:\github\test1\react-string-replace\node_modules\react-string-replace\index.js:11
? string.replace(reRegExpChar, '\$&')
^
TypeError: string.replace is not a function
So, I added some validation checks for the arguments so that instead of exiting with an error it provides the user with the warning message and exists swiftly.
PS F:\github\test1\react-string-replace> node app
Debugger attached.
Apt 111, phone number 555-5555-555.
function
undefined
Please check the function arguments, either the match or the function is missing
Waiting for the debugger to disconnect...
thanks,
Niki
Also, I have added the fix in my local system and wanted to check in with you
- Langage dominant
- JavaScript
- Étoiles
- 666
- Forks
- 54
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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 iansinnott/react-string-replace
-
not a replacement functionOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
iansinnott/react-string-replace#89 · 1 réaction ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 30/100
iansinnott/react-string-replace#64 · 2 commentaires · 2 réactions ·
-
Capture groups?Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
iansinnott/react-string-replace#58 · 4 commentaires · 15 réactions ·
Toutes les issues de iansinnott/react-string-replace
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 1 jour
-
Design only Leadership Survey SLFS
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
bcgov/digital-journeys#2293 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
tursodatabase/turso#9405 ·
Les mainteneurs répondent en général sous 1 jour
-
Toolkit
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
API Bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
ProjectSidewalk/SidewalkWebpage#5556 ·
Les mainteneurs répondent en général sous 1 jour