Browse Source

Fix plugin icon see BT#10139

Julio Montoya 9 years ago
parent
commit
ef371df086
2 changed files with 10 additions and 6 deletions
  1. 2 1
      main/inc/lib/course_home.lib.php
  2. 8 5
      main/inc/lib/plugin.class.php

+ 2 - 1
main/inc/lib/course_home.lib.php

@@ -858,7 +858,8 @@ class CourseHome
                         $tool_name = $plugin_info['title'];
                     }
 
-                    if (!file_exists(api_get_path(SYS_CODE_PATH).'img/'.$tool['image'])) {
+                    if (!file_exists(api_get_path(SYS_CODE_PATH).'img/'.$tool['image']) &&
+                        !file_exists(api_get_path(SYS_CODE_PATH).'img/icons/22/'.$tool['image'])) {
                         $tool['image'] = 'plugins.png';
                     }
                     $tool_link_params['href'] = api_get_path(WEB_PLUGIN_PATH).$tool['original_link'].'?'.api_get_cidreq();

+ 8 - 5
main/inc/lib/plugin.class.php

@@ -251,6 +251,7 @@ class Plugin
         }
         $result->setDefaults($defaults);
         $result->addButtonSave($this->get_lang('Save'), 'submit_button');
+
         return $result;
     }
 
@@ -365,6 +366,7 @@ class Plugin
     /**
      * Caller for the install_course_fields() function
      * @param int $courseId
+     *
      * @param boolean $addToolLink Whether to add a tool link on the course homepage
      *
      * @return void
@@ -379,6 +381,7 @@ class Plugin
      * @param int $courseId Course integer ID
      * @param boolean $add_tool_link Whether to add a tool link or not
      * (some tools might just offer a configuration section and act on the backend)
+     *
      * @return boolean  False on error, null otherwise
      */
     public function install_course_fields($courseId, $add_tool_link = true)
@@ -435,25 +438,25 @@ class Plugin
         }
 
         //Add an icon in the table tool list
-        $t_tool = Database::get_course_table(TABLE_TOOL_LIST);
-        $sql = "SELECT name FROM $t_tool
+        $table = Database::get_course_table(TABLE_TOOL_LIST);
+        $sql = "SELECT name FROM $table
                 WHERE c_id = $courseId AND name = '$plugin_name' ";
         $result = Database::query($sql);
         if (!Database::num_rows($result)) {
             $tool_link = "$plugin_name/start.php";
-            $visibility = AddCourse::string2binary(api_get_setting('course_create_active_tools', $plugin_name));
+            //$visibility = AddCourse::string2binary(api_get_setting('course_create_active_tools', $plugin_name));
 
             $cToolId = AddCourse::generateToolId($courseId);
 
             Database::insert(
-                $t_tool,
+                $table,
                 [
                     'id' => $cToolId,
                     'c_id' => $courseId,
                     'name' => $plugin_name,
                     'link' => $tool_link,
                     'image' => "$plugin_name.png",
-                    'visibility' => $visibility,
+                    'visibility' => 1,
                     'admin' => 0,
                     'address' => 'squaregrey.gif',
                     'added_tool' => 'NO',