From facef06d606416af2fb32cf757423dcf6f0fb157 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20Hanol?= <regis@hanol.fr>
Date: Wed, 18 Jul 2018 22:50:50 +0200
Subject: [PATCH] UX: reload the page when changing avatar

---
 .../discourse/routes/preferences.js.es6       | 28 ++++++++++---------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/app/assets/javascripts/discourse/routes/preferences.js.es6 b/app/assets/javascripts/discourse/routes/preferences.js.es6
index 84f2469b57..e63e3441e9 100644
--- a/app/assets/javascripts/discourse/routes/preferences.js.es6
+++ b/app/assets/javascripts/discourse/routes/preferences.js.es6
@@ -46,13 +46,18 @@ export default RestrictedUserRoute.extend({
 
     selectAvatar(url) {
       const user = this.modelFor("user");
-      const controller = this.controllerFor("avatar-selector");
+
+      this.controllerFor("avatar-selector").send("closeModal");
 
       user
         .selectAvatar(url)
-        .then(() => bootbox.alert(I18n.t("user.change_avatar.cache_notice")))
-        .catch(popupAjaxError)
-        .finally(() => controller.send("closeModal"));
+        .then(() => {
+          bootbox.alert(
+            I18n.t("user.change_avatar.cache_notice"),
+            () => window.location.reload()
+          );
+        })
+        .catch(popupAjaxError);
     },
 
     saveAvatarSelection() {
@@ -62,20 +67,17 @@ export default RestrictedUserRoute.extend({
       const selectedAvatarTemplate = controller.get("selectedAvatarTemplate");
       const type = controller.get("selected");
 
+      controller.send("closeModal");
+
       user
         .pickAvatar(selectedUploadId, type, selectedAvatarTemplate)
         .then(() => {
-          user.setProperties(
-            controller.getProperties(
-              "system_avatar_template",
-              "gravatar_avatar_template",
-              "custom_avatar_template"
-            )
+          bootbox.alert(
+            I18n.t("user.change_avatar.cache_notice"),
+            () => window.location.reload()
           );
-          bootbox.alert(I18n.t("user.change_avatar.cache_notice"));
         })
-        .catch(popupAjaxError)
-        .finally(() => controller.send("closeModal"));
+        .catch(popupAjaxError);
     }
   }
 });