Inconsistent return value for shortFmt
还没有人认领这个 Issue。
评估
- 难度
- 1/5
- 预计耗时
- 1 小时以内
- 新手友好度
- 35/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- angularjs, javascript
- 领域
- frontend
调研方向
在 angular-filter 源代码中查找 shortFmt 过滤器的实现,可能位于类似 lib/filters/math/shortFmt.js 的文件中。修复应确保它始终返回字符串,方法是将小于 1000 的数字强制转换为字符串。运行现有的 shortFmt 测试来验证行为,从而测试此更改。
由索引模型根据 Issue 内容生成。
描述
The current behavior
When given an Number less than 1000, shortFmt returns a Number. Otherwise, it returns a String.
Why is this a problem?
Consider the following case:
<span>{{ (n | shortFmt:1) || 'N/A'}}</span>
When n = 0, this will print "N/A" (Why? Because 0 is a false-y value in JS. Therefore, it evaluates the right-hand side of the "||"). I believe that most developers would expect this to print "0" instead. (Similar to the behavior of angular's number filter)
Proposed fix
When n < 1000, return n + ''. This will coerce the Number into a String.
If this seems like a reasonable behavior, then I'll go ahead and create a PR for this later today.
- 主要语言
- JavaScript
- 星标
- 2.9k
- 派生
- 319
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
a8m/angular-filter 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 70/100
a8m/angular-filter#267 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
a8m/angular-filter#260 · 1 个 reaction ·
-
KiB and MiB are wrongly used in place of kB and MB可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 2/5 1-3 小时 新手友好度 65/100
a8m/angular-filter#254 · 2 条评论 · 1 个 reaction ·
-
难度 2/5 1-3 小时 新手友好度 65/100
a8m/angular-filter#250 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 65/100
a8m/angular-filter#208 · 1 条评论 · 1 个 reaction ·
查看 a8m/angular-filter 的全部 Issue
相似的 Issue
-
status: waiting for maintainer
难度 2/5 1-3 小时 新手友好度 74/100
mui/material-ui#49547 ·
维护者通常 1 天内回复
-
type/bug
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 66/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 67/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
rescript-lang/rescript#8996 ·
维护者通常 1 天内回复