X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=bin%2Fcron%2Fcheckdb.php;h=a82dfbc16ceb98edce1f82ccecffe095c92c7762;hb=1549059d07f3e752e52ad5d794648fd389170f94;hp=510381780ee56c419e80aa5620eeec0a0090a982;hpb=7706cc73e140d7a154136cf1e6836091a25c69c5;p=platal.git

diff --git a/bin/cron/checkdb.php b/bin/cron/checkdb.php
index 5103817..a82dfbc 100755
--- a/bin/cron/checkdb.php
+++ b/bin/cron/checkdb.php
@@ -1,7 +1,7 @@
 #!/usr/bin/php5 -q
 <?php
 /***************************************************************************
- *  Copyright (C) 2003-2008 Polytechnique.org                              *
+ *  Copyright (C) 2003-2010 Polytechnique.org                              *
  *  http://opensource.polytechnique.org/                                   *
  *                                                                         *
  *  This program is free software; you can redistribute it and/or modify   *
@@ -19,197 +19,98 @@
  *  Foundation, Inc.,                                                      *
  *  59 Temple Place, Suite 330, Boston, MA  02111-1307  USA                *
  ***************************************************************************/
-/*
- * verifie qu'il n'y a pas d'incoherences dans les tables de jointures
- *
-*/
+/* Checks inconsistances in tables ans joins. */
 
-require('./connect.db.inc.php');
-require("Console/Getopt.php");
+require './connect.db.inc.php';
+require 'Console/Getopt.php';
 
-function check($sql, $commentaire='')
+function check($sql, $comment = '')
 {
     $it = XDB::iterRow($sql);
-    if ($err = XDB::error()) echo $err;
+    if ($err = XDB::error()) {
+        echo $err;
+    }
     if ($it->total() > 0) {
-        echo "Erreur pour la verification : $commentaire\n$sql\n\n";
+        echo "Erreur pour la verification : $comment\n$sql\n\n";
         echo "|";
         while($col = $it->nextField()) {
-            echo "\t".$col->name."\t|";
+            echo "\t" . $col->name . "\t|";
         }
         echo "\n";
 
         while ($arr = $it->next()) {
             echo "|";
-            foreach ($arr as $val) echo "\t$val\t|";
+            foreach ($arr as $val) {
+                echo "\t$val\t|";
+            }
             echo "\n";
         }
         echo "\n";
     }
 }
 
