Moodle

Assigning to Mark and addessing for 1.9.4. Thanks!

BUMP!
This is a major bug and should really be fixed ASAP.
See http://moodle.org/mod/forum/discuss.php?d=98346
Thanks in advance,
Joseph

This shows clearly the bug in the Gradebook.

This IS a problem. How ever many points you get on a lesson, that same number becomes your percentage score! 5 out of 5 points becomes a score of 5%. Thanks for your attention to this.

Any progress on this? This bug has completed disabled our ability to use Lesson as a graded item unless we complete rework the gradebook into weighted categories and completely redoing the gradebook for all our classes with graded lessons in the middle of the semester is really not an option. We have 1.9.2+ PLEASE HELP ASAP. This combined with other gradebook challenges is really decreasing the moodle enthusiasm to a level I have never seen. I fear serous mutiny. This lesson problem is the last straw for many but if it can be fixed quickly their might be hope. Appreciate whatever anyone could do. I would help if I had the skills.

Hi All:

I've encountered a bug in the lesson module that may be related to this issue in at least two builds of 1.9.4+. I'm currently running 1.9.4+ (Build: 20090218).

Environment: php 5.2.8, V1.9.4+ (Build: 20090218), MySQL 5.0.67-community, Apache 2.2.11 (Unix), Perl 5.8.8, Operating system Linux Kernel 2.6.28

Any time a question includes a short answer, the question attempt is recorded. However, the response is not recorded as correct even if it was correct. This causes the student score to be incorrect. The lesson pages process correctly. The correct feed back is displayed. Everything appears to work perfectly, except that he total score for the lesson is not incremented properly.

Between my hosting company and myself, we have done some fairly extensive trouble shooting, and it looks like it is a bug.

This is really becoming a problem on our site!

@ Carl Keller
More specifically this is what I have observed with latest 1.9.4 version.
if lesson parameter custom scoring is set to YES, then all question types will record the correct scores
if lesson parameter custom scoring is set to NO, then all question types will record the correct scores EXCEPT short answer!

There are obviously serious bugs in the scoring system in the Lesson module. Mark, please can you fix those ASAP?
Thanks
Joseph

Looks like the problem is that the lesson module stores percentages in its grade table and then was sending the percentage to the gradebook instead of the raw grade.

I'm attaching a patch that should now send the correct grade to the gradebook and also fix current grades via a lesson upgrade (so, need to visit Site Administration > Notifications). If anyone would like to test the patch, that would be fantastic! Just remember to use it on a copy of your database and that the patch is not finalized yet.

I'd like someone from core to review the patch before it goes in as I'm not the most familiar with gradebook nor am I certain if the version bump was done correctly (I think it's OK, but don't want to mess up upgrades to 2.0).

Also, in the patch, the same snippet of code is executed in upgrade.php for different versions (once for the gradebook update and now for this fix). I'm wondering if I should try to prevent that from happening twice or not (that being calling lesson_update_grades() twice in one upgrade).

Petr, from what I understand, you are familiar with the gradebook upgrade, etc. Could you take a look at the patch when you get a chance?

Updated the patch based on Petr's recommendations.

@Mark
I applied your most recent patch (lesson_grade2.patch) to my current moodle 1.9.4 local installation.
1- it works OK as far as grades are concerned
2- but it throws an error when I delete selected attempts:

Notice: Trying to get property of non-object in C:\Program Files\xampp\htdocs\moodle19dev\moodle\mod\lesson\lib.php on line 433
Fatal error: Cannot use object of type object as array in C:\Program Files\xampp\htdocs\moodle19dev\moodle\mod\lesson\lib.php on line 433

Joseph

Good catch. It appears that $grades can be several different values and types. Looking into a solution now.

Attaching a new patch (lesson_grade3.patch). Looks like grade_update() eventually converts $grades to an array of arrays. The lesson code now does this as it calculates the raw grade.

so am I correct in assuming the grade2 patch is for v1.9.2+ That is the version we are having issues with. If so will have remote learner install and I can go in and see if that worked. thanks for your efforts. This has been a nightmare for instructors with lots of graded lessons.

lesson_grade3,patch is the current one, not lesson_grade2.patch. The patch is against 1.9.4+ (Build: 20090224), which is the llatest in MOODLE_19_STABLE. The patch has not been finalized yet, Petr was planning on testing it today.

