publish_post() warns on PHP 8 when a feed has no subscribers
还没有人认领这个 Issue。
评估
调研方向
首先在插件源码中找到 publish_post() 和 get_subscribers();issue 指出,RSS2 和 Atom feed 的失败路径都是 subscriber 循环。检查空 subscriber 结果的处理方式,并在 PHP 8 上重现没有 subscriber 时发布的情况。完成标准是:没有 subscriber 时发布不会为任一 feed 产生数组偏移警告;存在 subscriber 时,subscriber 回调仍会运行。
由索引模型根据 Issue 内容生成。
描述
When a post is published and a feed has no subscribers, get_subscribers() returns FALSE. publish_post() then runs foreach ( (array) $subs as $callback => $data ). Casting FALSE gives [ false ], so the loop runs once with $data === false and $data['is_active'] raises "Trying to access array offset on value of type bool" on PHP 8 (a notice on 7.4).
This happens on every publish on any site with no subscribers, for both the RSS2 and Atom feeds.
Fix: skip the loop when there are no subscribers, e.g. wrap each loop in if ( ! empty( $subs ) ) { … }, or have get_subscribers() return array() instead of FALSE.
- 主要语言
- PHP
- 星标
- 15
- 派生
- 10
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
Automattic/pushpress 的其他 Issue
-
难度 1/5 1-3 小时 新手友好度 85/100
Automattic/pushpress#18 ·
-
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 天内回复