Fix: Use entity
This commit is contained in:
parent
86ea79c860
commit
023b6f2e8b
@ -1233,13 +1233,15 @@ function dol_most_recent_file($dir,$regexfilter='',$excludefilter=array('\.meta$
|
|||||||
*
|
*
|
||||||
* @param string $modulepart Module of document
|
* @param string $modulepart Module of document
|
||||||
* @param string $original_file Relative path with filename
|
* @param string $original_file Relative path with filename
|
||||||
|
* @param string $entity Restrict onto entity
|
||||||
* @return mixed Array with access information : accessallowed & sqlprotectagainstexternals & original_file (as full path name)
|
* @return mixed Array with access information : accessallowed & sqlprotectagainstexternals & original_file (as full path name)
|
||||||
*/
|
*/
|
||||||
function dol_check_secure_access_document($modulepart,$original_file)
|
function dol_check_secure_access_document($modulepart,$original_file,$entity)
|
||||||
{
|
{
|
||||||
global $user, $conf;
|
global $user, $conf;
|
||||||
|
|
||||||
if (empty($modulepart)) return 'ErrorBadParameter';
|
if (empty($modulepart)) return 'ErrorBadParameter';
|
||||||
|
if (empty($entity)) $entity=0;
|
||||||
|
|
||||||
// We define $accessallowed and $sqlprotectagainstexternals
|
// We define $accessallowed and $sqlprotectagainstexternals
|
||||||
$accessallowed=0;
|
$accessallowed=0;
|
||||||
|
|||||||
@ -96,7 +96,7 @@ $refname=basename(dirname($original_file)."/");
|
|||||||
|
|
||||||
// Security check
|
// Security check
|
||||||
if (empty($modulepart)) accessforbidden('Bad value for parameter modulepart');
|
if (empty($modulepart)) accessforbidden('Bad value for parameter modulepart');
|
||||||
$check_access = dol_check_secure_access_document($modulepart,$original_file);
|
$check_access = dol_check_secure_access_document($modulepart,$original_file,$entity);
|
||||||
$accessallowed = $check_access['accessallowed'];
|
$accessallowed = $check_access['accessallowed'];
|
||||||
$sqlprotectagainstexternals = $check_access['sqlprotectagainstexternals'];
|
$sqlprotectagainstexternals = $check_access['sqlprotectagainstexternals'];
|
||||||
$original_file = $check_access['original_file'];
|
$original_file = $check_access['original_file'];
|
||||||
|
|||||||
@ -100,7 +100,7 @@ $original_file = str_replace("../","/", $original_file);
|
|||||||
|
|
||||||
// Security check
|
// Security check
|
||||||
if (empty($modulepart)) accessforbidden('Bad value for parameter modulepart');
|
if (empty($modulepart)) accessforbidden('Bad value for parameter modulepart');
|
||||||
$check_access = dol_check_secure_access_document($modulepart,$original_file);
|
$check_access = dol_check_secure_access_document($modulepart,$original_file,$entity);
|
||||||
$accessallowed = $check_access['accessallowed'];
|
$accessallowed = $check_access['accessallowed'];
|
||||||
$sqlprotectagainstexternals = $check_access['sqlprotectagainstexternals'];
|
$sqlprotectagainstexternals = $check_access['sqlprotectagainstexternals'];
|
||||||
$original_file = $check_access['original_file'];
|
$original_file = $check_access['original_file'];
|
||||||
|
|||||||
@ -221,10 +221,10 @@ function getDocument($authentication, $modulepart, $file)
|
|||||||
$refname=basename(dirname($original_file)."/");
|
$refname=basename(dirname($original_file)."/");
|
||||||
|
|
||||||
// Security check
|
// Security check
|
||||||
$accessallowed=0;
|
$check_access = dol_check_secure_access_document($modulepart,$original_file,$conf->entity);
|
||||||
$check_access = dol_check_secure_access_document($modulepart,$original_file);
|
|
||||||
$accessallowed = $check_access['accessallowed'];
|
$accessallowed = $check_access['accessallowed'];
|
||||||
$sqlprotectagainstexternals = $check_access['sqlprotectagainstexternals'];
|
$sqlprotectagainstexternals = $check_access['sqlprotectagainstexternals'];
|
||||||
|
$original_file = $check_access['original_file'];
|
||||||
|
|
||||||
// Basic protection (against external users only)
|
// Basic protection (against external users only)
|
||||||
if ($fuser->societe_id > 0)
|
if ($fuser->societe_id > 0)
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user