aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/Files
Commit message (Collapse)AuthorAgeFilesLines
...
* | | 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>
* | | chore: improve hash_file php usage in Local Storageartonge/fix/hash_return_typeJohn Molakvoæ2024-09-161-8/+1
| | | | | | | | | | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
* | | fix(files): Ensure that the hash method does not return nullLouis Chemineau2024-09-162-2/+10
|/ / | | | | | | | | | | | | | | To match https://github.com/nextcloud/server/blob/beececf66068f57c416225efcde9b44ce5c2e835/lib/private/Files/View.php#L1050 - Fix https://github.com/nextcloud/server/issues/44110 Signed-off-by: Louis Chemineau <louis@chmn.me>
* | Merge pull request #47852 from nextcloud/sharding-code-fixesJohn Molakvoæ2024-09-161-1/+1
|\ \
| * | fix: misc code fixes around db shardingsharding-code-fixesRobin Appelman2024-09-091-1/+1
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #48016 from nextcloud/fix/s3configtrait/proxy-falseKate2024-09-151-1/+1
|\ \ \
| * | | fix(S3ConfigTrait): Allow proxy field to take falsefix/s3configtrait/proxy-falseprovokateurin2024-09-151-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | fix(Storage\Local): Do not call getSourcePath() on SplFileInfofix/storage-local/get-source-path-spl-file-infoprovokateurin2024-09-151-1/+1
|/ / / | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | fix(files): Adjust Cache::searchQuery() parameter name to match interfaceprovokateurin2024-09-132-4/+4
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | Merge pull request #47853 from nextcloud/clearify-swift-exceptionRobin Appelman2024-09-101-4/+4
|\ \ \ | | | | | | | | fix: make swift connect exception message more informative
| * | | fix: make swift connect exception message more informativeRobin Appelman2024-09-091-4/+4
| |/ / | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* / / fix: Only write once to template instead of create/copyJulius Knorr2024-09-101-3/+3
|/ / | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | fix(files): Never return a null ETag in DAVprovokateurin2024-09-093-10/+3
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | chore(storage): refactor some code portionsGit'Fellow2024-09-061-30/+16
| | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> chore: revert portion Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | Merge pull request #47784 from nextcloud/fix/file-infoFerdinand Thiessen2024-09-061-9/+5
|\ \ | | | | | | fix(FileInfo): correctly implement `\ArrayAccess::offsetGet`
| * | fix(FileInfo): correctly implement `\ArrayAccess::offsetGet`Ferdinand Thiessen2024-09-061-9/+5
| | | | | | | | | | | | | | | | | | | | | We are targetting PHP 8.1+, since PHP 8.0 `offsetGet` has the `mixed` types for `offset` and return value. So this adds the types to correctly implement the interface. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | fix(files): Check if the target path is a descendant of the shared folder pathGit'Fellow2024-09-051-1/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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> fix: add tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests