Ошибка `NoMethodError in TopicsController#show` после слияния тем

[blu3id (Mike)]

Приоритет/Серьезность: Низкий приоритет / Высокая серьезность

Платформа: 3.5.0.beta8-dev с Discourse Calendar (and Event) - 0.5 0162ed5

Описание:

После объединения тем и удаления объединенного поста целевая тема становится недоступной из-за ошибки сервера ([NoMethodError: undefined method 'category_id' for nil).

Ожидается, что тема будет функционировать нормально и не вызовет ошибку сервера.

Полный трассировочный стек

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 `<top (required)>'
bin/unicorn:96:in `load'
bin/unicorn:96:in `block in <main>'
bin/unicorn:95:in `fork'
bin/unicorn:95:in `<main>'

Шаги для воспроизведения:

На 3.5.0.beta8-dev с включенным плагином Discourse Calendar (and Event) - 0.5 0162ed5:

1. Создайте новую тему topic 1 - that will be deleted с одним постом от имени user1, у которого роль tl1.
2. Создайте новую тему topic 2 - that will break с одним постом, убедившись, что в этом посте есть Calendar Event.
3. От имени user1 удалите свой пост в теме topic 1 - that will be deleted, заблокировав тему и заменив свой пост на “(topic deleted by author)”.
4. От имени user2, имеющего права moderator:
5. Выберите посты в теме topic 1 - that will be deleted, выберите первый (теперь удаленный) пост > переместить в > topic 2 - that will break, установив флажок preserve chronological order after merging.
6. От имени user2 удалите newly объединенный первый пост в теме topic 2 - that will break (пост, который был только что объединен и содержит текст “(topic deleted by author)”). Это приведет к сбою темы topic 2 - that will break и вызовет ошибку, указанную выше.

Обходное решение — отключить плагин calendar-and-event. Это позволит вам открыть сломанную тему и восстановить первый пост. Затем снова включите плагин.

Обсуждение:

Это также выявило менее серьезную ошибку в Discourse: при выполнении описанных выше действий с отключенным плагином calendar-and-event тема становится скрытой от всех пользователей, включая администраторов. См. отдельный отчет об ошибке:

Topic unlisted to all users including admins after merging topics

Priority/Severity: Low Priority / Medium Severity Platform: 3.5.0.beta8-dev Description: After merging topics and deleting a merged post the topic efectively becomes unlisted without being marked as such and doesn’t show up even to tl4 users or Admins. It is expected that the topic will function normally and not become effectively unlisted. Speculating - It may also be the case that this topic is now viewed as “deleted” and potentially be removed automatically as part of cleanup. Reproducib…

[sam (Sam Saffron)]

Огромное спасибо за подробный разбор. Согласен, что это нужно привести в порядок — пока повешу тег pr-welcome.

[Michael12 (Michael Phillips)]

Спасибо обоим за подробный отчёт и последующие уточнения. У меня только что возникла похожая проблема после объединения тем с событиями календаря, и я не мог понять, почему тема ломается. Рад видеть, что это задокументировано.

Я буду следить за любыми PR по этой теме. Тем временем полезно знать, что временное отключение плагина может служить временным решением. Дайте знать, если я смогу помочь с тестированием.

[JammyDodger]

Кстати, я думаю, что могу упростить воспроизведение:

• Включите оба параметра: Calendar enabled и Discourse post event enabled
• Как администратор, создайте Topic1
• Как администратор, создайте Topic2 и добавьте пост-событие, используя «Create event» из меню композера с кругом и плюсом
• Как администратор, перейдите в Topic1 и с помощью гаечного ключа темы выберите исходный пост (OP) и объедините его с Topic2, отметив галочкой опцию preserve chronological order after merging

На этом этапе пост-событие больше не является исходным постом (OP) темы Topic2

• Как администратор, удалите Topic2, используя либо удаление поста OP, либо опцию удаления темы

Результат: всё сломалось. Удалённая тема недоступна. Ошибка в /logs: NoMethodError (undefined method 'category_id' for nil)

---

Неужели решение здесь — предотвратить слияние, которое заменит событие как OP?

При попытке объединить пост-событие (правильно размещённое в OP) с другой темой процесс блокируется, и в логах появляется следующее:

Failed to process hijacked response correctly : ActiveRecord::RecordNotSaved : An event can only be in the first post of a topic

(Ошибка в интерфейсе более общая: «There was an error moving posts.», что было бы лучше, если бы она была более подробной)

Можно ли применить аналогичное ограничение, чтобы предотвратить перемещение события из позиции OP при слиянии?