Permalink.create(url: /my/path, post_id: 123) non funziona più

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

sembra aver interrotto il comportamento previsto di Permalink.create. Invece di poterlo chiamare con un post_id o topic_id come funzionava una volta (e come un programmatore Ruby si aspetterebbe?), ora devi chiamarlo così:

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

Questo interrompe tutti gli script di importazione che creano permalink ( ./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)
E anche le chiamate come find_or_create_by non funzionano più (interrompendo vbulletin3, smf1, xenforo e getsatisfaction).

È previsto?

Sembra che anche i test siano stati modificati per supportare questo nuovo comportamento, quindi forse sì?

6 Mi Piace

Ho creato una PR con una possibile soluzione:

7 Mi Piace

Questo argomento è stato chiuso automaticamente 2 giorni dopo l’ultima risposta. Non sono più consentite nuove risposte.