Permalink.create(url: /my/path, post_id: 123) não funciona mais

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

parece ter quebrado o comportamento esperado de Permalink.create. Em vez de poder chamá-lo com um post_id ou topic_id como funcionava antes (e o que um programador Ruby esperaria?), agora você precisa chamá-lo assim:

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

Isso quebra todos os scripts de importação que criam permalinks ( ./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 chamadas como find_or_create_by também não funcionam mais (quebrando vbulletin3, smf1, xenforo e getsatisfaction).

Isso é esperado?

Parece que os testes também foram alterados para suportar esse novo comportamento, então talvez sim?

6 curtidas

Criei um PR com uma possível solução:

7 curtidas

Este tópico foi fechado automaticamente 2 dias após a última resposta. Novas respostas não são mais permitidas.