Permalink.create(url: /my/path, post_id: 123) funktioniert nicht mehr

Dieser Commit UX: redesign admin permalinks page (#29634) · discourse/discourse@42b1ca8 · GitHub

scheint das erwartete Verhalten von Permalink.create gebrochen zu haben. Anstatt es mit einer post_id oder topic_id aufrufen zu können, wie es früher funktionierte (und was ein Ruby-Programmierer erwarten würde?), muss man es jetzt so aufrufen:

          Permalink.create!(url: "/my/path/#{import_id}.html", permalink_type_value: pcf.post_id, permalink_type: "post")

Dies bricht alle Import-Skripte, die Permalinks erstellen ( ./vanilla_mysql.rb ./zendesk.rb ./muut.rb ./vbulletin3.rb ./jforum.rb ./ipboard.rb ./bbpress.rb ./answerhub.rb ./vbulletin.rb ./mylittleforum.rb ./nodebb/nodebb.rb./telligent.rb ./mybb.rb ./question2answer.rb./smf2.rb ./vbulletin5.rb ./phpbb3/importers/permalink_importer.rb ./phorum.rb./answerbase.rb./drupal.rb./zendesk_api.rb)
Und Aufrufe wie find_or_create_by funktionieren ebenfalls nicht mehr (was vbulletin3, smf1, xenforo und getsatisfaction bricht).

Ist das erwartet?

Es scheint, dass die Tests ebenfalls geändert wurden, um dieses neue Verhalten zu unterstützen, also vielleicht doch?

6 „Gefällt mir“

Ich habe einen PR mit einer möglichen Lösung erstellt:

7 „Gefällt mir“

Dieses Thema wurde automatisch 2 Tage nach der letzten Antwort geschlossen. Neue Antworten sind nicht mehr möglich.