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

Survey should have activity completion condition on survey completed

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Manual Testing:
      1. Create a survey activity that requires activity completion on submit.
      2. Complete the survey as the student.
      3. Verify that the activity is marked complete.

      Automated testing:
      1. Run behat tests for mod_survey
      2. Run phpunit tests for mod_survey

      Show
      Manual Testing: 1. Create a survey activity that requires activity completion on submit. 2. Complete the survey as the student. 3. Verify that the activity is marked complete. Automated testing: 1. Run behat tests for mod_survey 2. Run phpunit tests for mod_survey
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_28_STABLE, MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull Master Branch:
      MDL-48944_m32v4

      Description

      Discovered from being a student on learn.moodle.. I had to manually mark my survey activity as complete.

      Helen tells me the only options are viewing the survey activity or manual completion.

      It would seem natural to have a condition on completing the survey - especially as once i've submitted the survey - I can't undo it.

      Steps to reproduce:

      1. Enable activity completion
      2. Add a survey to a course
      3. Attempt to choose 'survey completed' as a completion condiiton

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              sbourget Stephen Bourget
              Reporter:
              poltawski Dan Poltawski
              Peer reviewer:
              Sam Marshall
              Integrator:
              Dan Poltawski
              Tester:
              Jun Pataleta
              Participants:
              Component watchers:
              Sam Marshall, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                5/Dec/16