# HG changeset patch # User Pawel Solyga # Date 1236190456 0 # Node ID 85bbf50038b8ab12c3d86ef408c9fbca71f4a870 # Parent 6b65707f424897b978fd5a7718099e99cc95b7bf Remove unused imports and fix too long lines in soc.views.models.user_self module Patch by: Pawel Solyga Reviewed by: to-be-reviewed diff -r 6b65707f4248 -r 85bbf50038b8 app/soc/views/models/user_self.py --- a/app/soc/views/models/user_self.py Wed Mar 04 17:36:14 2009 +0000 +++ b/app/soc/views/models/user_self.py Wed Mar 04 18:14:16 2009 +0000 @@ -29,7 +29,6 @@ from google.appengine.api import users from django import forms -from django import http from django.utils.encoding import force_unicode from django.utils.safestring import mark_safe from django.utils.translation import ugettext @@ -42,15 +41,11 @@ from soc.views import helper from soc.views.helper import access from soc.views.helper import decorators -from soc.views.helper import dynaform from soc.views.helper import redirects from soc.views.helper import widgets from soc.views.models import base -from soc.views.models import user as user_view from soc.views.models import role as role_view -import soc.models.user - class View(base.View): """Views for User own profiles. @@ -108,8 +103,10 @@ new_params['sidebar_heading'] = 'User (self)' new_params['sidebar'] = [ (users.create_login_url("/"), 'Sign In', 'signIn'), - ('/' + new_params['url_name'] + '/create_profile', 'Create Profile', 'create_profile'), - ('/' + new_params['url_name'] + '/edit_profile', 'Edit Profile', 'edit_profile'), + ('/' + new_params['url_name'] + '/create_profile', + 'Create Profile', 'create_profile'), + ('/' + new_params['url_name'] + '/edit_profile', + 'Edit Profile', 'edit_profile'), ('/' + new_params['url_name'] + '/roles', 'Roles', 'roles'), ('/' + new_params['url_name'] + '/requests', 'Requests', 'requests'), ] @@ -155,8 +152,8 @@ user_entity = user_logic.getForCurrentAccount() link_id = user_entity.link_id - return self.edit(request, access_type, - page_name=page_name, params=params, seed=seed, link_id=link_id, **kwargs) + return self.edit(request, access_type, page_name=page_name, + params=params, seed=seed, link_id=link_id, **kwargs) def editGet(self, request, entity, context, seed, params=None): """Overwrite so we can add the contents of the ToS. @@ -168,7 +165,8 @@ if site_tos: context['tos_contents'] = site_tos.content - return super(View, self).editGet(request, entity, context, seed, params=params) + return super(View, self).editGet(request, entity, context, + seed, params=params) def _editGet(self, request, entity, form): """Sets the content of the agreed_to_tos_on field and replaces. @@ -181,8 +179,8 @@ form.fields['agreed_to_tos_on'].initial = entity.agreed_to_tos_on # replace the 'agreed_to_tos' field with a hidden field so # that the form checks still pass - form.fields['agreed_to_tos'] = forms.fields.BooleanField(widget=forms.HiddenInput, - initial=entity.agreed_to_tos, required=True) + form.fields['agreed_to_tos'] = forms.fields.BooleanField( + widget=forms.HiddenInput, initial=entity.agreed_to_tos, required=True) def editPost(self, request, entity, context, params=None): """Overwrite so we can add the contents of the ToS. @@ -246,8 +244,8 @@ i = 0 - for name, view in sorted(role_view.ROLE_VIEWS.iteritems()): - list_params = view.getParams().copy() + for name, loop_view in sorted(role_view.ROLE_VIEWS.iteritems()): + list_params = loop_view.getParams().copy() list_params['list_action'] = (redirects.getEditRedirect, list_params) list_params['list_description'] = self.DEF_ROLE_LIST_MSG_FMT % list_params @@ -280,7 +278,8 @@ count = len(list(notifications)) if count > 0: - link_title = '%s (%d)' % (force_unicode(link_title), count) + link_title = '%s (%d)' % ( + force_unicode(link_title), count) link_title = mark_safe(link_title) items = [('/' + 'notification/list', link_title, 'notification')]