UPDATE suivi d'un SELECT, mauvais synchronisme ?

Version imprimable

16/09/2008, 20h14
FrankOVD

UPDATE suivi d'un SELECT, mauvais synchronisme ?

Bonjour,

SITUATION :

J'ai développé une application Web où se situe un formulaire à partir duquel on peut modifier des informations pour un produit et lorsque l'événement "onUnload" est appelé, la page appele en AJAX un script PHP qui effectue une requête UPDATE dans la base de données.

PROBLÈMATIQUE :

Que la requête update soit appelée pour un rafraîchissement de la page pour pour retourner à ma liste des produits, la page suivante n'affiche pas les informations à jour. Ainsi, il faut que je rafraîchisse ma page à nouveau pour voir les bonnes informations.

QUESTION :

Est-il possible dans ce cas que la requête d'UPDATE n'aie pas terminé son exécution alors qu'une requête SELECT est appelée dans la page suivante ou existe-t-il une autre situation qui pourrait causer ce problème. Si oui, comment pourrais-je corriger la situation?

PISTES :

1 - Même si je fait attendre ma première page jusqu'à ce que l'exécution du script PHP qui appelle ma requête UPDATE soit terminée, le problème reste entier.

2 - Peu importe la page où je me retrouve et la requête SELECT qui est appelée, l'information retournée n'est pas à jours.

3 - J'ai appelé mes pages avec des paramètres générés aléatoirement pour écarter la possibilité d'une mauvaise gestion de la cache.
17/09/2008, 11h47
Mr N.

J'ai pas tout compris ton enchaînement de pages...

Tu peux nous poster les bouts de code qui concernent ton problème ?

D'accord,

Voici comment ça fonctionne. Le tout est déclanché onUnload. La fonction save_final() appelle save_produit() qui appelle en AJAX le code PHP plus bas. Ensuite bien entendu la page se décharge et le navigateur charge la nouvelle page où les informations ne seront pas à jour...

Code:

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
35
36
37
38
39
40
41
42
43
 
function save_final() {
  if(document.forms['mod_produit'].unsaved.value > 0) {
    save_recette();
    var msg = "Souhaitez-vous enregistrer les modifications?\n\nCliquez sur OK pour enregistrer les modifications\nCliquez sur ANNULER pour quitter sans enregstrer";
    if(confirm(msg)) {
      save_produit("all");
      alert("Enregistrement terminé");
    }
  }
}
 
/**
 * Enregistre les informations relatives à un produit. 
 * Si le paramètre obj est un objet, l'enregistrement sera fait spécifiquement pour cet objet.
 * Si cependant obj == "all", toutes les informations relatives au produit seront enregistrées
 * Paramètre : obj
 **/
function save_produit(obj) {
  //AJAX
  xmlHttp=GetXmlHttpObject();
  if(xmlHttp==null) {
    alert("Browser does not support HTTP Request")
    return
  }
 
  var id_produit = document.forms['mod_produit'].id_produit.value;
 
  if(obj == "all") {
    var frm = document.forms['mod_produit'];
    var cible = "produits.php?action=modifier&all="+id_produit;
    cible += "&code_produit="+escape(frm.code_produit.value);
    cible += "&desc_fr="+escape(frm.desc_fr.value);
    //...						
    $asynchronous = true;
    xmlHttp.open("GET",cible,$asynchronous); //change to true for asynchronous
    xmlHttp.send(null);
  }
  else {
    //...
  }
  return true;
}

Code PHP

Code:

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
 
