PullHotlinkedImages não substitui as imagens novamente após a edição do post

Arkshine · Junho 8, 2020, 1:33pm

Olá,

Relacionado a:

github.com/discourse/discourse

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

committed 12:07PM - 29 May 20 UTC

davidtaylorhq

+12 -11

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.

Na verdade, o problema com o download de imagens em tópicos editados com system não parece ter sido totalmente resolvido. Desculpe por não ter notado isso antes. Por exemplo, este ou este.

Como lembrete, no nosso caso, trata-se de uma alteração automática de categoria (usando a API).

Meu cliente ainda tem tópicos onde PullHotlinkedImages não faz nada, não importa o que se faça:

Apenas com um Save Edit ou até mesmo
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)

No entanto, funciona se você criar um novo tópico com o mesmo conteúdo.

@david

david · Junho 8, 2020, 2:07pm

Isso é estranho, deve haver algo diferente sobre o tópico criado via API.

Como exatamente as postagens são criadas? Você pode compartilhar os parâmetros da API que está usando?

Arkshine · Junho 8, 2020, 2:24pm

Quando digo “novo tópico”, refiro-me diretamente ao Discourse, não via API.

Originalmente, é um tópico criado a partir do WP-Discourse; então deve ser este código:

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

Por exemplo, se você observar a primeira edição do tópico vinculado acima, verá claramente o sistema substituindo as imagens. Portanto, o WP-Discourse provavelmente não é o problema.

Mais tarde, o autor original atualizou a postagem no WordPress, o tópico no Discourse foi atualizado e as imagens originais voltaram:

Então ocorreu uma mudança de categoria e, a partir daí, o PullGotlinkedImages não faz mais nada:

Se você quiser o código usado para a mudança de categoria, é algo assim:

david · Junho 8, 2020, 2:34pm

Perfeito, obrigado pelo resumo detalhado! Acredito que o problema esteja aqui:

Consegui criar um caso de teste que falha para isso. Parece que substituímos a URL apenas quando a imagem é primeiramente baixada. Se já baixamos a imagem uma vez, ela não será substituída novamente.

Este é um caso de teste que falha e demonstra o problema:

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

Vou adicioná-lo à minha lista de correções. Deveria conseguir tratá-lo nas próximas semanas.

Arkshine · Junho 8, 2020, 2:41pm

Incrível!

Parece que você está certo. Verifiquei outros tópicos e posso ver que, anteriormente, o sistema substituía as imagens corretamente.

Arkshine · Julho 10, 2020, 12:16am

Alguma novidade/status sobre isso?

david · Julho 10, 2020, 8:19am

Ainda não, mas ainda está na lista!

david · Agosto 6, 2020, 9:03am

Isso deve ser resolvido após

Eu adicionei um teste para este caso para evitar que isso volte a ocorrer no futuro.

Arkshine · Agosto 7, 2020, 12:50am

Acabei de editar/salvar rapidamente alguns tópicos manualmente que haviam sido editados pelo system, e parece que o PullHotlinkedImages está funcionando agora. Vou tentar o comando pela linha de comando mais tarde, mas não há dúvida de que funcionará perfeitamente.

Obrigado!

david · Agosto 8, 2020, 5:00pm

Este tópico foi fechado automaticamente após 2 dias. Novas respostas não são mais permitidas.

Tópico		Respostas	Visualizações
Discourse not downloading remote images anymore? Support	7	875	4 de Julho de 2022
Thumbnail generation & markdown rendering issue Support	14	2002	30 de Maio de 2020
Cannot PullHotlinkedImages for some domains Support	6	563	16 de Março de 2022
Image_url is null after update Support	6	1113	10 de Junho de 2020
Oneboxed image links do not update after being downloaded to local Bug	18	1365	6 de Julho de 2022

PullHotlinkedImages não substitui as imagens novamente após a edição do post

Tópicos relacionados