aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib
Commit message (Collapse)AuthorAgeFilesLines
* fix(federatedfilesharing): get share by token fallbackbackport/49973/masterskjnldsv2024-12-281-4/+58
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(activity): make emails for link share uploads true by defaultskjnldsv2024-12-061-1/+1
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* feat(ocm): signing ocm requestsMaxence Lange2024-12-041-0/+1
| | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* Merge pull request #49569 from nextcloud/shared-watcher-perfStephan Orbaugh2024-12-031-13/+8
|\ | | | | perf: improve performance of SharedStorage::getWatcher
| * perf: improve performance of SharedStorage::getWatcherRobin Appelman2024-11-291-13/+8
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | 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>
* Merge pull request #43025 from ↵Côme Chilliet2024-11-252-0/+89
|\ | | | | | | | | nextcloud/bugfix/error-on-reshare-after-transfer-ownership fix: update re-share if shared-by user has been revoked
| * fix(sharing): add command to fix broken shares after ownership transferringbugfix/error-on-reshare-after-transfer-ownershipLuka Trovic2024-11-182-0/+89
| | | | | | | | Signed-off-by: Luka Trovic <luka@nextcloud.com>
* | fix(files_sharing,files): Do not validate shares before adjusting the ownerprovokateurin2024-11-251-7/+11
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | 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>
* feat(files_sharing): use share node or label as page titlefeat/sharing-titleskjnldsv2024-11-081-0/+4
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(OpenAPI): Adjust array syntax to avoid ambiguitiesfix/openapi/array-syntaxprovokateurin2024-11-058-62/+66
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Adjust preview for view-only sharesfix/view-only-previewFerdinand Thiessen2024-10-282-12/+17
| | | | | | | | | | | | | | | | | 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>
* Merge pull request #48918 from ↵Kate2024-10-281-9/+9
|\ | | | | | | nextcloud/fix/files_sharing/cleanup-error-messages
| * fix(files_sharing): Cleanup error messagesfix/files_sharing/cleanup-error-messagesprovokateurin2024-10-281-9/+9
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | Merge pull request #48614 from nextcloud/refactor/storage/constructorsKate2024-10-281-7/+7
|\ \ | |/ |/|
| * refactor(Storage): Align all Storage constructorsrefactor/storage/constructorsprovokateurin2024-10-231-7/+7
| | | | | | | | 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-2145-589/+296
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(activity): Fix download activity parametersbugfix/noid/fix-download-activity-parametersJoas Schilling2024-10-191-2/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* chore(deps): Bump nextcloud/coding-standard in /vendor-bin/cs-fixerdependabot/composer/vendor-bin/cs-fixer/nextcloud/coding-standard-1.3.2dependabot[bot]2024-10-192-21/+21
| | | | | | | | | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 1.3.1 to 1.3.2. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v1.3.1...v1.3.2) --- updated-dependencies: - dependency-name: nextcloud/coding-standard dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(db): Apply query prepared statementsdbQueriesExecStmt2Git'Fellow2024-10-172-16/+10
| | | | | | | | | | | | | | | | Fix: psalm fix: bad file fix: bug chore: add batch chore: add batch chore: add batch fix: psalm
* chore(db): Correctly apply query typesGit'Fellow2024-10-172-22/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | fix: psalm fix: error fix: add batch fix: fatal error fix: add batch chore: add batch chore: add batch fix: psalm fix: typo fix: psalm fix: return bool fix: revert Manager
* chore(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-1522-95/+126
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* revert: "fix(files_sharing): Make share labels nullable"fix/null-labelFerdinand Thiessen2024-10-141-1/+1
| | | | | | This reverts commit 01c4fa3ba8a7311da331e6a2b70fc866b0a9b39c. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #48494 from nextcloud/refactor/storage/strong-param-typesKate2024-10-082-37/+32
|\
| * refactor(files_sharing): Add Storage parameter strong typesprovokateurin2024-10-072-37/+32
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(BackgroundJobs): Adjust intervals and time sensitivitiesprovokateurin2024-10-083-5/+5
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* 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>
* fix(files_sharing): Fix all IStorage return typesprovokateurin2024-09-262-67/+30
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files_sharing): Check if propfind response is valid before accessing ↵fix/files_sharing/ocm-permissionsprovokateurin2024-09-261-0/+4
| | | | | | share permission props Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(files_sharing): Make permissions prop checks less error proneprovokateurin2024-09-261-6/+9
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(files_sharing): Parse OCM share permissions from OCM and not OCS propprovokateurin2024-09-261-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #48223 from nextcloud/feat/add-rector-configCôme Chilliet2024-09-231-4/+4
|\ | | | | Add rector config
| * chore: Apply rector configuration to apps folderCôme Chilliet2024-09-201-4/+4
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | chore(files): Remove legacy style filesFerdinand Thiessen2024-09-201-1/+0
|/ | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore: Remove deprecated `Share_Helper::generateTarget` `$exclude` parameterchore/add-deprecation-dateFerdinand Thiessen2024-09-202-13/+4
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(federation): Do not overwrite certificate bundlefix/federation-certificate-storeJulius Härtl2024-09-201-14/+17
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-1913-19/+22
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: add sharding compatible version of share reminder jobshare-reminder-shardingRobin Appelman2024-09-191-12/+101
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: fix share reminder job for oracleoracle-share-reminderRobin Appelman2024-09-181-2/+2
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* refactor: move to new activity exceptionrefactor/move-to-new-activity-exceptionAnna Larch2024-09-171-2/+3
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* fix(Storage): Fix method docs inheritancefix/storage/method-docs-inheritanceprovokateurin2024-09-172-21/+0
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Storage): Document getOwner() can return falseprovokateurin2024-09-172-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: Fix psalm issues, put back private versions of getter in private ↵Côme Chilliet2024-09-161-0/+1
| | | | | | Storage interface Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Fix other uses of removed Storage interfaceCôme Chilliet2024-09-161-3/+4
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>