aboutsummaryrefslogtreecommitdiffstats
path: root/apps/files_sharing/lib
Commit message (Collapse)AuthorAgeFilesLines
* 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>
* fix: Handle null checks with the ?? operatorrefactor/elvisChristoph Wurst2024-09-151-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* refactor: Use the elvis operatorChristoph Wurst2024-09-151-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #47339 from nextcloud/fix/leave-share-instead-of-deleteJohn Molakvoæ2024-09-112-0/+23
|\ | | | | fix: Display 'Leave share' instead of 'Delete'
| * 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-062-0/+6
| | | | | | | | Signed-off-by: Marcel Klehr <mklehr@gmx.net>
* | fix(INotification): Restrict rich object parameters typeprovokateurin2024-09-101-2/+2
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | Merge pull request #47841 from nextcloud/fix/files-sharing-openfileJonas2024-09-091-0/+1
|\ \ | | | | | | fix(files_sharing): Open single-file-share by default
| * | fix(files_sharing): Open single-file-share by defaultFerdinand Thiessen2024-09-091-0/+1
| |/ | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* / chore(files_sharing): Add migration attribute for `share.reminder_sent` columnFerdinand Thiessen2024-09-061-0/+7
|/ | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* 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>
* fix(files): Do not allow copy action on public shares without `create` ↵Ferdinand Thiessen2024-09-061-0/+1
| | | | | | permission Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* feat(files_sharing): Migrate public share to use Vue files listFerdinand Thiessen2024-09-061-170/+133
| | | | | | 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>
* chore: adjust code to adhere to coding standardAnna Larch2024-09-051-1/+1
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* Merge pull request #47412 from Luka-sama/feat/shares-reminderJohn Molakvoæ2024-09-032-0/+254
|\
| * fix(files_sharing): Add check for nullStefan Cherniakov2024-09-031-0/+3
| | | | | | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
| * fix(files_sharing): Add missing check for null & use bool instead of int for ↵Stefan Cherniakov2024-09-031-1/+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-031-133/+85
| | | | | | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
| * feat(files_sharing): reminder for link shares with expiration dateStefan Cherniakov2024-09-032-0/+297
| | | | | | | | Signed-off-by: Stefan Cherniakov <luka-sama@pm.me>
* | refactor(IMenuAction): Make public menu actions use the new Vue UIFerdinand Thiessen2024-09-031-6/+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>
* Merge pull request #47551 from nextcloud/fix/files_sharing/share-nullable-labelJoas Schilling2024-08-291-1/+1
|\ | | | | fix(files_sharing): Make share labels nullable