Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

Inconsistent return value for shortFmt

未关闭
#205 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 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,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

a8m/angular-filter 的其他 Issue

查看 a8m/angular-filter 的全部 Issue

相似的 Issue

更多 JavaScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。