Simplify gradebook (remove due_date and publish_date)
jackbravo - May 29, 2009 - 22:26
| Project: | Gradebook |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | postponed |
Description
I think these fields (due_date and publish_date) should be removed from the gradebook module.
Some assignments may have due and publish dates, but some don't. Maybe I just want to grade a course.
I think using date and scheduler modules you can pretty much do everything you may want to do with those fields.
That would specialize and simplify this module.
#1
Perhaps provide the teacher options to select the gradebook fields to include would be the best approach. That way its complete, but not required?
#2
Could be. That would solve my problem, where I don't want to use that fields. But that adds more code that has nothing to do with grading, and that its already implemented using CCK with other modules (adding and removing fields, and doing stuff with them).
That's why I think it would be better to remove that functionality from the gradebook module.
#3
Here's a PATCH in case you want to move forward with these =P.
#4
Thanks for the patch. I am moving forward with this, but taking a different approach to allow the teacher to decide whether or not to use dates for assignments, and more general control over what is displayed on the gradebook grade page. I've already refactored a lot of code, so the patch won't help me at this point...but thanks for the effort! I am hoping to start updating the 6.x-2.x-dev code sometime before August.
Marking this as postponed until then.