(12-08-2010, 10:31 AM)Sephi-Chan a écrit : La partie Javascript n'est là qu'en bonus, pour être plus agréable pour l'utilisateur en l'empêchant de se confronter à une erreur s'il clique plusieurs fois.

(12-08-2010, 10:31 AM)Sephi-Chan a écrit : L'authenticity token est un grand classique (implémenté dans tous les frameworks sérieux), mais je ne vois pas ce qu'il a de plus propre/moins bidouille, c'est équivalent (avec un champ caché côté formulaire en plus).

(12-08-2010, 11:46 AM)garf a écrit :

(12-08-2010, 10:31 AM)Sephi-Chan a écrit : La partie Javascript n'est là qu'en bonus, pour être plus agréable pour l'utilisateur en l'empêchant de se confronter à une erreur s'il clique plusieurs fois.

Tu le dis bien, le javascript c'est du bonus... J'essaye de trouver les solutions les plus propres. si JS désactivé, cette solution ne fonctionne pas. Laissons au serveur ce qui doit être gérer par le serveur. et au client, ce qui doit l'être. Pour ma part, le JS n'est là que pour améliorer l'expérience utilisateur.

(12-08-2010, 10:31 AM)Sephi-Chan a écrit : L'authenticity token est un grand classique (implémenté dans tous les frameworks sérieux), mais je ne vois pas ce qu'il a de plus propre/moins bidouille, c'est équivalent (avec un champ caché côté formulaire en plus).

C'est plus propre, dans le sens ou la solution du token prend aussi en charge le cas du refresh, je ne pense pas que ce soit le cas du javascript. Les solutions sont donc non équivalente. CQFD ^^

(12-08-2010, 07:22 AM)Sephi-Chan a écrit : Côté serveur, tu peux tout simplement créer une variable de session que tu nommeras par exemple last_form_submitted_at qui contient le timestamp du dernier envoi de formulaire, ainsi sur les formulaires critique il est très simple de vérifier l'écart entre le timestamp courant et le timestamp du dernier envoi de formulaire.

(12-08-2010, 11:52 AM)Sephi-Chan a écrit : Je crois que tu n'as pas lu la solution que je proposais. Je me cite :

(12-08-2010, 11:52 AM)Sephi-Chan a écrit : Et Javascript est donc bien un bonus d'utilisabilité.

function DisabledInput()

{

	setTimeout(DisabledInputNow,1);

	function DisabledInputNow()

	{

		tag=document.getElementsByTagName('input')

		for(i=0;i<tag.length;i++)

		{

			if(tag[i].getAttribute('type')=='submit')

			{

				tag[i].disabled=true;

			}

		}

	}

}

(12-08-2010, 12:53 PM)garf a écrit :

(12-08-2010, 11:52 AM)Sephi-Chan a écrit : Je crois que tu n'as pas lu la solution que je proposais. Je me cite :

J'avais du lire que la première partie de ta réponse. Cependant ton last_form_submitted_at, j'aime pas c'est trop long comme nom de champ ^^
Je l'utiliserai plus pour imposer un délai plutôt que pour empêcher un resoumission de formulaire. ca fait faire plus d'opération.

t1 - t2 > X <=> comparaison des timestamps
T1 != T2 <=> comparaison de token

La comparaison de timestamp permet plus de chose mais elle est plus gourmande en ressources (je pinaille pour rien du tout mais répond "correctement" à la question )

(12-08-2010, 11:52 AM)Sephi-Chan a écrit : Et Javascript est donc bien un bonus d'utilisabilité.

:respect: :good:, en gros j'approuve => tu devrais changer le Chan en Sama ou Sensei

(12-08-2010, 01:05 PM)php_addict a écrit : voici pour le JS:

function DisabledInput()

{

	setTimeout(DisabledInputNow,1);

	function DisabledInputNow()

	{

		tag=document.getElementsByTagName('input')

		for(i=0;i<tag.length;i++)

		{

			if(tag[i].getAttribute('type')=='submit')

			{

				tag[i].disabled=true;

			}

		}

	}

}

function disableInputForDuration(elementId, duration){

  var element = document.getElementById(elementId);

  element.disabled = true;

  setTimeout(function(){ element.disabled = false; }, duration);

}

(12-08-2010, 02:13 PM)Sephi-Chan a écrit : Je trouve cette fonction inutilement compliquée et je verrais plutôt quelque chose comme :

$('form').submit(function(event){

  var submitButton = $(this).find('input[type=submit]:first');

  submitButton.attr('disabled', 'disabled').delay(1000).removeAttr('disabled');

  event.preventDefault();

});