Embedding è ROTTO in 3.0.4 (stabile)

Bug
7

Quindi main ha questo

TopicEmbed.import_remote(@embed_url, user: User.find_by(username_lower: username.downcase))

e stable ha questo

TopicEmbed.import_remote(user, @embed_url)

Nota l’ordine dei parametri.

Ora il backport della patch di sicurezza ha modificato la firma della funzione su stable al nuovo ordine dei parametri, quindi

def self.import_remote(import_user, url, opts = nil)

è diventato

def self.import_remote(url, opts = nil)

e ora il parametro url riceve un oggetto User.

La modifica della chiamata alla funzione risolve il problema

diff --git a/lib/topic_retriever.rb b/lib/topic_retriever.rb
index b798df6cd7..6186ce5868 100644
--- a/lib/topic_retriever.rb
+++ b/lib/topic_retriever.rb
@@ -50,6 +50,6 @@ class TopicRetriever
     user = User.where(username_lower: username.downcase).first
     return if user.blank?

-    TopicEmbed.import_remote(user, @embed_url)
+    TopicEmbed.import_remote(@embed_url, user: user)
   end
 end

@blake

8 Mi Piace