improve modal register error display

This commit is contained in:
Bachir Soussi Chiadmi 2013-10-30 18:36:19 +01:00
parent e960755e20
commit ca6c5bd4fa

View File

@ -83,7 +83,7 @@ MaterioUser = function(){
dataType:'json', dataType:'json',
url: Drupal.settings.basePath+Drupal.settings.pathPrefix+"materio_user/register/submit", url: Drupal.settings.basePath+Drupal.settings.pathPrefix+"materio_user/register/submit",
data: $('#user-register-form', $modal).serialize(), data: $('#user-register-form', $modal).serialize(),
success: function(data) { onUserLoginRegisterSubmit(data, destination);}, success: function(data) { onUserLoginRegisterSubmit($('#user-register-form'), data, destination);},
error: function() {trace('error');} error: function() {trace('error');}
}); });
return false; return false;
@ -96,7 +96,7 @@ MaterioUser = function(){
dataType:'json', dataType:'json',
url: Drupal.settings.basePath+Drupal.settings.pathPrefix+"materio_user/login/submit", url: Drupal.settings.basePath+Drupal.settings.pathPrefix+"materio_user/login/submit",
data: $('#user-login', $modal).serialize(), data: $('#user-login', $modal).serialize(),
success: function(data){ onUserLoginRegisterSubmit(data, destination); }, success: function(data){ onUserLoginRegisterSubmit($('#user-login'), data, destination); },
error: function() { trace('error'); } error: function() { trace('error'); }
}); });
return false; return false;
@ -115,12 +115,12 @@ MaterioUser = function(){
}); });
}; };
function onUserLoginRegisterSubmit(data, destination){ function onUserLoginRegisterSubmit($form, data, destination){
cleanModalErrors(); cleanModalErrors();
if(data.errors != null){ if(data.errors != null){
for(field in data.errors){ for(field in data.errors){
$('#user-login input[name='+field+']', '.modal-content').addClass('error'); $('input[name='+field+']', $form).addClass('error');
$('.modal-content').prepend($('<div>').addClass('message-error').html(data.errors[field])); $form.prepend($('<div>').addClass('message-error').html(data.errors[field]));
} }
}else{ }else{
window.location.pathname = destination; window.location.pathname = destination;