跳到内容
新问题

对这个项目有疑问吗？注册一个免费的GitHub帐户以打开一个问题，并联系其维护者和社区。

注册GitHub

单击“注册GitHub”，表示您同意我们的服务条款和隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了？登录到您的帐户
跳到底部

交互API-如果筛选器中解析的块发生更改，则修复SSR。 #6245

关闭
想要合并7提交到从
关闭

交互API-如果筛选器中解析的块发生更改，则修复SSR。 #6245

想要合并7提交到从

对话

卡布拉沃伯纳尔
复制链接

Trac票：
https://core.trac.wordpress.org/ticket/60743#票证

如中所评论WordPress/gutenberg#59057（评论）
如果有人编辑$解析块变量，它将在Interactivity API SSR处理比较中失败：

wordpress-develop/src/wp-includes/interactivity-api/interactity-api.php

第49行在里面9a616a5

如果($根块名称===$解析块['区块名称'] &&$解析块===$根路径块) {

在6.6中似乎不需要进行这种比较，因为md5错误不允许向该变量添加键。

与合著@桑托斯·吉拉莫特

此Pull请求仅用于代码审查。请将所有其他讨论保留在Trac记录单中。不要合并此Pull请求。请参见GitHub请求代码审查有关更多详细信息，请参阅《核心手册》。

@github-操作 GitHub操作
复制链接

github-操作 机器人程序 评论2024年3月11日 •
已编辑

以下账户与本公关和/或相关问题进行了互动。当活动发生时，我将继续更新这些列表。您还可以通过添加道具机器人标签。

核心提交人：在SVN中提交时，使用此行作为道具的基础：

道具cbravobernal、瑞士斯皮迪、flixos90、乔麦吉尔、gziolo。

为了理解WordPress项目对贡献者的期望，请查看《核心手册》中的“贡献者归因”页面.

@github-操作 GitHub操作
复制链接

github-操作 机器人程序 评论2024年3月11日

使用WordPress Playground进行测试

可以使用WordPress游戏场实例。

WordPress游戏场是一个完全在浏览器中创建完整WordPress实例的实验项目。

需要注意的一些事项

  • 无法在Playground中访问插件和主题目录。
  • 关闭带有Playground实例的选项卡时，所有更改都将丢失。
  • 刷新页面时，所有更改都将丢失。
  • 每次单击下面的链接时，都会创建一个新的实例。
  • 每次更新此拉请求时，都会创建一个包含所有更改的新ZIP文件。如果Playground实例中没有反映更改，
    最新的构建可能失败或尚未完成。检查要确保的工作流运行列表.

有关这些限制的更多详细信息，请查看限制页面在WordPress Playground文档中。

用WordPress Playground测试这个拉请求.

@瑞士风格
复制链接
成员

复写的副本@吉奥罗牌手表 @路易希兰兹供审阅

费利萨尔茨
费利萨尔茨 检验过的 2024年3月11日
查看已审阅的更改
复制链接
成员

@费利萨尔茨 费利萨尔茨 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

LGTM，只有一个反馈。

src/wp-includes/interactivity-api/interactionity-api.php 过时的 显示已解决 隐藏已解决
@吉奥罗牌手表
复制链接
成员

我没有足够的洞察力来判断优先级是20、99还是PHP_INT_最大值在这里会更好。最后的方法很可能会影响render_block（渲染块）_*此函数中使用的过滤器使用了完全相同的推理：

wordpress-develop/src/wp-includes/interactivity-api/interactity-api.php

第60至63行在里面3c2a87e公司

*使用20的优先级以确保其他筛选器可以添加其他
*处理开始之前的指令。
*/
添加筛选器('渲染块_'.$块_名称,$进程交互块,20,2);

要记住的一个重要方面是，我们一直在讨论当HTML API能够处理所有HTML标记时，我们可以探索基于为<body>标记。在这种情况下，我们不需要使用这些过滤器。无论如何，我们还没有达到目标，我们还必须确认在切换到该方法之前，性能是相等的或有所提高的。

瑞士风格
瑞士风格 批准了这些更改 2024年3月13日
查看已审阅的更改
复制链接
成员

@瑞士风格 瑞士风格 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

在我看来不错，但正在等待来自@吉奥罗牌手表

吉奥罗牌手表
吉奥罗牌手表 批准了这些更改 2024年3月13日
查看已审阅的更改
复制链接
成员

@齐奥罗 吉奥罗牌手表 留下了评论

选择隐藏此评论的原因

将显示原因，以便向其他人描述此评论。了解更多信息.

一切都好。我的反馈得到了解决。所有内容都与包含的更改很好地一致👍🏻

@瑞士风格
复制链接
成员

已承诺中继https://core.trac.wordpress.org/changeset/57826

需要双重签字(已审查开发)用于向6.5分支进行反向移植。

已合并
免费注册 在GitHub上加入此对话.已经有帐户了吗？登录以发表评论
审阅者

@吉奥罗牌手表 吉奥罗牌手表 gziolo批准了这些修改

@瑞士风格 瑞士风格 瑞士政府批准了这些修改

@费利萨尔茨 费利萨尔茨 费利萨尔茨批准了这些修改

@乔麦吉尔 乔麦吉尔 乔麦吉尔留下了评论

受让人
未分配人员
标签
还没有
项目
还没有
里程碑
无里程碑
5名参与者
@中央银行行长 @瑞士风格 @吉奥罗牌手表 @乔麦吉尔 @费利萨尔茨