UI Update: "Take Course" should change to "Review Course" after user enrolled

What if the user is enrolled but not finished?

Maybe we should have 3 states (and think of labels for them) - not started, in progress, completed

take
continue/resume
review

?

I am looking for this feature too: for the "take course" to change to "course outline" or "resume course" once the user is enrolled.

Or a way for the 'Take Course' tab to disappear after the user enrolled.

Or add a class to the 'Take Course' tab once the user is enrolled, like 'class='enrolled''

I found this in 633 of course.module:

function course_takecourse_title($node) {
     global $user;
     $report = course_report_load($node, $user);
     return ($user->uid > 1 && isset($report->complete) && $report->complete) ? t('Review Course') : t('Take Course');
}

I was thinking I could change '$report->complete' to '$report->incomplete' (or 'started') in this code to make it so the 'Take Course' button changes to 'Review Course' once the user was enrolled and taking the course, but there were errors. I am just a newbie at PHP, so I guess I am trying to see if this is the right place to be working on this problem.

Thanks!

OK -- I've got a short term fix! This will make it so that the tab changes to 'Review Course' when the user is enrolled.

@ line 633 of course.module:

function course_takecourse_title($node) {
     global $user;
     $report = course_report_load($node, $user);
     return ($user->uid > 1 && isset($report->complete) && ($report->complete == 0)) ? t('Review Course') : t('Take Course');
}

I'm using 7.x-1.0-rc1

Shouldn't matter, that bit of code hasn't changed.

This all looks good I'm just pondering over what is a good default for all users.

I'm using 7.x-1.0-rc4

Hi djdevin

Thanks your great module. I also looking for the feature.

1. Take Course
2. Resume Course / Course Overview / Course Progress
3. Review Course

maybe you can reference teamtreehouse.com's logic

Perhaps not the most elegant solution, but here's a working patch. If anyone has ideas on how or if this can be improved let me know, otherwise I think this is a decent solution.

Here it is for 3.x