tim-goto-link: re-check perms, future access on countdown expiry

TIM-JYU / TIM

TIM (The Interactive Material) is an open-source cloud-based platform for creating interactive learning documents.

https://tim.education/view/about/en-US

MIT License

13 stars 4 forks source link

tim-goto-link: re-check perms, future access on countdown expiry #3656

Closed saviit closed 1 month ago

saviit commented 1 month ago

Tarkistaa käyttäjän oikeudet dokumenttiin tim-goto-linkeissä kun aikalaskuri menee nollaan. Jos käyttäjällä on tulevaisuudessa aukeavia oikeuksia, käynnistää uuden aikalaskurin.

Tämä on vasta nopea hutaisu, pitäisi siistiä, kommentteja otetaan vastaan.

varmaan pitäisi muokata itse countdown niin että sen voi alustaa uudelleen tim-goto:sta (ettei tarvi olla templatessa tuplasti)
paremmat tarkistukset parissa kohdassa
muuta?

dezhidki commented 1 month ago

Muokkasin tätä jonkin verran. Tähän toteutukseen riittää, että käytännössä laskuri resetoidaan ja aloitetaan alusta samalla logiikalla kuin start-metodissa. Muokkasin koodia sellaiseksi, eli nyt koodi uudelleenkäyttää samat varoitusviestit ja muun käsittelylogiikan.

Tämä taitaa ratkaista oikeastaan kaikki ylös nostetut kohdat ja poistaa templatesta duplikaatin. VKn kannalta tämän pitäisi riittää, mutta toki parempia virheviestejä tuohon olisi hyvä joskus laittaa.