# HG changeset patch # User Sverre Rabbelier # Date 1236292140 0 # Node ID fbf8101f6eff265d6e8d94bf034885f64db7fbab # Parent 2b28763da7a412c63535da6ae012b7f5b4761bd5 Import user and site logic globals directly Patch by: Sverre Rabbelier diff -r 2b28763da7a4 -r fbf8101f6eff app/soc/logic/cleaning.py --- a/app/soc/logic/cleaning.py Thu Mar 05 21:38:18 2009 +0000 +++ b/app/soc/logic/cleaning.py Thu Mar 05 22:29:00 2009 +0000 @@ -34,8 +34,8 @@ from soc.logic import rights as rights_logic from soc.logic import validate -from soc.logic.models import site as site_logic -from soc.logic.models import user as user_logic +from soc.logic.models.site import logic as site_logic +from soc.logic.models.user import logic as user_logic from soc.models import document as document_model @@ -132,7 +132,7 @@ """ agrees_to_tos = self.cleaned_data.get(field_name) - if not site_logic.logic.getToS(site_logic.logic.getSingleton()): + if not site_logic.getToS(site_logic.logic.getSingleton()): return agrees_to_tos # Site settings specify a site-wide ToS, so agreement is *required* @@ -157,7 +157,7 @@ """ link_id = clean_link_id(field_name)(self) - user_entity = user_logic.logic.getForFields({'link_id': link_id}, + user_entity = user_logic.getForFields({'link_id': link_id}, unique=True) if not user_entity: @@ -179,7 +179,7 @@ """ link_id = clean_link_id(field_name)(self) - user_entity = user_logic.logic.getForCurrentAccount() + user_entity = user_logic.getForCurrentAccount() if not user_entity or user_entity.link_id != link_id: # this user is not the current user @@ -200,7 +200,7 @@ """ link_id = clean_link_id(field_name)(self) - user_entity = user_logic.logic.getForFields({'link_id': link_id}, + user_entity = user_logic.getForFields({'link_id': link_id}, unique=True) if user_entity: @@ -223,7 +223,7 @@ clean_user_field = clean_existing_user(field_name) user_entity = clean_user_field(self) - current_user_entity = user_logic.logic.getForCurrentAccount() + current_user_entity = user_logic.getForCurrentAccount() if user_entity.key() == current_user_entity.key(): # users are equal @@ -262,9 +262,9 @@ user_account = users.User(email_adress) fields = {'account': user_account} - user_entity = user_logic.logic.getForFields(fields, unique=True) + user_entity = user_logic.getForFields(fields, unique=True) - if user_entity or user_logic.logic.isFormerAccount(user_account): + if user_entity or user_logic.isFormerAccount(user_account): raise forms.ValidationError("There is already a user " "with this email adress.") @@ -335,19 +335,19 @@ if link_id and user_account: # get the user from the link_id in the form fields = {'link_id': link_id} - user_entity = user_logic.logic.getForFields(fields, unique=True) + user_entity = user_logic.getForFields(fields, unique=True) # if it's not the user's current account if user_entity.account != user_account: # get the user having the given account fields = {'account': user_account} - user_from_account_entity = user_logic.logic.getForFields(fields, + user_from_account_entity = user_logic.getForFields(fields, unique=True) # if there is a user with the given account or it's a former account if user_from_account_entity or \ - user_logic.logic.isFormerAccount(user_account): + user_logic.isFormerAccount(user_account): # raise an error because this email address can't be used raise forms.ValidationError("There is already a user with " "this email address.") @@ -385,7 +385,7 @@ group_app_entity = group_app_logic.logic.getForFields(fields, unique=True) # get the current user - user_entity = user_logic.logic.getForCurrentAccount() + user_entity = user_logic.getForCurrentAccount() # make sure it's not the applicant creating the new group if group_app_entity and ( @@ -491,7 +491,7 @@ params = view.getParams() rights = params['rights'] - user = user_logic.logic.getForCurrentAccount() + user = user_logic.getForCurrentAccount() rights.setCurrentUser(user.account, user) checker = rights_logic.Checker(prefix)