Discussion :

[p0Kep0K]

Hello!

La question a déjà du être posée des dizaines de fois mais je ne trouve rien! En plus le problème m'était déjà arrivé mais je ne trouve plus rien dans mes archives!

Voilà, c'est tout simple... c'est un contrôle d'un champ dans un formulaire. Si l'image uploadée n'est pas un JPEG, le formulaire n'est pas envoyé. J'aimerais en faire une fonction globale et réutilisable pour tous mes formulaires. Je mets donc comme arguments dans ma fonction, le nom du formulaire et le champ sur lequel l'action doit être lancée.

Plus facile avec un peu de code! L'évènement:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

onSubmit="return ControlJPEG('this.form', 'NomChamp');"

Et le code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
	function ControlJPEG(Formulaire, Champ)
	{
		var NomImage = document.Formulaire.Champ.value;
		var Extension = NomImage.substring(NomImage.lastIndexOf("."));
 
		if( (Extension != '.jpg') && (Extension != '.JPG') && (Extension != '') )
		{
			alert('message');
			return false;
		}
		return true;
	}

Merci pour votre aide!

[nicocsgamer]

J'ai le meme probleme.

[SpaceFrog]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function ControlJPEG(Formulaire, Champ)
   {
      var NomImage = document.Formulaire.Champ.value;

normal Formulaire est déja un objet formulaire !!!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var NomImage = Formulaire.Elements[Champ].value;

tu as passé en paramètre l'objet formulaire donc tu accède directement à ses proriétés sans rajouter document devant

par contre tu aurais passé le nom de formulaire , c'est à dire un string, alors là il aurait fallu faire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var NomImage = document.forms[Formulaire].Elements[Champ].value;

[denisC]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

onsubmit="return ControlJPEG('this', 'NomChamp');"

Pas besoin de this.form, car tu es déjà dans ton element form, donc this refere au formulaire.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
function ControlJPEG(Formulaire, Champ)
	{
		var NomImage = Formulaire.elements[Champ].value;
		var testExtension=/\.(jpg|JPG|jpeg)$/.test(NomImage);
		if( !testExtension )
		{
			alert('message');
		}
		return testExtension;
	}

En utilisant les expressions régulières, c'est plus beau. J'ai juste un doute sur les ou (|) dans l'expression, je ne suis pas sur qu'on puisse en mettre 2 mais faut juste vérifier ce point.

Bonne chance

[p0Kep0K]

SpaceFrog, ta solution ne fonctionne pas!...
Entre-temps j'ai retrouvé ce que j'avais fait et qui fonctionne parfaitement:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var NomImage = eval("document."+Formulaire+"."+Champ+".value");

Vu que les arguments sont repris comme étant du texte (string), j'utilise la fonction eval(). Elle permet d'exécuter un chaîne de caractères comme s'il s'agissait de la ligne d'un code Javascript...

Voili, c'est tout bon!
Merci quand même pour l'aide!

[SpaceFrog]

desolé, juste un problème de majuscule à elements...
sinon ma solution focntionne

[denisC]

SpaceFrog, ta solution ne fonctionne pas!...

Et la mienne tout le monde s'en fout?????

Sinon, pour toi p0Kep0K, je ne suis pas sur que ta solution fonctionne sous FFX.... Tu testes uniquement sous IE ou sous les deux?

[p0Kep0K]

SpaceFrog, ta solution ne fonctionne pas!...

Et la mienne tout le monde s'en fout?????

Désolé on a répondu en même temps!!
Pour l'instant je teste tout sur IE6 étant donné que tous les utilisateurs de ma console d'admin (car c'est pour une console d'admin!) sont sous IE! En plus de ca, je ne sais même pas s'ils savent qu'il existe d'autres navigateurs! Donc tu vois, à la limite, même si ca ne fonctionne pas sous FireFox!...

[p0Kep0K]

Je viens de tester, le script fonctionne parfaitement sous FireFox 0.8+ et sous Netscape 7+
Nickel!