Code review feedback.

cms/djangoapps/contentstore/views/assets.py

@@ -145,7 +145,7 @@ def upload_asset(request, org, course, coursename):
     return JsonResponse(response_payload)
 
 
-@require_http_methods("DELETE")
+@require_http_methods(("DELETE",))
 @login_required
 @ensure_csrf_cookie
 def update_asset(request, org, course, name, asset_id):
@@ -158,7 +158,6 @@ def update_asset(request, org, course, name, asset_id):
     """
     get_location_and_verify_access(request, org, course, name)
 
-    if request.method == 'DELETE':
     # make sure the location is valid
     try:
         loc = StaticContent.get_location_from_path(asset_id)

cms/static/js/views/assets_view.js

@@ -10,7 +10,7 @@ CMS.Views.Assets = Backbone.View.extend({
         this.$el.empty();
 
         var self = this;
-        _.each(this.collection.models,
+        this.collection.each(
             function(asset) {
                 var view = new CMS.Views.Asset({model: asset});
                 self.$el.append(view.render().el);

cms/urls.py

@@ -76,7 +76,7 @@ urlpatterns = ('',  # nopep8
 
     url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/assets/(?P<name>[^/]+)$',
         'contentstore.views.asset_index', name='asset_index'),
-    url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/assets/(?P<name>[^/]+)/update/(?P<asset_id>.+)?.*$',
+    url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/assets/(?P<name>[^/]+)/(?P<asset_id>.+)?.*$',
         'contentstore.views.assets.update_asset', name='update_asset'),
     url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/textbooks/(?P<name>[^/]+)$',
         'contentstore.views.textbook_index', name='textbook_index'),