pep8/pyflakes cleanup

core/forms.py

@@ -16,4 +16,3 @@ def __init__(self, *args, **kwargs):
                 self.fields[name].widget.input_type = "number"
             if field.required:
                 self.fields[name].widget.attrs["required"] = ""
-

core/management/commands/socketio_server.py

@@ -11,5 +11,3 @@ class Command(BaseCommand):
     def handle(self, *args, **options):
         game.autodiscover()
         SocketIOServer(("0.0.0.0", 9000), Application()).serve_forever()
-
-

core/models.py

@@ -26,7 +26,7 @@ def user_saved(sender, **kwargs):
     """
     Create an initial account balance for new users.
     """
-    account = Account.objects.get_or_create(user=kwargs["instance"])
+    Account.objects.get_or_create(user=kwargs["instance"])
 
 
 @receiver(socialauth_registered, sender=None)
@@ -41,7 +41,8 @@ def avatar(sender, user, response, details, **kwargs):
         photo_url = "_reasonably_small".join(photo_url.rsplit("_normal", 1))
     except KeyError:
         # facebook
-        photo_url = "http://graph.facebook.com/%s/picture?type=large" % response["id"]
+        uid = response["id"]
+        photo_url = "http://graph.facebook.com/%s/picture?type=large" % uid
     path = join(settings.MEDIA_ROOT, "photos")
     if not exists(path):
         makedirs(path)

core/settings.py

@@ -6,4 +6,5 @@
 DEFAULT_BALANCE = getattr(settings, "DEFAULT_BALANCE", 5000)
 
 # Number of seconds until a game is played, after the first player joins.
-BETTING_PERIOD = getattr(settings, "BETTING_PERIOD", int(not settings.DEBUG) * 10)
+BETTING_PERIOD = int(not settings.DEBUG) * 10
+BETTING_PERIOD = getattr(settings, "BETTING_PERIOD", BETTING_PERIOD)

core/urls.py

@@ -1,5 +1,5 @@
 
-from django.conf.urls.defaults import patterns, include, url
+from django.conf.urls.defaults import patterns, url
 
 
 urlpatterns = patterns("core.views",

core/views.py

@@ -10,14 +10,17 @@
 def index(request):
     return render(request, "index.html", {"games": game_registry.values()})
 
+
 def logged_in(request):
     info(request, _("Logged in as %s" % request.user))
     return redirect("index")
 
+
 def login_error(request):
     error(request, _("An error occurred logging in"))
     return redirect("index")
 
+
 def logout(request):
     auth_logout(request)
     info(request, _("Successfully logged out"))

urls.py

@@ -1,6 +1,5 @@
 
 from django.conf.urls.defaults import patterns, include, url
-from django.conf import settings
 from django.contrib import admin
 
 from core import game