[PHP]AJAX logowanie, pomoc w zabezpieczeniu kodu Opcje

[casperii]

@nospor SignIn poprawiony na signIn

to:

[HTML] pobierz, plaintext 
 
if(responseData.status == 'error'){
	setTimeout(function(){
		responseMsg.fadeOut(200,function(){
			$(this).removeClass(klass);
		});
	},3000);
 
}
[HTML] pobierz, plaintext 

nie powoduje opóźnienia wyświetlenia błędu tylko usunięcie klasy z informacją o błędzie.
podobnie jest z chwilą success - wyświetlam informację o poprawnym zalogowaniu i przeładowuje stronę.

klass gdy zmienię na class to wywala błąd

[nospor]

To nazwij to np. messageClass tylko na jadro linuksa nie spolszaczaj tego w ten sposob

[casperii]

@viking fail() / done() jest w nowszej wersji jQuery , nie chce teraz ruszać strony by się nie posypała
@nospor usunąłem switcha i (c/k)lass

czy teraz wygląda Panowie kod optymalnie , nie czepiając się oczywiście nowego jQuery.

obecny JS wygląda tak:

[HTML] pobierz, plaintext 
	var signIn = function() {
		$('#LoginButton,#LoginButtonTop').on('click', function (){
			var form = $('#login').serialize(),
			responseMsg = $('#messageLogin,#messageLoginTop');
			responseMsg.hide().addClass('response-waiting').text('czekaj...').fadeIn(200);
 
			$.ajax({
				type: 'GET',
                url: "/ajax.php?p=login",
                data: form,
				dataType: 'json', 
 
				success: function(data, textStatus, jQxhr) {				
					responseMsg.fadeOut(200,function(){
						$(this).removeClass('response-waiting').addClass('response-success').html(data.message).fadeIn(200,function(){
								var url4 = window.location.href;
								setTimeout(function() {
									window.location.reload(url4);
									location.reload(true);
								}, 1000); /*automatyczne odświeżenie strony */
						});
					});
				},
				complete: function() {
					//ten fragment wykona się po zakończeniu łączenia - nie ważne czy wystąpił błąd, czy sukces
				},
				error: function(jqXHR, errorText, errorThrown) {
					console.log( errorThrown );
 
					responseMsg.fadeOut(200,function(){
						$(this).removeClass('response-waiting').addClass('response-error').html(jqXHR.responseJSON.message).fadeIn(200,function(){
 
							setTimeout(function(){
								responseMsg.fadeOut(200,function(){
									$(this).removeClass('response-error');
								});
							},3000);							
						});
					});
 
 
				}
			});
			return false;
		});
	}
[HTML] pobierz, plaintext 

Panowie jeśli treść do jQuery z php wysyłam w ten sposób:

[PHP] pobierz, plaintext 
echo json_encode(['message' => 'wiadomość przekaż']); 
return;
[PHP] pobierz, plaintext 

to jak powinno wyglądać poprawnie w samym jQuery odebranie tego i wstawienie do diva ?

[HTML] pobierz, plaintext 
.fail(function(jqXHR, errorText, errorThrown) {
	console.log( errorThrown );
	$(this).html(jqXHR.responseJSON.message);
});
return false;
[HTML] pobierz, plaintext 

niby dla powyższego działa, ale wydaje mi się, że nie jest to poprawnie, ponieważ w przypadku gdy wykona się done() to wówczas wywala bład z message
No i jak powinno zostać przekazane info z php do diva w przypadku done ?

Ten post edytował casperii 14.10.2018, 20:07:53