Bug (presque) résolu pour l'alerte Full Crawl

[djoan]

il y a bien un bug ligne 92 de googlestats.inc une fin de parenthèse en trop TO_DAYS(date)) à enlever bien sur ce qui donne TO_DAYS(date) et surtout plus de bug, ni email vide

[WebRankInfo]

merci.
mais apparemment ça ne suffit pas, on reçoit toujours des mails. Quelqu'un voit un pb dans la requete ? ça doit sauter aux yeux...
voici le code actuel :

Code: Tout sélectionner

// on va chercher s'il est déjà venu dans les $NB_J_DET_FULL_CRAWL
// derniers jours
$sql3  = "SELECT id";
$sql3 .= " FROM ".$TABLE_LOG;
$sql3 .= " WHERE ip LIKE '".$FULL_CRAWL_IP."%')";
$sql3 .= " AND TO_DAYS(NOW()) - TO_DAYS(date) < $NB_J_DET_FULL_CRAWL";
$res3  = mysql_query($sql3) or sendErrorMySQL($sql3);

// si la requete n'a donné aucun résultat, c'est sans doute le
// début du Full Crawl : on envoie un mail
if (mysql_num_rows($res3) == 0)
{
  @mail($ADRESSE_EMAIL,
    $LANG["FullCrawlBeginSubject"],
    $LANG["FullCrawlBeginBody"],
    "From: $ADRESSE_EMAIL");
}


avec $NB_J_DET_FULL_CRAWL = 2

[djoan]

il y a bien encore cela:
$sql3 .= " FROM $TABLE_LOG";
et non $sql3 .= " FROM ".$TABLE_LOG;

[WebRankInfo]

il y a bien encore cela:
$sql3 .= " FROM $TABLE_LOG";
et non $sql3 .= " FROM ".$TABLE_LOG;

non ça ce n'est pas une erreur, on a le choix d'inclure la variable ou pas dès lors qu'on utilise des double quotes...

[hetzeld]

Code: Tout sélectionner

$sql3 .= " WHERE ip LIKE '".$FULL_CRAWL_IP."%')";

Il te sert à quoi le ) fermant ?

Dan

[berpi]

$sql3 .= " WHERE ip LIKE ' ".$FULL_CRAWL_IP."%')";

Peut-être l'ordre des apostrophes et guillemets ?

berpi

[djoan]

c'est vrai j'avais également supprimé le guillemet, c'est peut-être pour cela que j'ai plus l'erreur:
non pas $sql3 .= " WHERE ip LIKE '".$FULL_CRAWL_IP."%')";

mais $sql3 .= " WHERE ip LIKE '".$FULL_CRAWL_IP."%'";

djoan

[apocalx]

C'est quoi le problème, exactement? Que tous les e-mails soient vides?

[WebRankInfo]

les mails sont vides à cause d'un oubli d'inclusion du fichier de langue.
par contre le bug est qu'autant de mails soient envoyés !
le pb venait peut-être de cette parenthèse de trop, je vais tester.

[apocalx]

une fin de parenthèse en trop TO_DAYS(date)) à enlever bien sur ce qui donne TO_DAYS(date)

Ca serait pas plutot le contraire, une parenthèse a rajouter?
c'est a dire mettre
(TO_DAYS(NOW()) - TO_DAYS(date))
à la place de
TO_DAYS(NOW()) - TO_DAYS(date))

Enfin ça demande vérification...

[WebRankInfo]

non à cet endroit c'est facultatif (mais c'est mieux avec je pense).
je parlais de la parenthèse fermante en trop en fin de la ligne

Code: Tout sélectionner

."%')";

[djoan]

une confirmation:

http://www.phpinfo.net/astuces/astuces202.html

djoan

[djoan]

ligne 92

$sql3 .= " AND ((TO_DAYS(NOW()) - TO_DAYS(date)) < $NB_J_DET_FULL_CRAWL)";

djoan

[djoan]

ligne 92

$sql3 .= " AND ((TO_DAYS(NOW()) - TO_DAYS(date)) < $NB_J_DET_FULL_CRAWL)";

djoan

je me réponds à moi-même, mais il semblerai que ce soit ça, à l'heure ou je poste cassandra passe sur mon site et je n'ai plus l'erreur (chouette)

djoan

[djoan]

$sql3 = "SELECT id";
$sql3 .= " FROM $TABLE_LOG";
$sql3 .= " WHERE ip LIKE '".$FULL_CRAWL_IP."%'";
$sql3 .= " AND ((TO_DAYS(NOW()) - TO_DAYS(date)) < $NB_J_DET_FULL_CRAWL)";
$res3 = mysql_query($sql3) or sendErrorMySQL($sql3);

alors Olivier, à ton avis, c'est ok ou pas ?