Neues Discourse-Update hat den Plugin-Ausgang auf seltsame Weise kaputt gemacht

LeoDavidson · 1. Dezember 2022 um 21:27

Einer der Änderungen an Discourse in der letzten Woche oder so hat mein Plugin beschädigt, und ich kann nicht herausfinden, was tatsächlich falsch ist oder wie ich es beheben kann. Das Plugin hat jahrelang einwandfrei funktioniert, bis zu diesem Zeitpunkt.

Das Plugin macht verschiedene Dinge, aber das hier scheint relevant zu sein: Es fügt der Benutzerprofilseite eine Schaltfläche hinzu.

Wenn ich diese Seite für einen beliebigen Benutzer besuche, wird der Großteil der Seite nicht gerendert (einschließlich der Schaltfläche, die mein Plugin hinzufügen soll):

(Wenn ich das Plugin über seine Einstellungs-Schaltfläche deaktiviere, wird die Seite korrekt gerendert.)

Die Javascript-Konsole in Chrome zeigt jedes Mal, wenn diese Seite aktualisiert wird, die folgende Ausnahme und den folgenden Stacktrace an:

Ich bin mir nicht sicher, was ich von diesem Fehler halten soll, da es nicht mein Code ist und das t, das not a function ist, nicht in der Zeile oder in deren Nähe steht, von der die Ausnahme ausgeht. Ziemlich verwirrend. Aber ich bin kein JS-Experte, vielleicht übersehe ich hier einfach etwas.

Beim Durchgehen mit dem Chrome-Debugger tritt das Problem in dieser Zeile auf, während mein Plugin-Template verarbeitet wird, aber ich kann nicht sehen, was tatsächlich das Problem verursacht oder ob es an etwas liegt, das ich ändern/aktualisieren muss, oder an einem Fehler in Discourse selbst.

Der Plugin-JS-Code und das HBS-Template, die meiner Meinung nach beteiligt sind, befinden sich in meinem user-profile-primary-Connector, der hier zu finden ist:

https://github.com/LeoDavidson/discourse-directoryopus/tree/master/assets/javascripts/discourse/templates/user/connectors/user-profile-primary

Ich weiß es nicht sicher, aber es schien, als könnten diese kürzlichen Discourse-Commits möglicherweise beteiligt sein:

Jeder Vorschlag wäre sehr willkommen!

david · 1. Dezember 2022 um 22:01

Hallo @LeoDavidson – danke, dass Sie dieses Problem angesprochen haben. Sie haben mit ziemlicher Sicherheit Recht, dass es mit diesen Commits zusammenhängt – ich werde mich damit befassen und versuchen herauszufinden, was sich geändert hat. Ich hatte nicht vor, Verhaltensänderungen einzuführen.

In der Zwischenzeit sollte dieser PR das Problem beheben, indem Ihre Plugin-Konnektoren in ein gängigeres Verzeichnislayout verschoben werden:

LeoDavidson · 1. Dezember 2022 um 22:38

Vielen, vielen Dank! Sowohl für den PR als auch dafür, dass Sie so schnell waren. Ich habe ihn gerade angewendet und jetzt sieht alles gut aus!

david · 2. Dezember 2022 um 13:05

Ich habe untersucht, was diese Verhaltensänderung verursacht hat. Die Connectors befanden sich wie folgt:

assets/javascripts/discourse/templates/user/connectors/{outlet-name}/{unique-name}.js
assets/javascripts/discourse/templates/user/connectors/{outlet-name}/{unique-name}.hbs

Das zusätzliche /user/ im Pfad bedeutet, dass es von unserer unColocateConnectors-Logik nicht erkannt wird.

github.com/discourse/discourse

app/assets/javascripts/discourse-plugins/index.js

f8aec4167


      
          const COLOCATED_CONNECTOR_REGEX =
            /^(?<prefix>.*)\/connectors\/(?<outlet>[^\/]+)\/(?<name>[^\/\.]+)\.(?<extension>.+)$/;
          
          // Having connector templates and js in the same directory causes a clash
          // when outputting es6 modules. This shim separates colocated connectors
          // into separate js / template locations.
          function unColocateConnectors(tree) {
            return new Funnel(tree, {
              getDestinationPath: function (relativePath) {
                const match = relativePath.match(COLOCATED_CONNECTOR_REGEX);
                if (
                  match &&
                  match.groups.extension === "hbs" &&
                  match.groups.prefix.split("/").pop() !== "templates"
                ) {
                  const { prefix, outlet, name } = match.groups;
                  return `${prefix}/templates/connectors/${outlet}/${name}.hbs`;
                }
                if (
                  match &&

This file has been truncated. show original

Ich habe all-the-plugins und all-the-themes durchsucht und sehe keine weiteren Vorkommen dieses Verzeichnishochlayouts. Daher werde ich die Dinge vorerst so belassen, es sei denn, wir erhalten weitere Berichte über Probleme.

JammyDodger · 8. Juni 2024 um 12:36

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

Thema		Antworten	Aufrufe
Developing Discourse Plugins - Part 2 - Connect to a plugin outlet Developer Guides plugin-guides , tutorial	38	29478	23. November 2025
Connectors not rendering Bug	6	698	20. Oktober 2022
Raw plugin outlets not looking in RAW_TEMPLATES in v1.7.0.beta11 Bug	3	1482	5. Januar 2017
Important changes to Plugin Outlets for Ember 2.10 Dev	23	6647	1. Juni 2018
Using a Discourse component in a plugin Dev	19	4497	26. Januar 2017

Neues Discourse-Update hat den Plugin-Ausgang auf seltsame Weise kaputt gemacht

Verwandte Themen