More flexible configuration.

Author: demiankatz

Diff for: config/vufind/SimulatedSSO.ini

@@ -8,6 +8,8 @@
 username = fakeuser1
 
 ; Map of user attributes to set (corresponding to columns in the user table).
+; Comment this out to use defaults, and set "attributes = false" to disable all
+; attributes.
 attributes[firstname] = Test
 attributes[lastname] = User
 attributes[email] = "[email protected]"

Diff for: module/VuFind/src/VuFind/Auth/SimulatedSSO.php

@@ -96,10 +96,15 @@ public function authenticate($request)
             throw new AuthException('Simulated failure');
         }
         $user = $this->getUserTable()->getByUsername($username);
-        $attribs = array_merge(
-            $this->defaultAttributes,
-            $this->simulatedSSOConfig['General']['attributes'] ?? []
-        );
+
+        // Get attribute configuration -- use defaults if no value is set, and use an
+        // empty array if something invalid was provided.
+        $attribs = $this->simulatedSSOConfig['General']['attributes']
+            ?? $this->defaultAttributes;
+        if (!is_array($attribs)) {
+            $attribs = [];
+        }
+
         $catPassword = null;
         foreach ($attribs as $attribute => $value) {
             if ($attribute == 'email') {