Fix: Corrections probl�mes sur le programme d'install. Ajout d'une aide sur chaque message d'erreur pour guider au diagnostique.

htdocs/install/etape1.php

@@ -151,7 +151,6 @@ if ($_POST["action"] == "set")
         $dir[7] = "$main_data_dir/rsscache";
         $dir[8] = "$main_data_dir/logo";
 
-
         if (! is_dir($main_dir))
         {
             print "<tr><td>";
@@ -208,6 +207,7 @@ if ($_POST["action"] == "set")
         }
     }
 
+
     /*
      * Base de données
      *
@@ -217,50 +217,39 @@ if ($_POST["action"] == "set")
     {
         include_once($dolibarr_main_document_root . "/conf/conf.class.php");
 
-      // TODO
+        $conf = new Conf();
-      // Il y a encore des pb ds la procédure d'install qui ne passe dans pas tous les cas
+        $conf->db->type = trim($dolibarr_main_db_type);
-      // (exemple, rien n'existe et on veut créer une base avec un compte admin != root)
+        $conf->db->host = trim($dolibarr_main_db_host);
-      // L'algorithme ne semble pas adapté à tous les cas, il devrait etre remplacé par le suivant:
+        $conf->db->name = trim($dolibarr_main_db_name);
-      //
+        $conf->db->user = trim($dolibarr_main_db_user);
-      // On essaie l'accès par le user admin dolibarr
+        $conf->db->pass = trim($dolibarr_main_db_pass);
-      //   si accès serveur ok et accès base ok, tout est ok, on ne va pas plus loin, on a même pas utilisé le compte root.
-      //   si accès serveur ok et accès base ko, warning 1
-      //   si accès serveur ko, warning 2
-      // Si warning, on essai de se connecter au serveur via le super user root
-      //   Si connexion serveur par root ok et accès base ko, on la créée
-      //     Si création ok, on y accède
-      //     Si création ko, erreur
-      //   Si connexion serveur par root ok et si accès base ok,
-      //     si compte admin existe deja et db_create_user positionné, on ajoute les droits,
-      //     si compte admin existe deja et db_create_user non positionné, erreur compte admin incorrect "Le compte admin indiqué existe mais n'a pas les droits sur la base. Veuillez cocher pour les ajouter"
-      //     si compte admin n'existe pas deja et db_create_user positionné, on crée le compte
-      //     si compte admin n'existe pas deja et db_create_user non positionné, erreur compte admin inexistant "Veuillez cocher pour le créer"
 
-      // Si creation utilisateur admin demandée, on le crée
+        $userroot=isset($_POST["db_user_root"])?$_POST["db_user_root"]:"";
+        $passroot=isset($_POST["db_pass_root"])?$_POST["db_pass_root"]:"";
+
+        $ok=0;
+
+
+        /*
+         * Si creation utilisateur admin demandée, on le crée
+         */
         if (isset($_POST["db_create_user"]) && $_POST["db_create_user"] == "on")
         {
             dolibarr_syslog ("Creation de l'utilisateur : ".$dolibarr_main_db_user);
 
-	  if($choix == 1)
+            if ($choix == 1)     //choix 1=mysql
-	    { //choix 1=mysql
+            {
-	      
-	      $conf = new Conf();
-	      $conf->db->host = $dolibarr_main_db_host;
-	      $conf->db->name = $dolibarr_main_db_name;
-	      $conf->db->name = $dolibarr_main_db_type;
-	      $conf->db->user = isset($_POST["db_user_root"])?$_POST["db_user_root"]:"";
-	      $conf->db->pass = isset($_POST["db_pass_root"])?$_POST["db_pass_root"]:"";
-	      
                 //print $conf->db->host." , ".$conf->db->name." , ".$conf->db->user." , ".$conf->db->pass;
+                $db = new DoliDb($conf->db->type,$conf->db->host,$userroot,$passroot,'mysql');
 
-	      $db = new DoliDb();
+                if ($db->connected) 
-	      
+                {
                     $sql = "INSERT INTO user ";
-	      $sql .= "(Host,User,password,elect_priv,Insert_priv,Update_priv,Delete_priv,Create_priv,Drop_priv,Index_Priv,Alter_priv)";
+                    $sql .= "(Host,User,password,Select_priv,Insert_priv,Update_priv,Delete_priv,Create_priv,Drop_priv,Index_Priv,Alter_priv)";
                     $sql .= " VALUES ('$dolibarr_main_db_host','$dolibarr_main_db_user',password('$dolibarr_main_db_pass')";
-	      $sql .= ",'Y','Y','Y','Y','Y',Y','Y','Y');";
+                    $sql .= ",'Y','Y','Y','Y','Y','Y','Y','Y');";
     
-	      print $sql;
+                    //print "$sql<br>\n";
     
                     $db->query($sql);
     
@@ -269,34 +258,63 @@ if ($_POST["action"] == "set")
                     $sql .= " VALUES ('$dolibarr_main_db_host','$dolibarr_main_db_name','$dolibarr_main_db_user'";
                     $sql .= ",'Y','Y','Y','Y','Y','Y','Y','Y');";
     
-	      print $sql;
+                    //print "$sql<br>\n";
 
                     if ($db->query($sql))
                     {
                         dolibarr_syslog("flush privileges");
-		  $db->query("flush privileges");
+                        $db->query("FLUSH Privileges;");
 
-		  print "<tr><td>Création de l'utilisateur : $dolibarr_main_db_user</td><td>OK</td></tr>";
+                        print '<tr><td>';
+                        print $langs->trans("UserCreation").' : ';
+                        print $dolibarr_main_db_user;
+                        print '</td>';
+                        print '<td>'.$langs->trans("OK").'</td></tr>';
                     }
                     else
                     {
-		  if ($db->errno() == 1062)
+                        if ($db->errno() == DB_ERROR_RECORD_ALREADY_EXISTS)
                         {
                             dolibarr_syslog("Utilisateur deja existant");
-		      print "<tr><td>Création de l'utilisateur : $dolibarr_main_db_user</td><td>Deja existant</td></tr>";
+                            print '<tr><td>';
+                            print $langs->trans("UserCreation").' : ';
+                            print $dolibarr_main_db_user;
+                            print '</td>';
+                            print "<td>Deja existant</td></tr>";
                         }
                         else
                         {
                             dolibarr_syslog("impossible de creer l'utilisateur");
-		      print "<tr><td>Création de l'utilisateur : $dolibarr_main_db_user</td><td>ERREUR ".$db->error()."</td></tr>";
+                            print '<tr><td>';
+                            print $langs->trans("UserCreation").' : ';
+                            print $dolibarr_main_db_user;
+                            print '</td>';
+                            print "<td>".$langs->trans("Error").' '.$db->error()."</td></tr>";
                         }
                     }
     
                     $db->close();
                 }
-	  else 
+                else {
-	    {  //choix 2=postgresql		
+                    print '<tr><td>';
+                    print $langs->trans("UserCreation").' : ';
+                    print $dolibarr_main_db_user;
+                    print '</td>';
+                    print '<td>'.$langs->trans("Error").'</td>';
+                    print '</tr>';
         
+                    // Affiche aide diagnostique
+                    print '<tr><td colspan="2"><br>Vous avez demandé la création du login Dolibarr ('.$dolibarr_main_db_user.') mais pour cela ';
+                    print 'Dolibarr doit se connecter sur le serveur ('.$dolibarr_main_db_host.') via le super utilisateur ('.$userroot.'), mot de passe ('.$passroot.'). ';
+                    print 'La connexion ayant échoué, les paramètres du serveur ou du super utilisateur sont peut-etre incorrects.<br>';
+                    print 'Revenez en arrière pour corriger les paramètres.<br>';
+                    print '</td></tr>';
+
+                    $ok=-1;
+                }
+            }
+            else        //choix 2=postgresql
+            {  
                 $nom = $dolibarr_main_db_user;
                 $con=pg_connect("host=localhost dbname=dolibarr user=postgres");
                 $query_str = "create user \"$nom\" with password '".$dolibarr_main_db_pass."';";
@@ -304,39 +322,144 @@ if ($_POST["action"] == "set")
                 $ret = pg_query($con,$query_str);
 
                 if ($ret)
-		print "<tr><td>Création de l'utilisateur:\"$nom\"</td><td>OK</td></tr>";
+                {
+                    print '<tr><td>';
+                    print $langs->trans("UserCreation").' : ';
+                    print $dolibarr_main_db_user;
+                    print '</td>';
+                    print '<td>'.$langs->trans("OK").'</td>';
+                    print '</tr>';
+                }
                 else
-		print "<tr><td>Création de l'utilisateur:\"$nom\"</td><td>ERREUR</td></tr>";			
+                {
-	      
+                    print '<tr><td>';
+                    print $langs->trans("UserCreation").' : ';
+                    print $dolibarr_main_db_user;
+                    print '</td>';
+                    print '<td>'.$langs->trans("Error").'</td>';
+                    print '</tr>';
+                }
             }
 
+        }   // Fin si "creation utilisateur"
+        
+        
+        /*
+         * Si creation database demandée, on la crée
+         */
+        if (isset($_POST["db_create_database"]) && $_POST["db_create_database"] == "on")
+        {
+            dolibarr_syslog ("Creation de la base : ".$dolibarr_main_db_name);
+
+            $db = new DoliDb($conf->db->type,$conf->db->host,$userroot,$passroot,$conf->db->name);
+
+            if ($db->connected) 
+            {
+                if ($db->create_db($dolibarr_main_db_pass))
+                {
+                    print '<tr><td>';
+                    print $langs->trans("DatabaseCreation").' : ';
+                    print $dolibarr_main_db_name;
+                    print '</td>';
+                    print "<td>".$langs->trans("OK")."</td></tr>";
                 }
+                else
+                {
+                    print '<tr><td>';
+                    print $langs->trans("DatabaseCreation").' : ';
+                    print $dolibarr_main_db_name;
+                    print '</td>';
+                    print "<td>".$langs->trans("Error")."</td></tr>";
 
-      // Tentative accès serveur et base par le user admin dolibarr
+                    // Affiche aide diagnostique
+                    print '<tr><td colspan="2"><br>La création de la base Dolibarr ('.$dolibarr_main_db_name.') a échoué.';
+                    print 'Si la base existe déjà, revenez en arrière et désactiver l\'option "Créer la base de donnée".<br>';
+                    print '</td></tr>';
                     
-      $conf = new Conf();
+                    $ok=-1;
-      $conf->db->host = $dolibarr_main_db_host;
+                }
-      $conf->db->name = $dolibarr_main_db_name;
+                $db->close();
-      $conf->db->user = $dolibarr_main_db_user;
+            }
-      $conf->db->pass = $dolibarr_main_db_pass;
+            else {
-      //print $conf->db->host.",".$conf->db->name.",".$conf->db->user.",".$conf->db->pass;
+                print '<tr><td>';
+                print $langs->trans("DatabaseCreation").' : ';
+                print $dolibarr_main_db_name;
+                print '</td>';
+                print '<td>'.$langs->trans("Error").'</td>';
+                print '</tr>';
     
-      $db = new DoliDb();
+                // Affiche aide diagnostique
-      $ok = 1;
+                print '<tr><td colspan="2"><br>Vous avez demandé la création de la base Dolibarr ('.$dolibarr_main_db_name.') mais pour cela ';
+                print 'Dolibarr doit se connecter sur le serveur ('.$dolibarr_main_db_host.') via le super utilisateur ('.$userroot.'), mot de passe ('.$passroot.'). ';
+                print 'La connexion ayant échoué, les paramètres du serveur ou du super utilisateur sont peut-etre incorrects.<br>';
+                print 'Revenez en arrière pour corriger les paramètres.<br>';
+                print '</td></tr>';
+
+                $ok=-1;
+            }
+        }   // Fin si "creation database"
+
+
+        /*
+         * On essaie l'accès par le user admin dolibarr
+         */
+        if ($ok == 0)
+        {
+
+            $db = new DoliDb($conf->db->type,$conf->db->host,$conf->db->user,$conf->db->pass,$conf->db->name);
     
             if ($db->connected == 1)
             {
-	  dolibarr_syslog("la connexion au serveur est reussie");
+                //   si accès serveur ok et accès base ok, tout est ok, on ne va pas plus loin, on a même pas utilisé le compte root.
+                if ($db->database_selected == 1)
+                {
+                    dolibarr_syslog("la connexion au serveur par le user dolibarr est reussie");
                     print "<tr><td>";
                     print $langs->trans("ServerConnection")." : ";
                     print $dolibarr_main_db_host;
                     print "</td><td>";
                     print $langs->trans("OK");
                     print "</td></tr>";
+    
+                    dolibarr_syslog("la connexion a la database par le user dolibarr est reussie");
+                    print "<tr><td>";
+                    print $langs->trans("DatabaseConnection")." : ";
+                    print $dolibarr_main_db_name;
+                    print "</td><td>";
+                    print $langs->trans("OK");
+                    print "</td></tr>";
+    
+                    $ok = 1;
                 }
                 else
                 {
-	  dolibarr_syslog("la connection au serveur est rate");
+                    dolibarr_syslog("la connection au serveur par le user dolibarr est reussie");
+                    print "<tr><td>";
+                    print $langs->trans("ServerConnection")." : ";
+                    print $dolibarr_main_db_host;
+                    print "</td><td>";
+                    print $langs->trans("OK");
+                    print "</td></tr>";
+    
+                    dolibarr_syslog("la connexion a la database par le user dolibarr a échoué");
+                    print "<tr><td>";
+                    print $langs->trans("DatabaseConnection")." : ";
+                    print $dolibarr_main_db_name;
+                    print "</td><td>";
+                    print $langs->trans("Error");
+                    print "</td></tr>";
+    
+                    // Affiche aide diagnostique
+                    print '<tr><td colspan="2"><br>Vérifier que le nom de base ('.$dolibarr_main_db_name.') est correct. ';
+                    print 'Si ce nom est correct et que cette base n\'existe pas déjà, vous devez cocher l\'option "Créer la base de donnée".<br>';
+                    print 'Revenez en arrière pour corriger les paramètres.<br>';
+                    print '</td></tr>';
+
+                    $ok = -1;
+                }
+            }
+            else {
+                dolibarr_syslog("la connection au serveur par le user dolibarr est rate");
                 print "<tr><td>";
                 print $langs->trans("ServerConnection")." : ";
                 print $dolibarr_main_db_host;
@@ -344,97 +467,21 @@ if ($_POST["action"] == "set")
                 print $langs->trans("Error");
                 print "</td></tr>";
     
-	  print '<tr><td colspan="2">Un des paramètres saisis est peut-être incorrect:<br>';
+                // Affiche aide diagnostique
-	  print 'host='.$conf->db->host.', name='.$conf->db->name.', user='.$conf->db->user.', pass='.$conf->db->pass.'<br>';
+                print '<tr><td colspan="2"><br>Le serveur ('.$conf->db->host.'), nom de base ('.$conf->db->name.'), login ('.$conf->db->user.'), ou mot de passe ('.$conf->db->pass.') de la base de donnée est peut-être incorrect.<br>';
-      print 'Revenez en arrière pour corriger les paramètres.';
+                print 'Si le login n\'existe pas encore, vous devez cocher l\'option "Créer l\'utilisateur".<br>';
+                print 'Revenez en arrière pour corriger les paramètres.<br>';
                 print '</td></tr>';
     
-	  $ok = 0;
+                $ok = -1;
-	}
-      
-      if ($ok)
-	{
-	  if($db->database_selected == 1)
-	    {
-	      //
-	      // Connexion base existante
-	      // 
-	      dolibarr_syslog("la connexion a la database est reussie");
-	      print "<tr><td>";
-      	  print $langs->trans("DatabaseConnection")." : ";
-          print $dolibarr_main_db_name;
-          print "</td><td>";
-          print $langs->trans("OK");
-          print "</td></tr>";
-	      $ok = 1 ;
-	    }
-	  else
-	    {
-	      //
-	      // Création de la base
-	      //
-	      dolibarr_syslog("la connexion a la database est rate");
-	      print "<tr><td>";
-	      print "Echec de connexion à la base : ";
-	      print $dolibarr_main_db_name;
-	      print "</td><td>";
-	      print $langs->trans("Warning");
-	      print "</td></tr>";
-
-	      print '<tr><td colspan="2">';
-	      print "On tente de créer la base : ";
-	      print $dolibarr_main_db_name;
-	      print '</td></tr>';
-	      
-	      $conf = new Conf();
-	      $conf->db->host = $dolibarr_main_db_host;
-	      $conf->db->name = $dolibarr_main_db_name;
-	      $conf->db->user = isset($_POST["db_user_root"])?$_POST["db_user_root"]:"";
-	      $conf->db->pass = isset($_POST["db_pass_root"])?$_POST["db_pass_root"]:"";
-	      
-	      $dbt = new DoliDb();  // Ne pas prendre $db comme nom de variable car dejà pris pour connexion précédente et bug mémoire sur php windows dans ce cas
-	      
-	      if ($ok)
-		{
-		  if ($dbt->connected == 1)
-		    {
-		      dolibarr_syslog("la connexion au serveur avec l'utilisateur root reussi");
-		      print "<tr><td>Connexion au serveur : $dolibarr_main_db_host, avec l'utilisateur : ".$_POST["db_user_root"]."</td><td>".$langs->trans("OK")."</td></tr>";
-		    }
-		  else
-		    {
-		      dolibarr_syslog("la connexion au serveur avec l'utilisateur root rate");
-		      if (! $conf->db->user) {
-		        print "<tr><td colspan='2'>".$langs->trans("Error").": ";
-		        print "Le super utilisateur de la base est requis pour créer la base.";
-		        print "Créer la base en manuel ou revenez en arrière pour saisir le super utilisateur";
-		        print "</tr>";
-		      } else {
-		        print "<tr><td>Connexion au serveur : $dolibarr_main_db_host avec l'utilisateur : ".$_POST["db_user_root"]."</td><td>".$langs->trans("Error")."</td></tr>";
-              }
-		      $ok = 0;
             }
         }
 
-     
-	      if ($ok)
-		{
-		  if ($dbt->create_db ($dolibarr_main_db_name))
-		    {      
-		      print "<tr><td>Création de la base : $dolibarr_main_db_name</td><td>".$langs->trans("OK")."</td></tr>";
-		    }
-		  else
-		    {
-		      print "<tr><td>Création de la base : $dolibarr_main_db_name</td><td>".$langs->trans("Error")."</td></tr>";
-		      $ok = 0;
-		    }
-		}
-	    }
-	}    
     }
 }
+
 ?>
 </table>
 <?php
-pFooter($err);
+pFooter($ok==1?0:1);
 ?>

htdocs/install/etape2.php

@@ -59,11 +59,13 @@ if ($_POST["action"] == "set")
     $error=0;
 
     $conf = new Conf();// on pourrait s'en passer
+    $conf->db->type = $dolibarr_main_db_type;
     $conf->db->host = $dolibarr_main_db_host;
     $conf->db->name = $dolibarr_main_db_name;
     $conf->db->user = $dolibarr_main_db_user;
     $conf->db->pass = $dolibarr_main_db_pass;
-    $db = new DoliDb();
+
+    $db = new DoliDb($conf->db->type,$conf->db->host,$conf->db->user,$conf->db->pass,$conf->db->name);
     $ok = 0;
     if ($db->connected == 1)
     {

htdocs/install/etape4.php

@@ -49,11 +49,13 @@ print '<table cellspacing="0" cellpadding="4" border="1" width="100%">';
 $error=0;
 
 $conf = new Conf();
+$conf->db->type = $dolibarr_main_db_type;
 $conf->db->host = $dolibarr_main_db_host;
 $conf->db->name = $dolibarr_main_db_name;
 $conf->db->user = $dolibarr_main_db_user;
 $conf->db->pass = $dolibarr_main_db_pass;
-$db = new DoliDb();
+
+$db = new DoliDb($conf->db->type,$conf->db->host,$conf->db->user,$conf->db->pass,$conf->db->name);
 $ok = 0;
 if ($db->ok == 1)
 {

htdocs/install/etape5.php

@@ -74,11 +74,13 @@ if ($_POST["action"] == "set")
   $error=0;
 
   $conf = new Conf();
+  $conf->db->type = $dolibarr_main_db_type;
   $conf->db->host = $dolibarr_main_db_host;
   $conf->db->name = $dolibarr_main_db_name;
   $conf->db->user = $dolibarr_main_db_user;
   $conf->db->pass = $dolibarr_main_db_pass;
-  $db = new DoliDb();
+
+  $db = new DoliDb($conf->db->type,$conf->db->host,$conf->db->user,$conf->db->pass,$conf->db->name);
   $ok = 0;
   if ($db->connected == 1)
     {

htdocs/install/fileconf.php

@@ -24,7 +24,8 @@
  *
  */
 
-/**     \file       htdocs/install/fileconf.php
+/**
+        \file       htdocs/install/fileconf.php
         \ingroup    install
         \brief      Demande les infos qui constituerons le contenu du fichier conf.php. Ce fichier sera remplie à l'étape suivante
         \version    $Revision$
@@ -154,7 +155,7 @@ $dolibarr_main_db_host = "localhost";
 <tr>
 <!-- moi-->
 <td valign="top" class="label">
-<?php echo $langs->trans("DatabaseChoice"); ?>
+<?php echo $langs->trans("DatabaseType"); ?>
 </td>
 
 <td class="label"><select name='db_type'>
@@ -190,7 +191,18 @@ Nom ou adresse ip du serveur de base de donn
 
 <td class="label" valign="top"><input type="text" name="db_name" value="<?php echo $dolibarr_main_db_name ?>"></td>
 <td class="comment">
-Nom de la base de données Dolibarr (sera créée si nécessaire)
+<?php echo $langs->trans("DatabaseName"); ?>
+</td>
+</tr>
+
+<tr>
+<td class="label" valign="top">
+<?php echo $langs->trans("CreateDatabase"); ?>
+</td>
+
+<td class="label"><input type="checkbox" name="db_create_database"></td>
+<td class="comment">
+<?php echo $langs->trans("CheckToCreateDatabase"); ?>
 </td>
 </tr>
 
@@ -221,7 +233,7 @@ Nom de la base de donn
 
 <td class="label"><input type="checkbox" name="db_create_user"></td>
 <td class="comment">
-<?php echo $langs->trans("CheckToCreate"); ?>
+<?php echo $langs->trans("CheckToCreateUser"); ?>
 </td>
 </tr>
 

htdocs/install/inc.php

@@ -95,13 +95,13 @@ function pFooter($nonext=0,$addlang=1)
 
 function dolibarr_syslog($message)
 {
+// \todo    Désactiver sous Windows (gros problème mémoire et faute de protections)
+//  if (1 == 2) {
       define_syslog_variables();
-
       openlog("dolibarr", LOG_PID | LOG_PERROR, LOG_USER);	# LOG_USER au lieu de LOG_LOCAL0 car non accepté par tous les php
-  
       syslog(LOG_WARNING, $message);
-
       closelog();
+//  }
 }
 
 ?>