diff --git a/admin/tool/brickfield/index.php b/admin/tool/brickfield/index.php index 3f9a4b038b029..02d2842ff7f8d 100644 --- a/admin/tool/brickfield/index.php +++ b/admin/tool/brickfield/index.php @@ -35,6 +35,7 @@ use tool_brickfield\task\process_analysis_requests; require('../../../config.php'); +require_once($CFG->libdir.'/adminlib.php'); // If this feature has been disabled, do nothing. accessibility::require_accessibility_enabled(); @@ -110,7 +111,12 @@ $tool->set_filter(new filter($courseid, $categoryid, $tab, $page, $perpage, $url, $target)); -$PAGE->navigation->override_active_url($navurl); +// Course and site require different navigation setups. +if ($courseid > SITEID) { + $PAGE->navigation->override_active_url($navurl); +} else { + admin_externalpage_setup('tool_brickfield_reports', '', null, '', ['pagelayout' => 'report']); +} $PAGE->set_context($context); $PAGE->set_url($url); $PAGE->set_pagelayout($layout); diff --git a/admin/tool/brickfield/lang/en/tool_brickfield.php b/admin/tool/brickfield/lang/en/tool_brickfield.php index 1c82ca88c9317..61bdeecc8aafe 100644 --- a/admin/tool/brickfield/lang/en/tool_brickfield.php +++ b/admin/tool/brickfield/lang/en/tool_brickfield.php @@ -83,7 +83,7 @@ $string['eventreport_downloadeddesc'] = 'Accessibility summary was downloaded for course id {$a}.'; $string['failed'] = 'Failed'; $string['failedcount'] = 'Failed: {$a}'; -$string['tools'] = 'Go to reports'; +$string['tools'] = 'Reports'; $string['formcorrectlysubmitted'] = 'Check Process successfully added'; $string['formdescription'] = 'This will add a new check process record which will run as a scheduled task'; $string['formtitle'] = 'Add Check Process'; diff --git a/admin/tool/brickfield/tests/behat/accessresults.feature b/admin/tool/brickfield/tests/behat/accessresults.feature index e6a6b8a6c9c4c..29bd478e21fcc 100644 --- a/admin/tool/brickfield/tests/behat/accessresults.feature +++ b/admin/tool/brickfield/tests/behat/accessresults.feature @@ -11,14 +11,14 @@ Feature: Brickfield activityresults | label | Label two | Click here | C1 | id002 | @javascript - Scenario: Test the Brickfield accessibility reports plugin + Scenario: Test the Brickfield accessibility tool plugin Given I log in as "admin" And I navigate to "Plugins > Admin tools > Accessibility > Brickfield registration" in site administration And I set the field "id_key" to "123456789012345678901234567890ab" And I set the field "id_hash" to "ab123456789012345678901234567890" And I press "Activate" Then I should see "Your accessibility toolkit is still being validated." - And I navigate to "Plugins > Admin tools > Accessibility > Go to reports" in site administration + And I navigate to "Plugins > Admin tools > Accessibility > Reports" in site administration And I press "Submit for analysis" Then I should see "The global (course independent) content has been scheduled for analysis." And I am on "Course 1" course homepage @@ -39,7 +39,7 @@ Feature: Brickfield activityresults And I set the field "id_key" to "123456789012345678901234567890ab" And I set the field "id_hash" to "ab123456789012345678901234567890" And I press "Activate" - And I navigate to "Plugins > Admin tools > Accessibility > Go to reports" in site administration + And I navigate to "Plugins > Admin tools > Accessibility > Reports" in site administration And I should see "Error details: All reviewed courses (2 courses)" And I follow "Activity breakdown" Then I should see "Results per activity: All reviewed courses (2 courses)"