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

Sub-optimal error message on course/modedit.php

XMLWordPrintable

    • MySQL
    • MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE
    • MOODLE_28_STABLE, MOODLE_29_STABLE
    • mdl-50963
    • Hide

      Do not attempt to set the "Maximum points" value to zero. Instead, make sure it is always greater than zero and less than 301.

      Show
      Do not attempt to set the "Maximum points" value to zero. Instead, make sure it is always greater than zero and less than 301.
    • Hide

      1. Create a course.
      2. Create an assignment.
      3. Change the "Maximum points" value to be less than one.
      4. Submit the form.
      5. Make sure you receive an error message that says "Invalid Grade Value. This must be an integer between 1 and 300."
      6. Change the "Maximum points" value to be greater than 300.
      7. Submit the form.
      8. Make sure you receive an error message that says "Invalid Grade Value. This must be an integer between 1 and 300."
      9. Change the "Maximum points" value to be between 1 and 300.
      10. Submit the form.
      11. Make sure the form submission is accepted and you do not receive any error message.

      Show
      1. Create a course. 2. Create an assignment. 3. Change the "Maximum points" value to be less than one. 4. Submit the form. 5. Make sure you receive an error message that says "Invalid Grade Value. This must be an integer between 1 and 300." 6. Change the "Maximum points" value to be greater than 300. 7. Submit the form. 8. Make sure you receive an error message that says "Invalid Grade Value. This must be an integer between 1 and 300." 9. Change the "Maximum points" value to be between 1 and 300. 10. Submit the form. 11. Make sure the form submission is accepted and you do not receive any error message.

      When you create or edit an assignment and make changes to the "Grade" section (specifically the "Maximum points"), if you attempt to give a maximum point value of zero, the form submission is rejected and you receive this message:

      Invalid Grade Value. This must be an integer between 0 and 300
      

      Now, if you instead give a maximum point value of 300, the form submission will be accepted. I propose changing the message to be:

      Invalid Grade Value. This must be an integer between 1 and 300.
      

      This better matches how people think when you use the word "between" and also makes valid the given minimum and maximum values.

            Unassigned Unassigned
            c.d.smith Clem Smith
            Adrian Greeve Adrian Greeve
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Frédéric Massart Frédéric Massart
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.