"on View" not working from the "add a resource" drop down

Thanks Mary. I'm hoping to work on this tomorrow. If I don't comment further here inside a week, please feel free to post another comment to hassle me.

Fixed now. Support is supposed to be in place for automatic completion of all Resource types except Label, i.e. File, Folder, IMS content package, Page, URL.

I was not able to test IMS CP as I don't have any content packages handy, but I checked the rest, think it's working.

Note the file download completion is a bit disappointing because it doesn't automatically update the page straight away, i.e. you only see the tick next time you reload the course page, not immediately when the download starts. This is obviously because it was technically difficult to do but maybe it should be improved...

Hi Sam - a few of those commits broke -
Fatal error: Class 'completion_info' not found in /home/dan/www/workspace/moodle/mod/imscp/view.php on line 49

I see Dongsheng fixed one - I'll fix the imscp now, but you should probably check the others to make sure completionlib is included...

Hi Sam,

I fixed mod/url/view.php to include completionlib.php.

Thanks Dan, Rossiani. Actually I think I didn't break it - except imscp I tested the other types before so they used to work. I guess probably some other file used to require completion and now doesn't (probably for efficiency), which means the require_once is now needed at this level when it wasn't before.

I tested resource, folder, page, and url again, setting all of them to use 'view' completion and checking it actually worked when I viewed it. Folder and URL (thanks for fix) already worked. I just added the require_once into resource and page. I put it next to the lines that use it so hopefully it won't get lost if the files are rearranged again...

yeah - I'm pretty sure imscp was working a couple of weeks ago too, I couldn't quickly find the commit that pulled completionlib but saw that the commits that went in with this bug probably all needed checking again.