In course_module_deleted::get_description is recording the instanceid instead of objectid. Would it make more sense to use the course module ID since that is a lot easier to lookup? (I'm thinking of a user trying to manually enter the URL and also I feel like it is more standard to refer to modules by their cmid rather than their instanceid)

The course_module_created::get_description is doing the same as above.

In course_module_created::get_url and course_module_created::get_legacy_logdata, it is using instanceid to form the URL, pretty sure this should be objectid.