[Dolibarr-dev] Correction bugs dans gestionnair e menu eldy (Patch + requête SQL + question)

[Raphaël Bertrand (Résultic)]

Dans l'admin, Lionel avait détecté quelques anomalies d'affichage (tests 
sur droits non effectués).
http://www.dolibarr.fr/component/option,com_fireboard/Itemid,32/func,view/id,8882/catid,5/lang,fr/

Ce patch ci joint corrige le bug d'affichage du menu commercial menant 
sur une page prévenant que l'on a pas les accès.

Ensuite Pour restreindre l'affichage du menu ECM, j'ai du effectuer 
cette requête SQL:

UPDATE `llx_menu` SET `perms` = '$user->rights->ecm->create || 
$user->rights->ecm->read || $user->rights->ecm->setup',
`tms` = NOW( '20080603183419' ) WHERE `rowid` = '5003' LIMIT 1 ;

Par contre je ne sais pas où l'intégrer.

En regardant le code, la fonction verifCond (lignes 498-521 dans 
htdocs\core\menubase.class.php)
m'a paru bizarre, bien que fonctionnant:
La boucle semble effectuer plusieurs fois le même test s’il y a un OU 
dans le test, sans prendre en compte la variable de boucle, alors que le 
code utilisé pour faire ce test ressemble à du code pour faire un test 
ET, si l’on utilisait le compteur de boucle pour parcourir le tableau 
défini...


--
*Raphaël Bertrand*
Résultic - Management & Informatique
Siège : 11 rue Tronchet - 69006 LYON
Bureaux : 11 pl Maréchal Lyautey - 69006 LYON
Fixe. 08 74 77 00 70
Fax. 08 25 24 85 02
E-Mail : address@hidden

Index: eldy_backoffice.php
===================================================================
RCS file: 
/sources/dolibarr/dolibarr/htdocs/includes/menus/barre_top/eldy_backoffice.php,v
retrieving revision 1.44
diff -u -r1.44 eldy_backoffice.php
--- eldy_backoffice.php 2 May 2008 16:54:15 -0000       1.44
+++ eldy_backoffice.php 16 Jun 2008 08:19:33 -0000
@@ -182,8 +182,10 @@
                {
                    $class = 'class="tmenu"';
                }
-        
-               print '<td class="tmenu"><a '.$class.' 
href="'.DOL_URL_ROOT.'/comm/index.php?mainmenu=commercial&amp;leftmenu="'.($this->atarget?"
 target=$this->atarget":"").'>'.$langs->trans("Commercial").'</a></td>';
+                       if($user->rights->societe->lire)
+                               print '<td class="tmenu"><a '.$class.' 
href="'.DOL_URL_ROOT.'/comm/index.php?mainmenu=commercial&amp;leftmenu="'.($this->atarget?"
 target=$this->atarget":"").'>'.$langs->trans("Commercial").'</a></td>';
+                       else
+                               print '<td class="tmenu"><font 
class="tmenudisabled">'.$langs->trans("Commercial").'</font></td>';
         }
 
         // Financial