fix(files_sharing): adjust IAttributes API and files_versions

skjnldsv · skjnldsv · commit 9b84831c8d0c · 2024-07-12T20:14:30.000+02:00
Signed-off-by: skjnldsv &lt;skjnldsv@protonmail.com&gt;
diff --git a/apps/files/js/fileinfomodel.js b/apps/files/js/fileinfomodel.js
@@ -128,7 +128,7 @@
 			for (const i in this.attributes.shareAttributes) {
 				const attr = this.attributes.shareAttributes[i]
 				if (attr.scope === 'permissions' && attr.key === 'download') {
-					return attr.enabled
+					return attr.value === true
 				}
 			}
 
diff --git a/apps/files/src/actions/downloadAction.ts b/apps/files/src/actions/downloadAction.ts
@@ -33,7 +33,7 @@ const isDownloadable = function(node: Node) {
 	if (node.attributes['mount-type'] === 'shared') {
 		const shareAttributes = JSON.parse(node.attributes['share-attributes'] ?? 'null')
 		const downloadAttribute = shareAttributes?.find?.((attribute: { scope: string; key: string }) => attribute.scope === 'permissions' && attribute.key === 'download')
-		if (downloadAttribute !== undefined && downloadAttribute.enabled === false) {
+		if (downloadAttribute !== undefined && downloadAttribute.value === false) {
 			return false
 		}
 	}
diff --git a/apps/files/src/actions/moveOrCopyActionUtils.ts b/apps/files/src/actions/moveOrCopyActionUtils.ts
@@ -24,7 +24,7 @@ export const getQueue = () => {
 }
 
 type ShareAttribute = {
-	enabled: boolean
+	value: any
 	key: string
 	scope: string
 }
@@ -48,7 +48,7 @@ export const canMove = (nodes: Node[]) => {
 export const canDownload = (nodes: Node[]) => {
 	return nodes.every(node => {
 		const shareAttributes = JSON.parse(node.attributes?.['share-attributes'] ?? '[]') as Array<ShareAttribute>
-		return !shareAttributes.some(attribute => attribute.scope === 'permissions' && attribute.enabled === false && attribute.key === 'download')
+		return !shareAttributes.some(attribute => attribute.scope === 'permissions' && attribute.value === false && attribute.key === 'download')
 
 	})
 }
diff --git a/apps/files/src/views/FilesList.vue b/apps/files/src/views/FilesList.vue
@@ -396,14 +396,14 @@ export default defineComponent({
 			return { ...this.$route, query: { dir } }
 		},
 
-		shareAttributes(): number[] | undefined {
+		shareTypesAttributes(): number[] | undefined {
 			if (!this.currentFolder?.attributes?.['share-types']) {
 				return undefined
 			}
 			return Object.values(this.currentFolder?.attributes?.['share-types'] || {}).flat() as number[]
 		},
 		shareButtonLabel() {
-			if (!this.shareAttributes) {
+			if (!this.shareTypesAttributes) {
 				return t('files', 'Share')
 			}
 
@@ -413,12 +413,12 @@ export default defineComponent({
 			return t('files', 'Shared')
 		},
 		shareButtonType(): Type | null {
-			if (!this.shareAttributes) {
+			if (!this.shareTypesAttributes) {
 				return null
 			}
 
 			// If all types are links, show the link icon
-			if (this.shareAttributes.some(type => type === Type.SHARE_TYPE_LINK)) {
+			if (this.shareTypesAttributes.some(type => type === Type.SHARE_TYPE_LINK)) {
 				return Type.SHARE_TYPE_LINK
 			}
 
diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -1055,7 +1055,7 @@ public function getInheritedShares(string $path): DataResponse {
 		}
 
 		if (!($node->getPermissions() & Constants::PERMISSION_SHARE)) {
-			throw new SharingRightsException('no sharing rights on this item');
+			throw new SharingRightsException($this->l->t('no sharing rights on this item'));
 		}
 
 		// The current top parent we have access to
@@ -1171,7 +1171,7 @@ public function updateShare(
 		}
 
 		if (!$this->canEditShare($share)) {
-			throw new OCSForbiddenException('You are not allowed to edit incoming shares');
+			throw new OCSForbiddenException($this->l->t('You are not allowed to edit incoming shares'));
 		}
 
 		if (
@@ -1218,7 +1218,7 @@ public function updateShare(
 			 */
 
 			if ($share->getSharedBy() !== $this->currentUser) {
-				throw new OCSForbiddenException('You are not allowed to edit link shares that you don\'t own');
+				throw new OCSForbiddenException($this->l->t('You are not allowed to edit link shares that you don\'t own'));
 			}
 
 			// Update hide download state
@@ -1640,7 +1640,7 @@ private function parseDate(string $expireDate): \DateTime {
 			// Make sure it expires at midnight in owner timezone
 			$date->setTime(0, 0, 0);
 		} catch (\Exception $e) {
-			throw new \Exception('Invalid date. Format must be YYYY-MM-DD');
+			throw new \Exception($this->l->t('Invalid date. Format must be YYYY-MM-DD'));
 		}
 
 		return $date;
@@ -1845,7 +1845,7 @@ private function getSharesFromNode(string $viewer, $node, bool $reShares): array
 	 */
 	private function confirmSharingRights(Node $node): void {
 		if (!$this->hasResharingRights($this->currentUser, $node)) {
-			throw new SharingRightsException('no sharing rights on this item');
+			throw new SharingRightsException($this->l->t('No sharing rights on this item'));
 		}
 	}
 
@@ -2008,21 +2008,14 @@ private function setShareAttributes(IShare $share, ?string $attributesString) {
 			$formattedShareAttributes = \json_decode($attributesString, true);
 			if (is_array($formattedShareAttributes)) {
 				foreach ($formattedShareAttributes as $formattedAttr) {
-					// Legacy handling of the 'enabled' attribute
-					if (array_key_exists('enabled', $formattedAttr)) {
-						$formattedAttr['value'] = is_string($formattedAttr['enabled'])
-							? (bool) \json_decode($formattedAttr['enabled'])
-							: $formattedAttr['enabled'];
-					}
-
 					$newShareAttributes->setAttribute(
 						$formattedAttr['scope'],
 						$formattedAttr['key'],
 						$formattedAttr['value'],
 					);
 				}
 			} else {
-				throw new OCSBadRequestException('Invalid share attributes provided: \"' . $attributesString . '\"');
+				throw new OCSBadRequestException($this->l->t('Invalid share attributes provided: "%s"', [$attributesString]));
 			}
 		}
 		$share->setAttributes($newShareAttributes);
@@ -2044,10 +2037,10 @@ private function checkInheritedAttributes(IShare $share): void {
 			if ($storage instanceof Wrapper) {
 				$storage = $storage->getInstanceOfStorage(SharedStorage::class);
 				if ($storage === null) {
-					throw new \RuntimeException('Should not happen, instanceOfStorage but getInstanceOfStorage return null');
+					throw new \RuntimeException($this->l->t('Should not happen, instanceOfStorage but getInstanceOfStorage return null'));
 				}
 			} else {
-				throw new \RuntimeException('Should not happen, instanceOfStorage but not a wrapper');
+				throw new \RuntimeException($this->l->t('Should not happen, instanceOfStorage but not a wrapper'));
 			}
 			/** @var \OCA\Files_Sharing\SharedStorage $storage */
 			$inheritedAttributes = $storage->getShare()->getAttributes();
@@ -2085,15 +2078,15 @@ public function sendShareEmail(string $id, $password = ''): DataResponse {
 			}
 	
 			if (!$this->canEditShare($share)) {
-				throw new OCSForbiddenException('You are not allowed to send mail notifications');
+				throw new OCSForbiddenException($this->l->t('You are not allowed to send mail notifications'));
 			}
 
 			// For mail and link shares, the user must be
 			// the owner of the share, not only the file owner.
 			if ($share->getShareType() === IShare::TYPE_EMAIL
 				|| $share->getShareType() === IShare::TYPE_LINK) {
 				if ($share->getSharedBy() !== $this->currentUser) {
-					throw new OCSForbiddenException('You are not allowed to send mail notifications');
+					throw new OCSForbiddenException($this->l->t('You are not allowed to send mail notifications'));
 				}
 			}
 
diff --git a/apps/files_sharing/tests/ApiTest.php b/apps/files_sharing/tests/ApiTest.php
@@ -960,7 +960,7 @@ public function testUpdateShare() {
 		$ocs = $this->createOCS(self::TEST_FILES_SHARING_API_USER1);
 		$ocs->updateShare(
 			$share1->getId(), 1, null, null, null, null, null, null, null,
-			'[{"scope": "app1", "key": "attr1", "enabled": true}]'
+			'[{"scope": "app1", "key": "attr1", "value": true}]'
 		);
 		$ocs->cleanup();
 
diff --git a/apps/files_versions/src/components/Version.vue b/apps/files_versions/src/components/Version.vue
@@ -259,7 +259,7 @@ export default defineComponent({
 				const downloadAttribute = this.fileInfo.shareAttributes
 					.find((attribute) => attribute.scope === 'permissions' && attribute.key === 'download') || {}
 				// If the download attribute is set to false, the file is not downloadable
-				if (downloadAttribute?.enabled === false) {
+				if (downloadAttribute?.value === false) {
 					return false
 				}
 			}
diff --git a/build/integration/features/bootstrap/Sharing.php b/build/integration/features/bootstrap/Sharing.php
@@ -286,7 +286,7 @@ public function createShare($user,
 		}
 
 		if ($viewOnly === true) {
-			$body['attributes'] = json_encode([['scope' => 'permissions', 'key' => 'download', 'enabled' => false]]);
+			$body['attributes'] = json_encode([['scope' => 'permissions', 'key' => 'download', 'value' => false]]);
 		}
 
 		$options['form_params'] = $body;
diff --git a/core/src/files/fileinfo.js b/core/src/files/fileinfo.js
@@ -147,7 +147,7 @@
 			for (const i in this.shareAttributes) {
 				const attr = this.shareAttributes[i]
 				if (attr.scope === 'permissions' && attr.key === 'download') {
-					return attr.enabled
+					return attr.value === true
 				}
 			}
 
diff --git a/cypress/e2e/files_sharing/filesSharingUtils.ts b/cypress/e2e/files_sharing/filesSharingUtils.ts
@@ -33,6 +33,8 @@ export function createShare(fileName: string, username: string, shareSettings: P
 export function updateShare(fileName: string, index: number, shareSettings: Partial<ShareSetting> = {}) {
 	openSharingPanel(fileName)
 
+	cy.intercept({ times: 1, method: 'PUT', url: '**/apps/files_sharing/api/v1/shares/*' }).as('updateShare')
+
 	cy.get('#app-sidebar-vue').within(() => {
 		cy.get('[data-cy-files-sharing-share-actions]').eq(index).click()
 		cy.get('[data-cy-files-sharing-share-permissions-bundle="custom"]').click()
@@ -82,6 +84,8 @@ export function updateShare(fileName: string, index: number, shareSettings: Part
 		}
 
 		cy.get('[data-cy-files-sharing-share-editor-action="save"]').click({ scrollBehavior: 'nearest' })
+
+		cy.wait('@updateShare')
 	})
 }
 
diff --git a/cypress/e2e/files_versions/filesVersionsUtils.ts b/cypress/e2e/files_versions/filesVersionsUtils.ts
@@ -4,7 +4,6 @@
  */
 /* eslint-disable jsdoc/require-jsdoc */
 import type { User } from '@nextcloud/cypress'
-import path from 'path'
 import { createShare, type ShareSetting } from '../files_sharing/filesSharingUtils'
 
 export const uploadThreeVersions = (user: User, fileName: string) => {
diff --git a/dist/files_versions-files_versions.js b/dist/files_versions-files_versions.js
diff --git a/dist/files_versions-files_versions.js.map b/dist/files_versions-files_versions.js.map
diff --git a/lib/private/Share20/ShareAttributes.php b/lib/private/Share20/ShareAttributes.php
@@ -20,7 +20,7 @@ public function __construct() {
 	/**
 	 * @inheritdoc
 	 */
-	public function setAttribute($scope, $key, $value) {
+	public function setAttribute(string $scope, string $key, mixed $value): IAttributes {
 		if (!\array_key_exists($scope, $this->attributes)) {
 			$this->attributes[$scope] = [];
 		}
@@ -31,7 +31,7 @@ public function setAttribute($scope, $key, $value) {
 	/**
 	 * @inheritdoc
 	 */
-	public function getAttribute($scope, $key) {
+	public function getAttribute(string $scope, string $key): mixed {
 		if (\array_key_exists($scope, $this->attributes) &&
 			\array_key_exists($key, $this->attributes[$scope])) {
 			return $this->attributes[$scope][$key];
@@ -42,14 +42,14 @@ public function getAttribute($scope, $key) {
 	/**
 	 * @inheritdoc
 	 */
-	public function toArray() {
+	public function toArray(): array {
 		$result = [];
 		foreach ($this->attributes as $scope => $keys) {
 			foreach ($keys as $key => $value) {
 				$result[] = [
 					"scope" => $scope,
 					"key" => $key,
-					"value" => $value
+					"value" => $value,
 				];
 			}
 		}
diff --git a/lib/public/Share/IAttributes.php b/lib/public/Share/IAttributes.php
@@ -13,40 +13,41 @@
  */
 interface IAttributes {
 	/**
-	 * Sets an attribute enabled/disabled. If the key did not exist before it will be created.
+	 * Sets an attribute. If the key did not exist before it will be created.
 	 *
 	 * @param string $scope scope
 	 * @param string $key key
 	 * @param bool|string|array|null $value value
 	 * @return IAttributes The modified object
 	 * @since 25.0.0
 	 */
-	public function setAttribute($scope, $key, $value);
+	public function setAttribute(string $scope, string $key, mixed $value): IAttributes;
 
 	/**
-	 * Returns if attribute is enabled/disabled for given scope id and key.
+	 * Returns the attribute for given scope id and key.
 	 * If attribute does not exist, returns null
 	 *
 	 * @param string $scope scope
 	 * @param string $key key
 	 * @return bool|string|array|null
 	 * @since 25.0.0
 	 */
-	public function getAttribute($scope, $key);
+	public function getAttribute(string $scope, string $key): mixed;
 
 	/**
 	 * Formats the IAttributes object to array with the following format:
 	 * [
 	 * 	0 => [
 	 * 			"scope" => <string>,
 	 * 			"key" => <string>,
-	 * 			"enabled" => <bool>
+	 * 			"value" => <bool|string|array|null>,
 	 * 		],
 	 * 	...
 	 * ]
 	 *
 	 * @return array formatted IAttributes
 	 * @since 25.0.0
+	 * @since 30.0.0, `enabled` was renamed to `value`
 	 */
-	public function toArray();
+	public function toArray(): array;
 }

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@`
`128`	`128`	`for (const i in this.attributes.shareAttributes) {`
`129`	`129`	`const attr = this.attributes.shareAttributes[i]`
`130`	`130`	`if (attr.scope === 'permissions' && attr.key === 'download') {`
`131`		`- return attr.enabled`
	`131`	`+ return attr.value === true`
`132`	`132`	`}`
`133`	`133`	`}`
`134`	`134`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ const isDownloadable = function(node: Node) {`
`33`	`33`	`if (node.attributes['mount-type'] === 'shared') {`
`34`	`34`	`const shareAttributes = JSON.parse(node.attributes['share-attributes'] ?? 'null')`
`35`	`35`	`const downloadAttribute = shareAttributes?.find?.((attribute: { scope: string; key: string }) => attribute.scope === 'permissions' && attribute.key === 'download')`
`36`		`- if (downloadAttribute !== undefined && downloadAttribute.enabled === false) {`
	`36`	`+ if (downloadAttribute !== undefined && downloadAttribute.value === false) {`
`37`	`37`	`return false`
`38`	`38`	`}`
`39`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ export const getQueue = () => {`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`type ShareAttribute = {`
`27`		`- enabled: boolean`
	`27`	`+ value: any`
`28`	`28`	`key: string`
`29`	`29`	`scope: string`
`30`	`30`	`}`
`@@ -48,7 +48,7 @@ export const canMove = (nodes: Node[]) => {`
`48`	`48`	`export const canDownload = (nodes: Node[]) => {`
`49`	`49`	`return nodes.every(node => {`
`50`	`50`	`const shareAttributes = JSON.parse(node.attributes?.['share-attributes'] ?? '[]') as Array<ShareAttribute>`
`51`		`- return !shareAttributes.some(attribute => attribute.scope === 'permissions' && attribute.enabled === false && attribute.key === 'download')`
	`51`	`+ return !shareAttributes.some(attribute => attribute.scope === 'permissions' && attribute.value === false && attribute.key === 'download')`
`52`	`52`
`53`	`53`	`})`
`54`	`54`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1055,7 +1055,7 @@ public function getInheritedShares(string $path): DataResponse {`
`1055`	`1055`	`}`
`1056`	`1056`
`1057`	`1057`	`if (!($node->getPermissions() & Constants::PERMISSION_SHARE)) {`
`1058`		`- throw new SharingRightsException('no sharing rights on this item');`
	`1058`	`+ throw new SharingRightsException($this->l->t('no sharing rights on this item'));`
`1059`	`1059`	`}`
`1060`	`1060`
`1061`	`1061`	`// The current top parent we have access to`
`@@ -1171,7 +1171,7 @@ public function updateShare(`
`1171`	`1171`	`}`
`1172`	`1172`
`1173`	`1173`	`if (!$this->canEditShare($share)) {`
`1174`		`- throw new OCSForbiddenException('You are not allowed to edit incoming shares');`
	`1174`	`+ throw new OCSForbiddenException($this->l->t('You are not allowed to edit incoming shares'));`
`1175`	`1175`	`}`
`1176`	`1176`
`1177`	`1177`	`if (`
`@@ -1218,7 +1218,7 @@ public function updateShare(`
`1218`	`1218`	`*/`
`1219`	`1219`
`1220`	`1220`	`if ($share->getSharedBy() !== $this->currentUser) {`
`1221`		`- throw new OCSForbiddenException('You are not allowed to edit link shares that you don\'t own');`
	`1221`	`+ throw new OCSForbiddenException($this->l->t('You are not allowed to edit link shares that you don\'t own'));`
`1222`	`1222`	`}`
`1223`	`1223`
`1224`	`1224`	`// Update hide download state`
`@@ -1640,7 +1640,7 @@ private function parseDate(string $expireDate): \DateTime {`
`1640`	`1640`	`// Make sure it expires at midnight in owner timezone`
`1641`	`1641`	`$date->setTime(0, 0, 0);`
`1642`	`1642`	`} catch (\Exception $e) {`
`1643`		`- throw new \Exception('Invalid date. Format must be YYYY-MM-DD');`
	`1643`	`+ throw new \Exception($this->l->t('Invalid date. Format must be YYYY-MM-DD'));`
`1644`	`1644`	`}`
`1645`	`1645`
`1646`	`1646`	`return $date;`
`@@ -1845,7 +1845,7 @@ private function getSharesFromNode(string $viewer, $node, bool $reShares): array`
`1845`	`1845`	`*/`
`1846`	`1846`	`private function confirmSharingRights(Node $node): void {`
`1847`	`1847`	`if (!$this->hasResharingRights($this->currentUser, $node)) {`
`1848`		`- throw new SharingRightsException('no sharing rights on this item');`
	`1848`	`+ throw new SharingRightsException($this->l->t('No sharing rights on this item'));`
`1849`	`1849`	`}`
`1850`	`1850`	`}`
`1851`	`1851`
`@@ -2008,21 +2008,14 @@ private function setShareAttributes(IShare $share, ?string $attributesString) {`
`2008`	`2008`	`$formattedShareAttributes = \json_decode($attributesString, true);`
`2009`	`2009`	`if (is_array($formattedShareAttributes)) {`
`2010`	`2010`	`foreach ($formattedShareAttributes as $formattedAttr) {`
`2011`		`- // Legacy handling of the 'enabled' attribute`
`2012`		`- if (array_key_exists('enabled', $formattedAttr)) {`
`2013`		`- $formattedAttr['value'] = is_string($formattedAttr['enabled'])`
`2014`		`- ? (bool) \json_decode($formattedAttr['enabled'])`
`2015`		`- : $formattedAttr['enabled'];`
`2016`		`- }`
`2017`		`-`
`2018`	`2011`	`$newShareAttributes->setAttribute(`
`2019`	`2012`	`$formattedAttr['scope'],`
`2020`	`2013`	`$formattedAttr['key'],`
`2021`	`2014`	`$formattedAttr['value'],`
`2022`	`2015`	`);`
`2023`	`2016`	`}`
`2024`	`2017`	`} else {`
`2025`		`- throw new OCSBadRequestException('Invalid share attributes provided: \"' . $attributesString . '\"');`
	`2018`	`+ throw new OCSBadRequestException($this->l->t('Invalid share attributes provided: "%s"', [$attributesString]));`
`2026`	`2019`	`}`
`2027`	`2020`	`}`
`2028`	`2021`	`$share->setAttributes($newShareAttributes);`
`@@ -2044,10 +2037,10 @@ private function checkInheritedAttributes(IShare $share): void {`
`2044`	`2037`	`if ($storage instanceof Wrapper) {`
`2045`	`2038`	`$storage = $storage->getInstanceOfStorage(SharedStorage::class);`
`2046`	`2039`	`if ($storage === null) {`
`2047`		`- throw new \RuntimeException('Should not happen, instanceOfStorage but getInstanceOfStorage return null');`
	`2040`	`+ throw new \RuntimeException($this->l->t('Should not happen, instanceOfStorage but getInstanceOfStorage return null'));`
`2048`	`2041`	`}`
`2049`	`2042`	`} else {`
`2050`		`- throw new \RuntimeException('Should not happen, instanceOfStorage but not a wrapper');`
	`2043`	`+ throw new \RuntimeException($this->l->t('Should not happen, instanceOfStorage but not a wrapper'));`
`2051`	`2044`	`}`
`2052`	`2045`	`/** @var \OCA\Files_Sharing\SharedStorage $storage */`
`2053`	`2046`	`$inheritedAttributes = $storage->getShare()->getAttributes();`
`@@ -2085,15 +2078,15 @@ public function sendShareEmail(string $id, $password = ''): DataResponse {`
`2085`	`2078`	`}`
`2086`	`2079`
`2087`	`2080`	`if (!$this->canEditShare($share)) {`
`2088`		`- throw new OCSForbiddenException('You are not allowed to send mail notifications');`
	`2081`	`+ throw new OCSForbiddenException($this->l->t('You are not allowed to send mail notifications'));`
`2089`	`2082`	`}`
`2090`	`2083`
`2091`	`2084`	`// For mail and link shares, the user must be`
`2092`	`2085`	`// the owner of the share, not only the file owner.`
`2093`	`2086`	`if ($share->getShareType() === IShare::TYPE_EMAIL`
`2094`	`2087`	`\|\| $share->getShareType() === IShare::TYPE_LINK) {`
`2095`	`2088`	`if ($share->getSharedBy() !== $this->currentUser) {`
`2096`		`- throw new OCSForbiddenException('You are not allowed to send mail notifications');`
	`2089`	`+ throw new OCSForbiddenException($this->l->t('You are not allowed to send mail notifications'));`
`2097`	`2090`	`}`
`2098`	`2091`	`}`
`2099`	`2092`
Original file line number	Diff line number	Diff line change
`@@ -259,7 +259,7 @@ export default defineComponent({`
`259`	`259`	`const downloadAttribute = this.fileInfo.shareAttributes`
`260`	`260`	`.find((attribute) => attribute.scope === 'permissions' && attribute.key === 'download') \|\| {}`
`261`	`261`	`// If the download attribute is set to false, the file is not downloadable`
`262`		`- if (downloadAttribute?.enabled === false) {`
	`262`	`+ if (downloadAttribute?.value === false) {`
`263`	`263`	`return false`
`264`	`264`	`}`
`265`	`265`	`}`
Original file line number	Diff line number	Diff line change
`@@ -286,7 +286,7 @@ public function createShare($user,`
`286`	`286`	`}`
`287`	`287`
`288`	`288`	`if ($viewOnly === true) {`
`289`		`- $body['attributes'] = json_encode([['scope' => 'permissions', 'key' => 'download', 'enabled' => false]]);`
	`289`	`+ $body['attributes'] = json_encode([['scope' => 'permissions', 'key' => 'download', 'value' => false]]);`
`290`	`290`	`}`
`291`	`291`
`292`	`292`	`$options['form_params'] = $body;`
Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@`
`147`	`147`	`for (const i in this.shareAttributes) {`
`148`	`148`	`const attr = this.shareAttributes[i]`
`149`	`149`	`if (attr.scope === 'permissions' && attr.key === 'download') {`
`150`		`- return attr.enabled`
	`150`	`+ return attr.value === true`
`151`	`151`	`}`
`152`	`152`	`}`
`153`	`153`
Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,8 @@ export function createShare(fileName: string, username: string, shareSettings: P`
`33`	`33`	`export function updateShare(fileName: string, index: number, shareSettings: Partial<ShareSetting> = {}) {`
`34`	`34`	`openSharingPanel(fileName)`
`35`	`35`
	`36`	`+ cy.intercept({ times: 1, method: 'PUT', url: '*/apps/files_sharing/api/v1/shares/' }).as('updateShare')`
	`37`	`+`
`36`	`38`	`cy.get('#app-sidebar-vue').within(() => {`
`37`	`39`	`cy.get('[data-cy-files-sharing-share-actions]').eq(index).click()`
`38`	`40`	`cy.get('[data-cy-files-sharing-share-permissions-bundle="custom"]').click()`
`@@ -82,6 +84,8 @@ export function updateShare(fileName: string, index: number, shareSettings: Part`
`82`	`84`	`}`
`83`	`85`
`84`	`86`	`cy.get('[data-cy-files-sharing-share-editor-action="save"]').click({ scrollBehavior: 'nearest' })`
	`87`	`+`
	`88`	`+ cy.wait('@updateShare')`
`85`	`89`	`})`
`86`	`90`	`}`
`87`	`91`