Moodle
  1. Moodle
  2. MDL-24518

Backup: Non-editing teachers cannot Import

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 1.9.9
    • Fix Version/s: None
    • Component/s: Backup
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE

      Description

      I'm hoping this is fixed in 2.0, but figured I would make an issue in part as a reminder to myself to follow up. I was trying to show someone how to import courses from another teacher's course. They were a non-editing teacher and I wanted them to be able to pull those resources in from the other teacher; however, the capabilities required were manage activities and course update which of course makes them the equivalent of a teacher and make the whole non-editing part obsolete. In the spirit of sharing, I would think that the non-editing teacher role would allow a teacher to see the resources and activities being used in one course and allow teachers to get them. Perhaps we need a new capability to handle this scenario or perhaps it is already handled in 2.0. Peace - Anthony

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Eloy Lafuente (stronk7) added a comment -

            Hi Anthony,

            under 2.0 there are two capabilities:

            • 'moodle/backup:backuptargetimport': Used to control from which courses one user can import information (source courses).
            • 'moodle/restore:restoretargetimport': To control to which courses one user can import information (target courses).

            By default both are only granted to editingteachers (and managers).

            under 1.9, there is a clear conflict, as far as we are trying to control 2 things (sources and targets) with only one capability. More yet, there are some conflicts (like MDL-16615), that I'm not sure if/when will be fixed.

            Ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Hi Anthony, under 2.0 there are two capabilities: 'moodle/backup:backuptargetimport': Used to control from which courses one user can import information (source courses). 'moodle/restore:restoretargetimport': To control to which courses one user can import information (target courses). By default both are only granted to editingteachers (and managers). under 1.9, there is a clear conflict, as far as we are trying to control 2 things (sources and targets) with only one capability. More yet, there are some conflicts (like MDL-16615 ), that I'm not sure if/when will be fixed. Ciao
            Hide
            Anthony Borrow added a comment - - edited

            Eloy - Thanks for responding and your impressive work on the backup/restore code. I'm less concerned about 1.9 if the issue has received adequate attention in 2.0. At least in 2.0 it would be a relatively simple modification of the nonediting teacher role to allow them to import courses. I have no objection if you wish to simply resolve this as fixed for 2.0. Peace - Anthony

            Show
            Anthony Borrow added a comment - - edited Eloy - Thanks for responding and your impressive work on the backup/restore code. I'm less concerned about 1.9 if the issue has received adequate attention in 2.0. At least in 2.0 it would be a relatively simple modification of the nonediting teacher role to allow them to import courses. I have no objection if you wish to simply resolve this as fixed for 2.0. Peace - Anthony
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Closing as won't fix. No much we can do in 1.9 for this, sorry.

            Show
            Eloy Lafuente (stronk7) added a comment - Closing as won't fix. No much we can do in 1.9 for this, sorry.

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: