Moodle
  1. Moodle
  2. MDL-33030

Progress Bar in a lesson only goes up

    Details

    • Testing Instructions:
      Hide
      1. create a lesson within a course
      2. enable progress bar on lesson setting
      3. add some contents and questions.
      4. login as student and attempt the lesson
      5. make sure progress bar doesn't keep increasing for the same page.

      Note: the progress bar is determined by attempted/visited page (this includes correct/wrong attempt for the question). If the current progress bar is displaying 50% (assuming the student is in the middle progress of a lesson) and the student re-visited the first page again, the progress bar won't be reset to 10% but it will display the current progress which is 50%.

      Show
      create a lesson within a course enable progress bar on lesson setting add some contents and questions. login as student and attempt the lesson make sure progress bar doesn't keep increasing for the same page. Note: the progress bar is determined by attempted/visited page (this includes correct/wrong attempt for the question). If the current progress bar is displaying 50% (assuming the student is in the middle progress of a lesson) and the student re-visited the first page again, the progress bar won't be reset to 10% but it will display the current progress which is 50%.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      40206

      Description

      When logged in as a student and you go to the first page of a lesson the progress bar says 10%, go to the second page it says 20%, however if you go back to the first page you would expect the progress bar to display 10% again. However what it does is display 30% and each time I go back and forth on page 1 and 2 of a Lesson the progress keeps going up, even pass 100%.

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          Thanks for reporting this.

          I've put that on the backlog.

          In the meantime feel free to help us work on this issue. If you are able to provide a patch, please add a patch label so we will spot it.

          Show
          Michael de Raadt added a comment - Thanks for reporting this. I've put that on the backlog. In the meantime feel free to help us work on this issue. If you are able to provide a patch, please add a patch label so we will spot it.
          Hide
          Michael de Raadt added a comment -

          Bumping this issue.

          Show
          Michael de Raadt added a comment - Bumping this issue.
          Hide
          Ankit Agarwal added a comment -

          Hi Rosie,
          Changes look good to me.
          Thanks

          Show
          Ankit Agarwal added a comment - Hi Rosie, Changes look good to me. Thanks
          Hide
          Rossiani Wijaya added a comment -

          Thanks Ankit.

          Submitting for integration review.

          Show
          Rossiani Wijaya added a comment - Thanks Ankit. Submitting for integration review.
          Hide
          Dan Poltawski added a comment -

          Thanks Rosie, integrated to master 23 and 22 (not 21)

          Show
          Dan Poltawski added a comment - Thanks Rosie, integrated to master 23 and 22 (not 21)
          Hide
          Frédéric Massart added a comment -

          Test passed. Thanks!

          Show
          Frédéric Massart added a comment - Test passed. Thanks!
          Hide
          Dan Poltawski added a comment -

          Congratulations!

          You've made it into the weekly release!

          Thanks for your contribution - here are some random drummers to keep you inspired for the next week!
          http://www.youtube.com/watch?v=_QhpHUmVCmY

          Show
          Dan Poltawski added a comment - Congratulations! You've made it into the weekly release! Thanks for your contribution - here are some random drummers to keep you inspired for the next week! http://www.youtube.com/watch?v=_QhpHUmVCmY

            People

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

              Dates

              • Created:
                Updated:
                Resolved: