Incorporated PR feedback

Author: Billy Clark

docker-compose.yml

@@ -19,6 +19,7 @@ services:
       ADMIN_EMAIL: "admin@example.org"
       SECRET_SALT: "FcJwl0zCDc4nuzOuQL9/7WPIj3hdfusGm2ny0dcRQm0="
       IDP_NAME: "The Hub"
+      ANALYTICS_ID: "UA-XXXX-Y"
   idp1:
     image: silintl/ssp-base:develop
     volumes:

themes/material/common-announcement.php

@@ -1,5 +1,5 @@
 <?php
-$announcement = htmlentities($this->data['announcement']);
+$announcement = htmlentities($this->data['announcement'] ?? null);
 
 if (! empty($announcement)) {
 ?>

themes/material/core/loginuserpass.php

@@ -6,7 +6,7 @@
     <?php include __DIR__ . '/../common-head-elements.php' ?>
 
     <?php
-    $siteKey = htmlentities($this->data['recaptcha.siteKey']);
+    $siteKey = htmlentities($this->data['recaptcha.siteKey'] ?? null);
     if (! empty($siteKey)) {
     ?>
     <script src='https://www.google.com/recaptcha/api.js?onload=onRecaptchaLoad&render=explicit'
@@ -40,9 +40,9 @@ function onRecaptchaLoad() {
 
         <form method="POST" action="<?= htmlentities($_SERVER['PHP_SELF']) ?>">
             <input type="hidden" name="AuthState"
-                   value="<?= htmlentities($this->data['stateparams']['AuthState']) ?>" />
+                   value="<?= htmlentities($this->data['stateparams']['AuthState']  ?? null) ?>" />
             <?php
-            $csrfToken = htmlentities($this->data['csrfToken']);
+            $csrfToken = htmlentities($this->data['csrfToken'] ?? null);
             ?>
             <input type="hidden" name="csrf-token" value="<?= $csrfToken ?>" />
 
@@ -67,7 +67,7 @@ function onRecaptchaLoad() {
                             <?= $this->t('{material:login:label_username}') ?>
                         </label>
                         <?php
-                        $username = htmlentities($this->data['username']);
+                        $username = htmlentities($this->data['username'] ?? null);
                         ?>
                         <input type="text" name="username" class="mdl-textfield__input"
                                value="<?= $username ?>"
@@ -87,7 +87,7 @@ function onRecaptchaLoad() {
                 $errorCode = htmlentities($this->data['errorcode']);
                 if ($errorCode == 'WRONGUSERPASS') {
                     $errorMessageKey = $this->data['errorparams'][1] ?? '{material:login:error_wronguserpass}';
-                    $errorMessageTokens = $this->data['errorparams'][2];
+                    $errorMessageTokens = $this->data['errorparams'][2] ?? null;
 
                     $message = htmlentities($this->t($errorMessageKey, $errorMessageTokens));
                 ?>
@@ -108,7 +108,7 @@ function onRecaptchaLoad() {
 
                 <div class="mdl-card__actions" layout-children="row">
                     <?php
-                    $forgotPasswordUrl = htmlentities($this->data['forgotPasswordUrl']);
+                    $forgotPasswordUrl = htmlentities($this->data['forgotPasswordUrl'] ?? null);
                     if (! empty($forgotPasswordUrl)) {
                     ?>
                     <a href="<?= $forgotPasswordUrl ?>" target="_blank"

themes/material/default/error.php

@@ -21,14 +21,14 @@
         </p>
 
         <?php
-        if ($this->data['showerrors']) {
+        if ($this->data['showerrors'] ?? false) {
         ?>
         <p class="mdl-typography--body-2">
-            <?= htmlentities($this->data['error']['exceptionMsg']) ?>
+            <?= htmlentities($this->data['error']['exceptionMsg'] ?? null) ?>
         </p>
 
         <pre class="mdl-typography--caption">
-            <?= htmlentities($this->data['error']['exceptionTrace']) ?>
+            <?= htmlentities($this->data['error']['exceptionTrace'] ?? null) ?>
         </pre>
         <?php
         }

themes/material/default/selectidp-links.php

@@ -40,11 +40,11 @@ function clickedAnyway(idpName) {
         <form action="<?= htmlentities($_SERVER['PHP_SELF']) ?>"
               layout-children="row" child-spacing="space-around">
             <input type="hidden" name="entityID"
-                   value="<?= htmlentities($this->data['entityID']) ?>" />
+                   value="<?= htmlentities($this->data['entityID'] ?? null) ?>" />
             <input type="hidden" name="return"
-                   value="<?= htmlentities($this->data['return']) ?>" />
+                   value="<?= htmlentities($this->data['return'] ?? null) ?>" />
             <input type="hidden" name="returnIDParam"
-                   value="<?= htmlentities($this->data['returnIDParam']) ?>" />
+                   value="<?= htmlentities($this->data['returnIDParam'] ?? null) ?>" />
 
             <?php
             // in order to bypass some built-in simplesaml behavior, an extra idp
@@ -59,8 +59,8 @@ function clickedAnyway(idpName) {
             }
 
             foreach ($enabledIdps as $idp) {
-                $name = htmlentities($this->t($idp['name']));
-                $idpId = htmlentities($idp['entityid']);
+                $name = htmlentities($this->t($idp['name'] ?? null));
+                $idpId = htmlentities($idp['entityid'] ?? null);
                 $hoverText = $this->t('{material:selectidp:enabled}', ['{idpName}' => $name]);
             ?>
             <div class="mdl-card mdl-shadow--8dp row-aware" title="<?= $hoverText ?>">
@@ -78,8 +78,8 @@ function clickedAnyway(idpName) {
 
             <?php
             foreach ($disabledIdps as $idp) {
-                $name = htmlentities($this->t($idp['name']));
-                $idpId = htmlentities($idp['entityid']);
+                $name = htmlentities($this->t($idp['name'] ?? null));
+                $idpId = htmlentities($idp['entityid'] ?? null);
                 $hoverText = $this->t('{material:selectidp:disabled}', ['{idpName}' => $name]);
             ?>
             <div class="mdl-card mdl-shadow--2dp disabled row-aware" title="<?= $hoverText ?>"