Permalink.create(url: /mi/ruta, post_id: 123) ya no funciona

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

parece haber roto el comportamiento esperado de Permalink.create. En lugar de poder llamarlo con un post_id o topic_id como solía funcionar (¿y es lo que un programador de Ruby esperaría?), ahora necesitas llamarlo así:

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

Esto rompe todos los scripts de importación que crean 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)
Y las llamadas como find_or_create_by tampoco funcionan ya (rompiendo vbulletin3, smf1, xenforo y getsatisfaction).

¿Es esto esperado?

Parece que las pruebas también se cambiaron para soportar este nuevo comportamiento, ¿así que tal vez sí?

6 Me gusta

Creé una PR con una posible solución:

7 Me gusta

Este tema se cerró automáticamente 2 días después de la última respuesta. Ya no se permiten nuevas respuestas.