Die Dimensionen stammen von oembed:
https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F
Welches hat:
Ich denke, der Fehler hier ist, dass wenn der Typ “rich” ist und wir nicht die gesamte Präsentation aus der oembed html-Payload abrufen, wir das Hinzufügen von Dimensionen überspringen sollten, da wir an diesen Daten nicht interessiert sind.
Dies behebt das Problem:
diff --git a/lib/onebox/engine/standard_embed.rb b/lib/onebox/engine/standard_embed.rb
index e3175d6247..fc8c300d81 100644
--- a/lib/onebox/engine/standard_embed.rb
+++ b/lib/onebox/engine/standard_embed.rb
@@ -159,8 +159,9 @@ module Onebox
@json_ld ||= Onebox::JsonLd.new(html_doc)
end
- def set_from_normalizer_data(normalizer)
+ def set_from_normalizer_data(normalizer, skip_dimensions: false)
normalizer.data.each do |k, _|
+ next if skip_dimensions && k.in?(%i[width height])
v = normalizer.public_send(k)
@raw[k] ||= v unless v.nil?
end
@@ -179,7 +180,8 @@ module Onebox
def set_oembed_data_on_raw
oembed = get_oembed
- set_from_normalizer_data(oembed)
+ skip_dimensions = oembed.data[:type] == "rich"
+ set_from_normalizer_data(oembed, skip_dimensions:)
end
def set_json_ld_data_on_raw
Ich bin mir jedoch nicht sicher, welche anderen Nebenwirkungen dies hätte. Ich markiere das Mitglied-Erfahrungsteam, das sich dies im nächsten Monat ansehen wird.
Ich bin zurückhaltend, einfach meinen Patch hinzuzufügen, da es hier viele Ebenen und Komplexität gibt. Jemand muss sicherstellen, dass wir den Patch auf eine sehr sichere und getestete Weise hinzufügen können.
Dies ist pri-medium, da die Auswirkungen dieses Fehlers angesichts der Tatsache, dass wp-json dies anzeigt, ziemlich weitreichend sind.
