Clean up code (after linting)

mediagoblin/federation/views.py

@@ -8,9 +8,7 @@ from mediagoblin.decorators import oauth_required
 from mediagoblin.db.models import User, MediaEntry, MediaComment
 from mediagoblin.tools.response import redirect, json_response
 from mediagoblin.meddleware.csrf import csrf_exempt
-from mediagoblin.notifications import add_comment_subscription
-from mediagoblin.submit.lib import (new_upload_entry, prepare_queue_task,
-                                    run_process_media)
+from mediagoblin.submit.lib import new_upload_entry
 
 #@oauth_required
 def profile(request, raw=False):
@@ -46,7 +44,7 @@ def user(request):
 #@oauth_required
 @csrf_exempt
 def uploads(request):
-    """ This is the endpoint which uploads can be sent to - /api/user/<username>/uploads """
+    """ Endpoint for file uploads """
     user = request.matchdict["username"]
     requested_user = User.query.filter_by(username=user)
 
@@ -97,7 +95,6 @@ def feed(request):
         if obj.get("objectType", None) == "comment":
             # post a comment
             media = int(data["object"]["inReplyTo"]["id"])
-            author = request.user
             comment = MediaComment(
                 media_entry=media,
                 author=request.user.id,
@@ -126,7 +123,11 @@ def feed(request):
             return json_response(error, status=400)
         else:
             # Oh no! We don't know about this type of object (yet)
-            error = {"error": "Unknown object type '{0}'.".format(obj.get("objectType", None))}
+            error_message = "Unknown object type '{0}'.".format(
+                obj.get("objectType", None)
+            )
+
+            error = {"error": error_message}
             return json_response(error, status=400)
 
 
@@ -137,7 +138,10 @@ def feed(request):
             )
 
     feed = {
-        "displayName": "Activities by {0}@{1}".format(request.user.username, request.host),
+        "displayName": "Activities by {user}@{host}".format(
+            user=request.user.username,
+            host=request.host
+        ),
         "objectTypes": ["activity"],
         "url": feed_url,
         "links": {
@@ -176,11 +180,6 @@ def feed(request):
     return json_response(feed)
 
 @oauth_required
-def inbox(request):
-    """ Handles the user's inbox - /api/user/<username>/inbox """
-    raise NotImplemented("Yet to implement looking up user's inbox")
-
-#@oauth_required
 def object(request, raw_obj=False):
     """ Lookup for a object type """
     objectType = request.matchdict["objectType"]