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.

CommentFileSizeAuthor
#3 no_dates.diff28.74 KBjackbravo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MGN’s picture

MGN CreditAttribution: MGN commented

Perhaps provide the teacher options to select the gradebook fields to include would be the best approach. That way its complete, but not required?

jackbravo’s picture

jackbravo CreditAttribution: jackbravo commented

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.

jackbravo’s picture

jackbravo CreditAttribution: jackbravo commented
FileSize
no_dates.diff28.74 KB

Here's a PATCH in case you want to move forward with these =P.

MGN’s picture

MGN CreditAttribution: MGN commented
Status: Active » Postponed

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.