X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=planetlab%2Ftags%2Ftag.php;h=dc228fe529fb69e4f1d27d2a8c83d3eb2ad4bf91;hb=584e71ac4c7a93aedc0ab48dd1833a4e40641adc;hp=2f915e99f282624098092ecbf18fdb1b6553c1f4;hpb=fa425f537d2e4aced1a2362672791aed274dd903;p=plewww.git

diff --git a/planetlab/tags/tag.php b/planetlab/tags/tag.php
index 2f915e9..dc228fe 100644
--- a/planetlab/tags/tag.php
+++ b/planetlab/tags/tag.php
@@ -111,7 +111,7 @@ $toggle->end();
 $form=new PlekitForm(l_actions(), array("tag_type_id"=>$tag_type_id));
 $form->start();
 
-$toggle=new PlekitToggle ('roles',count_english($roles,"role"),array('visible'=>get_arg('show_roles',false)));
+$toggle=new PlekitToggle ('roles',count_english($roles,"role"),array('visible'=>get_arg('show_roles')));
 $toggle->start();
 
 if (! $roles) plc_warning ("This tag type has no role !");
@@ -153,11 +153,11 @@ if ($can_manage_roles) {
   // add
   // compute the roles that can be added
   if ($can_manage_roles) 
-    // all roles
+    // all roles - don't exclude 'node' as it's actually meaningful for some tags
     $exclude_role_ids=array();
   else
-    // all roles except admin and pi
-    $exclude_role_ids=array(10,20);
+    // all roles except admin and pi, and node to avoid confusing people
+    $exclude_role_ids=array(10,20,50);
   $possible_roles = roles_except($api->GetRoles(),$exclude_role_ids);
   $roles_to_add = roles_except ($possible_roles,$role_ids);
   if ( $roles_to_add ) {