work in progress category UI

refactor hack from earlier

app/assets/javascripts/admin/models/site_content_type.js

@@ -10,14 +10,12 @@ Discourse.SiteContentType = Discourse.Model.extend({});
 
 Discourse.SiteContentType.reopenClass({
   findAll: function() {
-    var promise = new Em.Deferred();
-    Discourse.ajax("/admin/site_content_types").then(function(data) {
+    return Discourse.ajax("/admin/site_content_types").then(function(data) {
       var contentTypes = Em.A();
       data.forEach(function (ct) {
         contentTypes.pushObject(Discourse.SiteContentType.create(ct));
       });
-      promise.resolve(contentTypes);
+      return contentTypes;
     });
-    return promise;
   }
 });

app/assets/javascripts/admin/routes/admin_site_content_edit_route.js

@@ -14,7 +14,6 @@ Discourse.AdminSiteContentEditRoute = Discourse.Route.extend({
 
   model: function(params) {
     var list = this.controllerFor('adminSiteContents').get('model');
-    var model;
 
     // ember routing is fun ... this is what happens
     //
@@ -28,14 +27,10 @@ Discourse.AdminSiteContentEditRoute = Discourse.Route.extend({
     // we could avoid this hack if Ember just compared .serialize(model) with .serialize(context)
     //
     // alternatively we could use some sort of identity map
+    //
+    // see also: https://github.com/emberjs/ember.js/issues/3005
 
-    list.forEach(function(orig){
-      if(orig.get("content_type") === params.content_type){
-        model = orig;
-      }
-    });
-
-    return model;
+    return list.findProperty("content_type", params.content_type);
   },
 
   renderTemplate: function() {