composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-04-18 16:18:47 +02:00
Code Issues Packages Projects Releases Wiki Activity

N°1671 Portal: Fix Aggregate Brick when user profile is not allowed to see one of the sub-brick

Browse Source
This commit is contained in:
Stephen Abello
2019-08-19 11:28:53 +02:00
parent ce22dc9309
commit 046eeb03f2
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
datamodels/2.x/itop-portal-base/portal/src/Controller/AggregatePageBrickController.class.inc.php
View File

@@ -22,7 +22,8 @@
namespace Combodo\iTop\Portal\Controller;
use Exception;
use Combodo\iTop\Portal\Brick\BrickNotFoundException;
use IssueLog;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Exception\HttpException;
@@ -76,7 +77,6 @@ class AggregatePageBrickController extends BrickController
* @param array $aAggregatePageBricksConf
*
* @return array
* @throws \Combodo\iTop\Portal\Brick\BrickNotFoundException
* @throws \Exception
*/
private function GetOrderedAggregatePageBricksObjectsById($aAggregatePageBricksConf)
@@ -87,11 +87,19 @@ class AggregatePageBrickController extends BrickController
$aAggregatePageBricks = array();
foreach ($aAggregatePageBricksConf as $sBrickId => $iBrickRank)
{
$oPortalBrick = $oBrickCollection->GetBrickById($sBrickId);
if (!isset($oPortalBrick))
try
{
throw new Exception("AggregatePageBrick : non existing brick '$sBrickId'");
$oPortalBrick = $oBrickCollection->GetBrickById($sBrickId);
}
catch (BrickNotFoundException $oException)
{
if ($this->get('kernel')->isDebug())
{
IssueLog::Warning('AggregatePageBrick: Could not display "'.$sBrickId.'", either wrong id or user profile not allowed');
}
continue;
}
$aAggregatePageBricks[] = $oPortalBrick;
}