Setting gravatar on Email update && bug fixing issue #44

AppBakerZ · Apr 15, 2017 · 6cae787 · 6cae787
1 parent 3ad3c30
commit 6cae787
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 16 deletions.
diff --git a/imports/api/settings/methods.js b/imports/api/settings/methods.js
@@ -89,6 +89,12 @@ export const updateProfile = new ValidatedMethod({
             update['emails.0.address'] = users.email;
         }
         Meteor.users.update({_id: Meteor.userId()} , {$set: update});
+
+        if(users.email){
+            Meteor.users.update( { _id: Meteor.user()._id }, {
+                   $set: { "profile.md5hash": Gravatar.hash( users.email ) }
+                     });
+        }
     }
 });
 
diff --git a/imports/ui/components/settings/Settings.jsx b/imports/ui/components/settings/Settings.jsx
@@ -211,22 +211,25 @@ class SettingsPage extends Component {
             folder: "profiles",
             uploaderId: userId
         };
-        let uploader = new Slingshot.Upload('imageUploader', metaContext);
-        uploader.send(this.state.target,  (error, downloadUrl) => { // you can use refs if you like
-            if (error) {
-                // Log service detailed response
-                console.error('Error uploading', uploader.xhr.response);
-                alert (error); // you may want to fancy this up when you're ready instead of a popup.
-            }
-            else {
-                // we use $set because the user can change their avatar so it overwrites the url :)
-                Meteor.users.update(Meteor.userId(), {$set: {"profile.avatar": downloadUrl}});
-                console.log(downloadUrl);
-                this.setState({imageUrl: downloadUrl});
-                this.resetImageUpload();
-            }
-
-        });
+              if(this.state.target && this.uploadImageName != this.state.target.name) {
+                  this.uploadImageName = this.state.target.name;
+
+                  let uploader = new Slingshot.Upload('imageUploader', metaContext);
+                  uploader.send(this.state.target, (error, downloadUrl) => { // you can use refs if you like
+                      if (error) {
+                          // Log service detailed response
+                          console.error('Error uploading', uploader.xhr.response);
+                          alert(error); // you may want to fancy this up when you're ready instead of a popup.
+                      }
+                      else {
+                          // we use $set because the user can change their avatar so it overwrites the url :)
+                          Meteor.users.update(Meteor.userId(), {$set: {"profile.avatar": downloadUrl}});
+                          console.log(downloadUrl);
+                          this.setState({imageUrl: downloadUrl});
+                      }
+                      this.resetImageUpload();
+                  });
+              }
     }
 
     updateAccountSettings (event) {