Merge pull request #3144 from darvelo/sort-new-posts

New posts pass PostsController sorting function at the top

ghost/admin/controllers/posts.js

@@ -34,9 +34,25 @@ var PostsController = Ember.ArrayController.extend({
             return Ember.compare(item1.get('published_at').valueOf(), item2.get('published_at').valueOf());
         }
 
-        var statusResult = Ember.compare(item1.get('status'), item2.get('status')),
+        var updated1 = item1.get('updated_at'),
-            updatedAtResult = Ember.compare(item1.get('updated_at').valueOf(), item2.get('updated_at').valueOf()),
+            updated2 = item2.get('updated_at'),
-            publishedAtResult = publishedAtCompare();
+            statusResult,
+            updatedAtResult,
+            publishedAtResult;
+
+        // when `updated_at` is undefined, the model is still
+        // being written to with the results from the server
+        if (item1.get('isNew') || !updated1) {
+            return -1;
+        }
+
+        if (item2.get('isNew') || !updated2) {
+            return 1;
+        }
+
+        statusResult = Ember.compare(item1.get('status'), item2.get('status'));
+        updatedAtResult = Ember.compare(updated1.valueOf(), updated2.valueOf());
+        publishedAtResult = publishedAtCompare();
 
         if (statusResult === 0) {
             if (publishedAtResult === 0) {

ghost/admin/mixins/validation-engine.js

@@ -56,6 +56,9 @@ var ValidationEngine = Ember.Mixin.create({
         if (Ember.isArray(errors)) {
             // get validation error messages
             message = errors.mapBy('message').join('<br />');
+        } else if (errors instanceof Error) {
+            // we got some kind of error in Ember
+            message += ': ' + errors.message;
         } else if (typeof errors === 'object') {
             // Get messages from server response
             message += ': ' + getRequestErrorMessage(errors);

ghost/admin/utils/ajax.js

@@ -26,7 +26,7 @@ var getRequestErrorMessage = function (request) {
 
                 message = request.responseJSON.errors.map(function (errorItem) {
                     return errorItem.message;
-                }).join('; ');
+                }).join('<br />');
             } else {
                 message =  request.responseJSON.error || 'Unknown Error';
             }