normalize_url() relies on @ to hide an undefined index for URLs without a query string
还没有人认领这个 Issue。
评估
调研方向
找到 normalize_url()，检查它如何围绕 ? 拆分 URL；issue 描述了失败的情况，并建议提前返回。检查 check_topic() 中现有的行为，确认规范化后两个 URL 仍然匹配。当不含查询字符串的 URL 不再触发警告，且比较行为不受影响时，即为完成。
由索引模型根据 Issue 内容生成。
描述
For a URL with no ?, @list( $path, $query ) = explode( '?', $url ); reads index 1 of a one-element array. The @ hides the "Undefined array key 1" warning, but a custom error handler is still called, so debug and error loggers record it on every hub request.
Fix: return early when there's no query string, e.g.
if ( false === strpos( $url, '?' ) ) {
return rtrim( $url, '/' ) . '/';
}
Note this returns path/ instead of the current path/?. Both sides of the comparison in check_topic() go through normalize_url(), so matching is unaffected.
- 主要语言
- PHP
- 星标
- 15
- 派生
- 10
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Automattic/pushpress 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 83/100
Automattic/pushpress#17 ·
-
GUID normalisation bug, leading to duplicates可能已有人在做 @Alkarex 于 420 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 42/100
Automattic/pushpress#15 · 1 条评论 · 1 个 reaction ·
-
enhancement help wanted
难度 5/5 一周以上 新手友好度 30/100
Automattic/pushpress#14 ·
-
难度 3/5 1-2 天 新手友好度 35/100
Automattic/pushpress#2 · 1 条评论 ·
查看 Automattic/pushpress 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 3 天内回复
-
sync-en
难度 2/5 1-3 小时 新手友好度 74/100
维护者通常 1 天内回复
-
[Sync EN] Formatting improvements to "Large Objects (LOBs)" docs (#6056)可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭sync-en
难度 1/5 1 小时以内 新手友好度 74/100
维护者通常 1 天内回复
-
Перевод устарел
难度 2/5 1-3 小时 新手友好度 73/100
维护者通常 1 天内回复
-
repo:cms
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复