مشاكل في تجاوز الـ getters في وحدة تحكم (3.0.0)

merefield · 14 يناير 2023، 6:57م

لدي هذا الرمز في المُهيئ الخاص بي، وهو يحاول تجاوز المُستدعي لـ allAdminRoutes:

    api.modifyClass('controller:admin-plugins', {
      pluginId: PLUGIN_ID,

      get allAdminRoutes() {
        return this.model
          .filter(p => p?.enabled)
          .map(p => {
            return p.admin_route;
          })
          .filter(Boolean);
      },
    });

إنه يفشل مع:

TypeError: this.model is undefined

الآن، لن أمانع، لكنه مجرد رمز المُستدعي الأصلي (لم أقم بتغييره بعد!)

ما الذي أفتقده؟ هل تجاوز المُستدعي بهذه الطريقة غير مدعوم بأي شكل من الأشكال أو أنه يُدخل نوعًا من مشكلة النطاق؟

merefield · 16 يناير 2023، 9:45ص

يبدو أن هذه ليست مشكلة مع الأحدث. (كان هذا مجرد اختبار على الإنتاج.)

سأرى ما إذا كان بإمكاني إعادة إنتاج المشكلة على إصدار مستقر نظيف 3.0.0.

david · 16 يناير 2023، 10:38ص

يجب أن يعمل المثال، وأنا لست على علم بأي تغييرات ذات صلة بين الإصدار المستقر والآن. أخبرنا إذا تمكنت من إعادة إنتاج المشكلة!

merefield · 16 يناير 2023، 10:40ص

شكرا ديفيد. سأفعل. نعم لاحظت ذلك. الكرة بالتأكيد في ملعبي في الوقت الحالي لأن البيئة التي واجهت فيها هذا لم تكن نظيفة تمامًا.

merefield · 16 يناير 2023، 10:23م

حسنًا ديفيد، يبدو (وهذا غريب):

يعمل في بيئة الإنتاج
يفشل في بيئة التطوير 3.0.0 وما بعده (ربما قبل ذلك)
لقد اختبرت على بيئة تطوير عادية بدون دوكر، وأيضًا على بيئة تطوير دوكر مخصصة نظيفة جدًا (ولكن ربما غير مكتملة التكوين).

إليك الأخطاء:

حدث خطأ:

- أثناء التصيير:
  - المستوى الأعلى
    التطبيق
      جذر ديسكورس
        غلاف الشريط الجانبي
          مسؤول
            غلاف المسؤول
              عنصر التنقل
                رابط إلى
                  -رابط إلى

تم اكتشاف خطأ غير معالج (في الوعد) TypeError: أثناء إنشاء رابط للمسار "adminPlugins": هذا.نموذج غير معرف
    الحصول على جميع مسارات المسؤول plugins-admin.js:20

لاحظ أنه يمكنني وضع عبارة مصحح الأخطاء قبل return this.model ولن يتم تشغيل مصحح الأخطاء أبدًا.

لقد نظر @RGJ بلطف في هذا أيضًا ويمكنه إعادة إنتاج هذه النتيجة.

هذه مشكلة بعض الشيء، حيث نحتاج بوضوح إلى تطوير الأشياء قبل أن تنتقل إلى بيئة الإنتاج … ولكن ليس عليّ أن أخبرك بذلك! :وجه مبتسم متعرق: كما أنه مزعج بعض الشيء لأنه مشكلة في الإصدار المستقر …

تم نقله إلى Bug في الوقت الحالي …

merefield · 16 يناير 2023، 10:46م

حسنًا، نحن أكثر تأكيدًا بوجود مشكلة هنا… حيث تم ذكر mergeMixins في تتبع المكدس

github.com/emberjs/ember.js

Issue with `mergeMixins` when accessing a service in a getter

opened 06:29PM - 03 Apr 20 UTC

nlfurniss

