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

Event monitor: Allow teacher to duplicate System rule

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      Test 1
      1. Visit 'Site administration' > 'Reports' > 'Event monitoring rules'.
      2. Enable the event monitor if it isn't already enabled.
      3. Create a site wide rule.
      4. Visit a course as a teacher.
      5. Visit 'Course administration' > 'Reports' > 'Event monitoring rules'.
      6. Confirm that you can duplicate the site wide rule you created earlier.
      7. Confirm you can edit and delete the duplicated rule.
      Test 2
      1. Run all the behat and unit tests in admin/tool/monitor.
      Show
      Test 1 Visit 'Site administration' > 'Reports' > 'Event monitoring rules'. Enable the event monitor if it isn't already enabled. Create a site wide rule. Visit a course as a teacher. Visit 'Course administration' > 'Reports' > 'Event monitoring rules'. Confirm that you can duplicate the site wide rule you created earlier. Confirm you can edit and delete the duplicated rule. Test 2 Run all the behat and unit tests in admin/tool/monitor.
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE
    • Pull Master Branch:
      MDL-47871_master
    • Sprint:
      Team B Sprint 1, Team Beards Sprint 2
    • Issue size:
      Small

      Description

      When admin is inside the course and duplicates the System rule, the copy is created under the course.
      Why is it not allowed to do the same for the teacher if teacher can:

      • see the system rule
      • create the course rule

        Attachments

          Activity

            People

            Assignee:
            markn Mark Nelson
            Reporter:
            marina Marina Glancy
            Peer reviewer:
            Ankit Agarwal
            Integrator:
            Andrew Nicols
            Tester:
            David Monllaó
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              2/Feb/15