onClick='window.open('

je te montre ou est l erreur tu corriges comme un grand apres

t as édité entre temps
t as juste déplacé l erreur mais c est encore la meme

http://www.phpdebutant.org/article150.php

j'aurais fais :
 

echo "<td id=center><a href='#' onClick=\"window.open('employeur/cv.php?num=".$fetch_requeteur['personne_num'].",'_blank','toolbar=0, location=0, directories=0, status=0, scrollbars=1, resizable=1, copyhistory=0, menuBar=1, width=670, height=);return(false)\"><img src=images/design/donnees.png alt=voircv /> </a></td>";

 
essai de voir ?

C'est quoi cette manie de vouloir concaténer quand y'a pas besoin ? Ensuite, avec des simples-quotes ce sera déjà plus clair :

echo '<td id=center><a href="#" onClick="window.open(\'employeur/cv.php?num=', $fetch_requeteur['personne_num'], ',\'_blank\',\'toolbar=0, location=0, directories=0, status=0, scrollbars=1, resizable=1, copyhistory=0, menuBar=1, width=670, height=\');return(false)"><img src="images/design/donnees.png" alt="Voir le CV" /> </a></td>';

 
Ensuite y'a encore dix fois mieux, c'est de séparer code HTML et PHP, du style :
 

// plein de code
?><td id="center"><a href="#" onClick="window.open('employeur/cv.php?num='<?php echo htmlspecialchars($fetch_requeteur['personne_num']) ?>','_blank','toolbar=0, location=0, directories=0, status=0, scrollbars=1, resizable=1, copyhistory=0, menuBar=1, width=670, height=');return(false)"><img src="images/design/donnees.png" alt="Voir le CV" /> </a></td><?php
 
// suite du code

 
On pourra faire un fichier template séparé du reste du code. Ensuite pour le Javascript on peut encore faire mieux, là c'est pas accessible du tout. Sinon c'est quoi le id="center" sur le <td> ? Y'a-t-il un id="center" sur plein de <td> (ce qui serait évidemment invalide et dangereux), ou est-ce qu'il est que sur un ?

je suis d'accord avec toi mais il est parfois bien plus pratique de concaténer, personnellement, je préfère

Non Ca sert à rien de concaténer si on peut afficher directement... Et si faut vraiment concaténer, surtout pour du gros HTML, le mieux et de passer par de l'output buffering, ce sera beaucoup plus rapide...

Ca m'intéresse ce que tu dis FlorentG, tu peux expliquer en 3 mots en quoi consiste l'output buffering ?

Genre prend une boucle de base :

$string = '';
for($i = 0; $i < 1000; $i++) {
  $string .= 'pouet';
}

 
Vraiment pas performant, je t'explique pas. Avec output buffering, ça devient :

ob_start();
for($i = 0; $i < 1000; $i++) {
  echo 'pouet';
}
$string = ob_get_contents();
ob_end_clean();

Beaucoup plus performant (pour d'importantes concaténations)

Compris ( enfin pas pourquoi, mais le principe )

La concaténation de base va allouer de la mémoire pour chaque concaténation. Tu te retrouve avec une nouvelle allocation à chaque passage de la boucle, pas top.
 
L'output buffer lui initialise à la base un buffer de 40 Ko, renouvellable par tranches de 10 Ko, donc beaucoup plus efficient. Maintenant si c'est deux pauvres trucs à concaténer, c'est pas intéressant évidemment.

Ok, merci pour l'explication.

 
 
je ne savais pas, merci

Salut!
J'utilise des pop up dans mon appli et j'ai pas mal galéré...dc je devrai pouvoir t'aider.Essaye avec ce code
en remplaçant les bonnes valeurs...

et je te donne la fonction js:

Chez moi ça marche nickel, si tu as des questions n'hésite pas...
Avedici!;-)

Là on voit tout l'intérer de séparer encore plus les couches : structure et comportement.

Il faudrait que le lien soit de la forme :

Ensuite un JS parcourerait tous les liens, et si une classe popup y est, ajouterait automatiquement l'évènement onclick et tout.

La règle est du même style que "on ne mixe pas la présentation et le code HTML". Si on utilise du CSS pour la présentation, on utilisera du JS __séparé__ pour le comportement. C'est-à-dire zéro onclick ou onmouseover ou truc du style, juste certaines class spécifiques qui seraient gérées par un JS externe. Et on oublie pas de gérer aussi un évènement au clavier, parce que la il ne se passe rien si on appuie sur entrée alors que le focus est sur le lien