aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Share20
Commit message (Collapse)AuthorAgeFilesLines
* fix(files_sharing): fix share creation error handlingskjnldsv6 days1-1/+5
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* Merge pull request #51838 from nextcloud/feat/cleanup-oc-util-methodsJohn Molakvoæ7 days2-13/+3
|\ | | | | Mark all methods of OC_Util as deprecated
| * fix: Fix types for sharingDisabledForUserCôme Chilliet8 days2-7/+2
| | | | | | | | | | | | and use the non-deprecated version whenever possible Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix: Remove some call and references to deprecated OC_Util classCôme Chilliet8 days1-7/+2
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix: Handle missing share providers when promoting reshareshandle-missing-share-providers-when-promoting-resharesDaniel Calviño Sánchez8 days1-1/+6
|/ | | | | | | | The provider for mail shares is not available when the "sharebymail" app is disabled, and in that case a "ProviderException" is thrown when trying to get it. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* perf(sharing): Use oc_share.{item_type, share_type} IN instead of OR equalsrefactor/use-in-instead-of-orMarcel Klehr2025-03-251-79/+20
| | | | | | to improve performance by using the db index Signed-off-by: Marcel Klehr <mklehr@gmx.net>
* fix(Share20\Manager): Propagate user and group deletion to remote share ↵provokateurin2025-03-031-4/+16
| | | | | | providers Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Share20): Convert broken hooks to IEventListenerprovokateurin2025-03-033-20/+64
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Exclude non accepted shares when computing access listartonge/fix/access_list_unaccepted_sharesLouis Chemineau2025-01-301-0/+12
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(sharing): Move permission validation to share managerFerdinand Thiessen2025-01-281-0/+11
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Ensure `label` is always a stringFerdinand Thiessen2025-01-261-2/+1
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Revert "fix(Share20\Manager): Ensure node is still accessible when checking ↵revert/share-node-accessibleprovokateurin2025-01-241-9/+0
| | | | | | | | share" This reverts commit c2ca99e2f641396b6823c1e675afd3ecb486e744. Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(sharing): Set correct password context for share password validationfix/password-validationFerdinand Thiessen2025-01-201-1/+3
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* feat(sharing): Add `shareapi_allow_custom_tokens` configChristopher Ng2025-01-151-0/+6
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* feat(sharing): Fetch a unique generated tokenChristopher Ng2025-01-151-35/+41
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* fix(shares): Remove noisy text from emailsremoveNoisyTextEmailsGit'Fellow2024-12-161-4/+0
| | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* fix(shares): Wrap exceptions from password validation to set code to 400Côme Chilliet2024-12-121-2/+7
| | | | | | | This fixes a regression that bad password returned 403 instead of 400 because of previous changes. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(files_sharing): Do not wrap password policy exception into a generic oneCôme Chilliet2024-12-121-6/+2
| | | | | | Let the controller access the HintException and show the error to the user. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(share): Don't print twice the same informationprintOnlyOnceTextGit'Fellow2024-12-101-10/+9
| | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* fix(files_sharing,files): Do not validate shares before adjusting the ownerprovokateurin2024-11-251-10/+14
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Share20\Manager): Ensure node is still accessible when checking shareprovokateurin2024-11-251-0/+9
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* feat(Share20\Manager): Return all shares on IShareOwnerlessMountprovokateurin2024-11-251-11/+30
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Share20\DefaultShareProvider): Return link shares in getSharesByPath()provokateurin2024-11-251-1/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(Share20\DefaultShareProvider): Sort getSharesByPath() resultsprovokateurin2024-11-251-0/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #47425 from ↵Côme Chilliet2024-11-041-0/+88
|\ | | | | | | | | nextcloud/fix/avoid-invalid-share-on-transfer-ownership fix: promote re-shares when deleting the parent share
| * fix: Use getRelativePath method to check if node is inside folderfix/avoid-invalid-share-on-transfer-ownershipCôme Chilliet2024-10-141-1/+2
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix: Fix promotion of reshares from subsubfoldersCôme Chilliet2024-10-141-9/+16
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore: Add comment to make code clearerCôme Chilliet2024-10-141-0/+1
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * chore: Turn method description into phpdocCôme Chilliet2024-10-141-1/+1
| | | | | | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
| * fix(shares): Promote reshares into direct shares when share is deletedCôme Chilliet2024-10-141-6/+14
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix: Tidy up code for reshare deletionCôme Chilliet2024-10-141-14/+16
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * fix: delete re-shares when deleting the parent shareLuka Trovic2024-10-141-0/+69
| | | | | | | | | | | | | | Note: Removed part about fix command from original PR Signed-off-by: Luka Trovic <luka@nextcloud.com> Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Merge pull request #47329 from nextcloud/feat/add-datetime-qbmapper-supportFerdinand Thiessen2024-10-181-4/+4
|\ \ | | | | | | feat(AppFramework): Add full support for date / time / datetime columns
| * | fix: Prevent breaking change in IQueryBuilderFerdinand Thiessen2024-10-171-4/+4
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | fix: Adjust parameter type usage and add SQLite supportFerdinand Thiessen2024-10-171-4/+4
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | chore(db): Apply query prepared statementsdbQueriesExecStmt2Git'Fellow2024-10-171-31/+30
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix: psalm fix: bad file fix: bug chore: add batch chore: add batch chore: add batch fix: psalm
* / fix(share): Return empty string if no label is setFerdinand Thiessen2024-10-141-2/+2
|/ | | | | | | | | * Resolves: https://github.com/nextcloud/server/issues/48629 While the database supports NULL, the typing has always said it only returns *string*. So to not break any apps that might trust the typings we should return `''` if the database is set to `NULL`. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* feat(share): make sharelink token length configurableernolf/configurable_sharetoken_lengthernolf2024-09-281-7/+37
| | | | | | - ensure unique share token with dynamic length adjustment Signed-off-by: ernolf <raphael.gradenwitz@googlemail.com>
* fix(l10n): Rephrasing some exceptions to be translatedGit'Fellow2024-09-201-14/+13
| | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Fix tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* chore: Add proper deprecation dates where missingFerdinand Thiessen2024-09-201-2/+2
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-191-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(ProviderFactory): Remove usage of ILogger and use non-deprecated stuffFerdinand Thiessen2024-09-191-10/+9
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* chore(i18n): Spelling of IDrakekniven2024-09-061-2/+2
| | | | | Reported at Transifex Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
* chore: adjust code to adhere to coding standardAnna Larch2024-09-051-2/+2
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* fix: adjust sharing controller exceptions caseJohn Molakvoæ2024-09-041-2/+2
| | | | | Co-authored-by: F. E Noel Nfebe <fenn25.fn@gmail.com> Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
* fix(l10n): Translate all share exceptionsGit'Fellow2024-09-041-70/+63
| | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> Fix drone Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> Fix tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* fix(files_sharing): Add missing check for null & use bool instead of int for ↵Stefan Cherniakov2024-09-031-3/+3
| | | | | | reminder_sent field Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
* fix(files_sharing): Make share reminders more stable & fix issuesStefan Cherniakov2024-09-032-3/+6
| | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
* feat(files_sharing): reminder for link shares with expiration dateStefan Cherniakov2024-09-031-0/+11
| | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
* fix: fix share cleanup for deleted groups with shardingRobin Appelman2024-08-281-10/+18
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>