Merge pull request #307 from FraGoTe/7173

7173

main/inc/lib/plugin.class.php

@@ -485,11 +485,10 @@ class Plugin
     public function addTab($tabName, $url)
     {
         $sql = "SELECT * FROM settings_current
-                WHERE
-                    variable = 'show_tabs' AND
-                    subkey like 'custom_tab_%'";
+            WHERE
+            variable = 'show_tabs' AND
+            subkey like 'custom_tab_%'";
         $result = Database::query($sql);
-
         $customTabsNum = Database::num_rows($result);
 
         $tabNum = $customTabsNum + 1;
@@ -497,6 +496,22 @@ class Plugin
         //Avoid Tab Name Spaces
         $tabNameNoSpaces = preg_replace('/\s+/', '', $tabName);
         $subkeytext = "Tabs" . $tabNameNoSpaces;
+
+
+        //Check if it is already added
+        $checkCondition = array(
+            'where' =>
+                array(
+                    "variable = 'show_tabs' AND subkeytext = ?" => array(
+                        $subkeytext
+                    )
+                )
+        );
+        $checkDuplicate = Database::select('*', 'settings_current', $checkCondition);
+        if (!empty($checkDuplicate)) {
+            return false;
+        }
+        //End Check
         $subkey = 'custom_tab_' . $tabNum;
         $attributes = array(
             'variable' => 'show_tabs',
@@ -521,7 +536,6 @@ class Plugin
         $whereCondition = array(
             'id = ?' => key($settings)
         );
-
         Database::update('settings_current', $setData, $whereCondition);
 
         return $resp;
@@ -530,6 +544,8 @@ class Plugin
     /**
      * Delete a tab to chamilo's platform
      * @param string $key
+     *
+     * @return boolean $resp
      */
     public function deleteTab($key)
     {