Probleme mit Zeitwerk Reloading

[pfaffman (Jay Pfaffman)]

Ein scheinbar neuer Test hat ein Problem identifiziert, von dem ich glaube, dass ich es vielleicht schon kannte, aber immer ignoriert habe. Jetzt kann ich es nicht mehr ignorieren, da es die Tests fehlschlagen lässt. Hier sind meine Routen:

# frozen_string_literal: true
require_dependency "pfaffmanager_constraint"

Pfaffmanager::Engine.routes.draw do
  get "/" => "pfaffmanager#index", :constraints => PfaffmanagerConstraint.new
  get "/servers" => "servers#index", :constraints => PfaffmanagerConstraint.new
  #      get "#{root_path}/:username/messages/group/:group_name" => "user_actions#private_messages", constraints: { username: RouteFormat.username, group_name: RouteFormat.username }
  get "/servers/group/:group_name" => "servers#index",
      :constraints => PfaffmanagerConstraint.new
  get "/find/:s" => "servers#index", :constraints => PfaffmanagerConstraint.new
  get "/group/:group_name" => "servers#index",
      :constraints => PfaffmanagerConstraint.new
  get "/inventory/:group_name.json" => "servers#inventory",
      :constraints => PfaffmanagerConstraint.new
  get "/servers/:id" => "servers#show",
      :constraints => PfaffmanagerConstraint.new
  get "/servers/:id/edit" => "servers#edit",
      :constraints => PfaffmanagerConstraint.new
  put "/servers/:id" => "servers#update",
      :constraints => PfaffmanagerConstraint.new
  delete "/servers/:id" => "servers#delete",
         :constraints => PfaffmanagerConstraint.new
  put "/status/:id" => "servers#update_status",
      :constraints => AdminConstraint.new
  post "/api_key/:id" => "servers#set_api_key",
       :constraints => PfaffmanagerConstraint.new
  put "/update_server_status/:id" => "servers#update_server_status",
      :constraints => PfaffmanagerConstraint.new
  post "/upgrade/:id" => "servers#queue_upgrade",
       :constraints => PfaffmanagerConstraint.new
  put "/install/:id" => "servers#install",
      :constraints => PfaffmanagerConstraint.new
  get "/ssh_key/:id" => "serverkeys#get_pub_key"
  get "/ssh-key/:hostname" => "serverkeys#get_pub_key_by_hostname",
      :constraints => {
        hostname: %r{[^/]+}
      }
  post "/servers" => "servers#create",
       :constraints => PfaffmanagerConstraint.new
end

Discourse::Application.routes.append do
  mount ::Pfaffmanager::Engine, at: "/pfaffmanager"
end

Und es schlägt wie folgt fehl:

