PullHotlinkedImages ne remplace plus les images après modification du message

Arkshine · Juin 8, 2020, 1:33

Salut,

En lien avec :

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 fait, le problème de téléchargement des images dans les sujets modifiés avec system ne semble pas être entièrement résolu. Désolé de ne pas l’avoir remarqué plus tôt. Par exemple, celui-ci ou celui-là.

Pour rappel, dans notre cas, il s’agit d’un changement de catégorie automatique (via l’API).

Mon client a toujours des sujets où PullHotlinkedImages ne fait rien, peu importe ce qu’on fait :

Suite à un simple Save Edit ou même
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)

Cependant, cela fonctionne si vous créez un nouveau sujet avec le même contenu.

@david

david · Juin 8, 2020, 2:07

C’est étrange, il doit y avoir une différence dans la façon dont le sujet est créé via l’API.

Comment exactement les messages sont-ils créés ? Pouvez-vous partager les paramètres d’API que vous utilisez ?

Arkshine · Juin 8, 2020, 2:24

Quand je dis « nouveau sujet », je parle directement sur Discourse, pas via l’API.

À l’origine, il s’agit d’un sujet créé à partir de WP-Discourse ; le code concerné devrait donc être celui-ci :

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

Par exemple, si vous examinez la première modification du sujet lié ci-dessus, vous voyez clairement le système remplacer les images. Ainsi, WP-Discourse n’est probablement pas en cause.

Plus tard, l’auteur original a mis à jour l’article WordPress, le sujet Discourse a été mis à jour et les images originales sont revenues :

Ensuite, un changement de catégorie a eu lieu, et à partir de là, PullGotlinkedImages ne fait plus rien :

Si vous souhaitez voir le code utilisé pour le changement de catégorie, il ressemble à ceci :

david · Juin 8, 2020, 2:34

Parfait, merci pour le résumé détaillé ! Je pense que le problème se trouve ici :

J’ai réussi à créer un cas de test qui échoue pour ce problème. Il semble que nous ne remplacions l’URL que lorsque l’image est téléchargée pour la première fois. Si nous avons déjà téléchargé l’image une fois, elle ne sera pas remplacée à nouveau.

Voici un test qui échoue et qui démontre le problème

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

Je vais l’ajouter à ma liste de corrections. Je devrais pouvoir m’en occuper dans les prochaines semaines.

Arkshine · Juin 8, 2020, 2:41

Super !

Il semble que vous ayez raison. J’ai consulté d’autres sujets et je constate que le système remplaçait correctement les images auparavant.

Arkshine · Juillet 10, 2020, 12:16

Des nouvelles ou un état d’avancement à ce sujet ?

david · Juillet 10, 2020, 8:19

Rien pour l’instant, mais c’est toujours sur la liste !

david · Août 6, 2020, 9:03

Cela devrait être résolu suite à

J’ai ajouté un test pour ce cas afin d’éviter toute régression à l’avenir.

Arkshine · Août 7, 2020, 12:50

Je viens de modifier et d’enregistrer rapidement quelques sujets manuellement qui avaient été modifiés par system, et il semble que PullHotlinkedImages fonctionne maintenant. J’essaierai la ligne de commande plus tard, mais je ne doute pas que cela fonctionnera parfaitement.

Merci !

david · Août 8, 2020, 5:00

Ce sujet a été automatiquement fermé après 2 jours. De nouvelles réponses ne sont plus autorisées.

Sujet		Réponses	Vues
Discourse not downloading remote images anymore? Support	7	875	Juillet 4, 2022
Thumbnail generation & markdown rendering issue Support	14	2002	Mai 30, 2020
Cannot PullHotlinkedImages for some domains Support	6	563	Mars 16, 2022
Image_url is null after update Support	6	1113	Juin 10, 2020
Oneboxed image links do not update after being downloaded to local Bug	18	1365	Juillet 6, 2022

PullHotlinkedImages ne remplace plus les images après modification du message

Sujets connexes