صور Onebox لها نسبة عرض إلى ارتفاع غير صحيحة

[sam (Sam Saffron)]

تأتي الأبعاد من oembed:

https://hookproductivity.com/wp-json/oembed/1.0/embed?url=https%3A%2F%2Fhookproductivity.com%2Fhelp%2Fintegration%2Fother-app-developers%2F

والذي يحتوي على:

أعتقد أن الخطأ هنا هو أنه إذا كان النوع “rich”، ولم نقم بجلب العرض التقديمي بالكامل من حمولة html الخاصة بـ oembed، فيجب علينا تخطي إضافة الأبعاد، لأننا لسنا مهتمين بهذه البيانات.

هذا يصلح الأمر:

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

لكنني لست متأكدًا من الآثار الجانبية الأخرى التي قد تنجم عن ذلك، وسأقوم بالإشارة إلى فريق تجربة الأعضاء الذي سيقوم بالاطلاع على هذا الأمر خلال الشهر القادم.

أنا متردد في مجرد إضافة التصحيح الخاص بي لأنه توجد طبقات كثيرة هنا وتعقيد، يحتاج شخص ما إلى التأكد من أننا قادرون على إضافة التصحيح بطريقة آمنة ومختبرة للغاية.

---

هذا ذو أولوية متوسطة إلى عالية لأن تأثير هذا الخطأ واسع جدًا نظرًا لأن wp-json يكشف عنه.