From 001c6064eb750b6ad7d8e9f65543ab6fcbd898f6 Mon Sep 17 00:00:00 2001 From: Ricky Zhou Date: Fri, 14 Mar 2008 18:04:21 -0400 Subject: [PATCH] Move "do not agree" check up, make it redirect to /user/view --- fas/fas/cla.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fas/fas/cla.py b/fas/fas/cla.py index fc358eb..b7e55b6 100644 --- a/fas/fas/cla.py +++ b/fas/fas/cla.py @@ -68,15 +68,15 @@ class CLA(controllers.Controller): turbogears.flash(_('You have already signed the CLA.')) turbogears.redirect('/cla/') return dict() + if not agree: + turbogears.flash(_("You have not signed the CLA.")) + turbogears.redirect('/user/view/%s' % person.username) if not person.telephone or \ not person.postal_address: turbogears.flash(_('To sign the CLA we must have your telephone number and postal address. Please ensure they have been filled out.')) turbogears.redirect('/user/edit/%s' % username) groupname = config.get('cla_fedora_group') group = Groups.by_name(groupname) - if not agree: - turbogears.flash(_("You have not agreed to the CLA.")) - turbogears.redirect('/cla/') try: # Everything is correct. person.apply(group, person) # Apply...