aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
* 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>
| * | fix: Remove duplicated code in \OC\OCS\ApiHelper, use Response classes insteadCôme Chilliet2024-09-091-90/+18
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | fix: Move ApiHelper to \OC\OCS next to related classesCôme Chilliet2024-09-091-2/+1
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | fix: A bit of code cleanup in OC\AppFramework\OCS\ApiHelperCôme Chilliet2024-09-091-30/+17
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | fix: Move OC_API into \OC\ApiHelper in standard namespaceCôme Chilliet2024-09-091-1/+4
| | | | | | | | | | | | | | | | | | It’s only used by ocs/v1.php Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | fix: Add whiteboard to education and public sector bundleJulius Knorr2024-09-122-0/+2
| | | | | | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | | Merge pull request #47586 from nextcloud/fix/colorFerdinand Thiessen2024-09-113-2/+3
|\ \ \ | | | | | | | | fix(theming): Add migration to restore primary color after separating primary and background
| * | | fix: Adjust types of IQueryBuilder to properly allow joining with sub-queryFerdinand Thiessen2024-09-102-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | | chore: Use `IAppConfig` instead of `IConfig->getAppValue`Ferdinand Thiessen2024-09-101-0/+1
| | | | | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | Merge pull request #47662 from ↵Kate2024-09-102-2/+11
|\ \ \ \ | | | | | | | | | | | | | | | nextcloud/fix/notification/validate-rich-object-key-value-types
| * | | | fix(Activity): Restrict IEvent rich object parameters typeprovokateurin2024-09-101-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
| * | | | fix(RichObjectStrings/Validator): Validate key value types of rich object ↵provokateurin2024-09-101-0/+9
| |/ / / | | | | | | | | | | | | | | | | | | | | parameters 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>
* | | | Merge pull request #47834 from nextcloud/retAppsManDisJohn Molakvoæ2024-09-101-2/+9
|\ \ \ \
| * | | | fix(appstore): disabled appstore log level bump from debug to infoJohn Molakvoæ2024-09-101-1/+1
| | | | | | | | | | | | | | | | | | | | Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * | | | fix(appstore): return if appstore is manually disabledGit'Fellow2024-09-091-2/+9
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: camel case Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: log a debug log Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: typing Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: typo fix: typing
* | | | Merge pull request #47858 from nextcloud/fix/preview/handle-no-local-fileRichard Steinmetz2024-09-101-0/+3
|\ \ \ \ | | | | | | | | | | fix(preview): gracefully handle file not being opened in ProviderV2
| * | | | fix(preview): gracefully handle file not being opened in ProviderV2Richard Steinmetz2024-09-101-0/+3
| | |/ / | |/| | | | | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* | | | fix: Only write once to template instead of create/copyJulius Knorr2024-09-101-3/+3
| |_|/ |/| | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | | fix(NavigationManager): Skip invalid default navigation entriesprovokateurin2024-09-091-12/+25
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | fix(TemplateLayout): Use NavigationManager to get default app nameprovokateurin2024-09-091-5/+4
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | fix(URLGenerator): Use NavigationManager to generate default page URLprovokateurin2024-09-091-6/+19
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | refactor(AppManager): Deprecated default apps handlingprovokateurin2024-09-091-46/+36
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | feat(NavigationManager): Add default entries handlingprovokateurin2024-09-091-10/+87
|/ / | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* / fix(files): Never return a null ETag in DAVprovokateurin2024-09-093-10/+3
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore(i18n): Spelling of IDrakekniven2024-09-061-2/+2
| | | | | Reported at Transifex Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
* fix(TaskProcessing): increase error_message column lengthAnupam Kumar2024-09-061-2/+3
| | | | Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
* 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
* chore: adjust code to adhere to coding standardAnna Larch2024-09-0514-22/+22
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>