Completion status block shouldn't ever be blank

Howard, thanks for your report. How about it displaying the message "Completion tracking criteria not yet set."

Assigning to Aaron as course completion component lead.

Oh yeh, not critical... should have said "something like". The main thing is that it doesn't mysteriously not appear and you're sat there wondering why

Yes, I agree with you Howard. As an interim measure I've added a note to http://docs.moodle.org/en/Course_completion_status_block

Fix, will update with pull request tomorrow morning. Thanks for the bug report Howard.

I also fixed similar issues with the selfcompletion block.

https://github.com/srynot4sale/moodle/commit/0b15b43b3ed2f05aa7c84b6551d67255e1faff9c

Here is the fix for the PULL request

Aaron, thanks for your fix. Please could you create a pull request for it and include brief instructions for testing the fix.

http://tracker.moodle.org/browse/PULL-373 done

Hi PULL-373 has been integrated... just one thought about strings like 'completionnotenabled' perhaps being better if placed in lang/en/completion.php (centrally, only once), instead of repeated in all the blocks using it.

Obviously if the meaning is 100% the same, like seems to be in this case.

For your consideration...ciao

This issue is fixed in the latest 2.0.2+. Thanks everyone.

I would like to reopen this issue as I have some further improvements on the original patch

Reproduce problem:

• All users (inc students) see warnings when completion not enabled, or criteria not being set for both Course Completion Status and Self Completion blocks
• A user can still view a Course Completion Status block details page if completion was enabled for the course, but was then turned off site wide (accessible via the block and clicking the "More Details" link).
• If viewing the Course Completion Status block details page for a user not in a graded role in the course, shows the "You are not enroled as a student in this course" error message even for other users.

Reproduce fix:

• Only users with the course:edit capability should see warnings about completion not being enabled, or criteria not being set for both Course Completion Status and Self Completion blocks
• If course completion is enabled for a course, but disabled sitewide show an error when viewing a Course Completion Status block details page
• If viewing the Course Completion Status block details page for another user not in a graded role in the course, show the "User is not enroled as a student in this course" error message rather than "You are not enroled as a student in this course".
• Show message in the Self Completion block if the users is already complete in the course, or has already self completed themselves.

Aaron, thanks for working on this, but can you please not re-open issues once they have been integrated and closed?

Any further improvements should be done in a new tracker issue. You can link back to this one for context, of course.

Closing this for now, under the assumption you'll create a new one.