summaryrefslogtreecommitdiffstats
path: root/lib/private/DB
Commit message (Collapse)AuthorAgeFilesLines
* fix(workflowengine): use andWhere() not second where()Arthur Schiwon2023-11-131-1/+1
| | | | | | | - an unconditional where() condition is added right before - fixes also wrongly named methods in the Exception message Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix(install): Make installing more verboseJoas Schilling2023-11-031-0/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix: Check for wrapped retriable exceptionsJulius Härtl2023-05-021-0/+5
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Use typed version of IConfig::getSystemValue as much as possibleCôme Chilliet2023-04-053-3/+3
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix DBAL exception handling in setValuesArthur Schiwon2023-03-311-7/+11
| | | | | | | | | This seems to be a left over after abstracting DBAL. Nowadays, IQueryBuilder::executeStatement() only throws a \OCP\DB\Exception, where previously original DBAL exceptions where thrown. These are now wrapped, the orignal classes are now mapped to a reason. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix(DB): Remove not supported column comments when using SQLiteFerdinand Thiessen2023-02-221-1/+5
| | | | Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>
* log start time of queries for profilerRobin Appelman2023-02-141-0/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #36261 from ↵Joas Schilling2023-01-231-0/+6
|\ | | | | | | | | nextcloud/techdebt/noid/warn-devs-about-potential-bugs fix(querybuilder): Log a warning if where() is called again on a quer…
| * fix(querybuilder): Log a warning if where() is called again on a query ↵Joas Schilling2023-01-201-0/+6
| | | | | | | | | | | | builder object Signed-off-by: Joas Schilling <coding@schilljs.com>
* | composer run cs:fixCôme Chilliet2023-01-2016-17/+1
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Fix more psalm issues in OC\DBCôme Chilliet2022-11-211-2/+4
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Fix psalm issues following doctrine/dbal bumpCôme Chilliet2022-11-212-4/+5
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* typehint migrationservice closuresRoeland Jago Douma2022-10-091-4/+5
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Change option name again to be more like the other optionsRetidurc Silvernight2022-09-231-1/+1
| | | | Signed-off-by: Retidurc Silvernight <retidurc@silvernight.social>
* Change config option to db.persistentRetidurc Silvernight2022-09-231-1/+1
| | | | Signed-off-by: Retidurc Silvernight <retidurc@silvernight.social>
* Expose Doctrine settings in Netxcloud settingsRetidurc Silvernight2022-09-231-0/+4
| | | | Signed-off-by: Retidurc Silvernight <retidurc@silvernight.social>
* Declare strict types in all new Event classesCôme Chilliet2022-08-251-0/+3
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Migrate Repair events to OCP\EventDispatcher\EventCôme Chilliet2022-08-252-2/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Migrate Migrator::executeSql to OCP\EventDispatcher\EventCôme Chilliet2022-08-252-7/+57
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Switch to string keys for argument of GenericEvent for OC\DB\MigratorCôme Chilliet2022-08-251-21/+10
| | | | | | It seems checkTable is actually never dispatched? Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Revert "add case statement to sql function builder"Robin Appelman2022-08-161-11/+0
| | | | | | This reverts commit 2a68819a67045d87a369a8a6413f153b3b2bea5f. Signed-off-by: Robin Appelman <robin@icewind.nl>
* Revert "make expression build return IQueryFunction instead of string"Robin Appelman2022-08-166-92/+85
| | | | | | This reverts commit 813b50ed428a8bc36817d19c84444e96dbe3b668. Signed-off-by: Robin Appelman <robin@icewind.nl>
* Fix typos in lib/private subdirectoryluz paz2022-07-271-1/+1
| | | | | | Found via `codespell -q 3 -S l10n -L jus ./lib/private` Signed-off-by: luz paz <luzpaz@github.com>
* composer run cs:fixCôme Chilliet2022-06-211-1/+0
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Fix PHP 8.2 warnings about undeclared propertiesCôme Chilliet2022-06-211-21/+13
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* add case statement to sql function builderRobin Appelman2022-06-021-0/+11
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* make expression build return IQueryFunction instead of stringRobin Appelman2022-06-026-85/+92
| | | | | | this allows passing the expressions to further expressions without them being escaped as column names Signed-off-by: Robin Appelman <robin@icewind.nl>
* Fix psalm issues related to the user backendCarl Schwan2022-05-201-1/+1
| | | | | | | | | | | | | - 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>
* tell mysql to ignore the sort index for search queriesRobin Appelman2022-04-221-1/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Log missing primary key until we make it a requirementJoas Schilling2022-04-081-1/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add missing doc changesJoas Schilling2022-04-071-0/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix typos and empty tabsJoas Schilling2022-04-071-4/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Improve error logging on migration failureJohn Molakvoæ2022-04-071-4/+2
| | | | Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
* Ensure schema change before checking OracleConstraintsJohn Molakvoæ2022-04-071-12/+23
| | | | Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
* record backtrace when profiling db requestsRobin Appelman2022-04-043-5/+37
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #30823 from nextcloud/work/profilerCarl Schwan2022-04-043-0/+240
|\ | | | | | | | | | | Built-in profiler This adds the required API for collecting information about requests. This information can then be displayed with the new 'profiler' app.
| * Add a built-in profiler inside NextcloudCarl Schwan2022-04-043-0/+240
| | | | | | | | | | | | The webui is provided by a seperate application named profiler Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | Merge pull request #31679 from ↵Côme Chilliet2022-03-311-0/+4
|\ \ | |/ |/| | | | | nextcloud/bugfix/noid/ensure-string-columns-to-be-maximum-of-4000 Ensure string column limit of 4.000 characters
| * Ensure string column limit of 4.000 charactersJoas Schilling2022-03-251-0/+4
| | | | | | | | | | | | https://docs.oracle.com/en/database/oracle/oracle-database/19/refrn/datatype-limits.html#GUID-963C79C9-9303-49FE-8F2D-C8AAF04D3095 Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #31609 from nextcloud/fix/migrate-away-from-iloggerCôme Chilliet2022-03-293-52/+46
|\ \ | | | | | | Migrate from ILogger to LoggerInterface in lib/private
| * | Fix log level for exception in QueryBuilderCôme Chilliet2022-03-241-1/+1
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | Adapt more code to migration to LoggerInterfaceCôme Chilliet2022-03-242-36/+36
| | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | Migrate from ILogger to LoggerInterface in lib/privateCôme Chilliet2022-03-241-16/+10
| |/ | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Fix LENGTH function name across databasesCôme Chilliet2022-03-283-1/+13
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Use OCTET_LENGTH which is more common than LENGTHBCôme Chilliet2022-03-242-1/+7
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | Add octetLength and charLength to function builder, and testsCôme Chilliet2022-03-242-0/+18
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #31235 from nextcloud/techdebt/noid/extract-request-idJoas Schilling2022-03-221-2/+8
|\ | | | | Extract request id handling to dedicated class so it can be injected without DB dependency
| * Allow to prefix the Query log with the request idJoas Schilling2022-02-231-2/+8
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Disable for now again until more apps are fixedJoas Schilling2022-03-211-2/+2
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Limit missing primary key fail to new tablesJoas Schilling2022-03-211-1/+1
| | | | | | | | | | | | | | This will work on CI so devs notice it when they install the app for testing, and at the same time existing faulty tables don't break the upgrade to 24 Signed-off-by: Joas Schilling <coding@schilljs.com>
generated by cgit v1.2.3 (git 2.39.1) at 2025-02-12 22:08:22 +0000