Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-35581 META: tasks related to RTL theme fixes before 2.4 freeze (Week 39)
  3. MDL-35202

Unable to set a new grade or update old grade when using the new Assignment type (Assign) while in Hebrew UI

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Change the language to hebrew (he)
      2. Go to the grading table
      3. Click the grade button for a submission
      4. Change the grade
      5. Click save on the grading form (top right in the group of buttons)
        Expected result: The grading table should show the updated grade
        Actual result: The grading table shows the old grade
      Show
      Change the language to hebrew (he) Go to the grading table Click the grade button for a submission Change the grade Click save on the grading form (top right in the group of buttons) Expected result: The grading table should show the updated grade Actual result: The grading table shows the old grade
    • Workaround:
      Hide

      See code change in the description.

      Show
      See code change in the description.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:

      Description

      When UI is set to Hebrew (and probably other non English languages) It is not possible to update a grade or set a new grade in the new "Assign" (assignment type) submission form (mod/assign/view.php?id=XX&action=grading).

      Changing the following fixes this issue:
      moodle2/mod/assign/locallib.php (line 330)

                  } else if (optional_param('savegrade', null, PARAM_ALPHA)) {

      change to:

                  } else if (optional_param('savegrade', null, PARAM_RAW)) {

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              damyon Damyon Wiese
              Reporter:
              nadavkav Nadav Kavalerchik
              Integrator:
              Dan Poltawski
              Tester:
              Mark Nelson
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              3 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Nov/12