Skip to content

Commit

Permalink
Fix call to this.notification
Browse files Browse the repository at this point in the history
Refs #5351
- Injected services need to be accessed via .get().
  • Loading branch information
jaswilli committed Jun 4, 2015
1 parent 60e160d commit 164fa27
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions core/client/app/controllers/signin.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,11 @@ export default Ember.Controller.extend(ValidationEngine, {

forgotten: function () {
var email = this.get('model.identification'),
notifications = this.get('notifications'),
self = this;

if (!email) {
return this.notifications.showError('Enter email address to reset password.');
return notifications.showError('Enter email address to reset password.');
}

self.set('submitting', true);
Expand All @@ -58,10 +59,10 @@ export default Ember.Controller.extend(ValidationEngine, {
}
}).then(function () {
self.set('submitting', false);
self.get('notifications').showSuccess('Please check your email for instructions.');
notifications.showSuccess('Please check your email for instructions.');
}).catch(function (resp) {
self.set('submitting', false);
self.get('notifications').showAPIError(resp, {defaultErrorText: 'There was a problem with the reset, please try again.'});
notifications.showAPIError(resp, {defaultErrorText: 'There was a problem with the reset, please try again.'});
});
}
}
Expand Down

0 comments on commit 164fa27

Please sign in to comment.