summaryrefslogtreecommitdiffstats
path: root/lib/private/Files
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #38449 from nextcloud/backport/38415/stable27Arthur Schiwon2023-06-011-6/+18
|\ | | | | [stable27] don't always check if we need to setup the object store root
| * don't always check if we need to setup the object store rootRobin Appelman2023-05-241-6/+18
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix: execute throws docrine exceptions not our wrapped onesJulius Härtl2023-05-241-2/+2
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #38065 from nextcloud/fix-result-mimetype-loaderSimon L2023-05-161-16/+10
|\ | | | | fix(mimetype): Fix returning value when finding existing mimetype in MimeType Loader
| * fix(mimetype): Fix returning value when finding existing mimetype in ↵Thomas Citharel2023-05-041-16/+10
| | | | | | | | | | | | | | | | | | | | | | MimeType Loader Also, only return the ID from the transaction as the mimetype string is already used from the function argument value Fixes https://github.com/nextcloud/server/pull/35744/files#r1184644610 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | Merge pull request #38196 from nextcloud/fix/fix-32bits-freespace-and-sizesCôme Chilliet2023-05-157-14/+17
|\ \ | | | | | | Get rid of more int casts in file size manipulations
| * | Fix return types in phpdocCôme Chilliet2023-05-155-5/+5
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | Get rid of more int casts in file size manipulationsCôme Chilliet2023-05-113-9/+12
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | Merge pull request #32961 from nextcloud/more-debug-lazyuserfolderCôme Chilliet2023-05-152-6/+18
|\ \ \ | | | | | | | | Make it easier to debug issue #32304
| * | | Add back missing slash in LazyUserFolder pathCôme Chilliet2023-05-091-1/+1
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
| * | | Make it easier to debug issue #32304Carl Schwan2023-05-042-7/+19
| | |/ | |/| | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | Merge pull request #37709 from nextcloud/bugfix/deadlockJulius Härtl2023-05-131-2/+6
|\ \ \
| * | | fix: Check for wrapped retriable exceptionsJulius Härtl2023-05-021-2/+6
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | Merge pull request #37621 from nextcloud/enh/lock-readable-pathJulius Härtl2023-05-131-0/+3
|\ \ \ \ | |_|_|/ |/| | |
| * | | fix: Wrap filesystem LockedExceptions for holding the readable pathJulius Härtl2023-05-021-0/+3
| |/ / | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | Merge pull request #37961 from nextcloud/poc/noid/systemtags-perfArthur Schiwon2023-05-113-54/+102
|\ \ \ | | | | | | | | SystemTags endpoint to return tags used by a user with meta data
| * | | refactor: remove where specification from SELECT getterArthur Schiwon2023-05-102-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - search constraints are now fully in control of SystemTagsInFilesDetector::detectAssignedSystemTagsIn(), avoids duplication of a WHERE statement Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | | refactor: remove SystemTag logic from Folder into QuerySearchHelperArthur Schiwon2023-05-092-52/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - adds OC\SystemTag\SystemTagsInFilesDetector where the search logic is moved to Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | | feat: specify media type via url path: systemtags-current/$mediaTypeArthur Schiwon2023-05-092-2/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - only the media part of the mime type can be search, but not the full mime type. It can be added, should it become necessary. - thus fixes previously hardcoded selector for image/ types - also fixes a return type hint - adds a return type hint Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | | PoC: SystemTags endpoint to return tags used by a user with meta dataArthur Schiwon2023-05-093-46/+98
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Target case is photos app: when visiting the tags category, all systemtags of the whole cloud are retrieved. In subequent steps the next tag is requested until the browser view is filled with tag tiles (i.e. previews are requested just as well). With this approach, we incorpoate the dav search and look for user related tags that are used by them, and already returns the statistics (number of files tagged with the respective tag) as well as a file id for the purpose to load the preview. This defaults to the file with the highest id. Call: curl -s -u 'user:password' \ 'https://my.nc.srv/remote.php/dav/systemtags-current' \ -X PROPFIND -H 'Accept: text/plain' \ -H 'Accept-Language: en-US,en;q=0.5' -H 'Depth: 1' \ -H 'Content-Type: text/plain;charset=UTF-8' \ --data @/home/doe/request-systemtag-props.xml With request-systemtag-props.xml: <?xml version="1.0" encoding="UTF-8"?> <d:propfind xmlns:d="DAV:"> <d:prop xmlns:oc="http://owncloud.org/ns" xmlns:nc="http://nextcloud.org/ns"> <oc:id/> <oc:display-name/> <oc:user-visible/> <oc:user-assignable/> <oc:can-assign/> <nc:files-assigned/> <nc:reference-fileid/> </d:prop> </d:propfind> Example output: … <d:response> <d:href>/master/remote.php/dav/systemtags/84</d:href> <d:propstat> <d:prop> <oc:id>84</oc:id> <oc:display-name>Computer</oc:display-name> <oc:user-visible>true</oc:user-visible> <oc:user-assignable>true</oc:user-assignable> <oc:can-assign>true</oc:can-assign> <nc:files-assigned>42</nc:files-assigned> <nc:reference-fileid>924022</nc:reference-fileid> </d:prop> <d:status>HTTP/1.1 200 OK</d:status> </d:propstat> </d:response> <d:response> <d:href>/remote.php/dav/systemtags/97</d:href> <d:propstat> <d:prop> <oc:id>97</oc:id> <oc:display-name>Bear</oc:display-name> <oc:user-visible>true</oc:user-visible> <oc:user-assignable>true</oc:user-assignable> <oc:can-assign>true</oc:can-assign> <nc:files-assigned>1</nc:files-assigned> <nc:reference-fileid>923422</nc:reference-fileid> </d:prop> <d:status>HTTP/1.1 200 OK</d:status> </d:propstat> </d:response> … Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | Merge pull request #37691 from nextcloud/object-store-background-scanJohn Molakvoæ2023-05-114-90/+110
|\ \ \
| * | | repair -1 folder sizes for object store background scanRobin Appelman2023-05-104-90/+110
| |/ / | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* / / Make sure to never trigger files hooks on a null pathCôme Chilliet2023-05-041-1/+4
|/ / | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Merge pull request #37943 from nextcloud/fix/fix-getcontent-return-typeArthur Schiwon2023-05-037-11/+13
|\ \ | |/ |/| Fix file_get_content signatures to make it clear it can return false
| * Fix file_get_content signatures to make it clear it can return falseCôme Chilliet2023-04-277-11/+13
| | | | | | | | | | | | | | In File::getContent, which must return a string, throw an Exception instead of returning false. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Merge pull request #38014 from nextcloud/norm-logicSimon L2023-05-021-13/+13
|\ \ | | | | | | Normalize logical operators
| * | Normalize logical operatorsGit'Fellow2023-05-021-13/+13
| |/ | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* / allow storing multiple mounts for the same rootid in the mount cacheRobin Appelman2023-04-281-17/+19
|/ | | | | | | | | | | | currently `[$userId, $rootId]` is used as the unique key for storing mounts in the mount cache, however there are cases where the same rootid is mounted in multiple places for a user which currently leads to not all of those mounts being added to the cache. Previously this didn't matter as the mount cache was only used to list users with access to a specific file, so a user having access to the file multiple times didn' change anything. With 24 the mount cache is used for more cases and multiple mounts for the same id becomes relevant. While I think there isn't a real negative effect atm besides missing the optimized path we should ensure that the mounts are properly listed Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #37818 from nextcloud/dav-ext-storage-logRobin Appelman2023-04-251-0/+18
|\ | | | | debug log requests made by dav external storage/shares
| * debug log requests made by dav external storage/sharesRobin Appelman2023-04-241-0/+18
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Add missing taint analysis docblock comments, and improve escaping in some ↵Carl Schwan2023-04-241-0/+1
| | | | | | | | | | | | | | methods Signed-off-by: Carl Schwan <carl@carlschwan.eu> Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Merge pull request #36857 from nextcloud/scan-repair-unencrypted-sizeRobin Appelman2023-04-246-40/+54
|\ \ | | | | | | fix unencrypted_size for folders when scanning the filesystem with encryption enabled
| * | fix type hintsRobin Appelman2023-04-044-8/+6
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix unencrypted folder size when no children with unencrypted size set are leftRobin Appelman2023-04-042-31/+26
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix unencrypted_size for files when scanningRobin Appelman2023-04-042-1/+12
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix unencrypted_size for folders when scanning the filesystem with ↵Robin Appelman2023-04-042-6/+16
| | | | | | | | | | | | | | | | | | encryption enabled Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #37820 from nextcloud/parent-storage-mtime-ignore-errorRobin Appelman2023-04-211-1/+13
|\ \ \ | |_|/ |/| | ignore errors while trying to update parent storage_mtime
| * | ignore errors while trying to update parent storage_mtimeRobin Appelman2023-04-201-1/+13
| | | | | | | | | | | | | | | | | | in the worst case this should only cause an extra rescan later Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #37062 from nextcloud/free-spaceRobin Appelman2023-04-211-1/+1
|\ \ \ | | | | | | | | Check free space only if source exists and is dir
| * | | Add function_exists() checkGit'Fellow2023-03-071-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: Simon L. <szaimen@e.mail.de> Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * | | Fix free space problemsGit'Fellow2023-03-071-1/+1
| | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | | Merge branch 'master' into enh/type-iconfig-getter-callsCôme Chilliet2023-04-208-74/+153
|\ \ \ \ | | |/ / | |/| | Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
| * | | Merge pull request #36774 from nextcloud/bugfix/noid/sabre-nodesSimon L2023-04-181-1/+5
| |\ \ \ | | | | | | | | | | fix: always use proper path on node api when calling the view
| | * | | fix: Use proper path when creating node instancesJulius Härtl2023-04-171-1/+5
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| * | | | Merge pull request #37534 from Rsplwe/masterSimon L2023-04-181-7/+18
| |\ \ \ \ | | |/ / / | |/| | | Adjust the value of the "max-parts" parameter of the object storage 'ListPart' interface to 1000
| | * | | Update lib/private/Files/ObjectStore/S3.phpRsplwe2023-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Rsplwe <i@rsplwe.com>
| | * | | s3: get remaining truncation partRsplwe2023-04-011-7/+18
| | | | | | | | | | | | | | | Signed-off-by: Rsplwe <i@rsplwe.com>
| | * | | Adjust the value of 'maxparts' to 1000Rsplwe2023-04-011-1/+1
| | | | | | | | | | | | | | | Signed-off-by: Rsplwe <i@rsplwe.com>
| * | | | Merge pull request #32314 from nextcloud/object-mkdir-loggingSimon L2023-04-171-0/+3
| |\ \ \ \ | | | | | | | | | | | | additional logging when mkdir fails for object storage
| | * | | | additional logging when mkdir fails for object storageRobin Appelman2023-04-051-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>