Reviewed: https://reviews.mahara.org/c/mahara/+/13741 Committed: https://git.mahara.org/mahara/mahara/commit/511e22608a0f72b40ae0f764e6cfc0b8d1c92d1f Submitter: "Robert Lyon <[email protected]>" Branch: main
commit 511e22608a0f72b40ae0f764e6cfc0b8d1c92d1f Author: Gold <[email protected]> Date: Thu Feb 9 12:07:40 2023 +1300 Bug 2004852: LTI 1.1 grading vs some locales Ensure the format of the grade uses a dot regardless of the system locale. Signed-off-by: Gold <[email protected]> Change-Id: I4b669486c2f73eff73ce940300ba26db4586e914 -- You received this bug notification because you are a member of Mahara Contributors, which is subscribed to Mahara. Matching subscriptions: mahara-contributors https://bugs.launchpad.net/bugs/2004852 Title: LTI 1.1 grading not working for some locales Status in Mahara: Fix Committed Status in Mahara 21.10 series: In Progress Status in Mahara 22.04 series: In Progress Status in Mahara 22.10 series: In Progress Status in Mahara 23.04 series: Fix Committed Bug description: With a LTI to a Moodle instance, when the teachers grades a submission, the grade is not sent to Moodle in some cases. I think it has to do with my server's locale set to french. The grade of 94%, for example, si translated to 0,94 instead of 0.94. I printed some debug and was able to see the error "Score must be numeric". I also printed the xml from the $body variable (see attachment). I made a few changes in function publish_lti_outcome of htdocs/module/lti/lib.php around line 1001 and it now works correctly. $grade = $this->grade / 100; $grade = str_replace(',', '.', $grade); $smarty->assign('score', $grade); This was tested with Mahara 22.10 and Moodle 3.11. To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/2004852/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp
