diff --git a/analytics/classes/insights_generator.php b/analytics/classes/insights_generator.php index 24fdfaf1d8d..6dad5ebcba9 100644 --- a/analytics/classes/insights_generator.php +++ b/analytics/classes/insights_generator.php @@ -195,7 +195,8 @@ class insights_generator { $actionurl = $action->get_url(); if (!$actionurl->get_param('forwardurl')) { - $actiondoneurl = new \moodle_url('/report/insights/done.php'); + $params = ['actionvisiblename' => $action->get_text(), 'target' => '_blank']; + $actiondoneurl = new \moodle_url('/report/insights/done.php', $params); // Set the forward url to the 'done' script. $actionurl->param('forwardurl', $actiondoneurl->out(false)); } diff --git a/report/insights/done.php b/report/insights/done.php index d64d15120b4..4878ef0092d 100644 --- a/report/insights/done.php +++ b/report/insights/done.php @@ -26,5 +26,18 @@ require_once(__DIR__ . '/../../config.php'); require_login(); -$url = new \moodle_url('/'); -redirect($url, get_string('actionsaved', 'report_insights'), null, \core\output\notification::NOTIFY_SUCCESS); +$actionvisiblename = required_param('actionvisiblename', PARAM_NOTAGS); + +$PAGE->set_pagelayout('popup'); +$PAGE->set_context(\context_system::instance()); +$PAGE->set_title(get_site()->fullname); +$PAGE->set_url(new \moodle_url('/report/insights/done.php')); + +echo $OUTPUT->header(); + +$notification = new \core\output\notification(get_string('actionsaved', 'report_insights', $actionvisiblename), + \core\output\notification::NOTIFY_SUCCESS); +$notification->set_show_closebutton(false); +echo $OUTPUT->render($notification); + +echo $OUTPUT->footer(); diff --git a/report/insights/insights.php b/report/insights/insights.php index c45aaf9e79b..2aaab8b05f3 100644 --- a/report/insights/insights.php +++ b/report/insights/insights.php @@ -103,8 +103,12 @@ if (!$model->uses_insights()) { exit(0); } +if ($context->id == SYSCONTEXTID) { + $PAGE->set_heading(get_site()->shortname); +} else { + $PAGE->set_heading($insightinfo->contextname); +} $PAGE->set_title($insightinfo->insightname); -$PAGE->set_heading($insightinfo->contextname); // Some models generate one single prediction per context. We can directly show the prediction details in this case. if ($model->get_analyser()::one_sample_per_analysable()) { diff --git a/report/insights/lang/en/report_insights.php b/report/insights/lang/en/report_insights.php index 646a6e6694d..db7c32e2651 100644 --- a/report/insights/lang/en/report_insights.php +++ b/report/insights/lang/en/report_insights.php @@ -22,11 +22,9 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ -$string['actionsaved'] = 'Your action has been saved.'; -$string['calculatedvalue'] = 'Calculated value'; +$string['actionsaved'] = 'Your feedback of \'{$a}\' has been saved.'; $string['disabledmodel'] = 'Sorry, this model has been disabled by the administrator'; -$string['indicator'] = 'Indicator'; -$string['insightprediction'] = '{$a} prediction'; +$string['indicators'] = 'Indicators'; $string['insight'] = 'Insight'; $string['insights'] = 'Insights'; $string['justpredictions'] = 'Please note that the following insights are only predictions. It is not possible to predict the future with any certainty. The insights are provided so that action can be taken as necessary to prevent any negative predictions becoming reality.'; @@ -39,7 +37,6 @@ $string['outcomeverypositive'] = 'Very positive outcome'; $string['outcomeverynegative'] = 'Very negative outcome'; $string['pluginname'] = 'Insights'; $string['prediction'] = 'Prediction'; -$string['predictioncalculations'] = 'Indicator calculations'; $string['predictiondetails'] = 'Prediction details'; $string['nodetailsavailable'] = 'No prediction details are relevant.'; $string['timecreated'] = 'Time predicted'; diff --git a/report/insights/prediction.php b/report/insights/prediction.php index bb67a028487..6db5e394bc6 100644 --- a/report/insights/prediction.php +++ b/report/insights/prediction.php @@ -71,8 +71,12 @@ if (!$model->uses_insights()) { exit(0); } +if ($context->id == SYSCONTEXTID) { + $PAGE->set_heading(get_site()->shortname); +} else { + $PAGE->set_heading($insightinfo->contextname); +} $PAGE->set_title($insightinfo->insightname); -$PAGE->set_heading($insightinfo->contextname); echo $OUTPUT->header(); diff --git a/report/insights/templates/insight_details.mustache b/report/insights/templates/insight_details.mustache index ad61ef1b94a..33164dd933e 100644 --- a/report/insights/templates/insight_details.mustache +++ b/report/insights/templates/insight_details.mustache @@ -73,7 +73,7 @@ } }} -

{{#str}}insightprediction, report_insights, {{insightname}} {{/str}}

+

{{insightname}}

{{#showpredicionheading}} - + {{#timerange}} - + {{/timerange}}
@@ -101,29 +101,23 @@ {{#str}}predictiondetails, report_insights{{/str}}
{{#str}}timecreated, report_insights{{/str}}{{#str}}timecreated, report_insights{{/str}} {{timecreated}}
{{#str}}timerange, report_insights{{/str}}{{#str}}timerange, report_insights{{/str}} {{.}}
- - - - - - - + {{#calculations}} - + {{/calculations}}
{{#str}}predictioncalculations, report_insights{{/str}}
{{#str}}indicator, report_insights{{/str}}{{#str}}calculatedvalue, report_insights{{/str}}
{{#str}}indicators, report_insights{{/str}}
{{name}}{{name}} {{#outcomeicon}}{{> core/pix_icon}}{{/outcomeicon}} {{displayvalue}}