From c94d1758615ba6e593b9271d205b1d203a108053 Mon Sep 17 00:00:00 2001
From: Thijs Kinkhorst <thijs@kinkhorst.com>
Date: Wed, 24 Apr 2019 19:02:48 +0000
Subject: [PATCH] Refactor to avoid using pre-PHP 7 operator
---
modules/admin/lib/FederationController.php | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/modules/admin/lib/FederationController.php b/modules/admin/lib/FederationController.php
index 0f7ac4ca9..b0bcc47a4 100644
--- a/modules/admin/lib/FederationController.php
+++ b/modules/admin/lib/FederationController.php
@@ -330,11 +330,15 @@ class FederationController
/** @var \SimpleSAML\Module\saml\Auth\Source\SP $source */
foreach (\SimpleSAML\Auth\Source::getSourcesOfType('saml:SP') as $source) {
$metadata = $source->getHostedMetadata();
- $certificates = $metadata['keys'] ?? [];
- if (count($certificates) === 1) {
- $cert = array_pop($metadata['keys']);
- $metadata['certData'] = $cert['X509Certificate'];
- unset($metadata['keys']);
+ if (isset($metadata['keys']) ) {
+ $certificates = $metadata['keys'];
+ if (count($metadata['keys']) === 1) {
+ $cert = array_pop($metadata['keys']);
+ $metadata['certData'] = $cert['X509Certificate'];
+ unset($metadata['keys']);
+ }
+ } else {
+ $certificates = [];
}
// get the name
--
GitLab