#19: Hide the Edit button if no eXe URL is provided

Author: amanzano3ip

classes/exeonline/exescorm_redirector.php

@@ -26,7 +26,10 @@
 
 namespace mod_exescorm\exeonline;
 
+use coding_exception;
+use dml_exception;
 use mod_exescorm\exeonline\token_manager;
+use moodle_exception;
 use moodle_url;
 
 class exescorm_redirector {
@@ -58,6 +61,9 @@ public function __construct(string $action = 'edit', moodle_url $returnto = null
      * @param moodle_url|null $returnto
      *
      * @return moodle_url
+     * @throws dml_exception
+     * @throws coding_exception
+     * @throws moodle_exception
      */
     public static function get_redirection_url(int $cmid, moodle_url $returnto = null, string $action = null) {
         global $CFG, $USER;
@@ -87,7 +93,7 @@ public static function get_redirection_url(int $cmid, moodle_url $returnto = nul
         }
         $url = $exeonlineurl . $target;
 
-        return new \moodle_url($url, $params);
+        return new moodle_url($url, $params);
     }
 
 

renderer.php

@@ -103,14 +103,22 @@ public function user_report_actionbar(\mod_exescorm\output\userreportsactionbar
      * Generate the EXESCORM's "Exit activity" button
      *
      * @param string $url The url to be hooked up to the exit button
-     * @param \stdClass $cm The course module viewed.
+     * @param stdClass $cm The course module viewed.
      * @return string
+     * @throws dml_exception
+     * @throws coding_exception
+     * @throws moodle_exception
      */
     public function generate_editexitbar(string $url, \stdClass $cm): string {
         $context = ['returnaction' => $url];
         if (has_capability('moodle/course:update', context_course::instance($cm->course))) {
             $returnto = new moodle_url("/mod/exescorm/view.php", ['id' => $cm->id, 'forceview' => 1]);
-            $context['editaction'] = exescorm_redirector::get_redirection_url($cm->id, $returnto)->out(false);
+            $exeonlineurl = get_config('exescorm', 'exeonlinebaseuri');
+            if (empty($exeonlineurl)) {
+                $context['editaction'] = false;
+            } else {
+                $context['editaction'] = exescorm_redirector::get_redirection_url($cm->id, $returnto)->out(false);
+            }
         }
         return $this->render_from_template('mod_exescorm/player_editexitbar', $context);
     }