aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private/DB
Commit message (Collapse)AuthorAgeFilesLines
...
* | 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>
* | Enforce primary keys as they are required to be performant in MySQL clustersJoas Schilling2022-03-101-0/+2
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Add event logging for db and redis connectionJulius Härtl2022-02-281-1/+11
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Add --dry-run option for db:add-missing-* cmdLouis Chemineau2022-02-241-2/+10
|/ | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Allow specify a config prefix for another database connectionJoas Schilling2022-02-031-6/+7
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #30379 from nextcloud/feature/add-comments-reactionsJoas Schilling2022-01-211-2/+2
|\ | | | | Add comments reactions
| * Remove order from groupConcat as it is not working everywhereJoas Schilling2022-01-214-5/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Fix groupConcat and ordering on OracleVitor Mattos2022-01-214-5/+5
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
| * Prevent query error when use subqueryVitor Mattos2022-01-211-2/+2
| | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | Merge pull request #30471 from nextcloud/enh/cast-char-mysqlJoas Schilling2022-01-204-0/+22
|\ \ | |/ |/| Explicitly cast char in the query builder
| * Fix psalm complaintJoas Schilling2022-01-041-0/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Explicitly cast char in the query builderJulius Härtl2022-01-043-0/+21
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #30129 from nextcloud/mdb10.6-pipelineJohn Molakvoæ2022-01-191-1/+0
|\ \
| * | Attempt to fix CIacsfer2021-12-071-1/+0
| | |
* | | Merge pull request #30508 from nextcloud/fix/psaml-binCarl Schwan2022-01-132-2/+2
|\ \ \ | | | | | | | | Fix psalm not running
| * | | Check style updateCarl Schwan2022-01-132-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Unit testsVitor Mattos2022-01-071-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | | | Accept multipe args on concatVitor Mattos2022-01-064-6/+30
|/ / / | | | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | | Use quote functionJoas Schilling2022-01-041-6/+2
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Try escaping Oracle single quotesJoas Schilling2022-01-041-1/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Add a test with integerJoas Schilling2022-01-045-16/+26
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Fix oracle LISTAGGJoas Schilling2022-01-031-1/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Add unit testVitor Mattos2022-01-034-23/+11
| | | | | | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | | Add unit test and orderBy parameterVitor Mattos2022-01-035-13/+38
| | | | | | | | | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | | Add group_concat aggregator functionVitor Mattos2022-01-033-0/+18
| |/ |/| | | | | Signed-off-by: Vitor Mattos <vitor@php.rio>
* | Check if directoy is writable instead of possibly missing fileCarl Schwan2021-12-201-1/+1
|/ | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Allow to log DB, redis and LDAP requests into filesJoas Schilling2021-12-011-0/+14
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix default value for $type in ↵Côme Chilliet2021-11-231-2/+2
| | | | | | | | | OC\DB\QueryBuilder\ExpressionBuilder\ExpressionBuilder::literal This fixes PHP errors about passing null to PDO::quote second parameter. We may want to change IExpressionBuilder as well? Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Allow NULL as well for limit, not integer onlyJoas Schilling2021-11-152-7/+16
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Bump doctrine/dbal from 3.0.0 to 3.1.3Christoph Wurst2021-10-221-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* disable path prefix index on postgresql for nowRobin Appelman2021-10-081-0/+13
| | | | | | | having the index work properly for the queries we need it for requires some additional options which dbal does not support at the momement. to prevent making it harder to add the correct index later on we don't create the index for now on postgresql Signed-off-by: Robin Appelman <robin@icewind.nl>
* make it possible to override the default collationArthur Schiwon2021-09-013-9/+13
| | | | | | - allows admins to configure it for edge cases like accent sensitivity Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Allow casting query functions as wellJoas Schilling2021-07-073-5/+5
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Update php licensesJohn Molakvoæ (skjnldsv)2021-06-0443-88/+43
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Rename IQueryBuilder::executeUpdate to IQueryBuilder::executeStatementChristoph Wurst2021-05-051-2/+17
| | | | | | | | | Because executeUpdate wasn't a great name. And in DBAL they also use executeStatement more consistently now. Ref https://github.com/doctrine/dbal/issues/4607 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Do not check Oracle column constraints in unrelated migrationsChristoph Wurst2021-04-191-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Rename the method to match what it doesJoas Schilling2021-03-311-3/+3
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Don't allow Notnull for boolean columnsJoas Schilling2021-03-311-0/+5
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Document the constraints we test againstJoas Schilling2021-03-311-0/+17
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Fix exception messages spacingJoas Schilling2021-03-311-2/+2
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Cleanup unneeded code around database.xmlMorris Jobke2021-03-243-520/+24
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #24384 from nextcloud/cleanup/remove-old-migration-checkMorris Jobke2021-03-174-138/+5
|\ | | | | Remove dead code from checking core/apps before upgrades
| * Skip db migration simulation for core schema changesJulius Härtl2021-03-174-138/+5
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #25924 from ↵Morris Jobke2021-03-171-2/+4
|\ \ | |/ |/| | | | | nextcloud/bugfix/noid/only-mark-migrations-as-installed-after-executing-them-in-case-the-developer-broke-them Only mark migrations as installed after execution
| * Only mark migrations as installed after executionJoas Schilling2021-03-041-2/+4
| | | | | | | | | | | | | | The problem is that if a developer creates a structural error in their migration file, they will already be marked as executed and an not be rerun. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #25929 from nextcloud/techdept/psalm/25839/redundantcastsRoeland Jago Douma2021-03-051-2/+2
|\ \ | | | | | | Remove Redundantcasts
| * | Remove RedundantcastsRoeland Jago Douma2021-03-041-2/+2
| |/ | | | | | | | | | | For #25839 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Type CompositeExpression and ICompositeExpressionRoeland Jago Douma2021-03-041-5/+5
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Add executeQuery and executeUpdateRoeland Jago Douma2021-03-041-0/+37
|/ | | | | | | | | | | Names shamelessly copied from Doctrine itself. Internally it is still using the same flow. But I added some checks around it. This should make static analysis a bit more happy. Which in turn makes me more happy. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>