La data ricorrente mostra la data successiva sbagliata

Sto cercando di impostare una data ricorrente come segue:

[date=2021-01-24 time=08:30:00 timezone="America/New_York" recurring="1.weeks"]

Visto che questa data è nel futuro, dovrebbe mostrare il 24 gennaio 2021 come prossima occorrenza. Invece, mostra il 28 febbraio 2021.

Data effettiva:
2021-01-24T13:30:00Z

Screenshot dall’anteprima del post (nel caso in cui l’anteprima lo mostri diversamente):
image


Modificando la data al 2021-01-25, la data salta fino all’8 marzo 2021, saltando 8 giorni invece di 1. Ogni 7 giorni salta un’ulteriore settimana, il che sembra essere la ragione per cui la data è così sbagliata. Sto facendo qualcosa di sbagliato? Forse c’è un caso limite nella creazione di date ricorrenti future in generale o specificamente in un anno futuro.

2 Mi Piace

Questo sembra stranamente intenzionale nel codice. Sembra sempre basare la data ricorrente sulla differenza tra la data corrente e oggi:

https://github.com/discourse/discourse/blob/master/plugins/discourse-local-dates/assets/javascripts/lib/local-date-builder.js.es6#L55

Ho provato a modificare questo nel codice in localDate.datetime e l’output era corretto nel browser, ma 4 test sono falliti. Sembrava che stessero assicurandosi che funzionasse in questo modo, quindi ora mi chiedo se sto trascurando qualcosa @j.jaffeux? La ricorrenza dovrebbe essere successiva alla data inserita, non alla data corrente, giusto?

4 Mi Piace

Sì, effettivamente penso di aver trascurato questo caso, considerando principalmente la situazione in cui si tenta di creare una data nel passato e la ricorrenza la sposta nel futuro. Ne darò un’occhiata la prossima settimana.

7 Mi Piace

Questo è in attesa di revisione, ma dovrebbe risolvere il problema una volta unito:

Grazie per la segnalazione, sono felice di aver sistemato questo bug.

7 Mi Piace