More fixes in the skills managemet #1791, now we use the label_tag skill and label_tag notice see #1791

main/admin/skills_profile.php

@@ -13,7 +13,7 @@ require_once '../inc/global.inc.php';
 require_once api_get_path(LIBRARY_PATH).'skill.lib.php';
 $this_section = SECTION_PLATFORM_ADMIN;
 
-//api_protect_admin_script();
+api_protect_admin_script();
 
 $htmlHeadXtra[] = '<script src="'.api_get_path(WEB_LIBRARY_PATH).'javascript/tag/jquery.fcbkcomplete.js" type="text/javascript" language="javascript"></script>';
 $htmlHeadXtra[] = '<link  href="'.api_get_path(WEB_LIBRARY_PATH).'javascript/tag/style.css" rel="stylesheet" type="text/css" />';

main/inc/ajax/model.ajax.php

@@ -93,7 +93,7 @@ switch ($action) {
             $skills_string = '';
             if (!empty($skills)) {
                 foreach($skills as $skill) {
-                    $item['skills'] .= Display::span($skill['name'], array('class' => 'label_tag success'));  
+                    $item['skills'] .= Display::span($skill['name'], array('class' => 'label_tag skill'));  
                 }
             }          
             $new_result[] = $item;

main/inc/lib/skill.lib.php

@@ -133,7 +133,7 @@ class SkillRelSkill extends Model {
             
             foreach ($skills as &$skill) {
                 $skill['data'] = $skill_obj->get($skill['skill_id']);
-                if (isset($skill['data'] )) {
+                if (isset($skill['data']) && !empty($skill['data'])) {
                     if (!empty($done_skills)) {
                         $skill['data']['passed'] =  0;
                         if (in_array($skill['skill_id'], $done_skills)) {

main/inc/lib/skill.visualizer.lib.php

@@ -42,7 +42,7 @@
         if (!empty($skill['gradebooks'])) {
             foreach ($skill['gradebooks'] as $gradebook) {
                 //uncomment this to show the gradebook tags
-                $gradebook_string .= Display::span($gradebook['name'], array('class'=>'label_tag notice','style'=>'width:50px')).'<br />';    
+                $gradebook_string .= Display::span($gradebook['name'], array('class'=>'label_tag gradebook','style'=>'width:50px')).'<br />';    
             }
         }        
         $skill['name'] = Display::url($skill['name'], '#', array('id'=>'edit_block_'.$block_id, 'class'=>'edit_block'));

main/social/home.php

@@ -109,7 +109,7 @@ echo '<div id="social-content">';
             $lis = '';
             if (!empty($skills)) {
                 foreach($skills as $skill) {
-                    $lis .= Display::tag('li', Display::span($skill['name'], array('class'=>'label_tag success')));
+                    $lis .= Display::tag('li', Display::span($skill['name'], array('class'=>'label_tag skill')));
                 }                
                 echo Display::tag('ul', $lis);
             }

main/template/default/skill/profile.tpl

@@ -164,7 +164,7 @@ function checkLength( o, n, min, max ) {
                         {$user.total_found_skills} / {$total_search_skills}                
                     {foreach $user['skills'] as $skill_data}                 
                         <li>
-                            <span class="label_tag notice">{$skill_list[$skill_data.skill_id].name}</span>
+                            <span class="label_tag skill">{$skill_list[$skill_data.skill_id].name}</span>
                             {if $skill_data.found}
                                  * I have this skill * 
                             {/if} 

main/template/default/skill/skill_tree_student.tpl

@@ -105,7 +105,7 @@ jsPlumb.bind("ready", function() {
                 //filling the gradebook_id         
                 $("#gradebook_id").html('');
                 jQuery.each(skill.gradebooks, function(index, data) {                    
-                    $("#gradebook_id").append('<span class="label_tag notice">'+data.name+'</div>');
+                    $("#gradebook_id").append('<span class="label_tag gradebook">'+data.name+'</div>');
                 });
             },
         });

user_portal.php

@@ -235,9 +235,9 @@ $controller->tpl->assign('reservation_block', 			$controller->return_reservation
 $controller->tpl->assign('search_block', 				$controller->return_search_block());
 $controller->tpl->assign('classes_block', 				$controller->return_classes_block());
 
-if (api_is_platform_admin() || api_is_drh()) {
+//if (api_is_platform_admin() || api_is_drh()) {
     $controller->tpl->assign('skills_block',            $controller->return_skills_links());
-}
+//}
 
 
 $controller->tpl->display_two_col_template();