$query  = "UPDATE produits ";
$query .= "SET code_produit  = ".mysql_safe(urldecode($_GET["code_produit"])).",";
$query .= "    desc_fr       = ".mysql_safe(urldecode($_GET["desc_fr"])).",";
$query .= "    desc_en       = ".mysql_safe(urldecode($_GET["desc_en"])).",";
$query .= "    poids         = ".mysql_safe(urldecode($_GET["poids"])).",";
$query .= "    volume        = ".mysql_safe(urldecode($_GET["volume"])).",";
$query .= "    qte           = ".mysql_safe(urldecode($_GET["qte"])).",";
$query .= "    qte_piece     = ".mysql_safe(urldecode($_GET["qte_piece"])).",";
$query .= "    prix_vente_1  = ".mysql_safe(urldecode($_GET["prix_vente_1"])).",";
$query .= "    prix_vente_2  = ".mysql_safe(urldecode($_GET["prix_vente_2"])).",";
$query .= "    prix_vente_3  = ".mysql_safe(urldecode($_GET["prix_vente_3"])).",";
$query .= "    prix_vente_4  = ".mysql_safe(urldecode($_GET["prix_vente_4"])).",";
$query .= "    prix_vente_5  = ".mysql_safe(urldecode($_GET["prix_vente_5"])).",";
$query .= "    gl_inventaire = ".mysql_safe($_GET["gl_inventaire"]).",";
$query .= "    gl_vente      = ".mysql_safe($_GET["gl_vente"]).",";
$query .= "    gl_achat      = ".mysql_safe($_GET["gl_achat"])." ";
$query .= "WHERE id_produit  = ".mysql_safe($_GET["all"]);
if(!mysql_unbuffered_query($query)) {
  queryErrorReport("La modification du produit #".$_GET["all"]." à échoué", $query, mysql_error(), basename(__FILE__), __LINE__);
  exit;
}
else {
  echo("Enregistrement terminé");
  return true;
}

17/09/2008, 19h13
Mr N.

Ajax, comme son nom l'indique d'ailleurs, est utilisée de manière asynchrone.
ça veut dire que ton appel ajax part, et que le code javascript continue son chemin. Et le chemin amène à... unload !
Donc on a, en parallèle, 2 requêtes simultanées qui partent vers le serveur. L'une va faire un update, l'autre, visiblement, fera un select. Mais rien ne te garantie qu'une des requêtes va arriver ou être traitée avant l'autre.

Plusieurs solutions s'offrent à toi:
- utilisation de messages synchrones (bad idea, le navigateur est bloqué le temps de la requête)
- annulation du unload (arrêt de l'événement) en cas de sauvegarde. Ce qui te permettra d'afficher le feedback comme quoi l'update c'est bien passé ou pas, ce qui n'était pas le cas visiblement.

Bonjour,

J'ai essayé de plusieurs façon d'éviter la situation que tu décris. C'est à dire que je l'ai essayé de façon Synchrone sans résultat. J'ai aussi fait une boucle qui ralentissait le navigateur jusqu'à l'obtention du readystate 4 (complete). Dans les deux cas, bien que Javascript air attendu un certain moment avant le unload, la situation reste la mème.
Code:

1 2 3 4 5 6 7 8 9 10 11 xmlHttp.open("GET",cible,$asynchronous); //change to true for asynchronous xmlHttp.send(null); //Pour tests, synchronous manuel if($asynchronous == false) { var loop = 0; while(xmlHttp.readyState < 4) { loop = loop + 1; } }
Pour ce qui est d'annuler le unload, tout ce que j'en sais c'est qu'il est impossible de le faire à moins d'appeler un événement géré par le navigateur qui fait afficher une page du genre "Voulez vous fermer la page?". Je ne souhaite pas que mes utilisateurs aient à répondre à une telle question car il ignoreront pourquoi ils ont à le faire.

18/09/2008, 13h06
Mr N.

Citation:

Envoyé par FrankOVD

Pour ce qui est d'annuler le unload, tout ce que j'en sais c'est qu'il est impossible de le faire à moins d'appeler un événement géré par le navigateur qui fait afficher une page du genre "Voulez vous fermer la page?". Je ne souhaite pas que mes utilisateurs aient à répondre à une telle question car il ignoreront pourquoi ils ont à le faire.

il te suffit de leur expliquer. en effet, tu captures l'événement onBeforeUnload et tu rajoute à l'événement un message :

Code:

evt.returnValue = "Vous n'avez pas sauvegardé vos modifications.";

Ce message sera automatiquement affiché dans la fenêtre de confirmation du navigateur. (en tout cas Fx et IE)

ça répond pas au besoin initial mais fournit une solution de contournement.
Sinon t'as pas une version de test hébergée en ligne pour nous permettre de tester ?
18/09/2008, 22h43
FrankOVD

Bonjour,

Malheureusement non, je n'ai rien qui soit accessible via Internet et je ne peux pas le faire facilement car ce système utilise à tout moment des données comptables sensibles. Remplacer toutes ces données serait plutôt long.

Je vais faire des tests à savoir si je peux attraper un retour en AJAX pour m'assurer que ma requête UPDATe a bel et bien été effectuée et à fonctionné avant le unload.

Pour ce qui est du beforeUnload je vais voir ce que ça donne! Merci beaucoup! Je post mes résultats dès que j'ai fait mes tests demain.