aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | 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 #46140 from nextcloud/fix-nc-env-inclusionRobin Appelman2024-09-161-5/+13
|\ \ \ \ | | | | | | | | | | fix(config): Add missing handling for `envCache` in `getKeys()`
| * | | | Merge branch 'master' into fix-nc-env-inclusionfix-nc-env-inclusionRobin Appelman2024-09-16309-3533/+8141
| |\| | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | fix: switch from explode to substr (faster)Josh Richards2024-07-081-1/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
| * | | | fix(config): Add missing handling for envCache in getKeys()Josh Richards2024-07-081-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NC_ env variable overrides were not appearing in the output of `occ config:list system` nor `occ config:system:get xxx`. This was creating nearly impossible to diagnose configuration/ behavior disprepancies. - Refactored readData() so that we aren't saving the entire environment in the envCache anymore (only those prefixed "NC_") and so that we save NC_ provided config values under their real key. - Refactored getValue() to accommodate readData() refactor - Fixed getKeys() to properly return envCache keys too Environment provided config variables now appear in `occ config:list system` as expected. Environment provided config variables now appear when queried via `occ config:system:get KEY` envCache is now free of non-NC stuff. Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* | | | | Merge pull request #47316 from Farcry69/fix/metadata-refresh-crashJohn Molakvoæ2024-09-161-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | fix: Prevent crash when refreshing metadata for files without an ownerVincent FarZz2024-08-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This crash was mostly happening when using external storages and for example the command "occ memories:migrate-google-takeout" Signed-off-by: Vincent FarZz <farcry69@live.fr>
* | | | | Merge pull request #47852 from nextcloud/sharding-code-fixesJohn Molakvoæ2024-09-166-11/+11
|\ \ \ \ \
| * | | | | fix: misc code fixes around db shardingsharding-code-fixesRobin Appelman2024-09-096-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | | Merge pull request #48063 from nextcloud/bugfix/noid/fix-otf-loadingJoas Schilling2024-09-161-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix(a11y): Add OTF font loading check
| * | | | | | fix(a11y): Add OTF font loading checkbugfix/noid/fix-otf-loadingJoas Schilling2024-09-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | | | Merge pull request #48042 from nextcloud/refactor/elvisChristoph Wurst2024-09-162-2/+2
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | refactor: Use the elvis operator + fix: Handle null checks with the ?? operator
| * | | | | | fix: Handle null checks with the ?? operatorrefactor/elvisChristoph Wurst2024-09-152-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
| * | | | | | refactor: Use the elvis operatorChristoph Wurst2024-09-152-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | | Merge pull request #48046 from nextcloud/refactor/self-class-referenceAnna2024-09-152-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | refactor: Replace __CLASS__ with ::class references
| * | | | | | | refactor: Replace __CLASS__ with ::class referencesrefactor/self-class-referenceChristoph Wurst2024-09-152-2/+2
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | | Merge pull request #48010 from nextcloud/feat/switch-from-settype-to-castsAnna2024-09-151-8/+12
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | feat(db): switch from settype to casts
| * | | | | | | feat(db): switch from settype to castsfeat/switch-from-settype-to-castsAnna Larch2024-09-151-8/+12
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* / / / / / / refactor: migrate dirname to __DIR__ constantsrefactor/dirname-to-dirChristoph Wurst2024-09-151-1/+1
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | | 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>
* | | | | | Merge pull request #48013 from ↵Kate2024-09-151-1/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | nextcloud/fix/baseresponse/xml-element-value-string-cast
| * | | | | | fix(BaseResponse): Cast XML element values to stringfix/baseresponse/xml-element-value-string-castprovokateurin2024-09-151-1/+3
| |/ / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | | | Merge pull request #48014 from nextcloud/fix/tags/boolean-user-has-tagsKate2024-09-151-2/+1
|\ \ \ \ \ \
| * | | | | | fix(Tags): Return boolean for userHasTags()fix/tags/boolean-user-has-tagsprovokateurin2024-09-151-2/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | | | Merge pull request #48017 from nextcloud/fix/comment/children-count-integerKate2024-09-151-1/+1
|\ \ \ \ \ \
| * | | | | | fix(Comment): Initialize childrenCount as integerfix/comment/children-count-integerprovokateurin2024-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>
* | | | | Merge pull request #47971 from ↵Côme Chilliet2024-09-141-1/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/fix/do-not-try-to-change-response-code-too-late fix: Do not try to set HTTP response code on already closed connection
| * | | | | fix: Do not try to set HTTP response code on already closed connectionCôme Chilliet2024-09-141-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids a PHP warning in the logs about trying to set the response code while the output already started. It’s useless to try to print an error page anyway in this situation because the connection was closed already. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | | feat(PsrLoggerAdapter): Allow to use `Psr\Log\LogLevel` for `log` methodFerdinand Thiessen2024-09-141-89/+30
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is the `Psr\Log\LogLevel` class defining loglevel constants, to be fully compatible we should at least support those logging levels. Moreover this is the last part that was still required from `ILogger` interface, as we did not have alternatives for the loglevel constants. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.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 #47927 from nextcloud/fix/migrate-away-from-oc_appCôme Chilliet2024-09-138-70/+74
|\ \ \ \ \ | | | | | | | | | | | | Migrate away from OC_App to IAppManager
| * | | | | fix: Replace OC_App::getAllApps with a method in AppManagerCôme Chilliet2024-09-135-36/+38
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | chore: Migrate cleanAppId and getAppPath calls to IAppManager from OC_AppCôme Chilliet2024-09-135-34/+36
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | | Merge pull request #47865 from ↵Côme Chilliet2024-09-132-10/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/admin_audit/enh/move-to-event-listeners-v2 Move admin_audit to proper event listeners v2
| * | | | | | chore: Code cleanup in lib/private/Console/ApplicationThomas Citharel2024-09-101-10/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
| * | | | | | feat: Add mimetype into BeforePreviewFetchedEvent eventThomas Citharel2024-09-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | | | | | Merge pull request #47883 from nextcloud/fix/setup-checksFerdinand Thiessen2024-09-131-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | fix(setup-checks): Ensure URL with webroot works
| * | | | | | | fix(setup-checks): Ensure URL with webroot worksFerdinand Thiessen2024-09-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We basically mock the way `URLGenerator::getAbsoluteURL` works, so we must make sure that the URL might already contain the webroot. Because `baseURL` and `cliURL` also contain the webroot we need to remove the webroot from the URL first. Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Daniel <mail@danielkesselberg.de> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | | | | | Merge pull request #47933 from ↵Joas Schilling2024-09-131-3/+3
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/throw-precondition-failure-when-not-matching fix(config): Throw PreconditionException always when it didn't match
| * | | | | | | fix(config): Throw PreconditionException always when it didn't matchJoas Schilling2024-09-131-3/+3
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously even when the precondition did not match, the call "passed" when the after value was the expected one. This however can lead to race conditions, duplicate code excutions and other things. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | | | Merge pull request #47854 from nextcloud/validAppsFromAppStoreGit'Fellow2024-09-131-1/+2
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | fix(appstore): Ensure returned apps from AppStore are valid
| * | | | | | fix: check if key is setGit'Fellow2024-09-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * | | | | | fix(appstore): Ensure returned apps from AppStore are validGit'Fellow2024-09-111-1/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: lint chore: remove space Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: check if response array is null Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> chore: Add log
* | | | | | Merge pull request #47685 from nextcloud/fix/move-apihelper-to-oc-namespaceCôme Chilliet2024-09-124-318/+80
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix: Move OC_API into \OC\ApiHelper in standard namespace
| * | | | | | chore: Make parameter name more explicitCôme Chilliet2024-09-091-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | | fix: Force 503 HTTP status code for maintenance mode on v1Côme Chilliet2024-09-091-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | | chore: Deleted now unused classes from \OC\OCSCôme Chilliet2024-09-092-156/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>