Boundary condition checks and arguments check is missing
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 38/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- javascript, react
- 领域
- frontend
调研方向
首先，使用 issue 中函数参数被省略的示例，针对 index.js 复现缺少参数的调用。检查现有的参数处理，并验证无效输入会产生报告的警告，而不会出现 string.replace TypeError；issue 未指定测试文件。
由索引模型根据 Issue 内容生成。
描述
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
- 主要语言
- JavaScript
- 星标
- 666
- 派生
- 54
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
我们还没有检查这个项目的环境配置文件。先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
iansinnott/react-string-replace 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 35/100
-
难度 3/5 1-2 天 新手友好度 35/100
-
难度 3/5 1-2 天 新手友好度 35/100
iansinnott/react-string-replace#89 · 1 个 reaction ·
-
难度 3/5 1-2 天 新手友好度 30/100
iansinnott/react-string-replace#64 · 2 条评论 · 2 个 reaction ·
-
Capture groups?未关闭
难度 3/5 1-2 天 新手友好度 35/100
iansinnott/react-string-replace#58 · 4 条评论 · 15 个 reaction ·
查看 iansinnott/react-string-replace 的全部 Issue
相似的 Issue
-
feature task
难度 2/5 1-3 小时 新手友好度 72/100
RealDevSquad/website-www#1178 ·
-
难度 2/5 1-3 小时 新手友好度 68/100
netlify-labs/nax#56 · 2 条评论 · 1 个 reaction ·
-
accessibility bug revealjs
难度 2/5 1-3 小时 新手友好度 84/100
quarto-dev/quarto-cli#14961 ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 88/100
PnX-SI/GeoNature#4435 · 2 条评论 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 85/100
openlayers/openlayers#17648 ·
维护者通常 1 天内回复