Qual: Correction de mauvaises habitudes de codage.
This commit is contained in:
parent
1a44d0dbbc
commit
f9f91f1e43
@ -53,6 +53,8 @@ class DolibarrModules
|
|||||||
*/
|
*/
|
||||||
function _init($array_sql)
|
function _init($array_sql)
|
||||||
{
|
{
|
||||||
|
global $langs;
|
||||||
|
|
||||||
// Insère les constantes
|
// Insère les constantes
|
||||||
$err = 0;
|
$err = 0;
|
||||||
$sql_del = "DELETE FROM ".MAIN_DB_PREFIX."const WHERE name = '".$this->const_name."';";
|
$sql_del = "DELETE FROM ".MAIN_DB_PREFIX."const WHERE name = '".$this->const_name."';";
|
||||||
@ -73,13 +75,14 @@ class DolibarrModules
|
|||||||
|
|
||||||
$sql = "SELECT count(*) FROM ".MAIN_DB_PREFIX."boxes_def WHERE name ='".$titre."'";
|
$sql = "SELECT count(*) FROM ".MAIN_DB_PREFIX."boxes_def WHERE name ='".$titre."'";
|
||||||
|
|
||||||
if ( $this->db->query($sql) )
|
$result=$this->db->query($sql);
|
||||||
|
if ($result)
|
||||||
{
|
{
|
||||||
$row = $this->db->fetch_row($sql);
|
$row = $this->db->fetch_row($result);
|
||||||
if ($row[0] == 0)
|
if ($row[0] == 0)
|
||||||
{
|
{
|
||||||
$sql = "INSERT INTO ".MAIN_DB_PREFIX."boxes_def (name, file) VALUES ('".$titre."','".$file."')";
|
$sql = "INSERT INTO ".MAIN_DB_PREFIX."boxes_def (name, file) VALUES ('".$titre."','".$file."')";
|
||||||
if (! $this->db->query($sql) )
|
if (! $this->db->query($sql))
|
||||||
{
|
{
|
||||||
$err++;
|
$err++;
|
||||||
}
|
}
|
||||||
@ -102,9 +105,10 @@ class DolibarrModules
|
|||||||
|
|
||||||
$sql = "SELECT count(*) FROM ".MAIN_DB_PREFIX."const WHERE name ='".$name."'";
|
$sql = "SELECT count(*) FROM ".MAIN_DB_PREFIX."const WHERE name ='".$name."'";
|
||||||
|
|
||||||
if ( $this->db->query($sql) )
|
$result=$this->db->query($sql);
|
||||||
|
if ($result)
|
||||||
{
|
{
|
||||||
$row = $this->db->fetch_row($sql);
|
$row = $this->db->fetch_row($result);
|
||||||
|
|
||||||
if ($row[0] == 0)
|
if ($row[0] == 0)
|
||||||
{
|
{
|
||||||
@ -188,8 +192,8 @@ class DolibarrModules
|
|||||||
umask(0);
|
umask(0);
|
||||||
if (! @mkdir($dir, 0755))
|
if (! @mkdir($dir, 0755))
|
||||||
{
|
{
|
||||||
$this->error = "Erreur: Le répertoire '$dir' n'existe pas et Dolibarr n'a pu le créer.";
|
$this->error = $langs->trans("ErrorCanNotCreateDir",$dir);
|
||||||
dolibarr_syslog("Erreur: impossible de créer $dir");
|
dolibarr_syslog("DolibarrModules::_init error");
|
||||||
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user