aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/Controller
Commit message (Collapse)AuthorAgeFilesLines
* fix: Fix download activity for foldersCôme Chilliet2025-03-251-98/+5
| | | | | | | | | Remove duplicate activity publishing from share controller download, listen to BeforeZipCreatedEvent to publish activity for folders, and cache folders activity to avoid sending activity for each file in the folder. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(lookup-server): disable lookup server for non-global scale setupsFerdinand Thiessen2025-03-111-5/+5
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(lookup-server): do not query data by defaultFerdinand Thiessen2025-03-111-6/+3
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): Also list deleted team sharesfix/show-deleted-team-sharesFerdinand Thiessen2025-02-201-1/+2
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): block downloading if neededfix/download-permsFerdinand Thiessen2025-02-181-0/+5
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore: Apply new rector configuration to apps folderCôme Chilliet2025-02-132-3/+5
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: re-apply current rector configuration to apps folderCôme Chilliet2025-02-131-5/+5
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Replace isInstalled calls with isEnabledForAnyone or isEnabledForUserCôme Chilliet2025-02-131-1/+1
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor: Remove some deprecated containers and exceptionsnfebe2025-02-061-25/+28
| | | | Signed-off-by: nfebe <fenn25.fn@gmail.com>
* fix(sharing): Ensure download restrictions are not droppedfix/proper-download-checkFerdinand Thiessen2025-02-041-23/+41
| | | | | | | | When a user receives a share with share-permissions but also with download restrictions (hide download or the modern download permission attribute), then re-shares of that share must always also include those restrictions. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(files_sharing): Respect permissions passed when creating link sharesFerdinand Thiessen2025-01-281-101/+94
| | | | | | | | | | | | | | | | | Given: User creates a link or email share with permissions=4 (create only = file drop). Problem: Currently the permissions are automatically extended to permissions = 5 (READ + CREATE). Work around was to create the share and directly update it. Solution: Respect what the user is requesting, create a file drop share. Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Ensure `label` is always a stringFerdinand Thiessen2025-01-261-1/+4
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* 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>
generated by cgit v1.2.3 (git 2.39.1) at 2025-05-08 22:02:38 +0000