Skip to content

Commit 3de5bdc

Browse files
committed
Fix disabled federated scope
Signed-off-by: Christopher Ng <chrng8@gmail.com>
1 parent 7835223 commit 3de5bdc

File tree

4 files changed

+21
-12
lines changed

4 files changed

+21
-12
lines changed

apps/settings/lib/Settings/Personal/PersonalInfo.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,6 @@ public function getForm(): TemplateResponse {
138138
$messageParameters = $this->getMessageParameters($account);
139139

140140
$parameters = [
141-
'federationEnabled' => $federationEnabled,
142141
'lookupServerUploadEnabled' => $lookupServerUploadEnabled,
143142
'isFairUseOfFreePushService' => $this->isFairUseOfFreePushService(),
144143
'profileEnabledGlobally' => $this->profileManager->isProfileEnabled(),
@@ -171,6 +170,7 @@ public function getForm(): TemplateResponse {
171170
$accountParameters = [
172171
'avatarChangeSupported' => $user->canChangeAvatar(),
173172
'displayNameChangeSupported' => $user->canChangeDisplayName(),
173+
'federationEnabled' => $federationEnabled,
174174
'lookupServerUploadEnabled' => $lookupServerUploadEnabled,
175175
];
176176

apps/settings/src/components/PersonalInfo/shared/FederationControl.vue

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,10 @@ import {
5757
import { savePrimaryAccountPropertyScope } from '../../../service/PersonalInfo/PersonalInfoService.js'
5858
import logger from '../../../logger.js'
5959
60-
const { lookupServerUploadEnabled } = loadState('settings', 'accountParameters', {})
60+
const {
61+
federationEnabled,
62+
lookupServerUploadEnabled,
63+
} = loadState('settings', 'accountParameters', {})
6164
6265
export default {
6366
name: 'FederationControl',
@@ -120,15 +123,21 @@ export default {
120123
},
121124
122125
supportedScopes() {
123-
if (lookupServerUploadEnabled && !UNPUBLISHED_READABLE_PROPERTIES.includes(this.readable)) {
124-
return [
125-
...PROPERTY_READABLE_SUPPORTED_SCOPES_ENUM[this.readable],
126-
SCOPE_ENUM.FEDERATED,
127-
SCOPE_ENUM.PUBLISHED,
128-
]
126+
const scopes = PROPERTY_READABLE_SUPPORTED_SCOPES_ENUM[this.readable]
127+
128+
if (UNPUBLISHED_READABLE_PROPERTIES.includes(this.readable)) {
129+
return scopes
130+
}
131+
132+
if (federationEnabled) {
133+
scopes.push(SCOPE_ENUM.FEDERATED)
134+
}
135+
136+
if (lookupServerUploadEnabled) {
137+
scopes.push(SCOPE_ENUM.PUBLISHED)
129138
}
130139
131-
return PROPERTY_READABLE_SUPPORTED_SCOPES_ENUM[this.readable]
140+
return scopes
132141
},
133142
},
134143

dist/settings-vue-settings-personal-info.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/settings-vue-settings-personal-info.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)