From: Marc Fiuczynski <mef@cs.princeton.edu>
Date: Thu, 16 Jul 2009 03:17:47 +0000 (+0000)
Subject: no longer letting folks register as PI/Tech, as its basically NEVER used correctly
X-Git-Tag: PLEWWW-4.3-25~7
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=455513b9dd3a524bb50c40f5efb352c8cdd4c467;p=plewww.git

no longer letting folks register as PI/Tech, as its basically NEVER used correctly
---

diff --git a/planetlab/persons/register.php b/planetlab/persons/register.php
index 16ca4fa..c1840be 100644
--- a/planetlab/persons/register.php
+++ b/planetlab/persons/register.php
@@ -52,6 +52,7 @@ $form['email'] = array('title' => 'E-mail', 'required' => TRUE,
 $form['password'] = array('title' => 'Password', 'required' => TRUE,
 		       'maxlength' => 60, 'size' => 20);
 $form['site_ids'] = array('title' => 'Site', 'required' => TRUE);
+if (0)
 $form['roles'] = array('title' => 'Additional Roles', 'required' => FALSE);
 
 //////////////////// additional messages
@@ -64,6 +65,7 @@ $form['site_ids']['comment'] = <<< EOF
 Select the site where you belong 
 EOF;
 
+if (0)
 $form['roles']['comment'] = <<< EOF
 Do not select the <b>Principal Investigator</b> or <b>Technical
 Contact</b> roles unless you have spoken with the current PI of your
@@ -307,11 +309,13 @@ EOF;
     break;
 
   case 'roles':
+    if (0) { /* Not letting users select PI or Tech any more.  Its only lead to confusion and abuse. */
     print <<<EOF
         <td><select name="roles[]" multiple="multiple" id="edit-roles" class="form-select $class">
           $role_options
         </select></td>\n
 EOF;
+    }
     break;
 
   default: