lainwir3d/dolibarr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix: Protection accs incomplete.

Browse Source 
Modif ordre des test pour respecter le modele MVC (Test secu en premier, puis action, puis affichage).
This commit is contained in:
Laurent Destailleur 2007-01-24 18:24:34 +00:00
parent 31212d9d0b
commit 132ee0d9fb
1 changed files with 25 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
htdocs/comm/propal.php
View File

@ -36,10 +36,6 @@ require_once(DOL_DOCUMENT_ROOT."/lib/propal.lib.php");
$user->getrights('propale'); $user->getrights('propale');
if (!$user->rights->propale->lire)
accessforbidden();
$langs->load('companies'); $langs->load('companies');
$langs->load('propal'); $langs->load('propal');
$langs->load('compta'); $langs->load('compta');
@ -58,6 +54,7 @@ if (isset($_GET["msg"])) { $mesg=urldecode($_GET["mesg"]); }
$year=isset($_GET["year"])?$_GET["year"]:""; $year=isset($_GET["year"])?$_GET["year"]:"";
$month=isset($_GET["month"])?$_GET["month"]:""; $month=isset($_GET["month"])?$_GET["month"]:"";
// Sécurité accés client // Sécurité accés client
$socid=''; $socid='';
if ($_GET["socid"]) { $socid=$_GET["socid"]; } if ($_GET["socid"]) { $socid=$_GET["socid"]; }
@ -66,12 +63,26 @@ if ($user->societe_id > 0)
$action = ''; $action = '';
$socid = $user->societe_id; $socid = $user->societe_id;
} }
if (!$user->rights->propale->lire) accessforbidden();
if ($_GET['propalid'] > 0)
{
$propal = new Propal($db);
$result=$propal->fetch($_GET['propalid']);
if (! $result > 0)
{
dolibarr_print_error($db,$propal->error);
exit;
}
if (!$user->rights->commercial->client->voir && $user->societe_id > 0 && $propal->socid <> $user->societe_id)
accessforbidden();
}
// Nombre de ligne pour choix de produit/service prédéfinis // Nombre de ligne pour choix de produit/service prédéfinis
$NBLINES=4; $NBLINES=4;
$form=new Form($db); $form=new Form($db);
/******************************************************************************/ /******************************************************************************/
/* Actions */ /* Actions */
/******************************************************************************/ /******************************************************************************/
@ -703,28 +714,20 @@ if ($_GET['action'] == 'down' && $user->rights->propale->creer)
} }
/*
* Affichage page
*/
llxHeader('',$langs->trans('Proposal'),'Proposition');
$html = new Form($db); $html = new Form($db);
/*
* Affichage fiche propal en mode visu
*
*/
if ($_GET['propalid'] > 0) if ($_GET['propalid'] > 0)
{ {
$propal = new Propal($db); /*
* Affichage fiche propal en mode visu
$result=$propal->fetch($_GET['propalid']); *
if (! $result > 0) */
{
dolibarr_print_error($db,$propal->error);
exit;
}
if ($user->societe_id > 0 && $propal->socid <> $user->societe_id)
accessforbidden();
llxHeader('',$langs->trans('Proposal'),'Proposition');
if ($mesg) print "$mesg<br>"; if ($mesg) print "$mesg<br>";
@ -734,7 +737,6 @@ if ($_GET['propalid'] > 0)
$head = propal_prepare_head($propal); $head = propal_prepare_head($propal);
dolibarr_fiche_head($head, 'comm', $langs->trans('Proposal')); dolibarr_fiche_head($head, 'comm', $langs->trans('Proposal'));
/* /*
* Confirmation de la suppression de la propale * Confirmation de la suppression de la propale
*/ */
@ -1721,8 +1723,6 @@ if ($_GET['propalid'] > 0)
} }
else else
{ {
llxHeader('',$langs->trans('Proposal'),'Proposition');
/**************************************************************************** /****************************************************************************
* * * *
* Mode Liste des propales * * Mode Liste des propales *