Update listOfSps-tpl.php

Author: melanger

CHANGELOG.md

@@ -11,7 +11,9 @@ All notable changes to this project will be documented in this file.
 - Changed the way of getting attribute names for interfaces: through internal attribute names in perun_attributes.php config
 - Return sorted eduPersonEntitlement
 - Don't show previous selection when user show all entries on the discovery page
-- ListOfSps - Don't show the description by default
+- ListOfSps 
+    - Don't show the description by default
+    - Added required attribute 'listOfSps.serviceNameAttr' !!!
 
 ## [v3.9.0]
 #### Added

config-templates/module_perun.php

@@ -147,9 +147,8 @@
 
     /**
      * Specify attribute name for facility attribute with service name
-     * Defaults to facility name (urn:perun:facility:attribute-def:core:name)
      */
-    //'listOfSps.serviceNameAttr' => '',
+    'listOfSps.serviceNameAttr' => '',
 
     /**
      * Specify attribute name for facility attribute with loginUrL for service

templates/listOfSps-tpl.php

@@ -116,7 +116,7 @@
                         }
                         echo '<tr>';
                         echo '<td>'
-                            . ListOfSps::printServiceName($service['name'], $service['loginURL'] ?? null)
+                            . ListOfSps::printServiceName($service['name']['value'], $service['loginURL']['value'] ?? null)
                             . '</td>';
                         if (array_key_exists($service['facility']->getID(), $samlServices)) {
                             echo '<td>' . $this->t('{perun:listOfSps:saml}') . '</td>';

www/listOfSps.php

@@ -63,7 +63,13 @@
     );
 }
 
-$perunServiceNameAttr = $conf->getString(PERUN_SERVICE_NAME_ATTR_NAME, 'urn:perun:facility:attribute-def:core:name');
+$perunServiceNameAttr = $conf->getString(PERUN_SERVICE_NAME_ATTR_NAME, null);
+if (empty($perunServiceNameAttr)) {
+    throw new Exception(
+        'perun:listOfSps: missing mandatory config option \''
+        . PERUN_SERVICE_NAME_ATTR_NAME . '\'.'
+    );
+}
 $perunLoginURLAttr = $conf->getString(PERUN_LOGIN_URL_ATTR_NAME, null);
 $perunTestSpAttr = $conf->getString(PERUN_TEST_SP_ATTR_NAME, null);
 $perunShowOnServiceListAttr