-function info($sql,$commentaire='') {
+function info($sql, $comment = '') {
     global $opt_verbose;
-    if ($opt_verbose)
-        check($sql,$commentaire);
+    if ($opt_verbose) {
+        check($sql, $comment);
+    }
 }
 
-/* on parse les options */
-$opts = Console_GetOpt::getopt($argv, "v");
-$opt_verbose=false;
+/* Parses options. */
+$opts = Console_GetOpt::getopt($argv, 'v');
+$opt_verbose = false;
 
-if ( PEAR::isError($opts) ) {
+if (PEAR::isError($opts)) {
     echo $opts->getMessage();
 } else {
     $opts = $opts[0];
-    foreach ( $opts as $opt) {
+    foreach ($opts as $opt) {
         switch ($opt[0]) {
-          case "v":
-            $opt_verbose=true;
+          case 'v':
+            $opt_verbose = true;
             break;
         }
     }
 }
 
-/* Validite des flags de transmission */
-check("SELECT  u.user_id, nom, prenom, promo,
-               emails_alias_pub, profile_freetext_pub, profile_medals_pub
-         FROM  auth_user_md5 AS u
-   INNER JOIN  auth_user_quick AS q USING(user_id)
-        WHERE  (emails_alias_pub != 'private' AND emails_alias_pub != 'public')
-           OR  (profile_freetext_pub != 'private' AND profile_freetext_pub != 'public')
-           OR  (profile_medals_pub != 'private' AND profile_medals_pub != 'public')",
-    "Utilisateur n'ayant pas de flag de publicite pour leurs donnees de profil");
-check("select uid from adresses where pub != 'private' and pub !='ax' and pub != 'public'", "Utiliseur n'ayant pas de flag de publicite pour une adresse");
-check("select uid from profile_phones where pub != 'private' and pub != 'ax' and pub != 'public'", "Utiliseur n'ayant pas de flag de publicite pour un numero de téléphone");
-check("select uid from profile_networking where pub != 'private' and pub != 'public'", "Utiliseur n'ayant pas de flag de publicité pour une adresse de networking");
-
-/* validite de aliases */
-check("SELECT a.*
-        FROM aliases       AS a
-        LEFT JOIN auth_user_md5 AS u ON u.user_id=a.id
-        WHERE (a.type='alias' OR a.type='a_vie') AND u.prenom is null");
-
-/* validite de applis_ins */
-check("select a.* from applis_ins as a left join auth_user_md5 as u on u.user_id=a.uid where u.prenom is null");
-check("select a.* from applis_ins as a left join applis_def as ad on ad.id=a.aid where ad.text is null");
-
-/* validite de binet_users */
-check("select b.* from binets_ins as b left join auth_user_md5 as u on u.user_id=b.user_id where u.prenom is null");
-check("select b.* from binets_ins as b left join binets_def as bd on bd.id=b.binet_id where bd.text is null");
-
-/* validite de contacts */
-check("select c.* from contacts as c left join auth_user_md5 as u on u.user_id=c.uid where u.prenom is null");
-check("select c.* from contacts as c left join auth_user_md5 as u on u.user_id=c.contact where u.prenom is null");
-
-/* validite de emails */
-check("select e.* from emails as e left join auth_user_md5 as u on u.user_id=e.uid where e.uid and u.prenom is null");
-
-/* validite de forums */
-check("select f.* from forums.abos as f left join auth_user_md5 as u on u.user_id=f.uid where u.prenom is null");
-check("select f.* from forums.abos as f left join forums.list as fd on fd.fid=f.fid where fd.nom is null");
-check("select f.* from forums.respofaqs as f left join forums.list as fd on fd.fid=f.fid where fd.nom is null");
-check("select f.* from forums.respofaqs as f left join auth_user_md5 as u on u.user_id=f.uid where u.prenom is null");
-
-/* validite de groupesx_ins */
-check("select g.* from groupesx_ins as g left join auth_user_md5 as u on u.user_id=g.guid where u.prenom is null");
-check("select g.* from groupesx_ins as g left join groupesx_def as gd on g.gid=g.gid where gd.text is null");
+/* Checks rewriting on deleted aliases. */
+check("SELECT  a.alias, e.email, e.rewrite AS broken
+         FROM  aliases AS a
+   INNER JOIN  emails  AS e ON (a.uid = e.uid AND rewrite != '')
+    LEFT JOIN  aliases AS b ON (b.uid = a.uid AND rewrite LIKE CONCAT(b.alias, '@%') AND b.type != 'homonyme')
+        WHERE  a.type = 'a_vie' AND b.type IS NULL",
+      "Personnes qui ont des rewrite sur un alias perdu.");
 
-/* validite de photo */
-check("select p.* from photo as p left join auth_user_md5 as u on u.user_id=p.uid where u.prenom is null");
-
-/* validite des formats téléphoniques */
-check("SELECT DISTINCT g.phoneprf from geoloc_pays AS g
-          WHERE EXISTS (SELECT h.phoneprf
-                          FROM geoloc_pays AS h
-                         WHERE h.phoneprf = g.phoneprf AND h.phoneformat != (SELECT i.phoneformat
-                                                                               FROM geoloc_pays AS i
-                                                                              WHERE i.phoneprf = g.phoneprf
-                                                                              LIMIT 1))",
-      "Préfixes téléphoniques qui ont des formats de numéros de téléphones différents selon les pays");
-
-/* validite des champ pays et region */
-check("SELECT a.uid, a.country FROM adresses AS a LEFT JOIN geoloc_pays AS gp ON a.country = gp.a2 WHERE gp.pays IS NULL","donne la liste des pays dans les profils qui n'ont pas d'entree correspondante dans geoloc_pays");
-/* les régions ne sont valides que dans les adresses pros */
-//check("SELECT e.uid, e.country, e.region FROM entreprises AS e LEFT JOIN geoloc_region AS gr ON (e.country = gr.a2 AND e.region = gr.region) WHERE e.region != '' AND gr.name IS NULL","donne la liste des regions dans les profils pros qui n'ont pas d'entree correspondante dans geoloc_region");
-
-/* donne la liste des emails douteux que les administrateurs n'ont pas encore traité */
+/* Lists unsound emails that remain unprocessed by the administrators. */
 check("SELECT  a1.alias, a2.alias, e1.email, e2.flags
-        FROM  emails        AS e1
-        INNER JOIN  emails        AS e2 ON(e1.email = e2.email AND e1.uid!=e2.uid AND
-            (e1.uid<e2.uid  OR  NOT FIND_IN_SET('active', e2.flags))
-            )
-        INNER JOIN  emails_watch  AS w  ON(w.email = e1.email AND w.state = 'pending')
-        INNER JOIN  aliases       AS a1 ON(a1.id=e1.uid AND a1.type='a_vie')
-        INNER JOIN  aliases       AS a2 ON(a2.id=e2.uid AND a2.type='a_vie')
-        INNER JOIN  auth_user_md5 AS u1 ON(a1.id=u1.user_id)
-        INNER JOIN  auth_user_md5 AS u2 ON(a2.id=u2.user_id)
-        WHERE  FIND_IN_SET('active', e1.flags) AND u1.nom!=u2.nom_usage AND u2.nom!=u1.nom_usage
-        ORDER BY  a1.alias",
-        "donne la liste des emails douteux actuellement non traites par les administrateurs");
-
-/* donne la liste des emails dangereux ou douteux*/
+         FROM  emails      AS e1
+   INNER JOIN  emails      AS e2 ON (e1.email = e2.email AND e1.uid != e2.uid AND
+                                     (e1.uid < e2.uid OR NOT FIND_IN_SET('active', e2.flags)))
+   INNER JOIN  email_watch AS w  ON (w.email = e1.email AND w.state = 'pending')
+   INNER JOIN  aliases     AS a1 ON (a1.uid = e1.uid AND a1.type = 'a_vie')
+   INNER JOIN  aliases     AS a2 ON (a2.uid = e2.uid AND a2.type = 'a_vie')
+        WHERE  FIND_IN_SET('active', e1.flags)
+     ORDER BY  a1.alias",
+      "Donne la liste des emails douteux actuellement non traites par les administrateurs.");
+
+/* Lists dangerous and unsound emails. */
 info("SELECT  a1.alias, a2.alias, e1.email, e2.flags, w.state
-        FROM  emails        AS e1
-        INNER JOIN  emails        AS e2 ON(e1.email = e2.email AND e1.uid!=e2.uid AND
-            (e1.uid<e2.uid  OR  NOT FIND_IN_SET('active', e2.flags))
-            )
-        INNER JOIN  emails_watch  AS w  ON(w.email = e1.email AND w.state != 'safe')
-        INNER JOIN  aliases       AS a1 ON(a1.id=e1.uid AND a1.type='a_vie')
-        INNER JOIN  aliases       AS a2 ON(a2.id=e2.uid AND a2.type='a_vie')
-        INNER JOIN  auth_user_md5 AS u1 ON(a1.id=u1.user_id)
-        INNER JOIN  auth_user_md5 AS u2 ON(a2.id=u2.user_id)
-        WHERE  FIND_IN_SET('active', e1.flags) AND u1.nom!=u2.nom_usage AND u2.nom!=u1.nom_usage
-        ORDER BY  a1.alias",
-        "donne la liste des emails dangereux ou douteux");
-
-/* donne la liste des homonymes qui ont un alias égal à leur loginbis depuis plus d'un mois */
+        FROM  emails      AS e1
+  INNER JOIN  emails      AS e2 ON (e1.email = e2.email AND e1.uid != e2.uid AND
+                                    (e1.uid < e2.uid OR NOT FIND_IN_SET('active', e2.flags)))
+  INNER JOIN  email_watch AS w  ON (w.email = e1.email AND w.state != 'safe')
+  INNER JOIN  aliases     AS a1 ON (a1.uid = e1.uid AND a1.type = 'a_vie')
+  INNER JOIN  aliases     AS a2 ON (a2.uid = e2.uid AND a2.type = 'a_vie')
+       WHERE  FIND_IN_SET('active', e1.flags)
+    ORDER BY  a1.alias",
+     "Donne la liste des emails dangereux ou douteux.");
+
+/* Lists homonyms who have an alias equals to their loginbis for more than a month. */
 check("SELECT  a.alias AS username, b.alias AS loginbis, b.expire
-        FROM  aliases AS a
-        INNER JOIN  aliases AS b ON ( a.id=b.id AND b.type != 'homonyme' and b.expire < NOW() )
+         FROM  aliases AS a
+   INNER JOIN  aliases AS b ON (a.uid=b.uid AND b.type != 'homonyme' and b.expire < NOW())
         WHERE  a.type = 'a_vie'",
-        "donne la liste des homonymes qui ont un alias égal à leur loginbis depuis plus d'un mois, il est temps de supprimer leur alias");
-
-/* verifie qu'il n'y a pas de gens qui recrivent sur un alias qu'ils n'ont plus */
-
-check("SELECT  a.alias AS a_un_pb, email, rewrite AS broken
-        FROM  aliases AS a
-        INNER JOIN  emails  AS e ON (a.id=e.uid AND rewrite!='')
-        LEFT  JOIN  aliases AS b ON (b.id=a.id AND rewrite LIKE CONCAT(b.alias,'@%') AND b.type!='homonyme')
-        WHERE  a.type='a_vie' AND b.type IS NULL","gens qui ont des rewrite sur un alias perdu");
-
-/* validite du champ matricule_ax de la table auth_user_md5 */
-check("SELECT  matricule,nom,prenom,matricule_ax,COUNT(matricule_ax) AS c
-        FROM  auth_user_md5
-        WHERE  matricule_ax != '0'
-        GROUP BY  matricule_ax
-        having  c > 1", "à chaque personne de l'annuaire de l'AX (identification_ax) doit correspondre AU PLUS UNE personne de notre annuaire (auth_user_md5) -> si ce n'est pas le cas il faut regarder en manuel ce qui ne va pas !");
-        
-/* each alumni has one and only one display name by default and one and only one name when we talk to him directly */
-check("SELECT u.`user_id`, u.`nom`, u.`prenom`, COUNT(n.`display`) AS c
-        FROM `auth_user_md5` AS u
-        LEFT JOIN `profile_names_display` AS n ON(u.`user_id` = n.`user_id` AND FIND_IN_SET(n.`reason`, 'default'))
-        GROUP BY u.`user_id`
-        HAVING c != 1", "chaque personne doit avoir un et un seul nom par défaut");
-check("SELECT u.`user_id`, u.`nom`, u.`prenom`, COUNT(n.`display`) AS c
-        FROM `auth_user_md5` AS u
-        LEFT JOIN `profile_names_display` AS n ON(u.`user_id` = n.`user_id` AND FIND_IN_SET(n.`reason`, 'yourself'))
-        GROUP BY u.`user_id`
-        HAVING c != 1", "chaque personne doit avoir un et un seul nom quand on lui parle");
-
-/* no alumni is allowed to have empty names */
-check("SELECT u.`user_id`, u.`nom`, u.`prenom`
-        FROM `auth_user_md5` AS u
-        INNER JOIN `profile_names_search` AS n USING(`user_id`)
-        WHERE n.`search_name` = ''", "liste des personnes qui ont un de leur nom de recherche vide");
-        
-/* verifie qu'il n'y a pas d'utilisateurs ayant un compte Google Apps désactivé et une redirection encore active vers Google Apps */
-check("SELECT  a.alias, g.g_status, u.mail_storage
-         FROM  auth_user_md5 AS u
-   INNER JOIN  aliases AS a ON (a.id = u.user_id AND a.type = 'a_vie')
-   INNER JOIN  gapps_accounts AS g ON (g.l_userid = u.user_id)
-        WHERE  FIND_IN_SET('googleapps', u.mail_storage) > 0 AND g.g_status != 'active'",
-      "utilisateurs ayant une redirection vers Google Apps alors que leur compte GApps n'est pas actif");
+      "Donne la liste des homonymes qui ont un alias égal à leur loginbis depuis plus d'un mois, il est temps de supprimer leur alias.");
 
 // vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:
 ?>