search

brendanheywood / moodle-local_cleanurls

Lets drag Moodle's url structure into this century...
37 stars 24 forks source link

Summary renderer doesn't respect non-top-level links #126

Open Arantor opened 6 years ago

Arantor commented 6 years ago

I'm testing this out on an installation that lives at localhost/moodle34/ and on the list of tests, none of the links work because they all end up pointing at localhost/local/cleanurls/

Easy fix - in classes/output/webserver_summary_renderer.php, where there is the html_writer::link call, just have it call via moodle_url instantiation:

html_writer::link(new \moodle_url("/local/cleanurls/webservertest.php?details={$classname}"), $webtest->get_name()),