PullHotlinkedImages no vuelve a reemplazar las imágenes después de editar el post

Arkshine · 8 Junio, 2020 13:33

Hola,

Relacionado con:

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.

En realidad, el problema con la descarga de imágenes en temas editados con system no parece estar completamente resuelto. Perdón por no haberlo notado antes. Por ejemplo, este uno o este otro.

Como recordatorio, en nuestro caso, se trata de un cambio automático de categoría (usando la API).

Mi cliente todavía tiene temas donde PullHotlinkedImages no hace nada, sin importar qué:

Desde un simple Save Edit o incluso
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)

Sin embargo, funciona si creas un nuevo tema con el mismo contenido.

@david

david · 8 Junio, 2020 14:07

Esto es extraño, debe haber alguna diferencia en el tema creado a través de la API.

¿Cómo se crean exactamente los mensajes? ¿Puedes compartir los parámetros de la API que estás utilizando?

Arkshine · 8 Junio, 2020 14:24

Cuando digo “nuevo tema”, me refiero directamente en Discourse, no a través de la API.

Originalmente, es un tema creado desde WP-Discourse; por lo tanto, debería 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 ejemplo, si observas la primera edición del tema enlazado anteriormente, verás claramente cómo el sistema reemplaza las imágenes. Por lo tanto, es probable que WP-Discourse no sea el problema.

Más tarde, el autor original actualizó la publicación de WordPress, el tema de Discourse se actualizó y las imágenes originales volvieron a aparecer:

Luego ocurrió un cambio de categoría, y a partir de ahí PullGotlinkedImages no hace nada:

Si deseas ver el código utilizado para el cambio de categoría, es algo así:

david · 8 Junio, 2020 14:34

Perfecto, ¡gracias por el resumen detallado! Creo que el problema está aquí:

Logré crear un caso de prueba que falla para esto. Parece que solo realizamos la sustitución de la URL cuando la imagen se descarga por primera vez. Si ya hemos descargado la imagen una vez, no se volverá a sustituir.

Este es un caso de prueba que falla y demuestra el problema

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

Lo añadiré a mi lista de tareas para corregirlo. Debería poder ocuparme de ello en las próximas semanas.

Arkshine · 8 Junio, 2020 14:41

¡Genial!

Parece que tienes razón. He revisado otros temas y puedo ver que antes el sistema reemplazaba correctamente las imágenes.

Arkshine · 10 Julio, 2020 00:16

¿Hay alguna novedad o estado al respecto?

david · 10 Julio, 2020 08:19

¡Aún no, pero sigue en la lista!

david · 6 Agosto, 2020 09:03

Esto debería resolverse tras

He agregado una prueba para este caso para evitar que vuelva a ocurrir en el futuro.

Arkshine · 7 Agosto, 2020 00:50

Acabo de editar/guardar rápidamente algunos temas manualmente que fueron editados por system y parece que PullHotlinkedImages ya funciona. Probaré la línea de comandos más tarde, pero sin duda funcionará bien.

¡Gracias!

david · 8 Agosto, 2020 17:00

Este tema se cerró automáticamente después de 2 días. Ya no se permiten nuevas respuestas.

Tema		Respuestas	Vistas
Discourse not downloading remote images anymore? Support	7	875	4 Julio 2022
Thumbnail generation & markdown rendering issue Support	14	2002	30 Mayo 2020
Cannot PullHotlinkedImages for some domains Support	6	563	16 Marzo 2022
Image_url is null after update Support	6	1113	10 Junio 2020
Oneboxed image links do not update after being downloaded to local Bug	18	1365	6 Julio 2022

PullHotlinkedImages no vuelve a reemplazar las imágenes después de editar el post

Temas relacionados