/home/pfaffman/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/actionpack-7.0.7/lib/action_dispatch/routing/route_set.rb:588:in `add_route': Ungültiger Routenname, bereits in Verwendung: 'pfaffmana\nger' (ArgumentError)
Sie haben möglicherweise zwei Routen mit demselben Namen über die Option `:as` definiert oder Sie überschreiben eine Route, die bereits von einer Ressource mit demselben Namen definiert wurde. Für letzteres können Sie die mit `resources` erstellten Routen wie hier beschrieben einschränken:
https://guides.rubyonrails.org/routing.html#restricting-the-routes-created

          raise ArgumentError, "Invalid route name, already in use: '#{name}' \n" \
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        from /home/pfaffman/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/actionpack-7.0.7/lib/action_dispatch/routing/mapper.rb:1985:in `add_route'
        from /home/pfaffman/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/actionpack-7.0.7/lib/action_dispatch/routing/mapper.rb:1956:in `decomposed_match'

[david (David Taylor)]

Hast du dieses Thema gesehen? Wenn du der Struktur dort (und im Plugin-Skeleton) folgst, sollte alles ziemlich reibungslos funktionieren:

Structuring a plugin for Rails autoloading

Many plugins include lots of class definitions inside plugin.rb, or use require_relative to load ruby files. That works, but it comes with some disadvantages: No auto-reloading of changes in development. Any changes require a full server restart Getting the require calls in the right order can be painful If they are require’d outside the after_initialize block, then other autoloaded classes/modules may not be available There is a solution! Plugins can lean on the standard Rails autoloading sy…

Ein wichtiger Teil des Zeitwerk-Autoloadings sind die Dateinamen/-pfade, daher ist es schwierig, das Problem anhand des von dir geteilten Ausschnitts zu identifizieren. Befindet sich das in einer plugin.rb-Datei? Oder in einer anderen Datei?

[pfaffman (Jay Pfaffman)]

@david

Ist das in einer plugin.rb-Datei? Oder in einer anderen Datei?

Entschuldigung. Das ist config/routes.rb

Aber ich habe auch . . . Zeug . . . in lib/discourse-pfaffmanager/engine.rb, also muss ich vielleicht das Zeug aus routes.rb in lib/pfaffmanager/engine.rb verschieben. noch mehr die RTFM lesen.

[david (David Taylor)]

Ich denke, Sie müssen wahrscheinlich diese Änderung vornehmen:

- Discourse::Application.routes.append do
+ Discourse::Application.routes.draw do

Auf diese Weise kann die Datei mehrmals ‘neu geladen’ werden, ohne einen Konflikt zu verursachen (‘draw’ ersetzt jede vorhandene Route, während ‘append’ einen Fehler auslöst, wenn es einen Konflikt gibt).

Das würde auch mit dem aktuellen Plugin-Skeleton übereinstimmen:

config/routes.rb

[pfaffman (Jay Pfaffman)]

Das hat funktioniert!

@david

‘draw’ ersetzt jede bestehende Route, während ‘append’ einen Fehler auslöst, wenn es eine Kollision gibt)

Immer so offensichtlich, wenn es herausgefunden ist.

Tausend Dank.

(Wenn ich jetzt noch VSCode dazu bringen könnte, beim Speichern von Dateien zu linten, aber zumindest geschieht es jetzt lokal, bevor es zu GitHub geht. . . )

[merefield (Robert)]

Ja, das ist genau eine Änderung, die ich kürzlich vorgenommen habe.

[pfaffman (Jay Pfaffman)]

@merefield

Das ist genau eine Änderung, die ich kürzlich vorgenommen habe.

Das stimmt mich schon etwas gnädiger. . .

Aber hast du 4 Stunden gebraucht?

Danke nochmal, David. Ich weiß deine Unterstützung wirklich zu schätzen.

[merefield (Robert)]

Nicht dieses Mal, manchmal hat man Glück ;). Dieses Mal habe ich zufällig Davids Anleitung gelesen und mir den Unterschied zu dem Code, dem ich ursprünglich gefolgt war, notiert.

Aber sicher, manche Probleme lassen sich am nächsten Tag bei einem Spaziergang im Park lösen

[david (David Taylor)]

@pfaffman

Aber hat es 4 Stunden gedauert?

@merefield

zufällig Davids Anleitung gelesen

Nur zur Info, wenn ich eine Anleitung zu etwas poste, dann meistens, weil ich selbst viele Stunden/Tage gebraucht habe, um es herauszufinden . All diese Zeitwerk-Sachen fallen definitiv in diese Kategorie!

@merefield

manche Probleme lassen sich am nächsten Tag bei einem Spaziergang im Park lösen

[pfaffman (Jay Pfaffman)]

Zur Kenntnis genommen. Ich esse sie alle, wenn du sie postest. Ich hätte nur nach zeitwork suchen sollen und hätte es vielleicht gefunden. Aber selbst dann war das Problem buchstäblich nur ein Wort, das so harmlos schien.

Jetzt versuche ich, all meine {{blah... Sachen in \u003cblah... umzuwandeln. Es macht mich wahnsinnig. Aber ich glaube, ich verstehe es (gut genug, um es vielleicht in ein paar Tagen zu schaffen).