aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/tests
Commit message (Collapse)AuthorAgeFilesLines
* feat(files_sharing): use share node or label as page titlefeat/sharing-titleskjnldsv2024-11-081-0/+3
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix: Adjust preview for view-only sharesfix/view-only-previewFerdinand Thiessen2024-10-281-10/+111
| | | | | | | | | | | | | | | | | 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-282-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* refactor(apps): Use constructor property promotion when possibleprovokateurin2024-10-2116-29/+44
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* 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-191-1/+1
| | | | | | | | | | | | | | | | | 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(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-1527-505/+562
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor(files_sharing): Add Storage parameter strong typesprovokateurin2024-10-071-1/+1
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* test: fix share storage move test with object storeRobin Appelman2024-09-271-0/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(files_sharing): Fix all IStorage return typesprovokateurin2024-09-261-3/+4
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: Apply rector configuration to apps folderCôme Chilliet2024-09-202-5/+5
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-196-24/+24
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #48200 from nextcloud/share-reminder-shardingRobin Appelman2024-09-191-1/+3
|\ | | | | fix: add sharding compatible version of share reminder job
| * fix: add sharding compatible version of share reminder jobshare-reminder-shardingRobin Appelman2024-09-191-1/+3
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #48162 from nextcloud/bugfix/noid/array-keysJoas Schilling2024-09-191-5/+5
|\ \ | |/ |/| fix(autocomplete): Move known array keys to psalm docs
| * fix(autocomplete): Move known array keys to psalm docsbugfix/noid/array-keysJoas Schilling2024-09-181-5/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | test: fix ShareControllerTest.phpobject-store-move-fixesRobin Appelman2024-09-181-0/+1
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | test: fix ShareAPIControllerTest.phpRobin Appelman2024-09-181-1/+41
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | test: put parent items into cache in testsRobin Appelman2024-09-181-0/+2
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: Fix other uses of removed Storage interfaceCôme Chilliet2024-09-161-6/+0
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Remove OCP\Files\Storage interface deprecated since version 9Côme Chilliet2024-09-161-16/+16
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* refactor: Add void return type to PHPUnit test methodsChristoph Wurst2024-09-1533-285/+285
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* fix(files_sharing): adjust permissions from custom edit and delete check methodsskjnldsv2024-09-111-0/+46
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* chore(files_sharing): Remove now unused legacy scriptsFerdinand Thiessen2024-09-063-542/+0
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* feat(files_sharing): Migrate public share to use Vue files listFerdinand Thiessen2024-09-061-350/+148
| | | | | | 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>
* Merge pull request #47412 from Luka-sama/feat/shares-reminderJohn Molakvoæ2024-09-031-0/+190
|\
| * fix(files_sharing): Add missing check for null & use bool instead of int for ↵Stefan Cherniakov2024-09-031-1/+1
| | | | | | | | | | | | reminder_sent field Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
| * fix(files_sharing): Make share reminders more stable & fix issuesStefan Cherniakov2024-09-031-8/+2
| | | | | | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
| * feat(files_sharing): reminder for link shares with expiration dateStefan Cherniakov2024-09-031-0/+196
| | | | | | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
* | refactor(IMenuAction): Make public menu actions use the new Vue UIFerdinand Thiessen2024-09-031-4/+2
|/ | | | | | | | This removes custom rendering code an replaces it with the declarative menu actions. Also adjust the template to allow the Vue UI to mount. Custom entries still are possible. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* test: mark share test cleanup as running across all shardsRobin Appelman2024-08-281-1/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-2512-93/+93
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* feat(files_sharing): add public name prompt for files requestsskjnldsv2024-07-181-4/+10
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* test: update share tests to work with shardingRobin Appelman2024-07-161-22/+22
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(files_sharing): adjust IAttributes API and files_versionsskjnldsv2024-07-121-1/+1
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(files_sharing): phpunit & openapi fixesskjnldsv2024-07-123-27/+57
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* Merge pull request #44967 from nextcloud/perf/sharing-eventsFerdinand Thiessen2024-06-101-24/+18
|\ | | | | perf(files_sharing): Move events to listener classes and registration instead of boot
| * perf(files_sharing): Move events to listener classes and registration ↵Julius Härtl2024-05-131-24/+18
| | | | | | | | | | | | instead of boot Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | chore: Add SPDX headerAndy Scherzinger2024-06-0640-947/+113
|/ | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* refactor(shareApiController): use contrusctor property promotion & DI loggerfenn-cs2024-04-182-61/+46
| | | | Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* fix(files_sharing): ShareesAPI - Return empty response when user is not ↵Ferdinand Thiessen2024-03-151-40/+74
| | | | | | | | allowed to share Resolves: https://github.com/nextcloud/server/issues/20950 Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* refactor: make share manager buildableRobin Appelman2024-03-061-2/+0
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix(sharing): Avoid (dead)locking during orphan deletionChristoph Wurst2024-03-051-2/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #43471 from nextcloud/cache-path-by-idRobin Appelman2024-03-054-75/+107
|\ | | | | Cache path by id
| * perf: switch places that always use the first getById result to getFirstNodeByIdRobin Appelman2024-03-044-75/+107
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix: remove old test-only methodsRobin Appelman2024-02-261-4/+4
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* feat(share): save date and time for expirationBenjamin Gaussorgues2024-02-223-9/+17
| | | | | | | | 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: phpunitJohn Molakvoæ2024-02-131-2/+2
| | | | Signed-off-by: John Molakvoæ <skjnldsv@protonmail.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(tests): Adjust legacy tests for nextcloud-router updateFerdinand Thiessen2024-01-251-3/+3
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(share): use user timezone to parse share expiration dateBenjamin Gaussorgues2024-01-092-2/+19
| | | | | | | | 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>