publish_post() warns on PHP 8 when a feed has no subscribers

未关闭 适合新手
#17 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
83/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
php
领域
backend

调研方向

首先在插件源码中找到 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,通用步骤见我们的新手贡献指南。

从这里开始

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

Automattic/pushpress 的其他 Issue

查看 Automattic/pushpress 的全部 Issue

相似的 Issue

更多 PHP Issue

把新 issue 发到你的邮箱

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