diff -r 18d0e10b02f7 -r a98a165c1300 app/soc/views/models/role.py
--- a/app/soc/views/models/role.py	Sat Jan 31 19:02:53 2009 +0000
+++ b/app/soc/views/models/role.py	Sat Jan 31 19:33:40 2009 +0000
@@ -506,7 +506,7 @@
         'role_verbose' : params['name'],
         'status' : 'new'}
 
-    if self._isValidNewRequest(request_fields, params):
+    if not self._isValidNewRequest(request_fields, params):
       # not a valid request
       context['error_message'] = self.DEF_REQUEST_ERROR_MSG_FMT % (
           params)