aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Files
Commit message (Collapse)AuthorAgeFilesLines
* chore: use `$this->logger` and prevent Server::getfix/undefined-responseskjnldsv2024-11-081-2/+2
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* Fix Undefined variable: response notice in case of non-404 errorVictor Dubiniuk2024-11-081-2/+3
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(OpenAPI): Adjust array syntax to avoid ambiguitiesfix/openapi/array-syntaxprovokateurin2024-11-051-2/+5
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #47847 from ↵John Molakvoæ2024-10-291-2/+6
|\ | | | | | | nextcloud/fix-copying-or-moving-from-shared-groupfolders
| * fix: Fix copying or moving from shared groupfoldersDaniel Calviño Sánchez2024-10-101-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When copying or moving between two local storages the source path (on disk) to copy or move from is got from the unjailed path of the source storage. However, if the source storage has more than one jail getting the unjailed path resolves the most external jail, but the source path needs to be got from the most internal jail instead (the one closer to the local storage). This can happen, for example, with a shared groupfolder: in that case there is an external jail for the shared storage, and one internal jail for the groupfolder storage wrapped by the shared storage. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | feat(systemtags): add etag support and handle proppatchskjnldsv2024-10-291-1/+1
| | | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | Merge pull request #48614 from nextcloud/refactor/storage/constructorsKate2024-10-2820-67/+67
|\ \
| * | refactor(Storage): Align all Storage constructorsrefactor/storage/constructorsprovokateurin2024-10-2320-67/+67
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | fix: $uid type in Encryption streamartonge/fix/uid_typeLouis2024-10-231-1/+1
|/ / | | | | | | | | | | | | As explained by the comment. Signed-off-by: Louis <louis@chmn.me> Signed-off-by: Louis Chemineau <louis@chmn.me>
* | 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(db): Apply query prepared statementsdbQueriesExecStmt2Git'Fellow2024-10-174-49/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-173-16/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | feat(http-client): expose default request timeout constant to be used ↵add-default-request-timeout-constThomas Citharel2024-10-111-1/+2
|/ | | | | | | | | | instead of magic number 30 Will be used to set default where we are able to override the default timeout. See https://github.com/nextcloud/server/pull/48418 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* Merge pull request #48623 from nextcloud/fix/allow-quota-wrapper-checkKate2024-10-091-1/+1
|\
| * fix: Allow overriding shouldApplyQuota check from child classesfix/allow-quota-wrapper-checkJulius Knorr2024-10-091-1/+1
| | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | refactor(Storage): Make all parameter types strong typesprovokateurin2024-10-0721-601/+448
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | refactor(Wrapper\Encryption): Migrate to strong typesrefactor/stream-encryption/typingsprovokateurin2024-10-071-59/+15
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | refactor(Stream\Encryption): Migrate to strong typesprovokateurin2024-10-071-74/+42
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* Merge pull request #48454 from ↵Kate2024-10-015-6/+6
|\ | | | | | | nextcloud/fix/storage/get-directory-content-return-type
| * fix(Storage): Fix getDirectoryContent() return typefix/storage/get-directory-content-return-typeprovokateurin2024-09-305-6/+6
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | Merge pull request #48235 from nextcloud/readd-object-store-phpunitRobin Appelman2024-10-012-24/+82
|\ \ | | | | | | test: re-add object store primary storage phpunit tests
| * | fix: rework move into object store to better preserve fileidsreadd-object-store-phpunitRobin Appelman2024-09-271-19/+55
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix: preserve fileid when moving from objectstore to non-objectstoreRobin Appelman2024-09-272-6/+27
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix: ensure source folder is removed from cache when moving to objectstoreRobin Appelman2024-09-271-0/+1
| | | | | | | | | | | | | | | | | | otherwise this causes confusion down the line as it's contents will be moved to the new cache Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | fix: catch NotPermittedException in Folder.nodeExistsupdate-phpdoc-for-folder-getDaniel Kesselberg2024-09-301-1/+1
| | | | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | docs: Folder.get throws a NotPermittedException exceptionDaniel Kesselberg2024-09-303-23/+0
| |/ |/| | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | fix: Fix test of IConstructableStorage implentation by storage classesfix/fix-storage-interface-checkCôme Chilliet2024-09-301-1/+1
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix(Storage): Fix IStorage return typesprovokateurin2024-09-2624-1484/+518
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* feat(trashbin): Port files trashbin events to IEventDispatcher/IEventListenerCarl Schwan2024-09-231-0/+5
| | | | | | | | oc_hooks is deprecated and will at some point be removed Signed-off-by: Carl Schwan <carl@carlschwan.eu> Signed-off-by: Thomas Citharel <tcit@tcit.fr> Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: Move storage constructor to specific interfaceCôme Chilliet2024-09-232-1/+7
| | | | | | That allows Wrappers to use DI and not care about the constructor Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore: Add proper deprecation dates where missingFerdinand Thiessen2024-09-201-1/+1
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* perf: reuse cache info we already have when moving to object storeobject-store-move-dbRobin Appelman2024-09-191-2/+2
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-1921-35/+35
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: fix share reminder job for oracleoracle-share-reminderRobin Appelman2024-09-181-1/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: ensure parent folder exists when writing a file to object storageRobin Appelman2024-09-181-0/+4
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: verify that parent exists in cache when insertingRobin Appelman2024-09-181-0/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: fix object store id for test object storeRobin Appelman2024-09-181-2/+2
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: cleanup objectstore file_put_contentRobin Appelman2024-09-181-7/+4
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: create intermediate directories for objectstore moveFromStorageRobin Appelman2024-09-181-0/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: get source file before moving the cache item in objectstore moveFromStorageRobin Appelman2024-09-181-1/+5
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #47986 from cfiehe/fix_move_on_same_bucketStephan Orbaugh2024-09-171-0/+5
|\ | | | | perf(ObjectStoreStorage): Improve (slow) move on same object bucket
| * perf(ObjectStoreStorage): Improve (slow) move on same object bucketChristoph Fiehe2024-09-161-0/+5
| | | | | | | | | | | | | | | | | | This commit fixes the issue #47856. When you upload a file into a group folder and when you use a single S3 bucket as primary storage, the final move operation hangs for a long time. In the background, Nextcloud initiates a copy-delete sequence from the bucket into the bucket, with causes a lot unnecessary overhead. Nextcloud thinks that the file must be imported to another storage and does not recognize that everything is done on the same object bucket. In that case, the import step can be completely skipped, which saves time, network bandwidth and reduces the load on the object storage. The behavior improves a lot with https://github.com/nextcloud/server/pull/46013. However, there are still some put messages that are being sent to the object storage when you use an object storage as primary storage and upload files into a group folder. Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com> Signed-off-by: Christoph Fiehe <c.fiehe@eurodata.de>
* | fix(Storage): Fix method docs inheritancefix/storage/method-docs-inheritanceprovokateurin2024-09-1710-155/+16
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(Files): Handle getOwner() returning falsefix/storage/get-owner-falseprovokateurin2024-09-173-10/+51
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix(Storage): Document getOwner() can return falseprovokateurin2024-09-177-41/+8
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | chore: Check storage is an instance of the correct class instead of ↵fix/remove-references-to-deprected-storage-interfaceCôme Chilliet2024-09-161-6/+9
| | | | | | | | | | | | psalm-suppress Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | chore: Remove deprecated Storage::search method instead of moving it to IStorageCôme Chilliet2024-09-164-34/+0
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | chore: psalm-suppress legacy code weirdnessCôme Chilliet2024-09-161-2/+12
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | chore: Fix psalm issues, put back private versions of getter in private ↵Côme Chilliet2024-09-166-15/+71
| | | | | | | | | | | | Storage interface Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix: Fix other uses of removed Storage interfaceCôme Chilliet2024-09-162-84/+8
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>