So is a patch going to be created for version Moodle 1.9.2+ (Build: 20080813)
I am the original person who posted this on moodle.org and Joseph so kindly posted this in tracker.
We can not upgrade to 1.9.4+ because we have activity locking installed and it is not stable for v1.9.4+

looking at the code, I think the grade altering should would look much better in lesson_get_user_grades() , right?

I hope you backported all the security patches somehow, if not you are in big trouble

"looking at the code, I think the grade altering should would look much better in lesson_get_user_grades() , right? "

That would assume that the only grades that are passed to lesson_grade_item_update() are from lesson_get_user_grades(). Doesn't seem like a safe assumption, but maybe it is? I'm not sure what else might call lesson_grade_item_update().

Full text search is your best friend if you need to know how and where are xxx_grade_item_update() used

Hi All:

Will someone please point me toward a resource explaining how to install this patch on a shared server? Our online courses are really in a bind over this because they rely heavily on lessons for grading. Also please see MDL-18347.

Thanks!

Petr, where are we with this patch? Can it be applied yet?

Committed to 1.9 branch and merge to trunk. Had to modify the upgrade code to match the new API in Moodle 2.0.

so will this fix version 1.9.2+ we are dying! help! remote-learner is trying hard to assist. We can not upgrade to 1.9.4. Thanks for your help!

Not entirely sure what's in the 1.9.2+ code. If you cannot upgrade, then your best bet would probably be to try to apply lesson_grade3.patch (attached to this ticket). Please test on a copy of your code and database. If that doesn't work, then you'll need a programmer to look at your specific code and derive a solution for it.

Actually, I take some of that back. You may upgrade your lesson module past incremental upgrades that might be introduced between your version and the current 1.9 version. So, you'll have to have a programmer to look at your code and see what can be done.

When I upgrade HEAD I get:

Warning: Missing argument 1 for lesson_update_grades(), called in /home/tim/Workspace/moodle_head/mod/lesson/db/upgrade.php on line 52 and defined in /home/tim/Workspace/moodle_head/mod/lesson/lib.php on line 383

Also,

$DB->set_debug(false);
lesson_update_grades();
$DB->set_debug(true);

is unnecessary (and evil) in HEAD. We have stopped outputting successful SQL there, so actually, the $DB->set_debug(false); is irrelevant, and that $DB->set_debug(true); screws things up.

Ah, I think the call should be to lesson_upGRADE_grades, but I am guessing. I'll checking the changes, since they make the error messgage go away, but I will leave this bug reopened because:

• Someone who understands should review the chagnes.
• The change may need backporting to 1.9 branch, and I have not done that.

Tim, your right, the function that needs to be call in HEAD is lesson_upgrade_grades() - it's a new function in trunk which does what lesson_update_grades() did when no parameters are passed. Removing the debug lines is fine, I didn't know about that change.

Nothing should be backported as I believe these are all HEAD specific issues.

Let me know if your committing your code or if I should make the changes and commit.

Never mind, just noticed that you committed already. Looks good to me.

Hi Guys!

I have upgraded to the latest version, and I my lessons still won't increment properly when short answers are used! Any ideas? This is quite an issue for us.

Thanks!

Hello Everybody

I use Moodle 1.9.4+. I patched all three of files using patch 'lesson_grade3.patch', but lesson grades in gradebook are still displayed always as 100%.

Regards

The patch has been applied to the code already and is in the weekly download (http://download.moodle.org/). Did you visit the admin notifications area to run database updates? And did you try attempting a lesson to see if you get 100% in the gradebook while not getting a perfect score?

Thanks for Your response.

When i attempt a lesson after applying patch, and my score is not 100%, the grade shown in gradebook is displayed propertly. But attempts made before patching are still displayed always as 100%. Entering in admin notification does not applying database updates.

Finally I changed whole /mod/lesson/ folder to this taken from latest weekly download, but admin notification still do not apply database updates, and previous lesson attempts are shown as 100% in gradebook. I changed grade of this attempts manually, because there was very few of them.

Regards, ?ukasz

what is the status of this issue?

Hi Petr:

As far as I am concerned, the lesson part of the issue was fixed when I installed the updated download that Mark Nielsen mentioned in his comment on 24/Mar/09 01:39 AM.

resolving as fixed, please file a new issue if needed, thanks everybody

I created a lesson with a maximum grade of 5. Completed the lesson as a student and my grade appears to be being correctly represented on the grader report. I was also able to delete attempts at the lesson without an error. Closing.