PullHotlinkedImages ersetzt Bilder nach einer Bearbeitung des Beitrags nicht erneut

Arkshine · 8. Juni 2020 um 13:33

Hey,

Bezugnehmend auf:

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.

Tatsächlich scheint das Problem beim Herunterladen von Bildern in bearbeiteten Themen mit system nicht vollständig gelöst zu sein. Entschuldige, dass ich es nicht früher bemerkt habe. Zum Beispiel dieses oder dieses.

Als Erinnerung: In unserem Fall handelt es sich um eine automatische Kategorienänderung (über die API).

Mein Kunde hat immer noch Themen, bei denen PullHotlinkedImages nichts bewirkt, egal was man tut:

Durch einfaches Save Edit oder sogar
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)

Es funktioniert jedoch, wenn man ein neues Thema mit demselben Inhalt erstellt.

@david

david · 8. Juni 2020 um 14:07

Das ist seltsam, es muss einen Unterschied bei der Erstellung des Themas über die API geben.

Wie genau werden die Beiträge erstellt? Können Sie die von Ihnen verwendeten API-Parameter teilen?

Arkshine · 8. Juni 2020 um 14:24

Wenn ich von “neuem Thema” spreche, meine ich direkt in Discourse, nicht über die API.

Ursprünglich ist es ein Thema, das von WP-Discourse erstellt wurde; also sollte es diesen Code sein:

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

Wenn Sie sich beispielsweise die erste Bearbeitung des oben verlinkten Themas ansehen, sehen Sie deutlich, wie das System die Bilder ersetzt. Daher ist WP-Discourse wahrscheinlich nicht das Problem.

Später aktualisierte der ursprüngliche Autor den WordPress-Beitrag, das Discourse-Thema wurde aktualisiert und die ursprünglichen Bilder sind wieder da:

Dann fand eine Kategorienänderung statt, und ab diesem Zeitpunkt macht PullGotlinkedImages nichts mehr:

Wenn Sie den Code für die Kategorienänderung sehen möchten, sieht er ungefähr so aus:

david · 8. Juni 2020 um 14:34

Perfekt, danke für die detaillierte Zusammenfassung! Ich denke, das Problem liegt hier:

Ich habe es geschafft, einen fehlerhaften Testfall dafür zu erstellen. Es scheint, dass wir den URL-Ersatz nur durchführen, wenn das Bild zum ersten Mal heruntergeladen wird. Wenn wir das Bild bereits einmal heruntergeladen haben, wird es nicht erneut ersetzt.

Dies ist ein fehlerhafter Test, der das Problem verdeutlicht:

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

Ich werde ihn auf meine Liste für die Behebung setzen. Ich sollte in den nächsten paar Wochen dazu kommen.

Arkshine · 8. Juni 2020 um 14:41

Toll!

Es scheint, du hast recht. Ich habe andere Themen überprüft und kann sehen, dass das System früher die Bilder gut ersetzt hat.

Arkshine · 10. Juli 2020 um 00:16

Gibt es dazu Neuigkeiten oder einen Status?

david · 10. Juli 2020 um 08:19

Noch nichts, aber es steht noch auf der Liste!

david · 6. August 2020 um 09:03

Dies sollte nachfolgend behoben werden

Ich habe einen Testfall für diesen Fall hinzugefügt, damit es in Zukunft nicht wieder auftritt.

Arkshine · 7. August 2020 um 00:50

Ich habe gerade einige von system bearbeitete Themen manuell schnell bearbeitet/gespeichert, und es scheint, dass PullHotlinkedImages jetzt funktioniert. Ich werde den Befehl später ausprobieren, aber ich bin mir sicher, dass er ebenfalls einwandfrei funktionieren wird.

Vielen Dank!

david · 8. August 2020 um 17:00

Dieses Thema wurde automatisch nach 2 Tagen geschlossen. Neue Antworten sind nicht mehr erlaubt.

Thema		Antworten	Aufrufe
Discourse not downloading remote images anymore? Support	7	875	4. Juli 2022
Thumbnail generation & markdown rendering issue Support	14	2002	30. Mai 2020
Cannot PullHotlinkedImages for some domains Support	6	563	16. März 2022
Image_url is null after update Support	6	1113	10. Juni 2020
Oneboxed image links do not update after being downloaded to local Bug	18	1365	6. Juli 2022

PullHotlinkedImages ersetzt Bilder nach einer Bearbeitung des Beitrags nicht erneut

Verwandte Themen