Probleme beim Überschreiben von Gettern in einem Controller (3.0.0)

Beitragen Fehler
1

Ich habe diesen Code in meinem Initialisierer, der versucht, den Getter für allAdminRoutes zu überschreiben:

    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);
      },
    });

Er schlägt mit folgender Fehlermeldung fehl:

TypeError: this.model is undefined

Nun, es wäre mir egal, aber es ist nur der ursprüngliche Getter-Code (ich habe ihn noch nicht einmal geändert!)

Was übersehe ich? Wird das Überschreiben eines Getters auf diese Weise nicht unterstützt oder führt es eine Art von Gültigkeitsbereichsproblem ein?

4 „Gefällt mir“
api.modifyClass sometimes(!) not working
How to override the buildQuote function?
2

Seltsamerweise scheint dies kein Problem mit der neuesten Version zu sein. (Das war nur ein Test in der Produktion.)

Ich werde sehen, ob ich dies auf einer sauberen 3.0.0 Stable reproduzieren kann.

1 „Gefällt mir“
4

Das Beispiel sollte funktionieren, und mir sind keine relevanten Änderungen zwischen der stabilen Version und jetzt bekannt. Lassen Sie uns wissen, wenn es Ihnen gelingt, das Problem zu reproduzieren!

1 „Gefällt mir“
5

Danke David. Werde ich machen. Ja, das ist mir aufgefallen. Der Ball liegt definitiv bei mir, da die Umgebung, in der ich das erlebt habe, nicht ganz sauber war.

1 „Gefällt mir“
7

OK David, es sieht so aus (und das ist seltsam):

  • funktioniert in der Produktion
  • scheitert ab Entwicklung 3.0.0 (möglicherweise auch schon früher)
    • Ich habe es sowohl in einer standardmäßigen, nicht-dockerisierten Dev-Umgebung getestet als auch in einer sehr sauberen (aber möglicherweise unvollständig konfigurierten) maßgeschneiderten Docker-Dev-Umgebung.

Hier sind die Fehlermeldungen:

Fehler aufgetreten:

- Beim Rendern:
  -top-level
    application
      discourse-root
        sidebar-wrapper
        admin
          admin-wrapper
            nav-item
              link-to
                -link-to

Uncaught (in promise) TypeError: Beim Generieren des Links zur Route „adminPlugins": this.model ist undefined
    get allAdminRoutes admin-plugins.js:20

Beachte, dass ich eine Debugger-Anweisung vor return this.model einfügen kann, aber der Debugger wird nie ausgelöst.

@RGJ hat sich das ebenfalls freundlicherweise angesehen und kann dieses Ergebnis reproduzieren.

Das ist ein gewisses Problem, da wir offensichtlich Dinge entwickeln müssen, bevor sie in die Produktion gehen … aber das muss ich dir nicht sagen! :sweat_smile: Es ist auch etwas ärgerlich, dass das Problem in der Stable-Version auftritt …

Derzeit in Contribute > Bug verschoben …

2 „Gefällt mir“
8

OK, wir sind uns hier eines Problems noch sicherer … da mergeMixins im Stack-Trace erwähnt wird

image
image676×196 3.28 KB

(danke an @RGJ für diesen Fund)

4 „Gefällt mir“
10

Tolle Recherche – danke @merefield und @RGJ

Ich bin mir nicht sicher, wie viel wir innerhalb von Discourse beheben können – wie Sie bemerkt haben, scheint das Problem stromaufwärts in Ember zu liegen. Vielleicht können wir die problematische Methode patchen :thinking:

Um uns den Einstieg zu erleichtern, habe ich einen minimalen, fehlschlagenden Testfall hinzugefügt, damit wir den Fortschritt verfolgen können. Es gibt einige Kommentare im Test, die beschreiben, warum das Problem durch unser modifyClass-System ausgelöst wird:

4 „Gefällt mir“
11

Ja, danke auch für deine Bemühungen. Es ist seltsam, dass sie dieses Problem ungelöst gelassen haben? Wissen wir, ob (Ember) 4.x es lösen wird?

12

Leider scheint das Problem in Ember 4.x immer noch zu bestehen :cry:

1 „Gefällt mir“
13

Interessanterweise scheint das der einzige Fortschritt beim Ember-Projekt zu sein, innerhalb des zugehörigen PR, der sich noch im Entwurfsstadium befindet. Ich frage mich, ob alles, was erforderlich wäre, eine höfliche Bitte wäre, ihn zu priorisieren, wenn die Zeit verfügbar ist?:

Ich verstehe vollkommen, dass dies von einem Mitarbeiter eines Drittanbieters eröffnet wurde.

Lohnt es sich, ein Problem unter GitHub - emberjs/ember.js: Ember.js - A JavaScript framework for creating ambitious web applications zu melden und auf diesen PR, dieses Thema und Ihren Test zu verweisen?

14

Ja, mein Test basiert auf ihrem (aber mit der Discourse-spezifischen api.modifyClass-API). Ich glaube nicht, dass wir ein neues Issue eröffnen sollten – es ist zu 100 % dasselbe wie das, das bereits offen ist. Wir werden sehen, ob wir etwas tun können, um es zu priorisieren.

3 „Gefällt mir“
15

Hier machen wir Fortschritte – wir haben jetzt einen PR mit einem Fix offen unter [BUGFIX LTS] Don't run getters while applying mixins by wycats · Pull Request #20388 · emberjs/ember.js · GitHub

Sobald dieser zusammengeführt und nach Ember 3.28 zurückportiert wurde, werden wir ihn so schnell wie möglich auf Discourse anwenden.

3 „Gefällt mir“
16

Zurückportiert?! Das habe ich mich gefragt, tolle Neuigkeiten!

Danke, dass du das hier betreut hast, David!

18

Diese Korrektur wurde nun als Teil von Ember 3.28.12 veröffentlicht, und wir haben Discourse aktualisiert, um es zu verwenden. Bitte lassen Sie uns wissen, wenn Sie nach der Aktualisierung auf das neueste Discourse immer noch Probleme mit modifyClass feststellen.

6 „Gefällt mir“
19

Super David, vielen Dank!

2 „Gefällt mir“
20

Dieses Thema wurde nach 9 Tagen automatisch geschlossen. Neue Antworten sind nicht mehr möglich.