ulysia/szurubooru
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed double errors in registration

Browse source
This commit is contained in:
Marcin Kurczewski 2014-09-01 21:54:22 +02:00
parent ae93b187ac
commit 3fbd1b218e
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
public_html/js/Presenters/RegistrationPresenter.js
View file

@ -25,7 +25,8 @@ App.Presenters.RegistrationPresenter = function(
email: $el.find('[name=email]').val(), email: $el.find('[name=email]').val(),
}; };
validateRegistrationData(registrationData); if (!validateRegistrationData(registrationData))
return;
api.post('/users', registrationData) api.post('/users', registrationData)
.then(function(response) { .then(function(response) {
@ -61,18 +62,20 @@ App.Presenters.RegistrationPresenter = function(
function validateRegistrationData(registrationData) { function validateRegistrationData(registrationData) {
if (registrationData.userName.length == 0) { if (registrationData.userName.length == 0) {
messagePresenter.showError($messages, 'User name cannot be empty.'); messagePresenter.showError($messages, 'User name cannot be empty.');
return; return false;
} }
if (registrationData.password.length == 0) { if (registrationData.password.length == 0) {
messagePresenter.showError($messages, 'Password cannot be empty.'); messagePresenter.showError($messages, 'Password cannot be empty.');
return; return false;
} }
if (registrationData.password != registrationData.passwordConfirmation) { if (registrationData.password != registrationData.passwordConfirmation) {
messagePresenter.showError($messages, 'Passwords must be the same.'); messagePresenter.showError($messages, 'Passwords must be the same.');
return; return false;
} }
return true;
}; };
return { return {