Make the DASHBOARD_PREFIX language string more flexible for LTI deepl…

…inking
thexerteproject · Feb 25, 2024 · 66adde0 · 66adde0
1 parent 0a9ca6d
commit 66adde0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/languages/en-GB/load_registrations.inc b/languages/en-GB/load_registrations.inc
@@ -11,7 +11,7 @@
  */
 
 
-define("LTI_DEEPLINK_DASHBOARD_PREFIX", "Dashboard for ");
+define("LTI_DEEPLINK_DASHBOARD_PREFIX", "{0} - Dashboard");
 define("LTI_DEEPLINK_AUTHORS", "Author(s): ");
 define("LTI_DEEPLINK_COURSE", "Course   : ");
 define("LTI_DEEPLINK_ID", "Id: ");

diff --git a/load_registrations.php b/load_registrations.php
@@ -81,9 +81,12 @@ function loadRegistrations()
                 if ($template['tsugi_xapi_enabled']) {
                     $dtool = array();
                     $dtool['url'] = $site_url . "tools/dashboard/index.php?template_id=" . $template['template_id'];
-                    $dtool['name'] = LTI_DEEPLINK_DASHBOARD_PREFIX . $metadata->name;
-                    $dtool['short_name'] = LTI_DEEPLINK_DASHBOARD_PREFIX . $tool['short_name'];
-                    $dtool['description'] = LTI_DEEPLINK_DASHBOARD_PREFIX . $tool['description'];
+                    $name = str_replace("{0}", $metadata->name, LTI_DEEPLINK_DASHBOARD_PREFIX);
+                    $dtool['name'] = $name;
+                    $short_name = str_replace("{0}", $tool['short_name'], LTI_DEEPLINK_DASHBOARD_PREFIX);
+                    $dtool['short_name'] = $short_name;
+                    $description = str_replace("{0}", $tool['description'], LTI_DEEPLINK_DASHBOARD_PREFIX);
+                    $dtool['description'] = $description;
                     $dtool['FontAwesome'] = "fa-chart-bar";
                     $tools[] = $dtool;
                 }