From db74ae5c7e5e39c5aa9fcb5f1d5477809f936845 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Fri, 10 Oct 2008 23:05:27 +0000
Subject: [PATCH] Qual: Removed duplicated code
---
.../menus/barre_left/auguria_backoffice.php | 28 ++++++---------
.../menus/barre_left/auguria_frontoffice.php | 17 ++++-----
.../menus/barre_left/eldy_backoffice.php | 5 ---
.../menus/barre_left/eldy_frontoffice.php | 5 ---
htdocs/includes/menus/barre_left/rodolphe.php | 35 ++++++++-----------
.../menus/barre_top/auguria_backoffice.php | 6 ----
.../menus/barre_top/auguria_frontoffice.php | 7 ----
.../menus/barre_top/eldy_backoffice.php | 5 ---
.../menus/barre_top/eldy_frontoffice.php | 5 ---
htdocs/includes/menus/barre_top/rodolphe.php | 5 ---
htdocs/lib/antispamimage.php | 1 +
htdocs/main.inc.php | 1 -
12 files changed, 33 insertions(+), 87 deletions(-)
diff --git a/htdocs/includes/menus/barre_left/auguria_backoffice.php b/htdocs/includes/menus/barre_left/auguria_backoffice.php
index c4084a348a0..a7ce8507b59 100644
--- a/htdocs/includes/menus/barre_left/auguria_backoffice.php
+++ b/htdocs/includes/menus/barre_left/auguria_backoffice.php
@@ -23,11 +23,11 @@
\version $Id$
\remarks La construction d'un gestionnaire pour le menu de gauche est simple:
- \remarks A l'aide d'un objet $newmenu=new Menu() et des m�thode add et add_submenu,
- \remarks d�finir la liste des entr�es menu � faire apparaitre.
+ \remarks A l'aide d'un objet $newmenu=new Menu() et des m�thode add et add_submenu,
+ \remarks d�finir la liste des entr�es menu � faire apparaitre.
\remarks En fin de code, mettre la ligne $menu=$newmenu->liste.
- \remarks Ce qui est d�fini dans un tel gestionnaire sera alors prioritaire sur
- \remarks les d�finitions de menu des fichiers pre.inc.php
+ \remarks Ce qui est d�fini dans un tel gestionnaire sera alors prioritaire sur
+ \remarks les d�finitions de menu des fichiers pre.inc.php
*/
@@ -47,8 +47,8 @@ class MenuLeft {
/**
* \brief Constructeur
- * \param db Handler d'acc�s base de donn�e
- * \param menu_array Tableau des entr�e de menu d�fini dans les fichier pre.inc.php
+ * \param db Handler d'acc�s base de donn�e
+ * \param menu_array Tableau des entr�e de menu d�fini dans les fichier pre.inc.php
*/
function MenuLeft($db,&$menu_array)
{
@@ -65,20 +65,14 @@ class MenuLeft {
{
global $user,$conf,$langs,$dolibarr_main_db_name;
-
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start(); // En mode authentification PEAR, la session a d�j� �t� ouverte
- }
-
- // On r�cup�re mainmenu et leftmenu qui d�finissent le menu � afficher
+ // On recupere mainmenu et leftmenu qui definissent le menu a afficher
if (isset($_GET["mainmenu"])) {
// On sauve en session le menu principal choisi
$mainmenu=$_GET["mainmenu"];
$_SESSION["mainmenu"]=$mainmenu;
$_SESSION["leftmenuopened"]="";
} else {
- // On va le chercher en session si non d�fini par le lien
+ // On va le chercher en session si non defini par le lien
$mainmenu=$_SESSION["mainmenu"];
}
@@ -94,7 +88,7 @@ class MenuLeft {
$_SESSION["leftmenuopened"]=$this->leftmenu;
}
} else {
- // On va le chercher en session si non d�fini par le lien
+ // On va le chercher en session si non d�fini par le lien
$this->leftmenu=isset($_SESSION["leftmenu"])?$_SESSION["leftmenu"]:'';
}
@@ -113,7 +107,7 @@ class MenuLeft {
$this->newmenu = $menuArbo->menuLeftCharger($this->newmenu,$mainmenu,$this->leftmenu,0,'auguria');
/*
- * Menu AUTRES (Pour les menus du haut qui ne serait pas g�r�s)
+ * Menu AUTRES (Pour les menus du haut qui ne serait pas g�r�s)
*/
if ($mainmenu && ! in_array($mainmenu,$this->overwritemenufor)) { $mainmenu=""; }
}
@@ -123,7 +117,7 @@ class MenuLeft {
/**
- * Si on est sur un cas g�r� de surcharge du menu, on ecrase celui par defaut
+ * Si on est sur un cas g�r� de surcharge du menu, on ecrase celui par defaut
*/
if ($mainmenu) {
$this->menu_array=$this->newmenu->liste;
diff --git a/htdocs/includes/menus/barre_left/auguria_frontoffice.php b/htdocs/includes/menus/barre_left/auguria_frontoffice.php
index 69357af0874..6a4762734ea 100644
--- a/htdocs/includes/menus/barre_left/auguria_frontoffice.php
+++ b/htdocs/includes/menus/barre_left/auguria_frontoffice.php
@@ -23,11 +23,11 @@
\version $Id$
\remarks La construction d'un gestionnaire pour le menu de gauche est simple:
- \remarks A l'aide d'un objet $newmenu=new Menu() et des m�thode add et add_submenu,
- \remarks d�finir la liste des entr�es menu � faire apparaitre.
+ \remarks A l'aide d'un objet $newmenu=new Menu() et des m�thode add et add_submenu,
+ \remarks d�finir la liste des entr�es menu � faire apparaitre.
\remarks En fin de code, mettre la ligne $menu=$newmenu->liste.
- \remarks Ce qui est d�fini dans un tel gestionnaire sera alors prioritaire sur
- \remarks les d�finitions de menu des fichiers pre.inc.php
+ \remarks Ce qui est d�fini dans un tel gestionnaire sera alors prioritaire sur
+ \remarks les d�finitions de menu des fichiers pre.inc.php
*/
@@ -66,19 +66,14 @@ class MenuLeft {
{
global $user,$conf,$langs,$dolibarr_main_db_name;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start(); // En mode authentification PEAR, la session a d�j� �t� ouverte
- }
-
- // On r�cup�re mainmenu et leftmenu qui d�finissent le menu � afficher
+ // On recupere mainmenu et leftmenu qui definissent le menu a afficher
if (isset($_GET["mainmenu"])) {
// On sauve en session le menu principal choisi
$mainmenu=$_GET["mainmenu"];
$_SESSION["mainmenu"]=$mainmenu;
$_SESSION["leftmenuopened"]="";
} else {
- // On va le chercher en session si non d�fini par le lien
+ // On va le chercher en session si non defini par le lien
$mainmenu=$_SESSION["mainmenu"];
}
diff --git a/htdocs/includes/menus/barre_left/eldy_backoffice.php b/htdocs/includes/menus/barre_left/eldy_backoffice.php
index 748e3225c03..05c3ea4a61b 100644
--- a/htdocs/includes/menus/barre_left/eldy_backoffice.php
+++ b/htdocs/includes/menus/barre_left/eldy_backoffice.php
@@ -61,11 +61,6 @@ class MenuLeft {
{
global $user,$conf,$langs,$dolibarr_main_db_name;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
// On r�cup�re mainmenu et leftmenu qui d�finissent le menu � afficher
if (isset($_GET["mainmenu"]))
{
diff --git a/htdocs/includes/menus/barre_left/eldy_frontoffice.php b/htdocs/includes/menus/barre_left/eldy_frontoffice.php
index 48364378ac0..757d2731f56 100644
--- a/htdocs/includes/menus/barre_left/eldy_frontoffice.php
+++ b/htdocs/includes/menus/barre_left/eldy_frontoffice.php
@@ -59,11 +59,6 @@ class MenuLeft {
{
global $user,$conf,$langs,$dolibarr_main_db_name;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
// On r�cup�re mainmenu et leftmenu qui d�finissent le menu � afficher
if (isset($_GET["mainmenu"]))
{
diff --git a/htdocs/includes/menus/barre_left/rodolphe.php b/htdocs/includes/menus/barre_left/rodolphe.php
index 7d9b624e042..d38fd5ec0c5 100644
--- a/htdocs/includes/menus/barre_left/rodolphe.php
+++ b/htdocs/includes/menus/barre_left/rodolphe.php
@@ -17,22 +17,22 @@
*/
/** \file htdocs/includes/menus/barre_left/rodolphe.php
- \brief Gestionnaire par d�faut du menu de gauche
+ \brief Gestionnaire par d�faut du menu de gauche
\version $Id$
\remarks La construction d'un gestionnaire pour le menu de gauche est simple:
- \remarks A l'aide d'un objet $newmenu=new Menu() et des m�thode add et add_submenu,
- \remarks d�finir la liste des entr�es menu � faire apparaitre.
+ \remarks A l'aide d'un objet $newmenu=new Menu() et des m�thode add et add_submenu,
+ \remarks d�finir la liste des entr�es menu � faire apparaitre.
\remarks En fin de code, mettre la ligne $menu=$newmenu->liste.
- \remarks Ce qui est d�finir dans un tel gestionnaire sera alors prioritaire sur
- \remarks les d�finitions de menu des fichiers pre.inc.php
+ \remarks Ce qui est d�finir dans un tel gestionnaire sera alors prioritaire sur
+ \remarks les d�finitions de menu des fichiers pre.inc.php
*/
/** \class MenuLeft
- \brief Classe permettant la gestion par d�faut du menu du gauche
- \remarks Le gestionnaire par defaut ne fait rien: C'est donc le menu d�fini dans les
- \remarks fichiers pre.inc.php du r�pertoire de la page qui est utilis�.
+ \brief Classe permettant la gestion par d�faut du menu du gauche
+ \remarks Le gestionnaire par defaut ne fait rien: C'est donc le menu d�fini dans les
+ \remarks fichiers pre.inc.php du r�pertoire de la page qui est utilis�.
*/
class MenuLeft {
@@ -42,8 +42,8 @@ class MenuLeft {
/**
* \brief Constructeur
- * \param db Handler d'acc�s base de donn�e
- * \param menu_array Tableau des entr�e de menu d�fini dans les fichier pre.inc.php
+ * \param db Handler d'acc�s base de donn�e
+ * \param menu_array Tableau des entr�e de menu d�fini dans les fichier pre.inc.php
*/
function MenuLeft($db,&$menu_array)
{
@@ -59,12 +59,7 @@ class MenuLeft {
{
global $user, $conf, $langs, $dolibarr_main_db_name;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
- // On r�cup�re mainmenu et leftmenu qui d�finissent le menu � afficher
+ // On r�cup�re mainmenu et leftmenu qui d�finissent le menu � afficher
if (isset($_GET["mainmenu"]))
{
// On sauve en session le menu principal choisi
@@ -74,7 +69,7 @@ class MenuLeft {
}
else
{
- // On va le chercher en session si non d�fini par le lien
+ // On va le chercher en session si non d�fini par le lien
$mainmenu=$_SESSION["mainmenu"];
}
@@ -93,7 +88,7 @@ class MenuLeft {
$_SESSION["leftmenuopened"]=$leftmenu;
}
} else {
- // On va le chercher en session si non d�fini par le lien
+ // On va le chercher en session si non d�fini par le lien
$leftmenu=isset($_SESSION["leftmenu"])?$_SESSION["leftmenu"]:'';
}
@@ -108,14 +103,14 @@ class MenuLeft {
$newmenu = $this->menuArbo->menuLeftCharger($newmenu,$mainmenu,$leftmenu,0,'eldy');
/*
- * Menu AUTRES (Pour les menus du haut qui ne serait pas g�r�s)
+ * Menu AUTRES (Pour les menus du haut qui ne serait pas g�r�s)
*/
if ($mainmenu && ! in_array($mainmenu,$this->overwritemenufor)) { $mainmenu=""; }
}
/**
- * Si on est sur un cas g�r� de surcharge du menu, on ecrase celui par defaut
+ * Si on est sur un cas g�r� de surcharge du menu, on ecrase celui par defaut
*/
if ($mainmenu) {
$this->menu_array=$newmenu->liste;
diff --git a/htdocs/includes/menus/barre_top/auguria_backoffice.php b/htdocs/includes/menus/barre_top/auguria_backoffice.php
index d443e1148b2..25b42517c23 100644
--- a/htdocs/includes/menus/barre_top/auguria_backoffice.php
+++ b/htdocs/includes/menus/barre_top/auguria_backoffice.php
@@ -64,12 +64,6 @@ class MenuTop {
global $user,$conf,$langs,$dolibarr_main_db_name;;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
-
// On sauve en session le menu principal choisi
if (isset($_GET["mainmenu"])) $_SESSION["mainmenu"]=$_GET["mainmenu"];
if (isset($_GET["idmenu"])) $_SESSION["idmenu"]=$_GET["idmenu"];
diff --git a/htdocs/includes/menus/barre_top/auguria_frontoffice.php b/htdocs/includes/menus/barre_top/auguria_frontoffice.php
index cecf9983dce..39a2ec21cd2 100644
--- a/htdocs/includes/menus/barre_top/auguria_frontoffice.php
+++ b/htdocs/includes/menus/barre_top/auguria_frontoffice.php
@@ -63,13 +63,6 @@ class MenuTop {
global $user,$conf,$langs,$dolibarr_main_db_name;;
-
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
-
// On sauve en session le menu principal choisi
if (isset($_GET["mainmenu"])) $_SESSION["mainmenu"]=$_GET["mainmenu"];
if (isset($_GET["idmenu"])) $_SESSION["idmenu"]=$_GET["idmenu"];
diff --git a/htdocs/includes/menus/barre_top/eldy_backoffice.php b/htdocs/includes/menus/barre_top/eldy_backoffice.php
index 2b5befc5060..137930cd14b 100644
--- a/htdocs/includes/menus/barre_top/eldy_backoffice.php
+++ b/htdocs/includes/menus/barre_top/eldy_backoffice.php
@@ -62,11 +62,6 @@ class MenuTop {
{
global $user,$conf,$langs,$dolibarr_main_db_name;;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
// On sauve en session le menu principal choisi
if (isset($_GET["mainmenu"])) $_SESSION["mainmenu"]=$_GET["mainmenu"];
if (isset($_GET["idmenu"])) $_SESSION["idmenu"]=$_GET["idmenu"];
diff --git a/htdocs/includes/menus/barre_top/eldy_frontoffice.php b/htdocs/includes/menus/barre_top/eldy_frontoffice.php
index 10de5889305..5d2e53d65c7 100644
--- a/htdocs/includes/menus/barre_top/eldy_frontoffice.php
+++ b/htdocs/includes/menus/barre_top/eldy_frontoffice.php
@@ -61,11 +61,6 @@ class MenuTop {
{
global $user,$conf,$langs,$dolibarr_main_db_name;;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
// On sauve en session le menu principal choisi
if (isset($_GET["mainmenu"])) $_SESSION["mainmenu"]=$_GET["mainmenu"];
if (isset($_GET["idmenu"])) $_SESSION["idmenu"]=$_GET["idmenu"];
diff --git a/htdocs/includes/menus/barre_top/rodolphe.php b/htdocs/includes/menus/barre_top/rodolphe.php
index 341884d55f0..ee4bf9908f3 100644
--- a/htdocs/includes/menus/barre_top/rodolphe.php
+++ b/htdocs/includes/menus/barre_top/rodolphe.php
@@ -57,11 +57,6 @@ class MenuTop {
{
global $user,$conf,$langs,$dolibarr_main_db_name;
- if (! session_id()) {
- session_name("DOLSESSID_".$dolibarr_main_db_name);
- session_start();
- }
-
// On sauve en session le menu principal choisi
if (isset($_GET["mainmenu"])) $_SESSION["mainmenu"]=$_GET["mainmenu"];
if (isset($_GET["idmenu"])) $_SESSION["idmenu"]=$_GET["idmenu"];
diff --git a/htdocs/lib/antispamimage.php b/htdocs/lib/antispamimage.php
index b67a0d9e287..1eeb3426adb 100644
--- a/htdocs/lib/antispamimage.php
+++ b/htdocs/lib/antispamimage.php
@@ -39,6 +39,7 @@ require_once ARTICHOW."/AntiSpam.class.php";
$sessionname="DOLSESSID_".$dolibarr_main_db_name;
session_name($sessionname);
session_start();
+dolibarr_syslog("Session name=".$sessionname." Session id()=".session_id().", _SESSION['dol_login']=".$_SESSION["dol_login"]);
// On créé l'objet anti-spam
diff --git a/htdocs/main.inc.php b/htdocs/main.inc.php
index 75bed021629..4c3b70fd808 100644
--- a/htdocs/main.inc.php
+++ b/htdocs/main.inc.php
@@ -46,7 +46,6 @@ function stripslashes_deep($value)
{
return (is_array($value) ? array_map('stripslashes_deep', $value) : stripslashes($value));
}
-//if (! eregi('PHP/6', $_SERVER['SERVER_SOFTWARE']))
if (function_exists('get_magic_quotes_gpc')) // magic_quotes_* plus pris en compte dans PHP6
{
if (get_magic_quotes_gpc())
--
GitLab