From Eloy

1. get_string('no_' . $id, 'lti') ??
2. lti_get_url_thumbprint(): Why is doing that with "www" part of the hostname?
3. review lti_filter_get_types() uses, missing $course param
4. lti_filter_print_types() unused? Also it included old output code (harcoded paths to icons, echos...) would be great to update as much as possible to current output stuff.
5. $type needs use of new stdClass(), cannot be constructed implicitly.
lti_post_launch_html() cannot it build that form in another way. Are all the inserted values (endpoint/params...) safe/encoded enough?