Controller::CreatePage : use @list() intead of 3 lines with count() tests

Thanks Hipska !

Co-authored-by: Thomas Casteleyn <thomas.casteleyn@super-visions.com>
This commit is contained in:
Pierre Goiffon
2022-01-19 12:26:57 +01:00
parent 3ef3166bd5
commit 41f6e85673

View File

@@ -571,9 +571,7 @@ abstract class Controller
if ($this->IsOperationBreadCrumbEnabled()) {
$aBreadCrumbEntry = $this->GetOperationBreadCrumbEntry();
if ((false === empty($aBreadCrumbEntry)) && (is_array($aBreadCrumbEntry)) && (count($aBreadCrumbEntry) >= 3)) {
list($sId, $sTitle, $sDescription) = $aBreadCrumbEntry;
$sUrl = (count($aBreadCrumbEntry) >= 4) ? $aBreadCrumbEntry[3] : '';
$sIcon = (count($aBreadCrumbEntry) >= 5) ? $aBreadCrumbEntry[4] : '';
@list($sId, $sTitle, $sDescription, $sUrl, $sIcon) = $aBreadCrumbEntry;
$this->m_oPage->SetBreadCrumbEntry($sId, $sTitle, $sDescription, $sUrl, $sIcon);
}
} else {