diff --git a/lib/Controller/SettingsController.php b/lib/Controller/SettingsController.php index ed063d6..eb22f3d 100644 --- a/lib/Controller/SettingsController.php +++ b/lib/Controller/SettingsController.php @@ -73,7 +73,14 @@ class SettingsController extends Controller { if (strpos($category, 'security-') === 0) { $category = 'security'; } - $key = $prefix . $category . '-' . $setting; + // make sure we properly fetch the attribute mapping + // as this is the only category that has the saml- prefix on config keys + if (strpos($category, 'attribute-mapping') === 0) { + $category = 'attribute-mapping'; + $key = $prefix . 'saml-attribute-mapping' . '-' . $setting; + } else { + $key = $prefix . $category . '-' . $setting; + } $settings[$category][$setting] = $this->config->getAppValue('user_saml', $key, ''); } } diff --git a/lib/Settings/Admin.php b/lib/Settings/Admin.php index 60b7810..39af402 100644 --- a/lib/Settings/Admin.php +++ b/lib/Settings/Admin.php @@ -148,7 +148,7 @@ class Admin implements ISettings { 'security-required' => $securityRequiredFields, 'security-general' => $securityGeneral, 'general' => $generalSettings, - 'attributeMappings' => $attributeMappingSettings, + 'attribute-mapping' => $attributeMappingSettings, 'type' => $type, 'providers' => $providers ]; diff --git a/templates/admin.php b/templates/admin.php index 38268e4..da6d1dc 100644 --- a/templates/admin.php +++ b/templates/admin.php @@ -120,7 +120,7 @@ style('user_saml', 'admin');