Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-67422

clarify / unify calendar permissions evaluation


    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • 3.7.3, 3.8, 3.9
    • Calendar, Roles / Access

      This was considered when reviewing MDL-67300.

      Right now calendar has different methods deciding which type of events can be managed at every context. And it seems that they are used in a mixed way.

      A proposal to unify uses a little more way done there, see this commit:


      And it was a good step, but this issue is a little bit more ambitious:

      1) We should review all the methods within calendar that have responsibilities calculating permissions.

      2) We should unify uses as much as possible, clearly deprecating the methods that we consider "old".

      3) All the combinations should be well covered by automation tests.

      4) The methods should be well documented (phpdoc is ok), about which are they correct uses and so on.

            Unassigned Unassigned
            stronk7 Eloy Lafuente (stronk7)
            0 Vote for this issue
            0 Start watching this issue


                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.