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:

      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%.

        Gliffy Diagrams

          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:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: