Le dimensioni provengono da oembed:
https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F
Che ha:
Penso che il bug qui sia che se il tipo è “rich” e non stiamo recuperando l’intera presentazione dal payload html di oembed, dovremmo saltare l’aggiunta delle dimensioni, poiché non siamo interessati a questi dati.
Questo lo corregge:
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
Tuttavia, non sono sicuro di quali altri effetti collaterali potrebbe avere, segnalo al team member-experience che darà un’occhiata a questo nel corso del prossimo mese.
Sono riluttante ad aggiungere semplicemente la mia patch perché ci sono molti livelli qui e complessità, qualcuno deve assicurarsi che siamo in grado di aggiungere la patch in modo molto sicuro e testato.
Questo è pri-medium perché l’impatto di questo bug è piuttosto ampio dato che wp-json lo espone.