XAPI: Fix installation - refs BT#16742

chamilo · Nov 27, 2020 · 6d00e92 · 6d00e92
1 parent 33167e7
commit 6d00e92
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/plugin/xapi/src/XApiPlugin.php b/plugin/xapi/src/XApiPlugin.php
@@ -92,7 +92,10 @@ public function install()
         $em = Database::getManager();
 
         $tablesExists = $em->getConnection()->getSchemaManager()->tablesExist(
-            ['xapi_shared_statement']
+            [
+                'xapi_shared_statement',
+                'xapi_tool_launch',
+            ]
         );
 
         if ($tablesExists) {
@@ -101,7 +104,6 @@ public function install()
 
         $this->installPluginDbTables();
         $this->installUuid();
-        $this->deleteCourseTools();
         $this->addCourseTools();
     }
 
@@ -166,10 +168,10 @@ public function uninstallHook()
         $quizQuestionAnsweredHook = XApiQuizQuestionAnsweredHookObserver::create();
         $quizEndHook = XApiQuizEndHookObserver::create();
 
-        HookLearningPathItemViewed::create()->attach($learningPathItemViewedHook);
+        HookLearningPathItemViewed::create()->detach($learningPathItemViewedHook);
         HookLearningPathEnd::create()->detach($learningPathEndHook);
-        HookQuizQuestionAnswered::create()->attach($quizQuestionAnsweredHook);
-        HookQuizEnd::create()->attach($quizEndHook);
+        HookQuizQuestionAnswered::create()->detach($quizQuestionAnsweredHook);
+        HookQuizEnd::create()->detach($quizEndHook);
 
         return 1;
     }
@@ -287,9 +289,6 @@ public function performActionsAfterConfigure()
             $quizEndEvent->detach($quizEndHook);
         }
 
-        $this->deleteCourseTools();
-        $this->addCourseTools();
-
         return $this;
     }