Recently found an issue with mergeMixins. it affects Ember versions at least as …far back as 3.12 and also current release and LTS. As I understand it, the issue is when a mixin defines a property (doesn’t necessarily have a value) then we try to access the value during construction before owner is set. This breaks getters that access services. Shared findings with @pzuraq. Error message: `Attempting to lookup an injected property on an object without a container, ensure that the object was instantiated via a container. Error: Assertion Failed: Attempting to lookup an injected property on an object without a container, ensure that the object was instantiated via a container.` [Code where issue arrises ](https://github.com/emberjs/ember.js/blob/v3.17.3/packages/@ember/-internals/metal/lib/mixin.ts#L475) (Please note: the original link was to `master` 😞. I've changed the tag to the version of ember published the day before, but I have no memory of the original code it linked to) [Reproduction](https://ember-twiddle.com/3147343e2dbf0e4c6965a50fa7e7389d?openFiles=controllers.application%5C.js%2C) ([Gist](https://gist.github.com/nlfurniss/3147343e2dbf0e4c6965a50fa7e7389d))

(شكرًا لـ @RGJ على اكتشاف هذا)

david · 18 يناير 2023، 5:57م

حفر رائع - شكراً @merefield و @RGJ

لست متأكداً من مقدار ما يمكننا فعله لإصلاح هذا داخل Discourse - كما لاحظت، تبدو المشكلة في مكان أعلى في Ember. ربما يمكننا إصلاح الطريقة التي تسبب المشكلة

للبدء، أضفت حالة اختبار بسيطة تفشل حتى نتمكن من تتبع التقدم. هناك بعض التعليقات في الاختبار تصف سبب إثارة المشكلة بواسطة نظام modifyClass الخاص بنا:

merefield · 18 يناير 2023، 5:59م

نعم، شكراً لجهودكم أيضاً. هل من الغريب أنهم تركوا هذه المشكلة دون معالجة؟ هل نعرف ما إذا كان (Ember) 4.x سيحلها؟

david · 18 يناير 2023، 6:05م

للأسف، يبدو أن المشكلة لا تزال قائمة في Ember 4.x

merefield · 18 يناير 2023، 8:53م

من المثير للاهتمام أن هذا يبدو التقدم الوحيد في مشروع Ember، ضمن PR المرتبط الذي لا يزال قيد المسودة. أتساءل عما إذا كان كل ما هو مطلوب هو طلب مهذب لتحديد أولوياته عندما يصبح الوقت متاحًا؟:

github.com/emberjs/ember.js

[BUGFIX LTS] Fix mixin merging with getters

main ← mixin-merging-18860

opened 05:12PM - 23 Jun 22 UTC

chriskrycho

+35 -0

Our handling of getters and setters works correctly for cases where we have only… classic classes, or only native classes, or native classes extending classic classes. In some cases, however, it fails when we have "zebra-striping" of class types, with a classic class extending a native class which extends a (stack of) classic class(es). We correctly handle native getters in mixin and POJOs passed to `.extend()`, but—quite reasonably—do *not* handle getters on native classes when calling `.extend(SomeNativeClass)`. The result is that those getters are not decorated the way they are in mixins, so mixin merging ends up invoking the getter rather than defining a decorator which returns it. Fixes #18860

أتفهم تمامًا أن هذا تم فتحه بواسطة أحد الموظفين في طرف ثالث.

هل يستحق الأمر طرح مشكلة في GitHub - emberjs/ember.js: Ember.js - A JavaScript framework for creating ambitious web applications والإشارة إلى هذا PR، وهذا الموضوع، واختبارك؟

david · 18 يناير 2023، 10:06م

نعم، يعتمد اختباري على اختبارهم (ولكن مع واجهة برمجة تطبيقات api.modifyClass الخاصة بـ discourse). لا أعتقد أنه يجب علينا فتح مشكلة جديدة - إنها متطابقة 100٪ مع المشكلة المفتوحة بالفعل. سنرى ما إذا كان بإمكاننا فعل أي شيء للحصول على الأولوية.

david · 2 مارس 2023، 11:21ص

نحقق بعض التقدم هنا - لدينا الآن طلب سحب (PR) مع إصلاح مفتوح على [BUGFIX LTS] Don't run getters while applying mixins by wycats · Pull Request #20388 · emberjs/ember.js · GitHub

بمجرد دمج هذا ونقله إلى Ember 3.28، سنهدف إلى تطبيقه على Discourse في أسرع وقت ممكن.

merefield · 2 مارس 2023، 2:21م

إعادته؟! تساءلت عن ذلك، أخبار رائعة!

شكراً لك على توجيه هذا يا ديفيد!

david · 5 مايو 2023، 11:59ص

تم إصدار هذا الإصلاح الآن كجزء من Ember 3.28.12، ولقد حدثنا Discourse لاستخدامه. يرجى إخبارنا إذا كنت لا تزال ترى أي مشاكل مع modifyClass بعد التحديث إلى أحدث إصدار من Discourse.

merefield · 5 مايو 2023، 12:01م

رائع ديفيد، شكراً جزيلاً!

david · 15 مايو 2023، 7:00ص

تم إغلاق هذا الموضوع تلقائيًا بعد 9 أيام. لم يعد يُسمح بالردود الجديدة.

الموضوع		الردود	مرات العرض
api.modifyClass sometimes(!) not working Bug	14	2062	22 أبريل 2023
Blank page and error in Theme Admin Bug	7	434	27 فبراير 2023
Problem tracking nested structure Dev	7	83	9 أغسطس 2024
Adding a mixin to a class in a plugin? Dev	4	1194	17 ديسمبر 2019
How to Modify Ember Route Class Dev	3	1572	10 مارس 2020

مشاكل في تجاوز الـ getters في وحدة تحكم (3.0.0)

الموضوعات ذات الصلة