023ec9a976
Every time the password reset form was loaded, the form submit event listener was attached to a non-disposable DOM node rather than the DOM node whose life scope was bound to the viewed page. As such, submitting the form, leaving the page, returning back to it and sending the request again caused the 'submit' event to fire twice - one time from the non-disposed event handler and one from the current handler. This resulted in the request being sent twice, and getting two confirmation messages on the screen. Fortunately, since the password reset requests are GET requests, they're intercepted by the internal cache of the client API facade, so the client just saw duplicate messages without the requests being actually sent to the backend - meaning no extra mails were sent.
56 lines
1.3 KiB
JavaScript
56 lines
1.3 KiB
JavaScript
'use strict';
|
|
|
|
const events = require('../events.js');
|
|
const views = require('../util/views.js');
|
|
|
|
const template = views.getTemplate('password-reset');
|
|
|
|
class PasswordResetView extends events.EventTarget {
|
|
constructor() {
|
|
super();
|
|
this._hostNode = document.getElementById('content-holder');
|
|
|
|
views.replaceContent(this._hostNode, template());
|
|
views.syncScrollPosition();
|
|
|
|
views.decorateValidator(this._formNode);
|
|
this._formNode.addEventListener('submit', e => {
|
|
e.preventDefault();
|
|
this.dispatchEvent(new CustomEvent('submit', {
|
|
detail: {
|
|
userNameOrEmail: this._userNameOrEmailFieldNode.value,
|
|
},
|
|
}));
|
|
});
|
|
}
|
|
|
|
showSuccess(message) {
|
|
views.showSuccess(this._hostNode, message);
|
|
}
|
|
|
|
showError(message) {
|
|
views.showError(this._hostNode, message);
|
|
}
|
|
|
|
clearMessages() {
|
|
views.clearMessages(this._hostNode);
|
|
}
|
|
|
|
enableForm() {
|
|
views.enableForm(this._formNode);
|
|
}
|
|
|
|
disableForm() {
|
|
views.disableForm(this._formNode);
|
|
}
|
|
|
|
get _formNode() {
|
|
return this._hostNode.querySelector('form');
|
|
}
|
|
|
|
get _userNameOrEmailFieldNode() {
|
|
return this._formNode.querySelector('[name=user-name]');
|
|
}
|
|
}
|
|
|
|
module.exports = PasswordResetView;
|