PullHotlinkedImages non sostituisce più le immagini dopo la modifica del post

Arkshine · 8 Giugno 2020, 1:33pm

Ehi,

Riguardo 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.

In realtà, il problema relativo al download delle immagini nei topic modificati con system non sembra essere stato risolto completamente. Scusa per non averlo notato prima. Ad esempio, questo o questo.

Per ricordare, nel nostro caso si tratta di un cambio automatico di categoria (tramite API).

Il mio cliente ha ancora dei topic in cui PullHotlinkedImages non fa nulla, indipendentemente da cosa si faccia:

Da un semplice Save Edit o anche
Post.update_all(baked_version: nil) + Jobs.enqueue_in(10, :pull_hotlinked_images, post_id: <id>)

Tuttavia, funziona se si crea un nuovo topic con lo stesso contenuto.

@david

david · 8 Giugno 2020, 2:07pm

È strano, deve esserci qualcosa di diverso nell’argomento creato tramite l’API.

Come vengono creati esattamente i post? Puoi condividere i parametri dell’API che stai utilizzando?

Arkshine · 8 Giugno 2020, 2:24pm

Quando dico “nuovo argomento”, intendo direttamente su Discourse, non tramite API.

Originariamente, è un argomento creato da WP-Discourse; quindi dovrebbe essere questo codice:

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

Ad esempio, se guardi la prima modifica dell’argomento collegato sopra, vedi chiaramente il sistema che sostituisce le immagini. Quindi, WP-Discourse probabilmente non è il problema.

In seguito, l’autore originale ha aggiornato il post di WordPress, l’argomento su Discourse è stato aggiornato e le immagini originali sono tornate:

Poi è avvenuta una modifica della categoria e da quel momento in poi PullGotlinkedImages non fa nulla:

Se vuoi il codice utilizzato per la modifica della categoria, è più o meno questo:

david · 8 Giugno 2020, 2:34pm

Perfetto, grazie per il riassunto dettagliato! Penso che il problema sia qui:

Sono riuscito a creare un caso di test fallito per questo. Sembra che eseguiamo la sostituzione dell’URL solo quando l’immagine viene scaricata per la prima volta. Se abbiamo già scaricato l’immagine una volta, non verrà sostituita di nuovo.

Questo è un test fallito che dimostra il problema

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

Lo aggiungerò alla mia lista da correggere. Dovrei riuscire a occuparmene nelle prossime settimane.

Arkshine · 8 Giugno 2020, 2:41pm

Fantastico!

Sembra che tu abbia ragione: ho controllato altri argomenti e vedo che in precedenza il sistema sostituiva correttamente le immagini.

Arkshine · 10 Luglio 2020, 12:16am

Ci sono novità o aggiornamenti su questo?

david · 10 Luglio 2020, 8:19am

Non ancora, ma è ancora sulla lista!

david · 6 Agosto 2020, 9:03am

Questo dovrebbe essere risolto dopo

Ho aggiunto uno spec per questo caso in modo che non si verifichi una regressione in futuro.

Arkshine · 7 Agosto 2020, 12:50am

Ho appena modificato/salvato rapidamente alcuni argomenti manualmente che erano stati modificati da system e sembra che PullHotlinkedImages funzioni ora. Proverò la riga di comando più tardi, ma non c’è dubbio che funzionerà bene.

Grazie!

david · 8 Agosto 2020, 5:00pm

Questo argomento è stato automaticamente chiuso dopo 2 giorni. Non sono più ammesse nuove risposte.

Argomento		Risposte	Visualizzazioni
Discourse not downloading remote images anymore? Support	7	875	Luglio 4, 2022
Thumbnail generation & markdown rendering issue Support	14	2002	Maggio 30, 2020
Cannot PullHotlinkedImages for some domains Support	6	563	Marzo 16, 2022
Image_url is null after update Support	6	1113	Giugno 10, 2020
Oneboxed image links do not update after being downloaded to local Bug	18	1365	Luglio 6, 2022

PullHotlinkedImages non sostituisce più le immagini dopo la modifica del post

Argomenti correlati