Erro `NoMethodError in TopicsController#show` após mesclar tópicos

Bug
,
1

Prioridade/Gravidade: Baixa Prioridade / Alta Gravidade

Plataforma: 3.5.0.beta8-dev com Discourse Calendar (and Event) - 0.5 0162ed5

Descrição:

Após mesclar tópicos e excluir uma postagem mesclada, o tópico de destino se torna inacessível devido a um erro de servidor ([NoMethodError: undefined method 'category_id' for nil).

Espera-se que o tópico funcione normalmente e não cause um erro de servidor.

Rastreamento completo 
plugins/discourse-calendar/app/serializers/discourse_post_event/event_serializer.rb:137:in `category_id'
(eval at /home/discourse/.bundle/gems/ruby/3.3.0/gems/active_model_serializers-0.8.4/lib/active_model/serializer.rb:467):5:in `_fast_attributes'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:468:in `rescue in attributes'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:455:in `attributes'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:480:in `_serializable_hash'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:359:in `serializable_hash'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:347:in `as_json'
activesupport (7.2.2.1) lib/active_support/json/encoding.rb:23:in `encode'
activesupport (7.2.2.1) lib/active_support/json/encoding.rb:23:in `encode'
activesupport (7.2.2.1) lib/active_support/core_ext/object/json.rb:42:in `to_json'
active_model_serializers (0.8.4) lib/active_model/serializer.rb:331:in `to_json'
multi_json (1.15.0) lib/multi_json/adapters/oj.rb:56:in `dump'
multi_json (1.15.0) lib/multi_json/adapters/oj.rb:56:in `dump'
multi_json (1.15.0) lib/multi_json/adapter.rb:25:in `dump'
multi_json (1.15.0) lib/multi_json.rb:139:in `dump'
app/controllers/topics_controller.rb:1383:in `block (2 levels) in perform_show_response'
actionpack (7.2.2.1) lib/action_controller/metal/mime_responds.rb:224:in `respond_to'
app/controllers/topics_controller.rb:1377:in `perform_show_response'
app/controllers/topics_controller.rb:191:in `show'
actionpack (7.2.2.1) lib/action_controller/metal/basic_implicit_render.rb:8:in `send_action'
actionpack (7.2.2.1) lib/abstract_controller/base.rb:226:in `process_action'
actionpack (7.2.2.1) lib/action_controller/metal/rendering.rb:193:in `process_action'
actionpack (7.2.2.1) lib/abstract_controller/callbacks.rb:261:in `block in process_action'
activesupport (7.2.2.1) lib/active_support/callbacks.rb:121:in `block in run_callbacks'
app/controllers/application_controller.rb:428:in `block in with_resolved_locale'
i18n (1.14.7) lib/i18n.rb:353:in `with_locale'
app/controllers/application_controller.rb:428:in `with_resolved_locale'
activesupport (7.2.2.1) lib/active_support/callbacks.rb:130:in `block in run_callbacks'
activesupport (7.2.2.1) lib/active_support/callbacks.rb:141:in `run_callbacks'
actionpack (7.2.2.1) lib/abstract_controller/callbacks.rb:260:in `process_action'
actionpack (7.2.2.1) lib/action_controller/metal/rescue.rb:27:in `process_action'
actionpack (7.2.2.1) lib/action_controller/metal/instrumentation.rb:77:in `block in process_action'
activesupport (7.2.2.1) lib/active_support/notifications.rb:210:in `block in instrument'
activesupport (7.2.2.1) lib/active_support/notifications/instrumenter.rb:58:in `instrument'
activesupport (7.2.2.1) lib/active_support/notifications.rb:210:in `instrument'
actionpack (7.2.2.1) lib/action_controller/metal/instrumentation.rb:76:in `process_action'
actionpack (7.2.2.1) lib/action_controller/metal/params_wrapper.rb:259:in `process_action'
activerecord (7.2.2.1) lib/active_record/railties/controller_runtime.rb:39:in `process_action'
actionpack (7.2.2.1) lib/abstract_controller/base.rb:163:in `process'
actionview (7.2.2.1) lib/action_view/rendering.rb:40:in `process'
rack-mini-profiler (4.0.0) lib/mini_profiler/profiling_methods.rb:116:in `block in profile_method'
actionpack (7.2.2.1) lib/action_controller/metal.rb:252:in `dispatch'
actionpack (7.2.2.1) lib/action_controller/metal.rb:335:in `dispatch'
actionpack (7.2.2.1) lib/action_dispatch/routing/route_set.rb:67:in `dispatch'
actionpack (7.2.2.1) lib/action_dispatch/routing/route_set.rb:50:in `serve'
actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:53:in `block in serve'
actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:133:in `block in find_routes'
actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:126:in `each'
actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:126:in `find_routes'
actionpack (7.2.2.1) lib/action_dispatch/journey/router.rb:34:in `serve'
actionpack (7.2.2.1) lib/action_dispatch/routing/route_set.rb:896:in `call'
lib/middleware/omniauth_bypass_middleware.rb:35:in `call'
rack (2.2.17) lib/rack/tempfile_reaper.rb:15:in `call'
rack (2.2.17) lib/rack/conditional_get.rb:27:in `call'
rack (2.2.17) lib/rack/head.rb:12:in `call'
actionpack (7.2.2.1) lib/action_dispatch/http/permissions_policy.rb:38:in `call'
lib/content_security_policy/middleware.rb:12:in `call'
lib/middleware/csp_script_nonce_injector.rb:12:in `call'
config/initializers/008-rack-cors.rb:14:in `call'
rack (2.2.17) lib/rack/session/abstract/id.rb:266:in `context'
rack (2.2.17) lib/rack/session/abstract/id.rb:260:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/cookies.rb:704:in `call'
activerecord (7.2.2.1) lib/active_record/migration.rb:674:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/callbacks.rb:31:in `block in call'
activesupport (7.2.2.1) lib/active_support/callbacks.rb:101:in `run_callbacks'
actionpack (7.2.2.1) lib/action_dispatch/middleware/callbacks.rb:30:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/executor.rb:16:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/actionable_exceptions.rb:18:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/debug_exceptions.rb:31:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/show_exceptions.rb:32:in `call'
logster (2.20.1) lib/logster/middleware/reporter.rb:40:in `call'
lib/middleware/default_headers.rb:13:in `call'
railties (7.2.2.1) lib/rails/rack/logger.rb:41:in `call_app'
railties (7.2.2.1) lib/rails/rack/logger.rb:29:in `call'
config/initializers/100-quiet_logger.rb:20:in `call'
config/initializers/100-silence_logger.rb:29:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/remote_ip.rb:96:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/request_id.rb:33:in `call'
rack (2.2.17) lib/rack/method_override.rb:24:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/executor.rb:16:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/static.rb:27:in `call'
rack (2.2.17) lib/rack/sendfile.rb:110:in `call'
lib/middleware/missing_avatars.rb:22:in `call'
actionpack (7.2.2.1) lib/action_dispatch/middleware/host_authorization.rb:143:in `call'
rack-mini-profiler (4.0.0) lib/mini_profiler.rb:334:in `call'
lib/middleware/processing_request.rb:12:in `call'
message_bus (4.4.1) lib/message_bus/rack/middleware.rb:60:in `call'
railties (7.2.2.1) lib/rails/engine.rb:535:in `call'
railties (7.2.2.1) lib/rails/railtie.rb:226:in `public_send'
railties (7.2.2.1) lib/rails/railtie.rb:226:in `method_missing'
rack (2.2.17) lib/rack/urlmap.rb:74:in `block in call'
rack (2.2.17) lib/rack/urlmap.rb:58:in `each'
rack (2.2.17) lib/rack/urlmap.rb:58:in `call'
unicorn (6.1.0) lib/unicorn/http_server.rb:634:in `process_client'
unicorn (6.1.0) lib/unicorn/http_server.rb:739:in `worker_loop'
unicorn (6.1.0) lib/unicorn/http_server.rb:547:in `spawn_missing_workers'
unicorn (6.1.0) lib/unicorn/http_server.rb:143:in `start'
unicorn (6.1.0) bin/unicorn:128:in `\u003ctop (required)\u003e'
bin/unicorn:96:in `load'
bin/unicorn:96:in `block in \u003cmain\u003e'
bin/unicorn:95:in `fork'
bin/unicorn:95:in `\u003cmain\u003e'

Passos para reproduzir:

Em 3.5.0.beta8-dev com Discourse Calendar (and Event) - 0.5 0162ed5 ativado:

  1. Crie um novo tópico tópico 1 - que será excluído com uma única postagem como user1 que é tl1
  2. Crie um novo tópico tópico 2 - que quebrará com uma única postagem, garantindo que Evento de Calendário faça parte desta postagem.
  3. Como user1, exclua sua postagem em tópico 1 - que será excluído, bloqueando o tópico e substituindo sua postagem por “(tópico excluído pelo autor)”
  4. Como user2 que tem permissões de moderador:
  5. Selecione as postagens em tópico 1 - que será excluído e selecione a primeira postagem (agora excluída)
  6. Como user2, exclua a primeira postagem recém-mesclada em tópico 2 - que quebrará (a postagem que acabou de ser mesclada e que diz “(tópico excluído pelo autor)”), isso fará com que tópico 2 - que quebrará falhe e gere o erro acima.

Uma solução alternativa - desative o plugin calendar-and-event. Isso permitirá que você visite o tópico quebrado e restaure a primeira postagem. Em seguida, reative o plugin.

Discussão:

Isso também identificou um bug ligeiramente menos grave no Discourse, onde a execução do acima com o plugin calendar-and-event desativado fará com que o tópico seja listado como não listado - veja o relatório de bug separado

2 curtidas
2

Muito obrigado pela descrição detalhada. Concordo que devemos limpar isso, vou colocar um pr-welcome nisso por enquanto.

1 curtida
3

Obrigado a ambos pelo relatório detalhado e pelo acompanhamento. Acabei de encontrar um problema semelhante depois de mesclar tópicos com eventos de calendário e não consegui descobrir por que o tópico estava quebrando. Fico feliz em ver que isso está documentado.

Ficarei de olho em quaisquer PRs para isso. Enquanto isso, saber que desativar o plugin temporariamente como uma solução alternativa é útil. Me avisem se houver algo que eu possa ajudar a testar.

4

FWIW, acho que consigo simplificar a reprodução:

  • Ative Calendário ativado e Evento de postagem do Discourse ativado
  • Como administrador, crie o Tópico1
  • Como administrador, crie o Tópico2 e inclua um evento de postagem usando ‘Criar evento’ no menu do compositor com o círculo com um mais
  • Como administrador, vá para o Tópico1 e use a chave inglesa do tópico para selecionar o OP e mesclar no Tópico2 com preservar ordem cronológica após mesclagem marcado

Neste estágio, o evento de postagem não é mais o OP do tópico para o Tópico2

  • Como administrador, exclua o Tópico2 usando a exclusão de postagem do OP ou a opção de exclusão de tópico

Resultado: borkity bork. Tópico excluído inacessível. Erro em /logs de NoMethodError (undefined method ‘category_id’ for nil)

O conserto aqui seria impedir a mesclagem que substituiria o evento como OP?

Ao tentar mesclar um post de evento (um corretamente posicionado em um OP) em outro tópico, ele é bloqueado com isto nos logs:

Falha ao processar a resposta sequestrada corretamente: ActiveRecord::RecordNotSaved: Um evento só pode estar na primeira postagem de um tópico

(O erro da interface do usuário é mais genérico, Houve um erro ao mover postagens., o que seria bom se fosse mais detalhado)

Algo semelhante poderia ser aplicado para impedir uma mesclagem que mova o evento para fora da posição de OP?

3 curtidas