From c88ae37844c9024de99a59ce389e04d9dcf234f6 Mon Sep 17 00:00:00 2001
From: provokateurin <kate@provokateurin.de>
Date: Mon, 2 Sep 2024 09:55:32 +0200
Subject: [PATCH] fix(settings): Fix sharing exclude groups from password
 requirement option not working

Signed-off-by: provokateurin <kate@provokateurin.de>
---
 apps/settings/lib/Settings/Admin/Sharing.php              | 4 ++--
 apps/settings/src/components/AdminSettingsSharingForm.vue | 8 ++++----
 apps/settings/tests/Settings/Admin/SharingTest.php        | 8 ++++----
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/apps/settings/lib/Settings/Admin/Sharing.php b/apps/settings/lib/Settings/Admin/Sharing.php
index 9fc6e1fe4c5..34c91f3bce9 100644
--- a/apps/settings/lib/Settings/Admin/Sharing.php
+++ b/apps/settings/lib/Settings/Admin/Sharing.php
@@ -53,8 +53,8 @@ class Sharing implements IDelegatedSettings {
 			'restrictUserEnumerationFullMatchEmail' => $this->getHumanBooleanConfig('core', 'shareapi_restrict_user_enumeration_full_match_email', true),
 			'restrictUserEnumerationFullMatchIgnoreSecondDN' => $this->getHumanBooleanConfig('core', 'shareapi_restrict_user_enumeration_full_match_ignore_second_dn'),
 			'enforceLinksPassword' => Util::isPublicLinkPasswordRequired(false),
-			'passwordExcludedGroups' => json_decode($excludedPasswordGroups) ?? [],
-			'passwordExcludedGroupsFeatureEnabled' => $this->config->getSystemValueBool('sharing.allow_disabled_password_enforcement_groups', false),
+			'enforceLinksPasswordExcludedGroups' => json_decode($excludedPasswordGroups) ?? [],
+			'enforceLinksPasswordExcludedGroupsEnabled' => $this->config->getSystemValueBool('sharing.allow_disabled_password_enforcement_groups', false),
 			'onlyShareWithGroupMembers' => $this->shareManager->shareWithGroupMembersOnly(),
 			'onlyShareWithGroupMembersExcludeGroupList' => json_decode($onlyShareWithGroupMembersExcludeGroupList) ?? [],
 			'defaultExpireDate' => $this->getHumanBooleanConfig('core', 'shareapi_default_expire_date'),
diff --git a/apps/settings/src/components/AdminSettingsSharingForm.vue b/apps/settings/src/components/AdminSettingsSharingForm.vue
index 8a274af88a8..93f30b2262c 100644
--- a/apps/settings/src/components/AdminSettingsSharingForm.vue
+++ b/apps/settings/src/components/AdminSettingsSharingForm.vue
@@ -45,9 +45,9 @@
 				<NcCheckboxRadioSwitch :checked.sync="settings.enforceLinksPassword" :disabled="!settings.enableLinkPasswordByDefault">
 					{{ t('settings', 'Enforce password protection') }}
 				</NcCheckboxRadioSwitch>
-				<label v-if="settings.passwordExcludedGroupsFeatureEnabled" class="sharing__labeled-entry sharing__input">
+				<label v-if="settings.enforceLinksPasswordExcludedGroupsEnabled" class="sharing__labeled-entry sharing__input">
 					<span>{{ t('settings', 'Exclude groups from password requirements') }}</span>
-					<NcSettingsSelectGroup v-model="settings.passwordExcludedGroups"
+					<NcSettingsSelectGroup v-model="settings.enforceLinksPasswordExcludedGroups"
 						style="width: 100%"
 						:disabled="!settings.enforceLinksPassword || !settings.enableLinkPasswordByDefault" />
 				</label>
@@ -222,8 +222,8 @@ interface IShareSettings {
 	restrictUserEnumerationFullMatchEmail: boolean
 	restrictUserEnumerationFullMatchIgnoreSecondDN: boolean
 	enforceLinksPassword: boolean
-	passwordExcludedGroups: string[]
-	passwordExcludedGroupsFeatureEnabled: boolean
+	enforceLinksPasswordExcludedGroups: string[]
+	enforceLinksPasswordExcludedGroupsEnabled: boolean
 	onlyShareWithGroupMembers: boolean
 	onlyShareWithGroupMembersExcludeGroupList: string[]
 	defaultExpireDate: boolean
diff --git a/apps/settings/tests/Settings/Admin/SharingTest.php b/apps/settings/tests/Settings/Admin/SharingTest.php
index d988ed15aed..9db7a257806 100644
--- a/apps/settings/tests/Settings/Admin/SharingTest.php
+++ b/apps/settings/tests/Settings/Admin/SharingTest.php
@@ -136,9 +136,9 @@ class SharingTest extends TestCase {
 						'remoteExpireAfterNDays' => '7',
 						'enforceRemoteExpireDate' => false,
 						'allowLinksExcludeGroups' => [],
-						'passwordExcludedGroups' => [],
-						'passwordExcludedGroupsFeatureEnabled' => false,
 						'onlyShareWithGroupMembersExcludeGroupList' => [],
+						'enforceLinksPasswordExcludedGroups' => [],
+						'enforceLinksPasswordExcludedGroupsEnabled' => false,
 					]
 				],
 			);
@@ -229,9 +229,9 @@ class SharingTest extends TestCase {
 						'remoteExpireAfterNDays' => '7',
 						'enforceRemoteExpireDate' => false,
 						'allowLinksExcludeGroups' => [],
-						'passwordExcludedGroups' => [],
-						'passwordExcludedGroupsFeatureEnabled' => false,
 						'onlyShareWithGroupMembersExcludeGroupList' => [],
+						'enforceLinksPasswordExcludedGroups' => [],
+						'enforceLinksPasswordExcludedGroupsEnabled' => false,
 					]
 				],
 			);