Moodle
  1. Moodle
  2. MDL-27907

Calculated Simple Question has wrong or confusing option in Moodle 2.1

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1
    • Component/s: Quiz
    • Labels:
    • Environment:
      Using Google Chrome as browser
    • Testing Instructions:
      Hide

      Create a calculated and a calculated simple question.
      Verify that each of the fields that Steve shows in his screenshot is present, but that there is only one copy of each!
      Verify that all the values are saved accurately by re-loading the form and inspecting the values that were saved and re-loaded.

      Show
      Create a calculated and a calculated simple question. Verify that each of the fields that Steve shows in his screenshot is present, but that there is only one copy of each! Verify that all the values are saved accurately by re-loading the form and inspecting the values that were saved and re-loaded.
    • URL:
      qa.moodle.net
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      When creating a calculated simple question in a quiz, the section for choosing the number of digits to display asks for the number of digits in the correct answer twice. It's not clear what the purpose for the repeated information is. Is this a duplicate request for information or is the second set of fields referring to something else?

      NOTE: This problem doesn't show up in Calculated questions, only in Calculated simple.

      This bug arose while working on QA test 146. Since is wasn't related to the test per se, I passed the test, but wanted to report the problem here.

        Gliffy Diagrams

          Issue Links

            Activity

            Steve Turley created issue -
            Tim Hunt made changes -
            Field Original Value New Value
            Fix Version/s STABLE backlog [ 10463 ]
            Labels triaged
            Tim Hunt made changes -
            Link This issue blocks MDL-27738 [ MDL-27738 ]
            Hide
            Tim Hunt added a comment -

            Sorry, this was just a screw-up. Should be fixed now (once this is integrated).

            Show
            Tim Hunt added a comment - Sorry, this was just a screw-up. Should be fixed now (once this is integrated).
            Tim Hunt made changes -
            Status Open [ 1 ] Waiting for integration review [ 10010 ]
            Pull Master Diff URL https://github.com/timhunt/moodle/compare/master...MDL-27907
            Pull Master Branch MDL-27907
            Pull from Repository git://github.com/timhunt/moodle.git
            Fix Version/s 2.1 [ 10370 ]
            Fix Version/s STABLE backlog [ 10463 ]
            Testing Instructions Go the qa.moodle.net and create a quiz in one of the courses. (I used Course Fullname 101.) After creating a quiz, add a Calculated simple question. Create a calculated and a calculated simple question.
            Verify that each of the fields that Steve shows in his screenshot is present, but that there is only one copy of each!
            Verify that all the values are saved accurately by re-loading the form and inspecting the values that were saved and re-loaded.
            Sam Hemelryk made changes -
            Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
            Integrator samhemelryk
            Currently in integration Yes
            Hide
            Sam Hemelryk added a comment -

            Thanks Tim - integrated now

            Show
            Sam Hemelryk added a comment - Thanks Tim - integrated now
            Sam Hemelryk made changes -
            Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
            Rajesh Taneja made changes -
            Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
            Tester rajeshtaneja
            Hide
            Rajesh Taneja added a comment -

            Thanks for fixing this Tim
            Options are not being duplicated anymore.
            Although, field name "Tolerance +-" has been replaced with "Accepted error", which is not updated in documentation (http://docs.moodle.org/20/en/question/type/calculated).

            Show
            Rajesh Taneja added a comment - Thanks for fixing this Tim Options are not being duplicated anymore. Although, field name "Tolerance +-" has been replaced with "Accepted error", which is not updated in documentation ( http://docs.moodle.org/20/en/question/type/calculated ).
            Rajesh Taneja made changes -
            Status Testing in progress [ 10011 ] Tested [ 10006 ]
            Hide
            Tim Hunt added a comment -

            Hmm. That is a very minor point, but important. Well done for spotting that! I created MDL-27937 to track this.

            Show
            Tim Hunt added a comment - Hmm. That is a very minor point, but important. Well done for spotting that! I created MDL-27937 to track this.
            Hide
            Rajesh Taneja added a comment -

            I was slow in creating improvement MDL
            Created another one MDL-27938... will close it now.
            Thanks Tim

            Show
            Rajesh Taneja added a comment - I was slow in creating improvement MDL Created another one MDL-27938 ... will close it now. Thanks Tim
            Hide
            Eloy Lafuente (stronk7) added a comment -

            U.P.S.T.R.E.A.M

            Show
            Eloy Lafuente (stronk7) added a comment - U.P.S.T.R.E.A.M
            Eloy Lafuente (stronk7) made changes -
            Status Tested [ 10006 ] Closed [ 6 ]
            Resolution Fixed [ 1 ]
            Currently in integration Yes
            Integration date 20/Jun/11

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: