summaryrefslogtreecommitdiffstats
path: root/lib/public
Commit message (Collapse)AuthorAgeFilesLines
* Allow to expire comments of multiple objects with one callJoas Schilling2022-07-011-2/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add status automationJoas Schilling2022-07-011-0/+12
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #33007 from nextcloud/cleanup/federation-appCarl Schwan2022-06-281-0/+48
|\ | | | | Summer cleanup of the federation app
| * Summer cleanup of the federation appCarl Schwan2022-06-241-0/+48
| | | | | | | | | | | | | | | | | | | | - Use IEventDispatcher instead of deprecated symfony dispatcher - Use LoggerInterface where possible - Use php 7.4 properties - Add type hinting where possible - Move federation hooks to a seperate listener Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | Merge pull request #32863 from nextcloud/feature/add-comments-expire-dateblizzz2022-06-242-0/+28
|\ \ | | | | | | Add comments expire date
| * | Add comments expire dateVitor Mattos2022-06-152-0/+28
| |/ | | | | | | | | | | https://github.com/nextcloud/spreed/pull/7327 Signed-off-by: Vitor Mattos <vitor@php.rio>
* / load widgets only of enabled appsArthur Schiwon2022-06-221-1/+1
|/ | | | | | | | | | | | | - per design, all enabled apps have their registration run - limitations, e.g. enabled by group, are not considered in that state, because we do not have a session (and might need apps?) - before instantiation of widget it has to be checked whether the providing app is actually enabled for the logged in user. - a public interface is being changed, but it is not meant to be implemented or used outside of the core handling. Therefore save to backport. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #31966 from nextcloud/unencrypted-sizeVincent Petry2022-06-134-36/+58
|\ | | | | store unencrypted size in the unencrypted_size column
| * store unencrypted size in the unencrypted_size columnRobin Appelman2022-06-021-0/+10
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * add case statement to sql function builderRobin Appelman2022-06-021-0/+12
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * make expression build return IQueryFunction instead of stringRobin Appelman2022-06-022-36/+36
| | | | | | | | | | | | this allows passing the expressions to further expressions without them being escaped as column names Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #32364 from nextcloud/dav-listenersVincent Petry2022-06-102-2/+3
|\ \ | | | | | | 🧹 Remove all legacy event dispatchers from CalDAV & CardDAV backends
| * | Fix a doctype in OCP for IAddressBookThomas Citharel2022-05-172-2/+3
| | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | Merge pull request #26582 from nextcloud/enhancement/logger-functionChristoph Wurst2022-06-071-0/+73
|\ \ \ | |_|/ |/| | Add a helper function that makes it easier to log from anywhere
| * | Add a helper function that makes it easier to log from anywhereChristoph Wurst2022-05-231-0/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our DI is able to inject a logger implementation to any server and app class if they want one. However, sometimes DI isn't applicable or hard to add. In those cases we typically fell back to the *service locator* pattern where we acquired a logger from the server via a global variable. There were some issues with that * `\OC` is a private class, apps are not supposed to use it * `\OC::$server` is a global variable, a well known anti-pattern * `\OC::$server->get(...)` uses the service locator anti-pattern * `\OC::$server->get(...)` may throw * `\OC::$server->get(LoggerInterface::class)` is not scoped to an app With this patch I'm proposing a new helper function ``\OCP\Log\logger`` that can be used to acquire a logger more easily. This function is meant to be public API and therefore apps may use it and there is an optional parameter to specifiy the app ID. The function hides all the ugly details about the global variable and the potentially thrown exceptions from the user. Therefore it's guaranteed that you always get a logger instance. In the worst case you get a noop, though those occasions should be rare. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Merge pull request #32485 from nextcloud/debt/noid/psalm-streamer-fhblizzz2022-05-311-1/+1
|\ \ \ | | | | | | | | [Psalm] Fix docblock for addFileFromStream
| * | | Fix type for resourceDaniel Kesselberg2022-05-241-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | | Merge pull request #32628 from nextcloud/bugfix/noid/json-throwJoas Schilling2022-05-311-7/+1
|\ \ \ \ | | | | | | | | | | Use JSON_THROW_ON_ERROR instead of custom error handling
| * | | | Use JSON_THROW_ON_ERROR instead of custom error handlingJulius Härtl2022-05-301-7/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | | Add the ISizeEstimationMigrator interface for method getExportEstimatedSizeCôme Chilliet2022-05-302-8/+43
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | Implement getExportEstimatedSize in migratorsCôme Chilliet2022-05-301-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | Add a method to get estimated export size in IMigratorCôme Chilliet2022-05-301-0/+8
|/ / / / | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | Merge pull request #32349 from nextcloud/enh/projects-eventCarl Schwan2022-05-271-0/+40
|\ \ \ \ | |/ / / |/| | | Add event to load additional scripts for projects
| * | | Introduce event for loading additional script on projectsJulius Härtl2022-05-121-0/+40
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | Merge pull request #32555 from nextcloud/fix/revert-api-breakCôme Chilliet2022-05-241-2/+2
|\ \ \ \ | | | | | | | | | | Revert api break
| * | | | Revert api breakCarl Schwan2022-05-241-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | | Merge pull request #32560 from nextcloud/feature/noid/preferences-apiJoas Schilling2022-05-242-0/+175
|\ \ \ \ \ | | | | | | | | | | | | Add API to change preferences
| * | | | | Add API to change preferencesJoas Schilling2022-05-232-0/+175
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* / / / / Adjust docs to the same as the implementationJoas Schilling2022-05-231-1/+1
|/ / / / | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Merge pull request #32378 from nextcloud/fix/psalm-userbackendCarl Schwan2022-05-235-9/+8
|\ \ \ \ | |_|/ / |/| | | Fix psalm issues related to the user backend
| * | | Fix psalm issues related to the user backendCarl Schwan2022-05-205-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Reflect the actual return value returned by the implementation in the the interface. E.g. IUser|bool -> IUser|false - Remove $hasLoggedIn parameter from private countUser implementation. Replace the two call with the equivalent countSeenUser - getBackend is nuallable, add this to the interface - Use backend interface to make psalm happy about call to undefined methods. Also helps with getting rid at some point of the old implementActions Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Make appName of TemplateResponse accessible in BeforeTemplateRenderedEventJoas Schilling2022-05-201-0/+9
| | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Merge pull request #32406 from nextcloud/debt/noid/psalm-warning-idbconnectionJoas Schilling2022-05-201-1/+1
|\ \ \ \ | |_|_|/ |/| | | [Psalm] Fix spelling for PreConditionNotMetException
| * | | Fix spelling for PreConditionNotMetExceptionDaniel Kesselberg2022-05-151-1/+1
| |/ / | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #32404 from nextcloud/debt/noid/psalm-warning-iaddressbookCôme Chilliet2022-05-171-1/+0
|\ \ \ | | | | | | | | [Psalm] Drop redundanten return comment
| * | | Drop redundanten return commentDaniel Kesselberg2022-05-151-1/+0
| |/ / | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #32405 from nextcloud/debt/noid/psalm-warning-isearchrequestJohn Molakvoæ2022-05-171-1/+1
|\ \ \
| * | | Fix spelling for return typeDaniel Kesselberg2022-05-151-1/+1
| |/ / | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Allow to tweak default scopes for accountsThomas Citharel2022-05-161-0/+36
| | | | | | | | | | | | | | | | | | Close #6582 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | Fix psalm errors fron the end of the baseline fileCarl Schwan2022-05-163-5/+5
| | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | Merge pull request #31662 from nextcloud/modernize-contactmenuJohn Molakvoæ2022-05-133-17/+13
|\ \ \
| * | | Modernize contacts menuThomas Citharel2022-05-123-17/+13
| | | | | | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | | | Merge pull request #32384 from nextcloud/improve-psalm-server-getJohn Molakvoæ2022-05-131-2/+3
|\ \ \ \
| * | | | Improve psalm annotation to make it stricterCarl Schwan2022-05-131-2/+3
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | Now using class-string<T> as input will only return T, and any other string will return mixed Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* / | | Fix psalm warning for zip response due wrong typeDaniel Kesselberg2022-05-131-4/+4
|/ / / | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #32077 from nextcloud/cleanup/lockCarl Schwan2022-05-121-19/+17
|\ \ \ | |/ / |/| | Cleanup lock related code
| * | Cleanup lock related codeCarl Schwan2022-05-121-19/+17
| | | | | | | | | | | | | | | | | | | | | | | | - Port to QueryBuilder - Improve the doc a bit - Add type hinting Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | Merge pull request #32344 from ↵Joas Schilling2022-05-121-9/+8
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/fix/always-free-db-result-qbmapper-find-entities Always free the DB result in QBMapper::findEntities
| * | | Always free the DB result in QBMapper::findEntitiesChristoph Wurst2022-05-121-9/+8
| | |/ | |/| | | | | | | | | | | | | | | | Without this patch it only happened if the code ran through without any errors. Now the result is also freed in the case of an error. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Merge pull request #32339 from nextcloud/appdatafactory-publicCarl Schwan2022-05-121-0/+21
|\ \ \ | |/ / |/| | Make it possible to get the appdata folder using the public API
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-10 09:00:54 +0000