From b23341e886e8ae84a4e9945d78745964a5961444 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Mon, 21 Mar 2005 19:53:50 +0000 Subject: [PATCH] Fix: Suppression de nombreux warnings en mode php error_reporting E_ALL --- htdocs/index.php | 2 +- htdocs/main.inc.php | 7 +++-- htdocs/master.inc.php | 56 +++++++++++++++++++------------------- htdocs/translate.class.php | 8 +++--- htdocs/user.class.php | 51 ++++++++++++++++++---------------- 5 files changed, 64 insertions(+), 60 deletions(-) diff --git a/htdocs/index.php b/htdocs/index.php index da147c99a7c..506022ef12f 100644 --- a/htdocs/index.php +++ b/htdocs/index.php @@ -80,7 +80,7 @@ print ""; $db->close(); -llxFooter("Dernière modification $Date$ révision $Revision$"); +llxFooter('$Date$ - $Revision$'); ?> diff --git a/htdocs/main.inc.php b/htdocs/main.inc.php index 495f24a84cb..fd863681ed7 100644 --- a/htdocs/main.inc.php +++ b/htdocs/main.inc.php @@ -24,7 +24,8 @@ * */ -/** \file htdocs/main.inc.php +/** + \file htdocs/main.inc.php \brief Fichier de formatage générique des ecrans Dolibarr \version $Revision$ */ @@ -115,7 +116,7 @@ if (defined("MAIN_NOT_INSTALLED")) function top_menu($head, $title="", $target="") { - global $user, $conf, $langs; + global $user, $conf, $langs, $db; print ''; print "\n"; @@ -219,7 +220,7 @@ function top_menu($head, $title="", $target="") function left_menu($menu_array, $help_url='', $form_search='') { - global $user, $conf, $langs; + global $user, $conf, $langs, $db; print '
'."\n"; diff --git a/htdocs/master.inc.php b/htdocs/master.inc.php index f6c65d2dfd0..4717600ba6c 100644 --- a/htdocs/master.inc.php +++ b/htdocs/master.inc.php @@ -112,7 +112,7 @@ $user = new User($db); // Pour utiliser une autre version de fpdf, définir la constante FPDF_PATH -if (! defined(FPDF_PATH)) { define('FPDF_PATH',DOL_DOCUMENT_ROOT .'/includes/fpdf/fpdf152/'); } +if (! defined('FPDF_PATH')) { define('FPDF_PATH',DOL_DOCUMENT_ROOT .'/includes/fpdf/fpdf152/'); } define('FPDF_FONTPATH', FPDF_PATH . 'font/'); // \todo mettre cet include uniquement sur les éléments qui manipulent du PDF require_once(FPDF_PATH . "fpdf.php"); @@ -125,7 +125,7 @@ $sql = "SELECT name, value FROM ".MAIN_DB_PREFIX."const"; $result = $db->query($sql); if ($result) { - $numr = $db->num_rows(); + $numr = $db->num_rows($result); $i = 0; while ($i < $numr) @@ -135,18 +135,18 @@ if ($result) $i++; } } - +$db->free($result); /* * Positionne les gestionnaires de menu */ -if (! defined(MAIN_MENU_BARRETOP)) +if (! defined('MAIN_MENU_BARRETOP')) { - define("MAIN_MENU_BARRETOP","default.php"); + define('MAIN_MENU_BARRETOP',"default.php"); } -if (! defined(MAIN_MENU_BARRELEFT)) +if (! defined('MAIN_MENU_BARRELEFT')) { - define("MAIN_MENU_BARRELEFT","default.php"); + define('MAIN_MENU_BARRELEFT',"default.php"); } $conf->top_menu=MAIN_MENU_BARRETOP; $conf->left_menu=MAIN_MENU_BARRELEFT; @@ -155,9 +155,9 @@ $conf->left_menu=MAIN_MENU_BARRELEFT; * Positionne le langage et localisation dans $conf->langage * et charge l'objet de traduction */ -if (! defined(MAIN_LANG_DEFAULT)) +if (! defined('MAIN_LANG_DEFAULT')) { - define(MAIN_LANG_DEFAULT,"fr_FR"); + define('MAIN_LANG_DEFAULT',"fr_FR"); } $conf->langage=MAIN_LANG_DEFAULT; @@ -178,8 +178,8 @@ $conf->users->dir_output=DOL_DATA_ROOT."/users"; * Activation des modules * et inclusion de librairies dépendantes */ -define(MAIN_MODULE_BOOKMARK4U,1); -if (defined("MAIN_MODULE_BOOKMARK4U")) +define('MAIN_MODULE_BOOKMARK4U',1); +if (defined('MAIN_MODULE_BOOKMARK4U')) { $conf->bookmark4u->enabled=MAIN_MODULE_BOOKMARK4U; } @@ -210,7 +210,7 @@ if (defined("MAIN_MODULE_SOCIETE")) } $conf->societe->dir_output=DOL_DATA_ROOT."/societe"; $conf->societe->dir_images=DOL_DATA_ROOT."/societe/images"; -if (defined(SOCIETE_OUTPUTDIR) && SOCIETE_OUTPUTDIR) { $conf->societe->dir_output=SOCIETE_OUTPUTDIR; } # Pour passer outre le rep par défaut +if (defined('SOCIETE_OUTPUTDIR') && SOCIETE_OUTPUTDIR) { $conf->societe->dir_output=SOCIETE_OUTPUTDIR; } # Pour passer outre le rep par défaut if (defined("MAIN_MODULE_COMMERCIAL")) { $conf->commercial->enabled=MAIN_MODULE_COMMERCIAL; @@ -245,7 +245,7 @@ if (defined("MAIN_MODULE_FICHEINTER")) } $conf->fichinter->dir_output=DOL_DATA_ROOT."/ficheinter"; $conf->fichinter->dir_images=DOL_DATA_ROOT."/ficheinter/images"; -if (defined(FICHEINTER_OUTPUTDIR) && FICHEINTER_OUTPUTDIR) { $conf->fichinter->dir_output=FICHEINTER_OUTPUTDIR; } # Pour passer outre le rep par défaut +if (defined('FICHEINTER_OUTPUTDIR') && FICHEINTER_OUTPUTDIR) { $conf->fichinter->dir_output=FICHEINTER_OUTPUTDIR; } # Pour passer outre le rep par défaut if (defined("MAIN_MODULE_ADHERENT")) { $conf->adherent->enabled=MAIN_MODULE_ADHERENT; @@ -310,15 +310,15 @@ if (defined("MAIN_MODULE_PRELEVEMENT")) } $conf->prelevement->dir_output=DOL_DATA_ROOT."/prelevement"; $conf->prelevement->dir_images=DOL_DATA_ROOT."/prelevement/images"; -if (defined("MAIN_MODULE_WEBCALENDAR")) +if (defined('MAIN_MODULE_WEBCALENDAR')) { $conf->webcal->enabled=MAIN_MODULE_WEBCALENDAR; - $conf->webcal->db->type=defined(PHPWEBCALENDAR_TYPE)?PHPWEBCALENDAR_TYPE:'mysql'; - $conf->webcal->db->host=PHPWEBCALENDAR_HOST; - $conf->webcal->db->user=PHPWEBCALENDAR_USER; - $conf->webcal->db->pass=PHPWEBCALENDAR_PASS; - $conf->webcal->db->name=PHPWEBCALENDAR_DBNAME; - $conf->webcal->syncro=PHPWEBCALENDAR_SYNCRO; + $conf->webcal->db->type=defined('PHPWEBCALENDAR_TYPE')?PHPWEBCALENDAR_TYPE:'mysql'; + $conf->webcal->db->host=defined('PHPWEBCALENDAR_HOST')?PHPWEBCALENDAR_HOST:''; + $conf->webcal->db->user=defined('PHPWEBCALENDAR_USER')?PHPWEBCALENDAR_USER:''; + $conf->webcal->db->pass=defined('PHPWEBCALENDAR_PASS')?PHPWEBCALENDAR_PASS:''; + $conf->webcal->db->name=defined('PHPWEBCALENDAR_DBNAME')?PHPWEBCALENDAR_DBNAME:''; + $conf->webcal->syncro=defined('PHPWEBCALENDAR_SYNCRO')?PHPWEBCALENDAR_SYNCRO:''; } if (defined("MAIN_MODULE_FACTURE")) { @@ -327,7 +327,7 @@ if (defined("MAIN_MODULE_FACTURE")) } $conf->facture->dir_output=DOL_DATA_ROOT."/facture"; $conf->facture->dir_images=DOL_DATA_ROOT."/facture/images"; -if (defined(FAC_OUTPUTDIR) && FAC_OUTPUTDIR) { $conf->facture->dir_output=FAC_OUTPUTDIR; } # Pour passer outre le rep par défaut +if (defined('FAC_OUTPUTDIR') && FAC_OUTPUTDIR) { $conf->facture->dir_output=FAC_OUTPUTDIR; } # Pour passer outre le rep par défaut if (defined("MAIN_MODULE_PROPALE")) { $conf->propal->enabled=MAIN_MODULE_PROPALE; @@ -339,7 +339,7 @@ if (defined("MAIN_MODULE_PROPALE")) } $conf->propal->dir_output=DOL_DATA_ROOT."/propale"; $conf->propal->dir_images=DOL_DATA_ROOT."/propale/images"; -if (defined(PROPALE_OUTPUTDIR) && PROPALE_OUTPUTDIR) { $conf->propal->dir_output=PROPALE_OUTPUTDIR; } # Pour passer outre le rep par défaut +if (defined('PROPALE_OUTPUTDIR') && PROPALE_OUTPUTDIR) { $conf->propal->dir_output=PROPALE_OUTPUTDIR; } # Pour passer outre le rep par défaut @@ -357,7 +357,7 @@ else { * Option du module Compta: Defini le mode de calcul des etats comptables (CA,...) */ $conf->compta->mode = 'RECETTES-DEPENSES'; // Par défaut -if (defined("COMPTA_MODE") && COMPTA_MODE) { +if (defined('COMPTA_MODE') && COMPTA_MODE) { // Peut etre 'RECETTES-DEPENSES' ou 'CREANCES-DETTES' $conf->compta->mode = COMPTA_MODE; } @@ -371,7 +371,7 @@ $conf->compta->tva=1; /* * Option du module Facture */ -if (defined("FACTURE_TVAOPTION") && FACTURE_TVAOPTION == 'franchise') { +if (defined('FACTURE_TVAOPTION') && FACTURE_TVAOPTION == 'franchise') { $conf->defaulttx='0'; // Taux par défaut des factures clients } else { @@ -381,7 +381,7 @@ else { /* * SIZE_LISTE_LIMIT : constante de taille maximale des listes */ -if (defined("SIZE_LISTE_LIMIT")) +if (defined('SIZE_LISTE_LIMIT')) { $conf->liste_limit=SIZE_LISTE_LIMIT; } @@ -389,12 +389,12 @@ else { $conf->liste_limit=20; } -if ($user->limite_liste > 0) +if ($user->liste_limit > 0) { - $conf->liste_limit = $user->limite_liste; + $conf->liste_limit = $user->liste_limit; } -if (defined("MAIN_THEME")) +if (defined('MAIN_THEME')) { $conf->theme=MAIN_THEME; $conf->css = "theme/".$conf->theme."/".$conf->theme.".css"; diff --git a/htdocs/translate.class.php b/htdocs/translate.class.php index 0837acc464b..3c32159e486 100644 --- a/htdocs/translate.class.php +++ b/htdocs/translate.class.php @@ -60,7 +60,7 @@ class Translate { function Load($domain,$alt=0) { - if ($this->tab_loaded[$domain]) { return; } // Le fichier de ce domaine est deja chargé + if (isset($this->tab_loaded[$domain]) && $this->tab_loaded[$domain]) { return; } // Le fichier de ce domaine est deja chargé // Repertoire de traduction $scandir = $this->dir."/".$this->defaultlang; @@ -82,7 +82,7 @@ class Translate { if ($ligne[0] != "\n" && $ligne[0] != " " && $ligne[0] != "#") { $tab=split('=',$ligne,2); //print "Domain=$domain, found a string for $tab[0] with value $tab[1]
"; - if (! $this->tab_translate[$tab[0]]) $this->tab_translate[$tab[0]]=trim($tab[1]); + if (! isset($this->tab_translate[$tab[0]])) $this->tab_translate[$tab[0]]=trim($tab[1]); } } fclose($fp); @@ -137,7 +137,7 @@ class Translate { */ function transnoentities($str, $param1='', $param2='', $param3='') { - if ($this->tab_translate[$str]) { + if (isset($this->tab_translate[$str]) && $this->tab_translate[$str]) { // Si la traduction est disponible return sprintf($this->tab_translate[$str],$param1,$param2,$param3); } @@ -188,7 +188,7 @@ class Translate { if (! $charset) $charset="iso-8859-1"; //header("Content-Type: text/html; charset=$charset"); - $texte .= "\n\n"; + $texte = "\n\n"; return $texte; } diff --git a/htdocs/user.class.php b/htdocs/user.class.php index bb5a81c348c..155b3269267 100644 --- a/htdocs/user.class.php +++ b/htdocs/user.class.php @@ -78,7 +78,7 @@ class User $this->id = $id; // Preference utilisateur - $this->userpreflimite_liste = 0; + $this->liste_limit = 0; $this->clicktodial_enabled = 0; $this->all_permissions_are_loaded = 0; @@ -95,7 +95,7 @@ class User function fetch($login='') { - $sql = "SELECT u.rowid, u.name, u.firstname, u.email, u.code, u.admin, u.login, u.pass, u.webcal_login, u.note"; + $sql = "SELECT u.rowid, u.name, u.firstname, u.email, u.code, u.admin, u.login, u.pass, u.webcal_login, u.note, u.fk_societe, u.fk_socpeople"; $sql .= ", ".$this->db->pdate("u.datec")." as datec, ".$this->db->pdate("u.tms")." as datem"; $sql .= " FROM ".MAIN_DB_PREFIX."user as u"; if ($this->id) @@ -133,9 +133,8 @@ class User $this->webcal_login = $obj->webcal_login; $this->societe_id = $obj->fk_societe; - $this->egroupware_id = $obj->egroupware_id; } - $this->db->free(); + $this->db->free($result); } else @@ -143,26 +142,30 @@ class User dolibarr_print_error($this->db); } - $sql = "SELECT param, value FROM ".MAIN_DB_PREFIX."user_param"; - $sql .= " WHERE fk_user = ".$this->id; - $sql .= " AND page = '".$_SERVER["SCRIPT_URL"]."'"; - - if ( $this->db->query($sql) ); - { - $num = $this->db->num_rows(); - $i = 0; - $page_param_url = ''; - $this->page_param = array(); - while ($i < $num) - { - $obj = $this->db->fetch_object(); - $this->page_param[$obj->param] = $obj->value; - $page_param_url .= $obj->param . "=".$obj->value."&"; - $i++; - } - $this->page_param_url = $page_param_url; - } - + if (isset($_SERVER['SCRIPT_URL'])) { + // \todo $_SERVER['SCRIPT_URL'] n'existe pas sous tout os/server web + $sql = "SELECT param, value FROM ".MAIN_DB_PREFIX."user_param"; + $sql .= " WHERE fk_user = ".$this->id; + $sql .= " AND page = '".$_SERVER['SCRIPT_URL']."'"; + + $result=$this->db->query($sql); + if ($result); + { + $num = $this->db->num_rows($result); + $i = 0; + $page_param_url = ''; + $this->page_param = array(); + while ($i < $num) + { + $obj = $this->db->fetch_object($result); + $this->page_param[$obj->param] = $obj->value; + $page_param_url .= $obj->param . "=".$obj->value."&"; + $i++; + } + $this->page_param_url = $page_param_url; + $this->db->free($result); + } + } } /**