a few cleanups

pyweb/mumble/mmobjects.py

@@ -170,10 +170,10 @@ class mmChannel( object ):
 				"id":   self.id,
 				"rel":  "channel",
 				},
-			'data':   self.name,
-			'children': [ sc.asMvJson() for sc in self.subchans ] + \
+			"data": self.name,
+			"children": [ sc.asMvJson() for sc in self.subchans ] + \
 				    [ pl.asMvJson() for pl in self.players  ],
-			'state': { False: "closed", True: "open" }[self.top_or_not_empty],
+			"state": { False: "closed", True: "open" }[self.top_or_not_empty],
 			}
 
 

pyweb/mumble/urls.py

@@ -22,7 +22,6 @@ urlpatterns = patterns(
 	( r'(?P<server>\d+)/users',				'users' 		),
 	
 	( r'(?P<server>\d+)/(?P<userid>\d+)/texture.png',	'showTexture'		),
-	( r'(?P<server>\d+)/texture.png',			'showTexture'		),
 	
 	( r'murmur/tree/(?P<server>\d+)',			'mmng_tree'		),
 	( r'mumbleviewer/(?P<server>\d+).xml',			'mumbleviewer_tree_xml' ),

pyweb/mumble/views.py

@@ -236,21 +236,14 @@ def mobile_show( request, server ):
 
 
 
-def showTexture( request, server, userid = None ):
+def showTexture( request, server, userid ):
 	""" Pack the given user's texture into an HttpResponse.
 	
 	    If userid is none, use the currently logged in User.
 	"""
 	
-	srv  = get_object_or_404( Mumble, id=int(server) );
-	
-	if userid is None:
-		if request.user.is_authenticated():
-			user = get_object_or_404( MumbleUser, server=srv, owner=request.user );
-		else:
-			raise Http404();
-	else:
-		user = get_object_or_404( MumbleUser, server=srv, id=int(userid) );
+	srv  = get_object_or_404( Mumble,     id=int(server) );
+	user = get_object_or_404( MumbleUser, server=srv, id=int(userid) );
 	
 	try:
 		img  = user.getTexture();