Moodle

When an activity is created win "no grade" as maximum grade, it does not appear in gradebook 1.9Beta3 (WITH PATCH)

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Won't Fix
  • Affects Version/s: 1.9
  • Fix Version/s: 1.9
  • Component/s: Gradebook
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

When a new assignment is created with a maximum grade of "no grade", it does not appear in the gradebook. However, when you change this item to have a value for maximum grade it does appear as a gradebook item, and it continues to be a gradebook item if you re-assign it to no grade.

The correct behavior is to have it be a gradebook item even if "no grade" is set upon creation. It will simply play no role in a computation. Otherwise it will (and has) caused teachers much confusion when the believe that some of their assignments have been lost in the gradebook.

It is not uncommon to have an assignment listed as no-grade to be a place holder until the teacher adjusts that assignment the number of points desired. So the behavior suggested is important. But in any case, the current behavior is inconsistent regardless of ones view of whether assignments with not grade should be listed.

Issue Links

Activity

Hide
Gary Anderson added a comment -

This issue is because of lines 120 to 123 of lib/gradelib.php.

In my view and for the reasons described above, these lines should be removed.

if ($k == 'gradetype' and $v == GRADE_TYPE_NONE) { // no grade item needed! return GRADE_UPDATE_OK; }

To summarize:

1. It is confusing when activities don't appear in the gradebook (in the field, teachers report missing items)
2. No grade might be temporary
3. There is inconsistency in changing back and forth between max grades
4. There is no adverse effect on computations of keeping the grade item in
5. The teacher can always hide the grade item from students if s/he wishes

Show
Gary Anderson added a comment - This issue is because of lines 120 to 123 of lib/gradelib.php. In my view and for the reasons described above, these lines should be removed. if ($k == 'gradetype' and $v == GRADE_TYPE_NONE) { // no grade item needed! return GRADE_UPDATE_OK; } To summarize: 1. It is confusing when activities don't appear in the gradebook (in the field, teachers report missing items) 2. No grade might be temporary 3. There is inconsistency in changing back and forth between max grades 4. There is no adverse effect on computations of keeping the grade item in 5. The teacher can always hide the grade item from students if s/he wishes
Hide
Petr Škoda (skodak) added a comment -

This is intentional - I remember discussing it with MD.
"No grade" means no grade will be given, I can imagine other ppl complaining that assignments appear in gradebook even if no grade expected.
The items are not removed when changing to no-grade because they may contain locked or overridden grades, I think there should be a way to remove them though.

I propose to alow manual deleting of items linked to activities with no-grade setting instead.

Show
Petr Škoda (skodak) added a comment - This is intentional - I remember discussing it with MD. "No grade" means no grade will be given, I can imagine other ppl complaining that assignments appear in gradebook even if no grade expected. The items are not removed when changing to no-grade because they may contain locked or overridden grades, I think there should be a way to remove them though. I propose to alow manual deleting of items linked to activities with no-grade setting instead.
Hide
Gary Anderson added a comment -

No, I think that allowing manual deletion of items linked to such activities would cause more confusion, rather than less.

I think that there should be a 1:1 correspondence with grade items and activities. There is no mystery to teachers of why such items are in the gradebook, and they have options if they don't like it such as hiding the items from students, putting it in an ungraded category, or using something like a label rather than an assignment for such tasks.

And certainly teachers who are transitioning from previous versions of Moodle will not be surprised that such items are in the gradebook.

I can report from the field that the current behavior caused considerable panic among teachers who reported that their assignments were suddenly missing, and the patch is a solution.

Show
Gary Anderson added a comment - No, I think that allowing manual deletion of items linked to such activities would cause more confusion, rather than less. I think that there should be a 1:1 correspondence with grade items and activities. There is no mystery to teachers of why such items are in the gradebook, and they have options if they don't like it such as hiding the items from students, putting it in an ungraded category, or using something like a label rather than an assignment for such tasks. And certainly teachers who are transitioning from previous versions of Moodle will not be surprised that such items are in the gradebook. I can report from the field that the current behavior caused considerable panic among teachers who reported that their assignments were suddenly missing, and the patch is a solution.
Hide
Petr Škoda (skodak) added a comment -

With all respect one institution is not a representative sample. Did somebody taught them to use no-grade for something else than "Not graded"?
New gradebook is not supposed to have bug per bug compatibility with old implementation, also the grading changes in activities and not fully implemented yet, I expect there will be some more changes in 2.0

I do not really like the idea of having tens of not-graded activities in gradebook when only one or two activities are expected to be graded in fact. This might be another configuration option, but the defaults should be IMHO no-grade activities do not appear in gradebok.

Show
Petr Škoda (skodak) added a comment - With all respect one institution is not a representative sample. Did somebody taught them to use no-grade for something else than "Not graded"? New gradebook is not supposed to have bug per bug compatibility with old implementation, also the grading changes in activities and not fully implemented yet, I expect there will be some more changes in 2.0 I do not really like the idea of having tens of not-graded activities in gradebook when only one or two activities are expected to be graded in fact. This might be another configuration option, but the defaults should be IMHO no-grade activities do not appear in gradebok.
Hide
Gary Anderson added a comment -

Let's just keep it as a patch available in the tracker and the decision can be deferred until you can get a larger sample of it being used in the field.

That is the nice things about the software being written in a scripting language: if you don't like the way something works, it can be easily changed.

--Gary

Show
Gary Anderson added a comment - Let's just keep it as a patch available in the tracker and the decision can be deferred until you can get a larger sample of it being used in the field. That is the nice things about the software being written in a scripting language: if you don't like the way something works, it can be easily changed. --Gary
Hide
Petr Škoda (skodak) added a comment -

yes, as usually we will get much wider testing just after 1.9.0 release - thanks for any reports and suggestions!

Show
Petr Škoda (skodak) added a comment - yes, as usually we will get much wider testing just after 1.9.0 release - thanks for any reports and suggestions!
Hide
Petr Škoda (skodak) added a comment -

I have enabled deleting of items attached to activities with "No grade" - so that ppl may delete them from gradebook.
Also there is a new meta bug for this kind of patches - I agree these might be useful in real life.

thanks again!

Show
Petr Škoda (skodak) added a comment - I have enabled deleting of items attached to activities with "No grade" - so that ppl may delete them from gradebook. Also there is a new meta bug for this kind of patches - I agree these might be useful in real life. thanks again!

People

Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: