fixed runtime errors for main action.

Course selection is not yet working.
DatabayAG · Jul 11, 2023 · d69e085 · d69e085
1 parent d0830d3
commit d69e085
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 9 deletions.
diff --git a/classes/class.ilVhbShibAuthConfig.php b/classes/class.ilVhbShibAuthConfig.php
@@ -127,7 +127,7 @@ public function __construct($a_plugin_object)
         $params[] = ilVhbShibAuthParam::_create(
             'evaluator_role',
             'Evaluatorenrolle',
-            'Suchmuster für Namen der ILIAS-Kursrolle, die Evaluatoren zugewiesen werden soll. Evaluatoren haben im Shibboleth-Attribut "eduPersonEntitlement" die Nutzerrolle "evaluator". Sie sollen im ILIAS-Kurs eine entsprechende Kursrolle bekommen. Mit dem Suchmuster wird in den Titeln aller Rollen des gefundenen Kurses nach der Evaluatoren-Rolle gesucht. Das Muster kann "?" oder "*" als Platzhalter für einzelne oder beliebig viele Zeichen enthalten.',
+            'Suchmuster für Namen der ILIAS-Kursrolle, die Evaluatoren zugewiesen werden soll. Evaluatoren haben im Shibboleth-Attribut "eduPersonEntitlement" die Nutzerrolle "evaluation". Sie sollen im ILIAS-Kurs eine entsprechende Kursrolle bekommen. Mit dem Suchmuster wird in den Titeln aller Rollen des gefundenen Kurses nach der Evaluatoren-Rolle gesucht. Das Muster kann "?" oder "*" als Platzhalter für einzelne oder beliebig viele Zeichen enthalten.',
             ilVhbShibAuthParam::TYPE_TEXT,
             'Kursgast*'
         );

diff --git a/classes/class.ilVhbShibAuthConfigGUI.php b/classes/class.ilVhbShibAuthConfigGUI.php
@@ -3,8 +3,10 @@
 
 /**
  * vhb Shibboleth Authentication configuration user interface class
+ * 
+ *  @ilCtrl_IsCalledBy ilVhbShibAuthConfigGUI: ilObjComponentSettingsGUI
  */
-class ilvhbShibAuthConfigGUI extends ilPluginConfigGUI
+class ilVhbShibAuthConfigGUI extends ilPluginConfigGUI
 {
 	/** @var ilVhbShibAuthPlugin $plugin */
 	protected $plugin;
@@ -84,7 +86,7 @@ protected function initConfigurationForm()
                     break;
                 case ilVhbShibAuthParam::TYPE_BOOLEAN:
                     $input = new ilCheckboxInputGUI($title, $postvar);
-                    $input->setChecked($param->value);
+                    $input->setChecked((bool) $param->value);
                     break;
                 case ilVhbShibAuthParam::TYPE_FLOAT:
                     $input = new ilNumberInputGUI($title, $postvar);

diff --git a/classes/class.ilVhbShibAuthData.php b/classes/class.ilVhbShibAuthData.php
@@ -80,6 +80,9 @@ public function configure($config, $plugin)
         // extract the relevant values from aggregated fields
         // should not be necessary if SP is correctly configured at vhb
         foreach (array_keys(get_class_vars('shibConfig')) as $field) {
+            if ($field == 'cache') {
+                continue;
+            }
             $values = explode(self::DELIM, (string) $this->{$field});
 
             if ($this->config->get('resolve_aggregation')) {
@@ -163,7 +166,7 @@ public function getData()
     {
         $data = [];
         foreach (array_keys(get_class_vars('shibConfig')) as $field) {
-            if (substr($field, 0, 7) != 'update_') {
+            if ($field !== 'cache' && substr($field, 0, 7) != 'update_') {
                 $data[$field] = $this->{$field};
             }
         }

diff --git a/classes/class.ilVhbShibAuthMatching.php b/classes/class.ilVhbShibAuthMatching.php
@@ -315,11 +315,11 @@ protected function getEntitledVhbCourses()
         foreach ($entitlements as $entitlement)
         {
             $parts = explode(':',$entitlement);
-            $role = $parts[5];
-            $scope = $parts[6];
-            $lvnr = $parts[7];
+            $role = $parts[5] ?? '';
+            $scope = $parts[6] ?? '';
+            $lvnr = $parts[7] ?? '';
 
-            if ($scope == $this->config->get('local_scope'))
+            if ($scope == $this->config->get('local_scope') && !empty($role) && !empty($lvnr))
             {
                 $courses[$lvnr] = $role;
             }

diff --git a/plugin.php b/plugin.php
@@ -5,7 +5,7 @@
 $id = "vhbshib";
 
 // code version; must be changed for all code changes
-$version = "1.2.0";
+$version = "1.2.1";
 
 // ilias min and max version; must always reflect the versions that should
 // run with the plugin