dsims
(Daniel)
2 Aprile 2025, 12:41pm
1
Se metto le virgolette nel nome dell’evento di Discourse Calendar (and Event) come Nome evento: RSVP “Parteciperò” per unirti
Al salvataggio causa un errore 500 a PUT /posts/123 (probabilmente a causa di markdown errato), ma la parte peggiore è che tutti i partecipanti sembrano essere stati rimossi dall’evento.
Nota: questo è stato utilizzato con Discourse Livestream ma presumo che non sia correlato.
3 Mi Piace
sam
(Sam Saffron)
2 Aprile 2025, 11:37pm
2
Accidenti, daremo un’occhiata a questo la prossima settimana.
Dato che ricevi un 500, puoi controllare in /logs e dirci cosa vedi lì?
1 Mi Piace
dsims
(Daniel)
3 Aprile 2025, 3:11am
3
Penso che ciò che sta accadendo sia che le virgolette non escapate interrompano il markup dell’evento, quindi nessun evento viene rilevato e l’evento viene distrutto . Quindi, quando rimuovo le virgolette e salvo di nuovo, viene creato un nuovo evento.
Il codice di errore 500 durante la serializzazione è probabilmente un effetto collaterale correlato.
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 Mi Piace
Grazie per il report @dsims , questo verrà risolto una volta che questa PR verrà unita.
main ← fix-escape-quotes
opened 01:12PM - 08 Apr 25 UTC
…kdown
This is not the best UX as user might enter an event name with double … quotes and they'll be deleted once they click save. But at least it won't break their event because the double quote breaks the BBCode/Markdown.
A proper fix would be to manipulate an AST instead of using regular expressions on a a string.
Meta - https://meta.discourse.org/t/-/360010
Non è la “migliore” soluzione, ma almeno l’evento non si interromperà se un utente inserisce un \" in uno dei campi della finestra modale dell’evento.
1 Mi Piace