aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib/Controller/ShareAPIController.php
Commit message (Collapse)AuthorAgeFilesLines
* 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-051-18/+18
| | | | Signed-off-by: provokateurin <kate@provokateurin.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-211-79/+77
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-151-15/+18
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-191-4/+4
| | | | 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-251-9/+10
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* fix(share): Rename Circle to TeamGit'Fellow2024-08-061-5/+5
| | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* fix(inherited-shares): ignore top root folderMaxence Lange2024-08-051-1/+4
| | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* refactor(files_sharing): Replace security annotations with respective attributesprovokateurin2024-07-271-18/+10
| | | | 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-061-40/+3
| | | | 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>
* fix(shareManager): Respect empty `expireDate` in serverfenn-cs2024-05-231-8/+15
| | | | | | If `expireDate` is an empty string and not `null` then the server should not set a default. Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* fix(sharing): Don't change the type of the controller argumentJoas Schilling2024-04-181-2/+2
| | | | | | [EA] New value type (\DateTime) is not matching the resolved parameter type and might introduce types-related false-positives. Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(shareApiController): avoid duplicated expiryDate operationsfenn-cs2024-04-181-27/+10
| | | | | | | | `expireDate` can be set once and used anywhere needed, the current implementation, duplicates this behavior which leads to `parseDate` receiving an a date object it parsed and returend earlier in the createShare method. Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* refactor(shareApiController): use contrusctor property promotion & DI loggerfenn-cs2024-04-181-56/+20
| | | | Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* fix: Apply new coding standard to all filesCôme Chilliet2024-04-021-16/+16
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* perf: switch places that always use the first getById result to getFirstNodeByIdRobin Appelman2024-03-041-18/+12
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #43428 from nextcloud/feat/share-expiration-with-timeBenjamin Gaussorgues2024-02-261-1/+4
|\
| * feat(share): save date and time for expirationBenjamin Gaussorgues2024-02-221-1/+4
| | | | | | | | | | | | | | | | Because of timezones, not saving time can lead to unexpected behaviour when sharing an item sooner than timezone offset Example: sharing a file before 9am when in UTC+9 Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | fix truncated circleId returned from Share APIAragon Gouveia2024-02-231-3/+4
|/
* Merge pull request #43461 from nextcloud/fix/get-rid-of-getloggerF. E Noel Nfebe2024-02-141-2/+2
|\ | | | | chore: Migrate away from OC::$server->getLogger
| * chore: Migrate away from OC::$server->getLoggerCôme Chilliet2024-02-131-2/+2
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | feat: rename users to account or personVincent Petry2024-02-131-2/+2
|/ | | | | | Replace translated text in most locations Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* fix(share): use user timezone to parse share expiration dateBenjamin Gaussorgues2024-01-091-17/+6
| | | | | | | | If an user in UTC+1 try to create a share at 00:00, it's day D for him, but D-1 for the server (UTC). This fix aims to apply the correct offset Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-231-4/+4
| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* refactor: Rename OpenAPI type aliases to their namespacesjld31032023-11-021-11/+11
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* Add single status code descriptions for OpenAPIjld31032023-09-271-0/+2
| | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>