diff options
| author | fenn-cs <fenn25.fn@gmail.com> | 2023-10-20 10:34:57 +0100 |
|---|---|---|
| committer | fenn-cs <fenn25.fn@gmail.com> | 2023-10-28 09:53:42 +0100 |
| commit | 170a83a77ee77fbb50a42a10fb814834f0d8a744 (patch) | |
| tree | b6a4d8f76aaa62034fe98528303f26b45ce4f496 /apps | |
| parent | a5a956406987fb3bb68200aab9ab94e2cdc4e609 (diff) | |
| download | nextcloud-server-170a83a77ee77fbb50a42a10fb814834f0d8a744.tar.gz nextcloud-server-170a83a77ee77fbb50a42a10fb814834f0d8a744.zip | |
Do not treat `isDefaultExpireDateEnforced` as fallback
`isDefaultExpireDateEnforced` and its corresponding `defaultExpirationDate`
is currently treated as the enforcement fallback when share type enforcements are not
set.
However, `isDefaultExpireDateEnforced` and `defaultExpirationDate` are actually more like
`isDefaultPublicExpireDateEnforced` and `defaultPublicExpirationDate` and therefore only applies
to public shares.
It might be ideal to rename this variables all the way from the backend config to the way we use
them in the frontend code.
Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/files_sharing/src/mixins/SharesMixin.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/apps/files_sharing/src/mixins/SharesMixin.js b/apps/files_sharing/src/mixins/SharesMixin.js index 2dfcc3e9c0e..860d1716fe0 100644 --- a/apps/files_sharing/src/mixins/SharesMixin.js +++ b/apps/files_sharing/src/mixins/SharesMixin.js @@ -140,9 +140,9 @@ export default { return this.config.isDefaultExpireDateEnforced } if (this.isRemoteShare) { - return this.config.isDefaultRemoteExpireDateEnforced || this.config.isDefaultExpireDateEnforced + return this.config.isDefaultRemoteExpireDateEnforced } - return this.config.isDefaultInternalExpireDateEnforced || this.config.isDefaultExpireDateEnforced + return this.config.isDefaultInternalExpireDateEnforced }, hasCustomPermissions() { const bundledPermissions = [ |