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
14 stars 4 forks source link

Calendar-pluginin bookers ja setters vaativat liian kattavia oikeuksia lisättäviin käyttäjäryhmiin #3221

Open saviit opened 2 years ago

saviit commented 2 years ago

Tapahtuman luominen kalenteriin vaatii käyttäjältä liian kattavia oikeuksia käyttäjäryhmiin:

Case 1: Bookers-attribuutti vaatii manage-oikeudet asetettuun bookers-ryhmään:

Case 2: Setters-attribuutti vaatii edit-oikeudet asetettuun setters-ryhmään:

Case 3: Bookers- ja setters- attribuutit vaativat oikeudet kaikkiin listattuihin ryhmiin:

saviit commented 2 years ago

Linkitys #2646 jatkokehityskorttiin.

dezhidki commented 2 years ago

Nykyiset vaatimukset tapahtumien luomiselle ovat listattu kalenteripluginin ohjeissa: https://tim.jyu.fi/view/tim/ohjeita/kalenteri#kalenterin-oikeuksien-asettaminen (erityisesti kohdat 3 ja 4).

Eli nykiset caset vastaavat kyllä ohjeiden toimintaa. Varsinainen ongelma taisi olla seuraava (liittyen case 1een):

Jotta tuollaisen tapahtuman voi lisätä. pitää tällä hetkellä olla oikeus ryhmään xxx ja vieläpä Manage. [...] [O]piskelijoilla ole tuohon mitään oikeutta, enkä uskaltaisi edes view-oikeutta antaa tuohon.

Tulkintani siis on, että ongelma on pikemmin siinä, että nykyisellä järjestelyllä oikeus luoda tapahtumia ryhmälle tarkoittaa, että pääset samalla hallinnoimaan kyseistä ryhmää ja näkemään jäsenet. Alkuun riittäisi varmaan pudottaa bookers sekä setters -vaatimus view-tasolle, mutta se sitten ei auta siihen, että silloin pääsee näkemään muut ryhmän jäsenet. En lähtisi tekemään täysin uutta oikeustasoa (ellei keksi sellaista yleiskäyttöä sille myös perusdokumenteille), vaan määrittelisin vaan ryhmille säädettävän minimitason, jolla ryhmän jäsenet voi katsoa. Silloin ryhmän omistaja voi valita esim. että pitää olla vähintään edit-oikeus, että näkee ryhmän jäsenet, mutta view antaa oikeuden jo luoda tapahtumat.

TL;DR: tähän korttiin varmaan riittäisi, että pudottaa vaatimuksen manage -> view ja edit -> view (case 1 ja case 2) ja sitten tehdään eri kortti, jossa mietitään tuota jäsenten näkyvyyttä ja mahdollisia ryhmän oikeuksien säätöä. Case 3ea ei ole tarkoitus korjata, koska sellaisen salliminen avaa tien aika helpolle väärinkäytölle. Sitä casea varten ainoastaan olisi hyvää laittaa muutama esimerkki käyttöohjeeseen siitä, mitä saa ja mitä ei saa tehdä tapahtumaryhmien kanssa.