From cc12fb701c38b3ac3392bdc31225f6db064954b2 Mon Sep 17 00:00:00 2001 From: Matt Enlow Date: Sat, 24 May 2014 08:43:53 -0600 Subject: [PATCH] Fix default user cover on settings pane Closes #2806 - Ensures that the user's cover property is a string, otherwise uses the default cover. --- ghost/admin/controllers/settings/user.js | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/ghost/admin/controllers/settings/user.js b/ghost/admin/controllers/settings/user.js index 1ba9eaebce..f14d82630e 100644 --- a/ghost/admin/controllers/settings/user.js +++ b/ghost/admin/controllers/settings/user.js @@ -1,10 +1,14 @@ /*global alert */ - var SettingsUserController = Ember.Controller.extend({ + coverDefault: '/shared/img/user-cover.png', cover: function () { // @TODO: add {{asset}} subdir path - return this.user.getWithDefault('cover', '/shared/img/user-cover.png'); - }.property('user.cover'), + var cover = this.user.get('cover'); + if (typeof cover !== 'string') { + cover = this.get('coverDefault'); + } + return cover; + }.property('user.cover', 'coverDefault'), coverTitle: function () { return this.get('user.name') + '\'s Cover Image';