aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/Controller
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #49898 from nextcloud/fix/noid/no-emails-for-user-sharesAnna2025-01-161-4/+7
|\ | | | | fix(sharing): send share emails for internal users too
| * fix(sharing): send share emails for internal users toofix/noid/no-emails-for-user-sharesAnna Larch2025-01-131-4/+7
| | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* | feat(sharing): Respect admin custom tokens settingChristopher Ng2025-01-151-0/+3
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | feat(sharing): Fetch a unique generated tokenChristopher Ng2025-01-151-0/+24
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | feat(sharing): Allow updating share tokenChristopher Ng2025-01-151-1/+21
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | fix(files_sharing): Handle null userIdprovokateurin2025-01-132-11/+2
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(OpenAPI): Adjust scopes to match previous behaviorprovokateurin2025-01-062-0/+5
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: remove legacy action to test remote endpointBenjamin Gaussorgues2024-11-291-66/+0
| | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* chore: stricter access to testremote endpointBenjamin Gaussorgues2024-11-281-1/+2
| | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* feat(files_sharing): Allow users with share permission to manage shares on ↵provokateurin2024-11-251-0/+13
| | | | | | IShareOwnerlessMount Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files_sharing): Remove duplicate link/email edit share logicprovokateurin2024-11-251-12/+0
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files_sharing): Fix error messages from password policyfix/fix-share-creation-error-messagesCôme Chilliet2024-11-191-4/+4
| | | | | | | | | Exceptions thrown from password_policy may bubble up in share creation or update when a password is used. Their message is meant to be shown to the user. This always the case for HintException so we catch that instead of the subclass GenericShareException. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(OpenAPI): Adjust array syntax to avoid ambiguitiesfix/openapi/array-syntaxprovokateurin2024-11-056-33/+33
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Adjust preview for view-only sharesfix/view-only-previewFerdinand Thiessen2024-10-281-2/+14
| | | | | | | | | | | | | | | | | Previously there was a different behavior for public shares (link-shares) and internal shares, if the user disabled the view permission. The legacy UI for public shares simply "disabled" the context menu and hided all download actions. With Nextcloud 31 all share types use the consistent permissions attributes, which simplifies code, but caused a regression: Images can no longer been viewed. Because on 30 and before the attribute was not set, previews for view-only files were still allowed. Now with 31 we need a new way to allow "viewing" shares. So this is allowing previews for those files, but only for internal usage. This is done by settin a special header, which only works with custom requests, and not by opening the URL directly. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): Cleanup error messagesfix/files_sharing/cleanup-error-messagesprovokateurin2024-10-281-9/+9
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(ShareApiController): Check for null and empty strings with empty()safety-net-null-checkfenn-cs2024-10-241-7/+4
| | | | | | Proactive measure to avoid warnings in higher php versions as well possible type errors Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* fix(ShareAPI): Send mails for mail shares by defaultfix/48012/fix-share-email-send-mail-sharefenn-cs2024-10-241-1/+10
| | | | | | | | | | | | | | It looks like, the frontend it needs to provide the `sendMail` param for the backend to decide wether mails would be sent. Our UI does not have that at the moment so it should default to sending emails always for mail shares. Not exactly sure how this was handled earlier but this is a good starting point. Resolves : https://github.com/nextcloud/server/issues/48012 Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* refactor(apps): Use constructor property promotion when possibleprovokateurin2024-10-217-166/+114
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-155-30/+40
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* feat(files_sharing): Make `ShareController` download route use the DAV ↵Ferdinand Thiessen2024-09-281-91/+24
| | | | | | `ZipFolderPlugin` Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-192-5/+5
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files_sharing): adjust permissions from custom edit and delete check methodsskjnldsv2024-09-111-0/+17
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix: Display 'Leave share' instead of 'Delete'Marcel Klehr2024-09-061-0/+4
| | | | Signed-off-by: Marcel Klehr <mklehr@gmx.net>
* fix(files_sharing): Handle download permission the same way for public and ↵Ferdinand Thiessen2024-09-061-0/+5
| | | | | | internal shares Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore: adjust code to adhere to coding standardAnna Larch2024-09-051-1/+1
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-253-12/+13
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* feat: Use new password context in sharing APIFerdinand Thiessen2024-08-221-2/+3
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #47070 from nextcloud/circToTeamAndy Scherzinger2024-08-061-5/+5
|\ | | | | fix(share): Rename Circle to Team
| * fix(share): Rename Circle to TeamGit'Fellow2024-08-061-5/+5
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | fix(files_sharing): show proper share not found error messageskjnldsv2024-08-061-4/+4
|/ | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* Merge pull request #47043 from ↵Maxence Lange2024-08-061-1/+4
|\ | | | | | | | | nextcloud/fix/noid/ignore-root-on-inherited-shares fix(inherited-shares): ignore top root folder
| * fix(inherited-shares): ignore top root folderMaxence Lange2024-08-051-1/+4
| | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | fix(files_sharing): filter out invalid shareWith in sharee APIskjnldsv2024-08-051-1/+1
|/ | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* refactor(files_sharing): Replace security annotations with respective attributesprovokateurin2024-07-2710-75/+57
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files_sharing): file request creation permissions and link copy displayskjnldsv2024-07-181-2/+4
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(files_sharing): remove undesirable translations from ShareAPIControllerskjnldsv2024-07-161-2/+2
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* chore(files_sharing): lint & refactor fixesskjnldsv2024-07-121-1/+1
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(files_sharing): adjust IAttributes API and files_versionsskjnldsv2024-07-121-17/+10
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(files_sharing): phpunit & openapi fixesskjnldsv2024-07-121-2/+8
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* chore(files_sharing): cleanup NewFileRequestDialog vueskjnldsv2024-07-121-1/+1
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix: psalm, cs, autoloader and lintskjnldsv2024-07-121-16/+19
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* chore(files_sharing): refactor share password mailskjnldsv2024-07-121-9/+29
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* feat(files_sharing): allow mixed values in share attributes and allow ↵skjnldsv2024-07-121-1/+8
| | | | | | storing email arrays Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* chore(files_sharing): refactor mail handlingskjnldsv2024-07-121-1/+32
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* feat(files_sharing): implement IShareProviderWithNotification and refactor ↵skjnldsv2024-07-121-36/+20
| | | | | | sendMailNotification Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* feat(files_sharing): add `new file request` optionskjnldsv2024-07-121-0/+49
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* docs(ShareApiController): Correct note about expiry date timezonefenn-cs2024-07-011-1/+1
| | | | Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* fix(files_sharing): Store the expiration date relative to the server's timezoneLouis Chemineau2024-06-261-3/+0
| | | | | | This is needed as we want to store the difference between the server's and the user's timezones. Signed-off-by: Louis Chemineau <louis@chmn.me>
* chore: Add SPDX headerAndy Scherzinger2024-06-0610-264/+25
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* fix(ShareApiClients): Maintain meaning of `''` and null for share api clientsfenn-cs2024-05-291-3/+3
| | | | Signed-off-by: fenn-cs <fenn25.fn@gmail.com>