PullHotlinkedImages 在帖子编辑后不再重新替换图片

Arkshine · 2020 年6 月 8 日 13:33

嘿，

FIX: Pull hotlinked images even when edited by system users (#9890)

committed 12:07PM - 29 May 20 UTC

Previously the pull hotlinked images job was skipped after system edits. This en…sured that we never had an infinite loop of system-edit/pull-hotlinked/system-edit/pull-hotlinked etc. A side effect was that edits made by system for any other reason (e.g. API, removing full quotes) would prevent pulling hotlinked images. This commit removes the system edit check, and replaces it with another method to avoid an infinite job scheduling loop.

实际上，在已编辑的话题中下载带有 system 标记的图片的问题似乎并未完全解决。很抱歉我之前没有注意到这一点。例如这个话题或这个话题。

提醒一下，在我们的案例中，这是通过 API 进行的自动分类变更。

我的客户仍然有一些话题，无论怎么做 PullHotlinkedImages 都没有任何效果：

无论是简单的 Save Edit，甚至是
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)

不过，如果你用相同的内容创建一个新话题，它就能正常工作。

@david

david · 2020 年6 月 8 日 14:07

这很奇怪，通过 API 创建的主题一定有什么不同的地方。

帖子具体是怎么创建的？你能分享一下你使用的 API 参数吗？

Arkshine · 2020 年6 月 8 日 14:24

当我提到“新主题”时，我指的是直接在 Discourse 上创建，而不是通过 API。

最初，该主题是由 WP-Discourse 创建的；因此相关代码应该是这个：

github.com/discourse/wp-discourse

lib/discourse-publish.php

main


      
          	 * @param string $raw The raw content of the post.
          	 */
          	public function sync_to_discourse_without_lock( $post_id, $title, $raw ) {
          		return $this->sync_to_discourse_work( $post_id, $title, $raw );
          	}
          
          	/**
          	 * Syncs a post to Discourse.
          	 *
          	 * @param int    $post_id The post id.
          	 * @param string $title The post title.
          	 * @param string $raw The content of the post.
          	 *
          	 * @return null
          	 */
          	protected function sync_to_discourse_work( $post_id, $title, $raw ) {
          		$options                     = $this->options;
          		$discourse_id                = $this->dc_get_post_meta( $post_id, 'discourse_post_id', true );
          		$post                        = get_post( $post_id );
          		$author_id                   = $post->post_author;

This file has been truncated. show original

例如，如果你查看上面链接的主题的首次编辑，你会清楚地看到系统替换了图片。因此，WP-Discourse 很可能不是问题所在。

后来，原始作者更新了 WordPress 帖子，Discourse 主题也随之更新，原始图片又回来了：

随后发生了分类变更，从那时起，PullGotlinkedImages 就不再执行任何操作：

如果你想知道用于分类变更的代码，大致如下：

david · 2020 年6 月 8 日 14:34

太好了，感谢详细的总结！我认为问题出在这里：

我成功复现了一个失败的测试用例。看来我们只在图片首次下载时才执行 URL 替换。如果图片之前已经下载过，就不会再次替换。

这是一个展示该问题的失败测试：

https://github.com/davidtaylorhq/discourse/blob/27ccf99d162005f82bf4d4e69f860cf78affcfcb/spec/jobs/pull_hotlinked_images_spec.rb#L63-L80

我会将其列入待修复清单。我预计在未来几周内能够着手处理。

Arkshine · 2020 年6 月 8 日 14:41

太棒了！

看来你说得对，我查看了其他主题，发现系统之前确实能正确替换图片。

Arkshine · 2020 年7 月 10 日 00:16

关于此事有什么最新消息或状态更新吗？

david · 2020 年7 月 10 日 08:19

还没有，但它仍在列表中！

david · 2020 年8 月 6 日 09:03

这应该在以下提交后得到解决

我为这种情况添加了一个测试规范，以防止将来出现回归。

Arkshine · 2020 年8 月 7 日 00:50

我刚刚手动快速编辑并保存了几个由 system 编辑的主题，看起来 PullHotlinkedImages 现在可以正常工作了。稍后我会尝试命令行操作，但毫无疑问它也会运行正常。

谢谢！

david · 2020 年8 月 8 日 17:00

本主题在 2 天后自动关闭。不再允许新的回复。

话题		回复	浏览量
Discourse not downloading remote images anymore? Support	7	868	2022 年7 月 4 日
Thumbnail generation & markdown rendering issue Support	14	1978	2020 年5 月 30 日
Cannot PullHotlinkedImages for some domains Support	6	542	2022 年3 月 16 日
Image_url is null after update Support	6	1113	2020 年6 月 10 日
Oneboxed image links do not update after being downloaded to local Bug	18	1302	2022 年7 月 6 日

PullHotlinkedImages 在帖子编辑后不再重新替换图片

相关话题