Fix: Wrong path of mysqldump command. Another try to fix this.

htdocs/admin/tools/dolibarr_export.php

@@ -175,17 +175,7 @@ if ($db->label == 'MySQL')
         <?php echo $langs->trans("FullPathToMysqldumpCommand");
             if (empty($conf->global->SYSTEMTOOLS_MYSQLDUMP))
             {
-				$resql=$db->query('SHOW VARIABLES LIKE \'basedir\'');
+				$fullpathofmysqldump=$db->getPathOfDump();
-				if ($resql)
-				{
-					$liste=$db->fetch_array($resql);
-					$basedir=$liste['Value'];
-					$fullpathofmysqldump=$basedir.'bin/mysqldump';
-				}
-				else
-				{
-					$fullpathofmysqldump='/pathtomysqldump/mysqldump';
-				}
             }
             else
             {
@@ -312,17 +302,7 @@ if ($db->label == 'PostgreSQL')
         <?php echo $langs->trans("FullPathToPostgreSQLdumpCommand");
             if (empty($conf->global->SYSTEMTOOLS_POSTGRESQLDUMP))
             {
-				$resql=$db->query('SHOW data_directory');
+				$fullpathofpgdump=$db->getPathOfDump();
-				if ($resql)
-				{
-					$liste=$db->fetch_array($resql);
-					$basedir=$liste['data_directory'];
-					$fullpathofpgdump=preg_replace('/data$/','bin',$basedir).'/pg_dump';
-				}
-				else
-				{
-					$fullpathofpgdump='/pathtopgdump/pg_dump';
-				}
             }
             else
             {

htdocs/admin/tools/dolibarr_import.php

@@ -94,12 +94,16 @@ print '<br>';
 
 
 <div id="div_container_sub_exportoptions">
+<?php
+if ($db->label == 'MySQL')
+{
+?>
 	<fieldset id="mysql_options">
     <legend>Import MySql</legend>
 	<div class="formelementrow">
 	<?php
 	// Parameteres execution
-	$command='mysql';
+	$command=$db->getPathOfRestore();
 	if (preg_match("/\s/",$command)) $command=$command=escapeshellarg($command);	// Use quotes on command
 
 	$param=$dolibarr_main_db_name;
@@ -122,6 +126,9 @@ print '<br>';
 	//else print '<br><a href="'.$_SERVER["PHP_SELF"].'?showpass=0&amp;radio_dump=mysql_options">'.$langs->trans("HidePassword").'</a>';
 	?>
 	</div>
+<?php
+}
+?>
 
 	<script type="text/javascript" language="javascript">
 //<![CDATA[

htdocs/lib/databases/mssql.lib.php

@@ -961,7 +961,8 @@ class DoliDb
 		return ''; // attente d<>buggage
 	}
 
-	function getDefaultCollationDatabase(){
+	function getDefaultCollationDatabase()
+	{
 		$resql=$this->query("SELECT SERVERPROPERTY('collation')");
 		if (!$resql)
 		{
@@ -971,7 +972,8 @@ class DoliDb
 		return $liste['computed'];
 	}
 
-	function getListOfCollation(){
+	function getListOfCollation()
+	{
 		/*
 		 $resql=$this->query('SHOW COLLATION');
 		 $liste = array();
@@ -992,6 +994,15 @@ class DoliDb
 		return ''; // attente debugage
 	}
 
+	function getPathOfDump()
+	{
+
+	}
+
+	function getPathOfRestore()
+	{
+
+	}
 }
 
 ?>

htdocs/lib/databases/mysql.lib.php

@@ -1097,6 +1097,42 @@ class DoliDb
 		return $liste;
 	}
 
+	/**
+	 *	\brief		Return full path of dump program
+	 *	\return		string		Full path of dump program
+	 */
+	function getPathOfDump()
+	{
+		$fullpathofdump='/pathtomysqldump/mysqldump';
+
+		$resql=$this->query('SHOW VARIABLES LIKE \'basedir\'');
+		if ($resql)
+		{
+			$liste=$this->fetch_array($resql);
+			$basedir=$liste['Value'];
+			$fullpathofdump=$basedir.'bin/mysqldump';
+		}
+		return $fullpathofdump;
+	}
+
+	/**
+	 *	\brief		Return full path of restore program
+	 *	\return		string		Full path of restore program
+	 */
+	function getPathOfRestore()
+	{
+		$fullpathofimport='/pathtomysql/mysql';
+
+		$resql=$this->query('SHOW VARIABLES LIKE \'basedir\'');
+		if ($resql)
+		{
+			$liste=$this->fetch_array($resql);
+			$basedir=$liste['Value'];
+			$fullpathofimport=$basedir.'bin/mysql';
+		}
+		return $fullpathofimport;
+	}
+
 }
 
 ?>

htdocs/lib/databases/mysqli.lib.php

@@ -1109,6 +1109,43 @@ class DoliDb
 		}
 		return $liste;
 	}
+
+	/**
+	 *	\brief		Return full path of dump program
+	 *	\return		string		Full path of dump program
+	 */
+	function getPathOfDump()
+	{
+		$fullpathofdump='/pathtomysqldump/mysqldump';
+
+		$resql=$this->query('SHOW VARIABLES LIKE \'basedir\'');
+		if ($resql)
+		{
+			$liste=$this->fetch_array($resql);
+			$basedir=$liste['Value'];
+			$fullpathofdump=$basedir.'bin/mysqldump';
+		}
+		return $fullpathofdump;
+	}
+
+	/**
+	 *	\brief		Return full path of restore program
+	 *	\return		string		Full path of restore program
+	 */
+	function getPathOfRestore()
+	{
+		$fullpathofimport='/pathtomysql/mysql';
+
+		$resql=$this->query('SHOW VARIABLES LIKE \'basedir\'');
+		if ($resql)
+		{
+			$liste=$this->fetch_array($resql);
+			$basedir=$liste['Value'];
+			$fullpathofimport=$basedir.'bin/mysql';
+		}
+		return $fullpathofimport;
+	}
+
 }
 
 ?>

htdocs/lib/databases/pgsql.lib.php

@@ -1038,5 +1038,42 @@ class DoliDb
 		return $liste;
 	}
 
+	/**
+	 *	\brief		Return full path of dump program
+	 *	\return		string		Full path of dump program
+	 */
+	function getPathOfDump()
+	{
+		$fullpathofdump='/pathtopgdump/pg_dump';
+
+		$resql=$db->query('SHOW data_directory');
+		if ($resql)
+		{
+			$liste=$db->fetch_array($resql);
+			$basedir=$liste['data_directory'];
+			$fullpathofdump=preg_replace('/data$/','bin',$basedir).'/pg_dump';
+		}
+
+		return $fullpathofdump;
+	}
+
+	/**
+	 *	\brief		Return full path of restore program
+	 *	\return		string		Full path of restore program
+	 */
+	function getPathOfRestore()
+	{
+		$fullpathofmysqldump='/pathtomysql/mysql';
+
+		$resql=$this->query('SHOW VARIABLES LIKE \'basedir\'');
+		if ($resql)
+		{
+			$liste=$this->fetch_array($resql);
+			$basedir=$liste['Value'];
+			$fullpathofmysqldump=$basedir.'bin/mysql';
+		}
+		return $fullpathofmysqldump;
+	}
+
 }
 ?>