From 455513b9dd3a524bb50c40f5efb352c8cdd4c467 Mon Sep 17 00:00:00 2001
From: Marc Fiuczynski <mef@cs.princeton.edu>
Date: Thu, 16 Jul 2009 03:17:47 +0000
Subject: [PATCH] no longer letting folks register as PI/Tech, as its basically
 NEVER used correctly

---
 planetlab/persons/register.php | 4 ++++
 1 file changed, 4 insertions(+)

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:
-- 
2.47.0