Chamilo LMS

olpc_peru_filter_na.png and olpc_peru_filter.png images need to be added

Otherwise an error will be shown in firebug

"NetworkError: 404 Not Found - main/img/olpc_peru_filter_na.png"

I think it would be better to modify the plugin.lib.php instead (because in this case I have added a feature to avoid showing the plugin in the course homepage, which should remove the need for a 64px image). I'll check that.

Julio A. Montoya wrote:

"NetworkError: 404 Not Found - main/img/olpc_peru_filter_na.png"

The location of this file here kind of implies that it loads it from the root of /main/img/... why would that be?

You can put the icon also in main/img/icons/32 or main/img/icons/64 we are using Display::return_icon to get the image

The php code is located in the course_home.lib.php file 744

// Including Courses Plugins

// Including Courses Plugins
                // Creating title and the link

                if (isset($tool['category']) &&  $tool['category'] == 'plugin') {
                    $plugin_info = $app_plugin->get_plugin_info($tool['name']);
                    if (isset($plugin_info) && isset($plugin_info['title'])) {
                        $tool_name = $plugin_info['title'];
                    }
                    $tool_link_params['href'] = api_get_path(WEB_PLUGIN_PATH).$tool['original_link'].'?'.api_get_cidreq();
                }

Remember that when you install the course plugin it also installs "the icon" in the tool table

You probably have something wrong or I didn't send something, but in short, you shouldn't see that tool appear there, because I modified the plugins system to avoid creating anything in the tools table.

I mistakenly "attached" my commits to task #4796, so all commits relative to this plugin should be checked there.

Yannick Warnier wrote:

...because I modified the plugins system to avoid creating anything in the tools table.

Only if requested through an explicit parameter, of course :-)

Ok, got it. On course creation, it still gets created. My fault. Checking this right away...

Ok I'm going to test it