Citações no nome do evento esclarecem os participantes

Bug
1

Se eu colocar aspas no nome do meu evento em Discourse Calendar (and Event) como Nome do evento: RSVP “Indo” para participar

Screenshot
Screenshot544×168 4.78 KB

Ao salvar, causa um erro 500 em PUT /posts/123 (provavelmente devido a markdown quebrado), mas a parte ruim é que todos os participantes parecem ter sido removidos do evento.

Observação: isso foi usado com Discourse Livestream, mas presumo que seja irrelevante.

3 curtidas
2

Eita, vamos dar uma olhada nisso na próxima semana.

Dado que você está recebendo um erro 500, você pode verificar em /logs e nos dizer o que vê lá?

1 curtida
3

Acho que o que está acontecendo é que as aspas não escapadas quebram a marcação do evento, então nenhum evento é detectado e o evento é destruído. Então, quando eu removo as aspas e salvo novamente, um novo evento é criado.
O erro 500 durante a serialização é provavelmente um efeito colateral relacionado.

NoMethodError (undefined method `end_of_day' for nil)
plugins/discourse-calendar/app/models/discourse_post_event/event.rb:93:in `expired?'
plugins/discourse-calendar/app/models/discourse_post_event/event.rb:279:in `can_user_update_attendance'
plugins/discourse-calendar/app/serializers/discourse_post_event/event_serializer.rb:93:in `can_update_attendance'
(eval at /var/www/discourse/vendor/bundle/ruby/3.3.0/gems/active_model_serializers-0.8.4/lib/active_model/serializer.rb:467):4: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/application_controller.rb:506:in `render_json_dump'
app/controllers/posts_controller.rb:301:in `update'
2 curtidas
5

Obrigado pelo relatório @dsims, isso será corrigido assim que este PR for mesclado.

Não é a “melhor” correção, mas pelo menos o evento não falhará se um usuário inserir um \" em um dos campos do modal de evento.

1 curtida