diff --git a/templates/selectidp-dropdown.php b/templates/selectidp-dropdown.php index 2ac0ef87bf7dd180798815e477e335353d3e7689..27fbc36e7b5a15f5adca9c6a4e3820509098481a 100644 --- a/templates/selectidp-dropdown.php +++ b/templates/selectidp-dropdown.php @@ -10,18 +10,18 @@ $this->data['autofocus'] = 'dropdownlist'; $this->includeAtTemplateBase('includes/header.php'); foreach ($this->data['idplist'] AS $idpentry) { - if (isset($idpentry['UIInfo']['DisplayName'])) { + if (!empty($idpentry['UIInfo']['DisplayName'])) { /* TODO: remove this branch, If ['UIInfo']['DisplayName'] is available, it will get through to 'name' in the * metadata parsed with SSP >= 1.13.0, so this code is no longer necessary. Keep it now to avoid breaking * metadata parsed with previous versions. */ $this->includeInlineTranslation('idpname_' . $idpentry['entityid'], $idpentry['UIInfo']['DisplayName']); - } elseif (isset($idpentry['name'])) { + } elseif (!empty($idpentry['name'])) { $this->includeInlineTranslation('idpname_' . $idpentry['entityid'], $idpentry['name']); - } elseif (isset($idpentry['OrganizationDisplayName'])) { + } elseif (!empty($idpentry['OrganizationDisplayName'])) { $this->includeInlineTranslation('idpname_' . $idpentry['entityid'], $idpentry['OrganizationDisplayName']); } - if (isset($idpentry['description'])) + if (!empty($idpentry['description'])) $this->includeInlineTranslation('idpdesc_' . $idpentry['entityid'], $idpentry['description']); }