Surchage de la fonction onClick d'un lien

**zizou39** · 19/01/2007, 14h06

Bonjour à tous.
Pour la mesure d'audiance de mon site j'utilise xiti.
Pour cela je appeler sur tous les liens sortant une fonction xiti sur la fonction onClick des mes liens.

J'aimerais faire un traitement automatique pour tous mes liens et surcharger leur fonction onClick.

(Pour tester si c'est un lien sortant je compare le hostname de la page et celui du lien).

Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
function generateXitiLinkTag() {
 
		var siteHostName = window.location.hostname;
 
	    for (i = 0; i < document.links.length; i++) {
	        var hostname = document.links[i].hostname;
	        if (hostname != '' && hostname != siteHostName) {
		        document.links[i].oldClick = (document.links[i].onclick) ? document.links[i].onclick : function() {};
 
	            document.links[i].onclick = function() {
	        		click_out();
		        	this.oldClick();
	        	};
 
	        }
 
	    }
	}
 
	generateXitiLinkTag();

Par exemple pour un lien comme celui-ci :
Code :

1.

Code :

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

<a href="http://www.google.fr" onClick="openThePopup('http://www.google.fr',0,0,'yes');return false;">test</a>

Une popup s'ouvre bien sur google mais ma fenetre principale aussi s'ouvre sous google.

En espérant avoir été le plus claire possible.

**ryan** · 19/01/2007, 14h37

Yop!

Envoyé par zizou39

<a href="http://www.google.fr" onClick="openThePopup('http://www.google.fr',0,0,'yes');return false;">test</a>

Une popup s'ouvre bien sur google mais ma fenetre principale aussi s'ouvre sous google.

Bin oui, si tu précises une URL dans le href, c'est normal.

**SpaceFrog** · 19/01/2007, 14h55

Code :

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

1
2
3
4
5
6
7
8
9
10
<script type='text/javascript'>
 
function openThePopup(Myurl){
window.open(Myurl)
}
</script>
<body>
 
      <a href="#" onclick="openThePopup('http:/www.google.fr');return false;">test</a>;
</body>

**zizou39** · 19/01/2007, 15h39

ryan -> Le javascript me permet d'ouvrir une popup sur google. Le fait de retourner "false" qui fait que le href n'est pas interprété. De plus, dans le cas ou le client déactive le javascript une popup n'est pas ouverte mais c'est la page qui se redirige vers google.
(cf : normes w3c http://www.w3.org/TR/WCAG10-HTML-TECHS/)

(Si j'enlève ma fonction de surchage de l'attribut onclick, mes liens fonctionnent parfaitement)

Envoyé par SpaceFrog

Code :

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

1
2
3
4
5
6
7
8
9
10
<script type='text/javascript'>
 
function openThePopup(Myurl){
window.open(Myurl)
}
</script>
<body>
 
      <a href="#" onclick="openThePopup('http:/www.google.fr');return false;">test</a>;
</body>

SpaceFrog -> mettre href="#" ne respectent pas les normes w3c et ca le fait pas pour un site pro. De plus, dans le cas ou le client déactive le javascript le lien ne redirigera jamais vers google.

Mon problème se trouve au niveau de la surchage de l'attribut onclick des liens. Je voudrais juste pouvoir rajouter une instruction.

**SpaceFrog** · 19/01/2007, 15h44

Code :

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

1
2
3
4
5
6
7
8
9
10
<script type='text/javascript'>
 
function openThePopup(Myurl){
window.open(Myurl)
}
</script>
<body>
 
      <a href="javascript:void(0)" oncllick="openThePopup('http:/www.google.fr');return false;">test</a>;
</body>

?

**zizou39** · 19/01/2007, 15h50

Envoyé par SpaceFrog

Code :

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

1
2
3
4
5
6
7
8
9
10
<script type='text/javascript'>
 
function openThePopup(Myurl){
window.open(Myurl)
}
</script>
<body>
 
      <a href="javascript:void(0)" oncllick="openThePopup('http:/www.google.fr');return false;">test</a>;
</body>

?

Tjs le même problème.
Selon les normes w3c tu n'as plus le droit du mettre du javascript dans l'attribut href.
Ce qui est logique car si le javascript est désactivé le lien ne marchera plus. (Le browser ne sera pas capable d'interpréter le onclick mais aussi le href)

**SpaceFrog** · 19/01/2007, 16h03

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<script type='text/javascript'>
 
function openThePopup(Myurl){
window.open(Myurl)
}
</script>
<body>
 
      <a href="http:/www.google.fr" onclick="this.href='#';openThePopup('http:/www.google.fr');return false;">test</a>;
</body>
 
</html>

**zizou39** · 19/01/2007, 16h20

Envoyé par SpaceFrog

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<script type='text/javascript'>
 
function openThePopup(Myurl){
window.open(Myurl)
}
</script>
<body>
 
      <a href="http:/www.google.fr" onclick="this.href='#';openThePopup('http:/www.google.fr');return false;">test</a>;
</body>
 
</html>

Ca c'est bon au niveau w3c.
Mais mon problème se situe au niveau de la surchage de l'attribut onclick dans ma fonction generateXitiLinkTag()

**SpaceFrog** · 19/01/2007, 16h27

arf ou j'oubliais... si ton onclick contient déja quelquechose ..

**SpaceFrog** · 19/01/2007, 16h43

Accrochez vous c'est un peu chaud ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
 
<html>
 
	<head>
		<meta http-equiv="content-type" content="text/html;charset=iso-8859-1">
		<title>Wazzzaaa</title>
		<script type="text/javascript"><!--
function addOnclick(id,func) {
	myEl = document.getElementById(id);
	if (typeof myEl.onclick == "function") {
		oldFunction = myEl.onclick;
		myEl.onclick = function() {
			oldFunction();
			func();
		}
	} else {
		myEl.onclick = func;
	}
}
 
 
 
function popme(myurl){
window.open(myurl)
}
</script>
	</head>
 
	<body>
		<a href="http://www.google.fr" id="btn" onclick="this.href='#';popme('http://www.google.com');return false;" ;>test</a>
	</body>
 
</html>

en fait faut juste concatener un nouveau onclick en prenant soin de conserver l'ancien onclick :d

**zizou39** · 25/01/2007, 11h00

Envoyé par SpaceFrog

Accrochez vous c'est un peu chaud ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
 
<html>
 
	<head>
		<meta http-equiv="content-type" content="text/html;charset=iso-8859-1">
		<title>Wazzzaaa</title>
		<script type="text/javascript"><!--
function addOnclick(id,func) {
	myEl = document.getElementById(id);
	if (typeof myEl.onclick == "function") {
		oldFunction = myEl.onclick;
		myEl.onclick = function() {
			oldFunction();
			func();
		}
	} else {
		myEl.onclick = func;
	}
}
 
 
 
function popme(myurl){
window.open(myurl)
}
</script>
	</head>
 
	<body>
		<a href="http://www.google.fr" id="btn" onclick="this.href='#';popme('http://www.google.com');return false;" ;>test</a>
	</body>
 
</html>

en fait faut juste concatener un nouveau onclick en prenant soin de conserver l'ancien onclick :d

L'idée est bonne merci. Par contre si ton ancienne fonction fait un return c'est foutu. Pour résoudre le problème j'ai fait :

Code :

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

1
2
3
4
5
 
document.links[i].onclick = function() {
   ma_nouvelle_fonction();
   return(this.oldClick());
};

**SpaceFrog** · 25/01/2007, 11h08

non en fait c'est ce que je pensais aussi mais le return ne fait que sortir de son niveau ... fais le test la seconde fonction s'executera même si elle est après ...

**Matthieu2000** · 25/01/2007, 14h41

**E.Bzz** · 25/01/2007, 14h47

Bonjour,

Code :

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

<a href="http://www.google.fr" target="_blank" onClick="openThePopup('http://www.google.fr',0,0,'yes');">test</a>

A+

Surchage de la fonction onClick d'un lien

JavaScript

Discussions similaires

Partager

Partager