mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #24049 from nextcloud/backport/23882/stable20 

[stable20] Run unit tests against oracle
This commit is contained in:
Morris Jobke 2020-11-12 19:54:36 +01:00 committed by GitHub
parent 140068e91d 9497dca2b0
commit 2e3637f826
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
67 changed files with 946 additions and 296 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

56
.github/workflows/oci.yml vendored Normal file
View File

@ -0,0 +1,56 @@
name: PHPUnit
on:
pull_request:
push:
branches:
- master
- stable*
jobs:
phpunit-oci8:
runs-on: ubuntu-latest
strategy:
# do not stop on another job's failure
fail-fast: false
matrix:
php-versions: [ '7.4' ]
databases: [ 'oci' ]
name: php${{ matrix.php-versions }}-${{ matrix.databases }}
services:
oracle:
image: deepdiver/docker-oracle-xe-11g # "wnameless/oracle-xe-11g-r2"
ports:
- "1521:1521"
steps:
- name: Checkout server
uses: actions/checkout@v2
- name: Checkout submodules
shell: bash
run: |
auth_header="$(git config --local --get http.https://github.com/.extraheader)"
git submodule sync --recursive
git -c "http.extraheader=$auth_header" -c protocol.version=2 submodule update --init --force --recursive --depth=1
- name: Set up php ${{ matrix.php-versions }}
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php-versions }}
extensions: mbstring, iconv, fileinfo, intl, sqlite, pdo_sqlite, oci8
tools: phpunit:8.5.2
coverage: none
- name: Set up Nextcloud
run: |
mkdir data
./occ maintenance:install --verbose --database=oci --database-name=XE --database-host=127.0.0.1 --database-port=1521 --database-user=autotest --database-pass=owncloud --admin-user admin --admin-pass admin
php -f index.php
- name: PHPUnit
working-directory: tests
run: phpunit --configuration phpunit-autotest.xml --group DB,SLOWDB

2
apps/accessibility/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/admin_audit/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/cloud_federation_api/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/comments/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/contactsinteraction/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/dav/appinfo/info.xml
View File

@ -5,7 +5,7 @@
<name>WebDAV</name>
<summary>WebDAV endpoint</summary>
<description>WebDAV endpoint</description>
<version>1.16.0</version>
<version>1.16.1</version>
<licence>agpl</licence>
<author>owncloud.org</author>
<namespace>DAV</namespace>

2
apps/dav/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

1
apps/dav/composer/composer/autoload_classmap.php
View File

@ -230,6 +230,7 @@ return array(
'OCA\\DAV\\Migration\\Version1011Date20190725113607' => $baseDir . '/../lib/Migration/Version1011Date20190725113607.php',
'OCA\\DAV\\Migration\\Version1011Date20190806104428' => $baseDir . '/../lib/Migration/Version1011Date20190806104428.php',
'OCA\\DAV\\Migration\\Version1012Date20190808122342' => $baseDir . '/../lib/Migration/Version1012Date20190808122342.php',
'OCA\\DAV\\Migration\\Version1016Date20201109085907' => $baseDir . '/../lib/Migration/Version1016Date20201109085907.php',
'OCA\\DAV\\Provisioning\\Apple\\AppleProvisioningNode' => $baseDir . '/../lib/Provisioning/Apple/AppleProvisioningNode.php',
'OCA\\DAV\\Provisioning\\Apple\\AppleProvisioningPlugin' => $baseDir . '/../lib/Provisioning/Apple/AppleProvisioningPlugin.php',
'OCA\\DAV\\RootCollection' => $baseDir . '/../lib/RootCollection.php',

1
apps/dav/composer/composer/autoload_static.php
View File

@ -245,6 +245,7 @@ class ComposerStaticInitDAV
'OCA\\DAV\\Migration\\Version1011Date20190725113607' => __DIR__ . '/..' . '/../lib/Migration/Version1011Date20190725113607.php',
'OCA\\DAV\\Migration\\Version1011Date20190806104428' => __DIR__ . '/..' . '/../lib/Migration/Version1011Date20190806104428.php',
'OCA\\DAV\\Migration\\Version1012Date20190808122342' => __DIR__ . '/..' . '/../lib/Migration/Version1012Date20190808122342.php',
'OCA\\DAV\\Migration\\Version1016Date20201109085907' => __DIR__ . '/..' . '/../lib/Migration/Version1016Date20201109085907.php',
'OCA\\DAV\\Provisioning\\Apple\\AppleProvisioningNode' => __DIR__ . '/..' . '/../lib/Provisioning/Apple/AppleProvisioningNode.php',
'OCA\\DAV\\Provisioning\\Apple\\AppleProvisioningPlugin' => __DIR__ . '/..' . '/../lib/Provisioning/Apple/AppleProvisioningPlugin.php',
'OCA\\DAV\\RootCollection' => __DIR__ . '/..' . '/../lib/RootCollection.php',

44
apps/dav/lib/CalDAV/CalDavBackend.php
View File

@ -241,8 +241,13 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
$principalUri = $this->convertPrincipal($principalUri, true);
$query = $this->db->getQueryBuilder();
$query->select($query->func()->count('*'))
->from('calendars')
->where($query->expr()->eq('principaluri', $query->createNamedParameter($principalUri)));
->from('calendars');
if ($principalUri === '') {
$query->where($query->expr()->emptyString('principaluri'));
} else {
$query->where($query->expr()->eq('principaluri', $query->createNamedParameter($principalUri)));
}
if ($excludeBirthday) {
$query->andWhere($query->expr()->neq('uri', $query->createNamedParameter(BirthdayService::BIRTHDAY_CALENDAR_URI)));
@ -292,13 +297,21 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
// Making fields a comma-delimited list
$query = $this->db->getQueryBuilder();
$query->select($fields)->from('calendars')
->where($query->expr()->eq('principaluri', $query->createNamedParameter($principalUri)))
$query->select($fields)
->from('calendars')
->orderBy('calendarorder', 'ASC');
$stmt = $query->execute();
if ($principalUri === '') {
$query->where($query->expr()->emptyString('principaluri'));
} else {
$query->where($query->expr()->eq('principaluri', $query->createNamedParameter($principalUri)));
}
$result = $query->execute();
$calendars = [];
while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
while ($row = $result->fetch()) {
$row['principaluri'] = (string) $row['principaluri'];
$components = [];
if ($row['components']) {
$components = explode(',',$row['components']);
@ -325,8 +338,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
$calendars[$calendar['id']] = $calendar;
}
}
$stmt->closeCursor();
$result->closeCursor();
// query for shared calendars
$principals = $this->principalBackend->getGroupMembership($principalUriOriginal, true);
@ -346,17 +358,19 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
$fields[] = 'a.transparent';
$fields[] = 's.access';
$query = $this->db->getQueryBuilder();
$result = $query->select($fields)
$query->select($fields)
->from('dav_shares', 's')
->join('s', 'calendars', 'a', $query->expr()->eq('s.resourceid', 'a.id'))
->where($query->expr()->in('s.principaluri', $query->createParameter('principaluri')))
->andWhere($query->expr()->eq('s.type', $query->createParameter('type')))
->setParameter('type', 'calendar')
->setParameter('principaluri', $principals, \Doctrine\DBAL\Connection::PARAM_STR_ARRAY)
->execute();
->setParameter('principaluri', $principals, \Doctrine\DBAL\Connection::PARAM_STR_ARRAY);
$result = $query->execute();
$readOnlyPropertyName = '{' . \OCA\DAV\DAV\Sharing\Plugin::NS_OWNCLOUD . '}read-only';
while ($row = $result->fetch()) {
$row['principaluri'] = (string) $row['principaluri'];
if ($row['principaluri'] === $principalUri) {
continue;
}
@ -427,6 +441,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
$stmt = $query->execute();
$calendars = [];
while ($row = $stmt->fetch(\PDO::FETCH_ASSOC)) {
$row['principaluri'] = (string) $row['principaluri'];
$components = [];
if ($row['components']) {
$components = explode(',',$row['components']);
@ -496,6 +511,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
->execute();
while ($row = $result->fetch()) {
$row['principaluri'] = (string) $row['principaluri'];
list(, $name) = Uri\split($row['principaluri']);
$row['displayname'] = $row['displayname'] . "($name)";
$components = [];
@ -562,6 +578,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
throw new NotFound('Node with name \'' . $uri . '\' could not be found');
}
$row['principaluri'] = (string) $row['principaluri'];
list(, $name) = Uri\split($row['principaluri']);
$row['displayname'] = $row['displayname'] . ' ' . "($name)";
$components = [];
@ -618,6 +635,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
return null;
}
$row['principaluri'] = (string) $row['principaluri'];
$components = [];
if ($row['components']) {
$components = explode(',',$row['components']);
@ -668,6 +686,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
return null;
}
$row['principaluri'] = (string) $row['principaluri'];
$components = [];
if ($row['components']) {
$components = explode(',',$row['components']);
@ -717,6 +736,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
return null;
}
$row['principaluri'] = (string) $row['principaluri'];
$subscription = [
'id' => $row['id'],
'uri' => $row['uri'],
@ -964,6 +984,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
'classification'=> (int)$row['classification']
];
}
$stmt->closeCursor();
return $result;
}
@ -994,6 +1015,7 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
->andWhere($query->expr()->eq('calendartype', $query->createNamedParameter($calendarType)));
$stmt = $query->execute();
$row = $stmt->fetch(\PDO::FETCH_ASSOC);
$stmt->closeCursor();
if (!$row) {
return null;

2
apps/dav/lib/Migration/Version1012Date20190808122342.php
View File

@ -69,7 +69,7 @@ class Version1012Date20190808122342 extends SimpleMigrationStep {
'length' => 11,
]);
$table->addColumn('is_recurring', Types::SMALLINT, [
'notnull' => true,
'notnull' => false,
'length' => 1,
]);
$table->addColumn('uid', Types::STRING, [

61
apps/dav/lib/Migration/Version1016Date20201109085907.php Normal file
View File

@ -0,0 +1,61 @@
<?php
declare(strict_types=1);
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @author Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OCA\DAV\Migration;
use Closure;
use OCP\DB\ISchemaWrapper;
use OCP\Migration\IOutput;
use OCP\Migration\SimpleMigrationStep;
class Version1016Date20201109085907 extends SimpleMigrationStep {
/**
* @param IOutput $output
* @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @param array $options
* @return null|ISchemaWrapper
*/
public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
/** @var ISchemaWrapper $schema */
$schema = $schemaClosure();
$result = $this->ensureColumnIsNullable($schema, 'calendar_reminders', 'is_recurring');
return $result ? $schema : null;
}
protected function ensureColumnIsNullable(ISchemaWrapper $schema, string $tableName, string $columnName): bool {
$table = $schema->getTable($tableName);
$column = $table->getColumn($columnName);
if ($column->getNotnull()) {
$column->setNotnull(false);
return true;
}
return false;
}
}

7
apps/dav/tests/unit/DAV/CustomPropertiesBackendTest.php
View File

@ -110,10 +110,13 @@ class CustomPropertiesBackendTest extends TestCase {
->where($query->expr()->eq('userid', $query->createNamedParameter($user)))
->where($query->expr()->eq('propertypath', $query->createNamedParameter($this->formatPath($path))));
$result = $query->execute();
$data = $result->fetchAll(\PDO::FETCH_KEY_PAIR);
$data = [];
while ($row = $result->fetch()) {
$data[$row['propertyname']] = $row['propertyvalue'];
}
$result->closeCursor();
return $data;
}

2
apps/encryption/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/federatedfilesharing/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

3
apps/federatedfilesharing/lib/Migration/Version1010Date20200630191755.php
View File

@ -52,7 +52,8 @@ class Version1010Date20200630191755 extends SimpleMigrationStep {
'notnull' => true,
'length' => 4,
]);
$table->addUniqueIndex(['share_id'], 'share_id_index');
$table->setPrimaryKey(['share_id'], 'federated_res_pk');
// $table->addUniqueIndex(['share_id'], 'share_id_index');
}
return $schema;
}

2
apps/federation/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/files/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/files_sharing/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/files_trashbin/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/files_versions/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/lookup_server_connector/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/oauth2/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

9
apps/oauth2/tests/Db/ClientMapperTest.php
View File

@ -40,6 +40,13 @@ class ClientMapperTest extends TestCase {
$this->clientMapper = new ClientMapper(\OC::$server->getDatabaseConnection());
}
protected function tearDown(): void {
$query = \OC::$server->getDatabaseConnection()->getQueryBuilder();
$query->delete('oauth2_clients')->execute();
parent::tearDown();
}
public function testGetByIdentifier() {
$client = new Client();
$client->setClientIdentifier('MyAwesomeClientIdentifier');
@ -51,7 +58,6 @@ class ClientMapperTest extends TestCase {
$this->assertEquals($client, $this->clientMapper->getByIdentifier('MyAwesomeClientIdentifier'));
}
public function testGetByIdentifierNotExisting() {
$this->expectException(\OCA\OAuth2\Exceptions\ClientNotFoundException::class);
@ -69,7 +75,6 @@ class ClientMapperTest extends TestCase {
$this->assertEquals($client, $this->clientMapper->getByUid($client->getId()));
}
public function testGetByUidNotExisting() {
$this->expectException(\OCA\OAuth2\Exceptions\ClientNotFoundException::class);

2
apps/provisioning_api/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/settings/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

11
apps/settings/lib/Controller/CheckSetupController.php
View File

@ -49,6 +49,7 @@ use OC\AppFramework\Http;
use OC\DB\Connection;
use OC\DB\MissingColumnInformation;
use OC\DB\MissingIndexInformation;
use OC\DB\MissingPrimaryKeyInformation;
use OC\DB\SchemaWrapper;
use OC\IntegrityCheck\Checker;
use OC\Lock\NoopLockingProvider;
@ -451,6 +452,15 @@ Raw output
return $indexInfo->getListOfMissingIndexes();
}
protected function hasMissingPrimaryKeys(): array {
$info = new MissingPrimaryKeyInformation();
// Dispatch event so apps can also hint for pending index updates if needed
$event = new GenericEvent($info);
$this->dispatcher->dispatch(IDBConnection::CHECK_MISSING_PRIMARY_KEYS_EVENT, $event);
return $info->getListOfMissingPrimaryKeys();
}
protected function hasMissingColumns(): array {
$indexInfo = new MissingColumnInformation();
// Dispatch event so apps can also hint for pending index updates if needed
@ -719,6 +729,7 @@ Raw output
'phpOpcacheDocumentation' => $this->urlGenerator->linkToDocs('admin-php-opcache'),
'isSettimelimitAvailable' => $this->isSettimelimitAvailable(),
'hasFreeTypeSupport' => $this->hasFreeTypeSupport(),
'missingPrimaryKeys' => $this->hasMissingPrimaryKeys(),
'missingIndexes' => $this->hasMissingIndexes(),
'missingColumns' => $this->hasMissingColumns(),
'isSqliteUsed' => $this->isSqliteUsed(),

5
apps/settings/tests/Controller/CheckSetupControllerTest.php
View File

@ -165,6 +165,7 @@ class CheckSetupControllerTest extends TestCase {
'isOpcacheProperlySetup',
'hasFreeTypeSupport',
'hasMissingIndexes',
'hasMissingPrimaryKeys',
'isSqliteUsed',
'isPHPMailerUsed',
'hasOpcacheLoaded',
@ -444,6 +445,9 @@ class CheckSetupControllerTest extends TestCase {
$this->checkSetupController
->method('hasMissingIndexes')
->willReturn([]);
$this->checkSetupController
->method('hasMissingPrimaryKeys')
->willReturn([]);
$this->checkSetupController
->method('isSqliteUsed')
->willReturn(false);
@ -587,6 +591,7 @@ class CheckSetupControllerTest extends TestCase {
'isSqliteUsed' => false,
'databaseConversionDocumentation' => 'http://docs.example.org/server/go.php?to=admin-db-conversion',
'missingIndexes' => [],
'missingPrimaryKeys' => [],
'missingColumns' => [],
'isPHPMailerUsed' => false,
'mailSettingsDocumentation' => 'https://server/index.php/settings/admin',

2
apps/sharebymail/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/systemtags/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/testing/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/twofactor_backupcodes/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/updatenotification/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/user_ldap/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/user_status/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

2
apps/workflowengine/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

216
build/psalm-baseline.xml
View File

@ -202,11 +202,10 @@
</NullableReturnStatement>
</file>
<file src="apps/dav/lib/CalDAV/CalDavBackend.php">
<ImplicitToStringCast occurrences="8">
<ImplicitToStringCast occurrences="7">
<code>$query-&gt;createParameter('principaluri')</code>
<code>$query-&gt;createNamedParameter(self::ACCESS_PUBLIC)</code>
<code>$query-&gt;createNamedParameter(self::ACCESS_PUBLIC)</code>
<code>$query-&gt;createNamedParameter($value)</code>
<code>$query-&gt;createParameter('uri')</code>
<code>$outerQuery-&gt;createFunction($innerQuery-&gt;getSQL())</code>
<code>$calendarObjectIdQuery-&gt;createNamedParameter($componentTypes, IQueryBuilder::PARAM_STR_ARRAY)</code>
@ -1204,18 +1203,7 @@
</TypeDoesNotContainNull>
</file>
<file src="apps/federatedfilesharing/lib/FederatedShareProvider.php">
<ImplicitToStringCast occurrences="17">
<code>$qb-&gt;createNamedParameter($shareType)</code>
<code>$qb-&gt;createNamedParameter($itemType)</code>
<code>$qb-&gt;createNamedParameter($itemSource)</code>
<code>$qb-&gt;createNamedParameter($itemSource)</code>
<code>$qb-&gt;createNamedParameter($shareWith)</code>
<code>$qb-&gt;createNamedParameter($uidOwner)</code>
<code>$qb-&gt;createNamedParameter($sharedBy)</code>
<code>$qb-&gt;createNamedParameter($permissions)</code>
<code>$qb-&gt;createNamedParameter($token)</code>
<code>$qb-&gt;createNamedParameter(time())</code>
<code>$qb-&gt;createNamedParameter('')</code>
<ImplicitToStringCast occurrences="6">
<code>$qb-&gt;createNamedParameter($this-&gt;supportedShareType, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$qb-&gt;createNamedParameter($this-&gt;supportedShareType, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$qb-&gt;createNamedParameter($this-&gt;supportedShareType, IQueryBuilder::PARAM_INT_ARRAY)</code>
@ -1703,17 +1691,8 @@
</TooManyArguments>
</file>
<file src="apps/files_external/lib/Service/DBConfigService.php">
<ImplicitToStringCast occurrences="10">
<ImplicitToStringCast occurrences="1">
<code>$builder-&gt;createNamedParameter($groupIds, IQueryBuilder::PARAM_STR_ARRAY)</code>
<code>$builder-&gt;createNamedParameter($mountId, IQueryBuilder::PARAM_INT)</code>
<code>$builder-&gt;createNamedParameter($key, IQueryBuilder::PARAM_STR)</code>
<code>$builder-&gt;createNamedParameter($value, IQueryBuilder::PARAM_STR)</code>
<code>$builder-&gt;createNamedParameter($mountId, IQueryBuilder::PARAM_INT)</code>
<code>$builder-&gt;createNamedParameter($key, IQueryBuilder::PARAM_STR)</code>
<code>$builder-&gt;createNamedParameter(json_encode($value), IQueryBuilder::PARAM_STR)</code>
<code>$builder-&gt;createNamedParameter($mountId)</code>
<code>$builder-&gt;createNamedParameter($type)</code>
<code>$builder-&gt;createNamedParameter($value)</code>
</ImplicitToStringCast>
<NullableReturnStatement occurrences="1">
<code>null</code>
@ -1755,10 +1734,9 @@
</UndefinedVariable>
</file>
<file src="apps/files_sharing/lib/AppInfo/Application.php">
<UndefinedClass occurrences="3">
<UndefinedClass occurrences="2">
<code>'ExternalMountProvider'</code>
<code>'OCP\Collaboration\Resources::loadAdditionalScripts'</code>
<code>'OCP\Share::postShare'</code>
<code>'\OCP\Collaboration\Resources::loadAdditionalScripts'</code>
</UndefinedClass>
</file>
<file src="apps/files_sharing/lib/Cache.php">
@ -2073,15 +2051,6 @@
<FalsableReturnStatement occurrences="1">
<code>false</code>
</FalsableReturnStatement>
<InvalidArgument occurrences="1">
<code>$ma</code>
</InvalidArgument>
<InvalidReturnStatement occurrences="1">
<code>$query-&gt;execute([$uid])</code>
</InvalidReturnStatement>
<InvalidReturnType occurrences="1">
<code>bool</code>
</InvalidReturnType>
<InvalidScalarArgument occurrences="2">
<code>$timestamp</code>
<code>$timestamp</code>
@ -2135,11 +2104,6 @@
<code>getURLGenerator</code>
</TooManyArguments>
</file>
<file src="apps/lookup_server_connector/lib/AppInfo/Application.php">
<UndefinedClass occurrences="1">
<code>'OC\AccountManager::userUpdated'</code>
</UndefinedClass>
</file>
<file src="apps/lookup_server_connector/lib/BackgroundJobs/RetryJob.php">
<InvalidArrayOffset occurrences="10">
<code>$publicData[IAccountManager::PROPERTY_DISPLAYNAME]['value']</code>
@ -2217,8 +2181,9 @@
</UndefinedInterfaceMethod>
</file>
<file src="apps/settings/lib/Controller/CheckSetupController.php">
<InvalidArgument occurrences="2">
<InvalidArgument occurrences="3">
<code>IDBConnection::CHECK_MISSING_INDEXES_EVENT</code>
<code>IDBConnection::CHECK_MISSING_PRIMARY_KEYS_EVENT</code>
<code>IDBConnection::CHECK_MISSING_COLUMNS_EVENT</code>
</InvalidArgument>
<InvalidOperand occurrences="1">
@ -2235,7 +2200,8 @@
<code>0</code>
<code>$lastCronRun</code>
</InvalidScalarArgument>
<TooManyArguments occurrences="2">
<TooManyArguments occurrences="3">
<code>dispatch</code>
<code>dispatch</code>
<code>dispatch</code>
</TooManyArguments>
@ -2308,21 +2274,7 @@
</NullableReturnStatement>
</file>
<file src="apps/sharebymail/lib/ShareByMailProvider.php">
<ImplicitToStringCast occurrences="15">
<code>$qb-&gt;createNamedParameter(IShare::TYPE_EMAIL)</code>
<code>$qb-&gt;createNamedParameter($itemType)</code>
<code>$qb-&gt;createNamedParameter($itemSource)</code>
<code>$qb-&gt;createNamedParameter($itemSource)</code>
<code>$qb-&gt;createNamedParameter($shareWith)</code>
<code>$qb-&gt;createNamedParameter($uidOwner)</code>
<code>$qb-&gt;createNamedParameter($sharedBy)</code>
<code>$qb-&gt;createNamedParameter($permissions)</code>
<code>$qb-&gt;createNamedParameter($token)</code>
<code>$qb-&gt;createNamedParameter($password)</code>
<code>$qb-&gt;createNamedParameter($sendPasswordByTalk, IQueryBuilder::PARAM_BOOL)</code>
<code>$qb-&gt;createNamedParameter(time())</code>
<code>$qb-&gt;createNamedParameter((int)$hideDownload, IQueryBuilder::PARAM_INT)</code>
<code>$qb-&gt;createNamedParameter('')</code>
<ImplicitToStringCast occurrences="1">
<code>$qb-&gt;createNamedParameter($ids, IQueryBuilder::PARAM_INT_ARRAY)</code>
</ImplicitToStringCast>
<InvalidReturnStatement occurrences="1">
@ -2485,19 +2437,11 @@
<InvalidReturnType occurrences="1">
<code>string[]</code>
</InvalidReturnType>
<InvalidScalarArgument occurrences="13">
<InvalidScalarArgument occurrences="4">
<code>[$attr =&gt; $result['values']]</code>
<code>$key</code>
<code>$key</code>
<code>$e-&gt;getCode()</code>
<code>$nameAttribute</code>
<code>$filter</code>
<code>$this-&gt;connection-&gt;ldapLoginFilter</code>
<code>$this-&gt;connection-&gt;ldapLoginFilter</code>
<code>$this-&gt;connection-&gt;ldapUserDisplayName</code>
<code>$this-&gt;connection-&gt;ldapUserDisplayName</code>
<code>$this-&gt;connection-&gt;ldapGroupDisplayName</code>
<code>$filter</code>
</InvalidScalarArgument>
<NullArgument occurrences="1">
<code>$cookie</code>
@ -2568,27 +2512,9 @@
<InvalidReturnType occurrences="1">
<code>bool</code>
</InvalidReturnType>
<InvalidScalarArgument occurrences="26">
<code>$gAssoc</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapLoginFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapDynamicGroupMemberURL</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGroupFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGroupMemberAssocAttr</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGidNumber</code>
<InvalidScalarArgument occurrences="2">
<code>$groupID</code>
<code>$groupID</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapDynamicGroupMemberURL</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGroupFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapUserDisplayName</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGroupMemberAssocAttr</code>
<code>[strtolower($this-&gt;access-&gt;connection-&gt;ldapGroupMemberAssocAttr), $this-&gt;access-&gt;connection-&gt;ldapGroupDisplayName, 'dn']</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapLoginFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapUserDisplayName</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapLoginFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapUserDisplayName</code>
<code>[$this-&gt;access-&gt;connection-&gt;ldapGroupDisplayName, 'dn']</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGroupFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapGroupDisplayName</code>
</InvalidScalarArgument>
<RedundantCondition occurrences="4">
<code>!is_array($members) || count($members) === 0</code>
@ -2724,10 +2650,6 @@
<InvalidDocblock occurrences="1">
<code>public function setLdapAccess(Access $access) {</code>
</InvalidDocblock>
<InvalidScalarArgument occurrences="2">
<code>$homeRule</code>
<code>$homeRule</code>
</InvalidScalarArgument>
</file>
<file src="apps/user_ldap/lib/User/User.php">
<FalsableReturnStatement occurrences="1">
@ -2740,34 +2662,18 @@
<code>null</code>
<code>null</code>
</InvalidReturnType>
<InvalidScalarArgument occurrences="15">
<code>$this-&gt;connection-&gt;ldapQuotaAttribute</code>
<code>$this-&gt;connection-&gt;ldapUserDisplayName</code>
<code>$this-&gt;connection-&gt;ldapUserDisplayName2</code>
<code>$this-&gt;connection-&gt;ldapEmailAttribute</code>
<code>$this-&gt;connection-&gt;homeFolderNamingRule</code>
<code>$this-&gt;connection-&gt;homeFolderNamingRule</code>
<InvalidScalarArgument occurrences="3">
<code>$this-&gt;getHomePath($ldapEntry[$attr][0])</code>
<code>$this-&gt;connection-&gt;ldapExtStorageHomeAttribute</code>
<code>$this-&gt;access-&gt;connection-&gt;homeFolderNamingRule</code>
<code>$this-&gt;access-&gt;connection-&gt;homeFolderNamingRule</code>
<code>true</code>
<code>1</code>
<code>$emailAttribute</code>
<code>$quotaAttribute</code>
<code>$this-&gt;connection-&gt;ldapExtStorageHomeAttribute</code>
</InvalidScalarArgument>
</file>
<file src="apps/user_ldap/lib/User_LDAP.php">
<ImplementedReturnTypeMismatch occurrences="1">
<code>string|false</code>
</ImplementedReturnTypeMismatch>
<InvalidScalarArgument occurrences="8">
<code>$this-&gt;access-&gt;connection-&gt;ldapUserFilter</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapUserFilter</code>
<InvalidScalarArgument occurrences="3">
<code>$path</code>
<code>$additionalAttribute</code>
<code>$this-&gt;access-&gt;connection-&gt;ldapUserDisplayName</code>
<code>$limit</code>
<code>$offset</code>
</InvalidScalarArgument>
@ -2979,7 +2885,7 @@
</file>
<file src="core/Command/Broadcast/Test.php">
<UndefinedClass occurrences="1">
<code>broadcasttest</code>
<code>'broadcasttest'</code>
</UndefinedClass>
</file>
<file src="core/Command/Config/App/GetConfig.php">
@ -3028,10 +2934,15 @@
<code>dispatch</code>
</TooManyArguments>
</file>
<file src="core/Command/Db/AddMissingPrimaryKeys.php">
<InvalidArgument occurrences="1">
<code>IDBConnection::ADD_MISSING_PRIMARY_KEYS_EVENT</code>
</InvalidArgument>
<TooManyArguments occurrences="1">
<code>dispatch</code>
</TooManyArguments>
</file>
<file src="core/Command/Db/ConvertType.php">
<ImplicitToStringCast occurrences="1">
<code>$insertQuery-&gt;createParameter($key)</code>
</ImplicitToStringCast>
<InternalMethod occurrences="1">
<code>setFilterSchemaAssetsExpression</code>
</InternalMethod>
@ -3213,7 +3124,6 @@
<InvalidScope occurrences="2">
<code>$this</code>
<code>$this</code>
<code>$this</code>
</InvalidScope>
</file>
<file src="core/templates/layout.public.php">
@ -3498,11 +3408,6 @@
<code>$default</code>
</MoreSpecificImplementedParamType>
</file>
<file src="lib/private/AppFramework/Services/InitialState.php">
<ImplementedParamTypeMismatch occurrences="1">
<code>$closure</code>
</ImplementedParamTypeMismatch>
</file>
<file src="lib/private/Archive/TAR.php">
<FalsableReturnStatement occurrences="1">
<code>false</code>
@ -3793,9 +3698,6 @@
</NullableReturnStatement>
</file>
<file src="lib/private/DB/Adapter.php">
<ImplicitToStringCast occurrences="1">
<code>$builder-&gt;createNamedParameter($value)</code>
</ImplicitToStringCast>
<InvalidReturnStatement occurrences="1">
<code>$builder-&gt;execute()</code>
</InvalidReturnStatement>
@ -3807,9 +3709,6 @@
<FalsableReturnStatement occurrences="1">
<code>$this-&gt;conn-&gt;fetchColumn('SELECT lastval()')</code>
</FalsableReturnStatement>
<ImplicitToStringCast occurrences="1">
<code>$builder-&gt;createNamedParameter($value)</code>
</ImplicitToStringCast>
</file>
<file src="lib/private/DB/Connection.php">
<ImplementedReturnTypeMismatch occurrences="1">
@ -4034,8 +3933,7 @@
</TooManyArguments>
</file>
<file src="lib/private/Files/Cache/Cache.php">
<ImplicitToStringCast occurrences="3">
<code>$builder-&gt;createNamedParameter($value)</code>
<ImplicitToStringCast occurrences="2">
<code>$fun-&gt;md5($newPathFunction)</code>
<code>$newPathFunction</code>
</ImplicitToStringCast>
@ -4091,9 +3989,6 @@
<code>$filesData</code>
<code>$data</code>
</UndefinedInterfaceMethod>
<UndefinedMagicMethod occurrences="1">
<code>closeCursor</code>
</UndefinedMagicMethod>
</file>
<file src="lib/private/Files/Cache/LocalRootScanner.php">
<InvalidNullableReturnType occurrences="2">
@ -4108,7 +4003,7 @@
<file src="lib/private/Files/Cache/Propagator.php">
<ImplicitToStringCast occurrences="4">
<code>$builder-&gt;func()-&gt;greatest('mtime', $builder-&gt;createNamedParameter((int)$time, IQueryBuilder::PARAM_INT))</code>
<code>$query-&gt;createFunction('GREATEST(' . $query-&gt;getColumnName('mtime') . ', ' . $query-&gt;createParameter('time') . ')')</code>
<code>$query-&gt;func()-&gt;greatest('mtime', $query-&gt;createParameter('time'))</code>
<code>$sizeQuery-&gt;func()-&gt;add('size', $sizeQuery-&gt;createParameter('size'))</code>
</ImplicitToStringCast>
</file>
@ -4902,7 +4797,9 @@
<InvalidScalarArgument occurrences="1">
<code>$position</code>
</InvalidScalarArgument>
<UndefinedInterfaceMethod occurrences="1"/>
<UndefinedInterfaceMethod occurrences="1">
<code>$cacheEntry</code>
</UndefinedInterfaceMethod>
</file>
<file src="lib/private/Files/Stream/SeekableHttpStream.php">
<FalsableReturnStatement occurrences="3">
@ -5004,12 +4901,6 @@
</TypeDoesNotContainNull>
</file>
<file src="lib/private/Group/Database.php">
<ImplicitToStringCast occurrences="4">
<code>$builder-&gt;createNamedParameter($gid)</code>
<code>$builder-&gt;createNamedParameter($gid)</code>
<code>$qb-&gt;createNamedParameter($uid)</code>
<code>$qb-&gt;createNamedParameter($gid)</code>
</ImplicitToStringCast>
<InvalidArgument occurrences="1"/>
<InvalidArrayOffset occurrences="1">
<code>$this-&gt;groupCache[$gid]['displayname']</code>
@ -5427,11 +5318,6 @@
<code>$permsFunc</code>
</ImplicitToStringCast>
</file>
<file src="lib/private/Repair/RepairMimeTypes.php">
<InvalidOperand occurrences="1">
<code>\OC_DB::executeAudited(self::updateByNameStmt(), [$mimetypeId, $this-&gt;folderMimeTypeId, $mimetypeId, '%.' . $extension])</code>
</InvalidOperand>
</file>
<file src="lib/private/Route/Router.php">
<InvalidClass occurrences="1">
<code>\OC_APP</code>
@ -5460,9 +5346,6 @@
</InvalidPropertyAssignmentValue>
</file>
<file src="lib/private/Security/Bruteforce/Throttler.php">
<ImplicitToStringCast occurrences="1">
<code>$qb-&gt;createNamedParameter($value)</code>
</ImplicitToStringCast>
<NullArgument occurrences="1">
<code>null</code>
</NullArgument>
@ -5587,9 +5470,8 @@
</file>
<file src="lib/private/Share/Helper.php">
<ImplicitToStringCast occurrences="3">
<code>$query->createNamedParameter($parents, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$query->createNamedParameter($changeParent, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$query->createNamedParameter($ids, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$query-&gt;createNamedParameter($changeParent, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$query-&gt;createNamedParameter($ids, IQueryBuilder::PARAM_INT_ARRAY)</code>
</ImplicitToStringCast>
</file>
<file src="lib/private/Share/Share.php">
@ -5613,27 +5495,7 @@
</UndefinedInterfaceMethod>
</file>
<file src="lib/private/Share20/DefaultShareProvider.php">
<ImplicitToStringCast occurrences="27">
<code>$qb-&gt;createNamedParameter($share-&gt;getShareType())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getSharedWith())</code>
<code>$qb-&gt;createNamedParameter(IShare::STATUS_PENDING)</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getExpirationDate(), 'datetime')</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getSharedWith())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getExpirationDate(), 'datetime')</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getLabel())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getToken())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getPassword())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getSendPasswordByTalk(), IQueryBuilder::PARAM_BOOL)</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getExpirationDate(), 'datetime')</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getParent())</code>
<code>$qb-&gt;createParameter('itemType')</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getNode()-&gt;getId())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getNode()-&gt;getId())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getPermissions())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getSharedBy())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getShareOwner())</code>
<code>$qb-&gt;createNamedParameter($share-&gt;getTarget())</code>
<code>$qb-&gt;createNamedParameter(time())</code>
<ImplicitToStringCast occurrences="7">
<code>$qb-&gt;createNamedParameter($ids, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$qb-&gt;createNamedParameter($chunk, IQueryBuilder::PARAM_INT_ARRAY)</code>
<code>$qb-&gt;createNamedParameter($chunk, IQueryBuilder::PARAM_INT_ARRAY)</code>
@ -5847,8 +5709,7 @@
<code>string</code>
<code>string</code>
</InvalidParamDefault>
<InvalidScalarArgument occurrences="4">
<code>\OC_User::getUser()</code>
<InvalidScalarArgument occurrences="3">
<code>\OC_User::getUser()</code>
<code>$appName</code>
<code>$appName</code>
@ -6211,11 +6072,6 @@
<code>$column</code>
</NullableReturnStatement>
</file>
<file src="lib/public/AppFramework/Db/QBMapper.php">
<ImplicitToStringCast occurrences="1">
<code>$qb-&gt;createNamedParameter($value, $type)</code>
</ImplicitToStringCast>
</file>
<file src="lib/public/AppFramework/Http/DataResponse.php">
<InvalidReturnStatement occurrences="1">
<code>$this-&gt;data</code>
@ -6247,14 +6103,6 @@
<code>$this-&gt;resources</code>
</InvalidPropertyAssignmentValue>
</file>
<file src="lib/public/AppFramework/Services/IInitialState.php">
<MismatchingDocblockParamType occurrences="1">
<code>Closure</code>
</MismatchingDocblockParamType>
<UndefinedDocblockClass occurrences="1">
<code>Closure</code>
</UndefinedDocblockClass>
</file>
<file src="lib/public/BackgroundJob/TimedJob.php">
<MoreSpecificImplementedParamType occurrences="1">
<code>$jobList</code>

58
core/Application.php
View File

@ -42,6 +42,7 @@ use OC\Authentication\Notifications\Notifier as AuthenticationNotifier;
use OC\Core\Notification\RemoveLinkSharesNotifier;
use OC\DB\MissingColumnInformation;
use OC\DB\MissingIndexInformation;
use OC\DB\MissingPrimaryKeyInformation;
use OC\DB\SchemaWrapper;
use OCP\AppFramework\App;
use OCP\EventDispatcher\IEventDispatcher;
@ -177,6 +178,63 @@ class Application extends App {
}
);
$oldEventDispatcher->addListener(IDBConnection::CHECK_MISSING_PRIMARY_KEYS_EVENT,
function (GenericEvent $event) use ($container) {
/** @var MissingPrimaryKeyInformation $subject */
$subject = $event->getSubject();
$schema = new SchemaWrapper($container->query(IDBConnection::class));
if ($schema->hasTable('federated_reshares')) {
$table = $schema->getTable('federated_reshares');
if (!$table->hasPrimaryKey()) {
$subject->addHintForMissingSubject($table->getName());
}
}
if ($schema->hasTable('systemtag_object_mapping')) {
$table = $schema->getTable('systemtag_object_mapping');
if (!$table->hasPrimaryKey()) {
$subject->addHintForMissingSubject($table->getName());
}
}
if ($schema->hasTable('comments_read_markers')) {
$table = $schema->getTable('comments_read_markers');
if (!$table->hasPrimaryKey()) {
$subject->addHintForMissingSubject($table->getName());
}
}
if ($schema->hasTable('collres_resources')) {
$table = $schema->getTable('collres_resources');
if (!$table->hasPrimaryKey()) {
$subject->addHintForMissingSubject($table->getName());
}
}
if ($schema->hasTable('collres_accesscache')) {
$table = $schema->getTable('collres_accesscache');
if (!$table->hasPrimaryKey()) {
$subject->addHintForMissingSubject($table->getName());
}
}
if ($schema->hasTable('filecache_extended')) {
$table = $schema->getTable('filecache_extended');
if (!$table->hasPrimaryKey()) {
$subject->addHintForMissingSubject($table->getName());
}
}
}
);
$oldEventDispatcher->addListener(IDBConnection::CHECK_MISSING_COLUMNS_EVENT,
function (GenericEvent $event) use ($container) {
/** @var MissingColumnInformation $subject */

181
core/Command/Db/AddMissingPrimaryKeys.php Normal file
View File

@ -0,0 +1,181 @@
<?php
declare(strict_types=1);
/**
* @copyright Copyright (c) 2017 Bjoern Schiessle <bjoern@schiessle.org>
*
* @author Bjoern Schiessle <bjoern@schiessle.org>
* @author Joas Schilling <coding@schilljs.com>
* @author Mario Danic <mario@lovelyhq.com>
* @author Morris Jobke <hey@morrisjobke.de>
* @author Robin Appelman <robin@icewind.nl>
* @author Roeland Jago Douma <roeland@famdouma.nl>
* @author Thomas Citharel <nextcloud@tcit.fr>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OC\Core\Command\Db;
use OC\DB\SchemaWrapper;
use OCP\IDBConnection;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Symfony\Component\EventDispatcher\GenericEvent;
/**
* Class AddMissingPrimaryKeys
*
* if you added primary keys to the database, this is the right place to add
* your update routine for existing instances
*
* @package OC\Core\Command\Db
*/
class AddMissingPrimaryKeys extends Command {
/** @var IDBConnection */
private $connection;
/** @var EventDispatcherInterface */
private $dispatcher;
public function __construct(IDBConnection $connection, EventDispatcherInterface $dispatcher) {
parent::__construct();
$this->connection = $connection;
$this->dispatcher = $dispatcher;
}
protected function configure() {
$this
->setName('db:add-missing-primary-keys')
->setDescription('Add missing primary keys to the database tables');
}
protected function execute(InputInterface $input, OutputInterface $output): int {
$this->addCorePrimaryKeys($output);
// Dispatch event so apps can also update indexes if needed
$event = new GenericEvent($output);
$this->dispatcher->dispatch(IDBConnection::ADD_MISSING_PRIMARY_KEYS_EVENT, $event);
return 0;
}
/**
* add missing indices to the share table
*
* @param OutputInterface $output
* @throws \Doctrine\DBAL\Schema\SchemaException
*/
private function addCorePrimaryKeys(OutputInterface $output) {
$output->writeln('<info>Check primary keys.</info>');
$schema = new SchemaWrapper($this->connection);
$updated = false;
if ($schema->hasTable('federated_reshares')) {
$table = $schema->getTable('federated_reshares');
if (!$table->hasPrimaryKey()) {
$output->writeln('<info>Adding primary key to the federated_reshares table, this can take some time...</info>');
$table->setPrimaryKey(['share_id'], 'federated_res_pk');
if ($table->hasIndex('share_id_index')) {
$table->dropIndex('share_id_index');
}
$this->connection->migrateToSchema($schema->getWrappedSchema());
$updated = true;
$output->writeln('<info>federated_reshares table updated successfully.</info>');
}
}
if ($schema->hasTable('systemtag_object_mapping')) {
$table = $schema->getTable('systemtag_object_mapping');
if (!$table->hasPrimaryKey()) {
$output->writeln('<info>Adding primary key to the systemtag_object_mapping table, this can take some time...</info>');
$table->setPrimaryKey(['objecttype', 'objectid', 'systemtagid'], 'som_pk');
if ($table->hasIndex('mapping')) {
$table->dropIndex('mapping');
}
$this->connection->migrateToSchema($schema->getWrappedSchema());
$updated = true;
$output->writeln('<info>systemtag_object_mapping table updated successfully.</info>');
}
}
if ($schema->hasTable('comments_read_markers')) {
$table = $schema->getTable('comments_read_markers');
if (!$table->hasPrimaryKey()) {
$output->writeln('<info>Adding primary key to the comments_read_markers table, this can take some time...</info>');
$table->setPrimaryKey(['user_id', 'object_type', 'object_id'], 'crm_pk');
if ($table->hasIndex('comments_marker_index')) {
$table->dropIndex('comments_marker_index');
}
$this->connection->migrateToSchema($schema->getWrappedSchema());
$updated = true;
$output->writeln('<info>comments_read_markers table updated successfully.</info>');
}
}
if ($schema->hasTable('collres_resources')) {
$table = $schema->getTable('collres_resources');
if (!$table->hasPrimaryKey()) {
$output->writeln('<info>Adding primary key to the collres_resources table, this can take some time...</info>');
$table->setPrimaryKey(['collection_id', 'resource_type', 'resource_id'], 'crr_pk');
if ($table->hasIndex('collres_unique_res')) {
$table->dropIndex('collres_unique_res');
}
$this->connection->migrateToSchema($schema->getWrappedSchema());
$updated = true;
$output->writeln('<info>collres_resources table updated successfully.</info>');
}
}
if ($schema->hasTable('collres_accesscache')) {
$table = $schema->getTable('collres_accesscache');
if (!$table->hasPrimaryKey()) {
$output->writeln('<info>Adding primary key to the collres_accesscache table, this can take some time...</info>');
$table->setPrimaryKey(['user_id', 'collection_id', 'resource_type', 'resource_id'], 'cra_pk');
if ($table->hasIndex('collres_unique_user')) {
$table->dropIndex('collres_unique_user');
}
$this->connection->migrateToSchema($schema->getWrappedSchema());
$updated = true;
$output->writeln('<info>collres_accesscache table updated successfully.</info>');
}
}
if ($schema->hasTable('filecache_extended')) {
$table = $schema->getTable('filecache_extended');
if (!$table->hasPrimaryKey()) {
$output->writeln('<info>Adding primary key to the filecache_extended table, this can take some time...</info>');
$table->setPrimaryKey(['fileid'], 'fce_pk');
if ($table->hasIndex('fce_fileid_idx')) {
$table->dropIndex('fce_fileid_idx');
}
$this->connection->migrateToSchema($schema->getWrappedSchema());
$updated = true;
$output->writeln('<info>filecache_extended table updated successfully.</info>');
}
}
if (!$updated) {
$output->writeln('<info>Done.</info>');
}
}
}

38
core/Migrations/Version13000Date20170718121200.php
View File

@ -672,7 +672,8 @@ class Version13000Date20170718121200 extends SimpleMigrationStep {
'default' => 0,
'unsigned' => true,
]);
$table->addUniqueIndex(['objecttype', 'objectid', 'systemtagid'], 'mapping');
$table->setPrimaryKey(['objecttype', 'objectid', 'systemtagid'], 'som_pk');
// $table->addUniqueIndex(['objecttype', 'objectid', 'systemtagid'], 'mapping');
}
if (!$schema->hasTable('systemtag_group')) {
@ -807,25 +808,26 @@ class Version13000Date20170718121200 extends SimpleMigrationStep {
'default' => '',
]);
$table->addIndex(['object_type', 'object_id'], 'comments_marker_object_index');
$table->addUniqueIndex(['user_id', 'object_type', 'object_id'], 'comments_marker_index');
$table->setPrimaryKey(['user_id', 'object_type', 'object_id'], 'crm_pk');
// $table->addUniqueIndex(['user_id', 'object_type', 'object_id'], 'comments_marker_index');
}
if (!$schema->hasTable('credentials')) {
$table = $schema->createTable('credentials');
$table->addColumn('user', 'string', [
'notnull' => true,
'length' => 64,
]);
$table->addColumn('identifier', 'string', [
'notnull' => true,
'length' => 64,
]);
$table->addColumn('credentials', 'text', [
'notnull' => false,
]);
$table->setPrimaryKey(['user', 'identifier']);
$table->addIndex(['user'], 'credentials_user');
}
// if (!$schema->hasTable('credentials')) {
// $table = $schema->createTable('credentials');
// $table->addColumn('user', 'string', [
// 'notnull' => false,
// 'length' => 64,
// ]);
// $table->addColumn('identifier', 'string', [
// 'notnull' => true,
// 'length' => 64,
// ]);
// $table->addColumn('credentials', 'text', [
// 'notnull' => false,
// ]);
// $table->setPrimaryKey(['user', 'identifier']);
// $table->addIndex(['user'], 'credentials_user');
// }
if (!$schema->hasTable('admin_sections')) {
$table = $schema->createTable('admin_sections');

1
core/Migrations/Version14000Date20180710092004.php
View File

@ -43,6 +43,7 @@ class Version14000Date20180710092004 extends SimpleMigrationStep {
if (!$table->hasColumn('password_by_talk')) {
$table->addColumn('password_by_talk', Types::BOOLEAN, [
'default' => 0,
'notnull' => false,
]);
}

4
core/Migrations/Version15000Date20180926101451.php
View File

@ -45,8 +45,8 @@ class Version15000Date20180926101451 extends SimpleMigrationStep {
$table = $schema->getTable('authtoken');
$table->addColumn('password_invalid','boolean', [
'notnull' => true,
'default' => false,
'default' => 0,
'notnull' => false,
]);
return $schema;

2
core/Migrations/Version15000Date20181015062942.php
View File

@ -45,7 +45,7 @@ class Version15000Date20181015062942 extends SimpleMigrationStep {
$table = $schema->getTable('share');
$table->addColumn('hide_download', 'smallint', [
'notnull' => true,
'notnull' => false,
'length' => 1,
'default' => 0,
]);

8
core/Migrations/Version16000Date20190207141427.php
View File

@ -77,7 +77,8 @@ class Version16000Date20190207141427 extends SimpleMigrationStep {
'length' => 64,
]);
$table->addUniqueIndex(['collection_id', 'resource_type', 'resource_id'], 'collres_unique_res');
$table->setPrimaryKey(['collection_id', 'resource_type', 'resource_id'], 'crr_pk');
// $table->addUniqueIndex(['collection_id', 'resource_type', 'resource_id'], 'collres_unique_res');
}
if (!$schema->hasTable('collres_accesscache')) {
@ -102,11 +103,12 @@ class Version16000Date20190207141427 extends SimpleMigrationStep {
'default' => '',
]);
$table->addColumn('access', Types::SMALLINT, [
'notnull' => true,
'notnull' => false,
'default' => 0,
]);
$table->addUniqueIndex(['user_id', 'collection_id', 'resource_type', 'resource_id'], 'collres_unique_user');
$table->setPrimaryKey(['user_id', 'collection_id', 'resource_type', 'resource_id'], 'cra_pk');
// $table->addUniqueIndex(['user_id', 'collection_id', 'resource_type', 'resource_id'], 'collres_unique_user');
$table->addIndex(['user_id', 'resource_type', 'resource_id'], 'collres_user_res');
$table->addIndex(['user_id', 'collection_id'], 'collres_user_coll');
}

2
core/Migrations/Version16000Date20190428150708.php
View File

@ -49,7 +49,7 @@ class Version16000Date20190428150708 extends SimpleMigrationStep {
if ($schema->hasTable('collres_accesscache')) {
$table = $schema->getTable('collres_accesscache');
$table->addColumn('access', Types::BOOLEAN, [
'notnull' => true,
'notnull' => false,
'default' => false
]);
}

3
core/Migrations/Version17000Date20190514105811.php
View File

@ -67,7 +67,8 @@ class Version17000Date20190514105811 extends SimpleMigrationStep {
'length' => 20,
'default' => 0,
]);
$table->addUniqueIndex(['fileid'], 'fce_fileid_idx');
$table->setPrimaryKey(['fileid'], 'fce_pk');
// $table->addUniqueIndex(['fileid'], 'fce_fileid_idx');
$table->addIndex(['creation_time'], 'fce_ctime_idx');
$table->addIndex(['upload_time'], 'fce_utime_idx');
}

1
core/Migrations/Version18000Date20191204114856.php
View File

@ -58,7 +58,6 @@ class Version18000Date20191204114856 extends SimpleMigrationStep {
'length' => 4000,
]);
return $schema;
}
}

65
core/Migrations/Version20000Date20201109081915.php Normal file
View File

@ -0,0 +1,65 @@
<?php
declare(strict_types=1);
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @author Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OC\Core\Migrations;
use Closure;
use OCP\DB\ISchemaWrapper;
use OCP\Migration\IOutput;
use OCP\Migration\SimpleMigrationStep;
class Version20000Date20201109081915 extends SimpleMigrationStep {
/**
* @param IOutput $output
* @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @param array $options
* @return null|ISchemaWrapper
*/
public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
/** @var ISchemaWrapper $schema */
$schema = $schemaClosure();
$result = $this->ensureColumnIsNullable($schema, 'share', 'password_by_talk');
$result = $this->ensureColumnIsNullable($schema, 'share', 'hide_download') || $result;
// $result = $this->ensureColumnIsNullable($schema, 'credentials', 'user') || $result;
$result = $this->ensureColumnIsNullable($schema, 'authtoken', 'password_invalid') || $result;
$result = $this->ensureColumnIsNullable($schema, 'collres_accesscache', 'access') || $result;
return $result ? $schema : null;
}
protected function ensureColumnIsNullable(ISchemaWrapper $schema, string $tableName, string $columnName): bool {
$table = $schema->getTable($tableName);
$column = $table->getColumn($columnName);
if ($column->getNotnull()) {
$column->setNotnull(false);
return true;
}
return false;
}
}

110
core/Migrations/Version20000Date20201109081918.php Normal file
View File

@ -0,0 +1,110 @@
<?php
declare(strict_types=1);
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @author Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OC\Core\Migrations;
use Closure;
use Doctrine\DBAL\Types\Type;
use OCP\DB\ISchemaWrapper;
use OCP\IDBConnection;
use OCP\Migration\IOutput;
use OCP\Migration\SimpleMigrationStep;
class Version20000Date20201109081918 extends SimpleMigrationStep {
/** @var IDBConnection */
protected $connection;
public function __construct(IDBConnection $connection) {
$this->connection = $connection;
}
/**
* @param IOutput $output
* @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @param array $options
* @return null|ISchemaWrapper
*/
public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
/** @var ISchemaWrapper $schema */
$schema = $schemaClosure();
if (!$schema->hasTable('storages_credentials')) {
$table = $schema->createTable('storages_credentials');
$table->addColumn('id', Type::BIGINT, [
'autoincrement' => true,
'notnull' => true,
'length' => 64,
]);
$table->addColumn('user', Type::STRING, [
'notnull' => false,
'length' => 64,
]);
$table->addColumn('identifier', Type::STRING, [
'notnull' => true,
'length' => 64,
]);
$table->addColumn('credentials', Type::TEXT, [
'notnull' => false,
]);
$table->setPrimaryKey(['id']);
$table->addUniqueIndex(['user', 'identifier'], 'stocred_ui');
$table->addIndex(['user'], 'stocred_user');
}
return $schema;
}
/**
* {@inheritDoc}
*
* @since 13.0.0
*/
public function postSchemaChange(IOutput $output, \Closure $schemaClosure, array $options): void {
if (!$this->connection->tableExists('credentials')) {
return;
}
$query = $this->connection->getQueryBuilder();
$query->select('*')
->from('credentials');
$insert = $this->connection->getQueryBuilder();
$insert->insert('storages_credentials')
->setValue('user', $insert->createParameter('user'))
->setValue('identifier', $insert->createParameter('identifier'))
->setValue('credentials', $insert->createParameter('credentials'));
$result = $query->execute();
while ($row = $result->fetch()) {
$insert->setParameter('user', (string) $row['user'])
->setParameter('identifier', (string) $row['identifier'])
->setParameter('credentials', (string) $row['credentials']);
$insert->execute();
}
$result->closeCursor();
}
}

53
core/Migrations/Version20000Date20201109081919.php Normal file
View File

@ -0,0 +1,53 @@
<?php
declare(strict_types=1);
/**
* @copyright Copyright (c) 2020 Joas Schilling <coding@schilljs.com>
*
* @author Joas Schilling <coding@schilljs.com>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OC\Core\Migrations;
use Closure;
use OCP\DB\ISchemaWrapper;
use OCP\Migration\IOutput;
use OCP\Migration\SimpleMigrationStep;
class Version20000Date20201109081919 extends SimpleMigrationStep {
/**
* @param IOutput $output
* @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
* @param array $options
* @return null|ISchemaWrapper
*/
public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
/** @var ISchemaWrapper $schema */
$schema = $schemaClosure();
if ($schema->hasTable('credentials')) {
$schema->dropTable('credentials');
return $schema;
}
return null;
}
}

15
core/js/setupchecks.js
View File

@ -357,6 +357,21 @@
type: OC.SetupChecks.MESSAGE_TYPE_INFO
})
}
if (data.missingPrimaryKeys.length > 0) {
var listOfMissingPrimaryKeys = "";
data.missingPrimaryKeys.forEach(function(element){
listOfMissingPrimaryKeys += "<li>";
listOfMissingPrimaryKeys += t('core', 'Missing primary key on table "{tableName}".', element);
listOfMissingPrimaryKeys += "</li>";
});
messages.push({
msg: t(
'core',
'The database is missing some primary keys. Due to the fact that adding primary keys on big tables could take some time they were not added automatically. By running "occ db:add-missing-primary-keys" those missing primary keys could be added manually while the instance keeps running.'
) + "<ul>" + listOfMissingPrimaryKeys + "</ul>",
type: OC.SetupChecks.MESSAGE_TYPE_INFO
})
}
if (data.missingColumns.length > 0) {
var listOfMissingColumns = "";
data.missingColumns.forEach(function(element){

16
core/js/tests/specs/setupchecksSpec.js
View File

@ -240,6 +240,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -294,6 +295,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -349,6 +351,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -402,6 +405,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -453,6 +457,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -504,6 +509,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -557,6 +563,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -608,6 +615,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: false,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -659,6 +667,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -731,6 +740,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -783,6 +793,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -835,6 +846,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -887,6 +899,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: false,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -938,6 +951,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -989,6 +1003,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {
@ -1041,6 +1056,7 @@ describe('OC.SetupChecks tests', function() {
isSettimelimitAvailable: true,
hasFreeTypeSupport: true,
missingIndexes: [],
missingPrimaryKeys: [],
missingColumns: [],
cronErrors: [],
cronInfo: {

2
lib/composer/composer/ClassLoader.php
View File

@ -60,7 +60,7 @@ class ClassLoader
public function getPrefixes()
{
if (!empty($this->prefixesPsr0)) {
return call_user_func_array('array_merge', $this->prefixesPsr0);
return call_user_func_array('array_merge', array_values($this->prefixesPsr0));
}
return array();

5
lib/composer/composer/autoload_classmap.php
View File

@ -796,6 +796,7 @@ return array(
'OC\\Core\\Command\\Config\\System\\SetConfig' => $baseDir . '/core/Command/Config/System/SetConfig.php',
'OC\\Core\\Command\\Db\\AddMissingColumns' => $baseDir . '/core/Command/Db/AddMissingColumns.php',
'OC\\Core\\Command\\Db\\AddMissingIndices' => $baseDir . '/core/Command/Db/AddMissingIndices.php',
'OC\\Core\\Command\\Db\\AddMissingPrimaryKeys' => $baseDir . '/core/Command/Db/AddMissingPrimaryKeys.php',
'OC\\Core\\Command\\Db\\ConvertFilecacheBigInt' => $baseDir . '/core/Command/Db/ConvertFilecacheBigInt.php',
'OC\\Core\\Command\\Db\\ConvertMysqlToMB4' => $baseDir . '/core/Command/Db/ConvertMysqlToMB4.php',
'OC\\Core\\Command\\Db\\ConvertType' => $baseDir . '/core/Command/Db/ConvertType.php',
@ -919,6 +920,9 @@ return array(
'OC\\Core\\Migrations\\Version18000Date20191014105105' => $baseDir . '/core/Migrations/Version18000Date20191014105105.php',
'OC\\Core\\Migrations\\Version18000Date20191204114856' => $baseDir . '/core/Migrations/Version18000Date20191204114856.php',
'OC\\Core\\Migrations\\Version19000Date20200211083441' => $baseDir . '/core/Migrations/Version19000Date20200211083441.php',
'OC\\Core\\Migrations\\Version20000Date20201109081915' => $baseDir . '/core/Migrations/Version20000Date20201109081915.php',
'OC\\Core\\Migrations\\Version20000Date20201109081918' => $baseDir . '/core/Migrations/Version20000Date20201109081918.php',
'OC\\Core\\Migrations\\Version20000Date20201109081919' => $baseDir . '/core/Migrations/Version20000Date20201109081919.php',
'OC\\Core\\Notification\\RemoveLinkSharesNotifier' => $baseDir . '/core/Notification/RemoveLinkSharesNotifier.php',
'OC\\Core\\Service\\LoginFlowV2Service' => $baseDir . '/core/Service/LoginFlowV2Service.php',
'OC\\DB\\Adapter' => $baseDir . '/lib/private/DB/Adapter.php',
@ -936,6 +940,7 @@ return array(
'OC\\DB\\Migrator' => $baseDir . '/lib/private/DB/Migrator.php',
'OC\\DB\\MissingColumnInformation' => $baseDir . '/lib/private/DB/MissingColumnInformation.php',
'OC\\DB\\MissingIndexInformation' => $baseDir . '/lib/private/DB/MissingIndexInformation.php',
'OC\\DB\\MissingPrimaryKeyInformation' => $baseDir . '/lib/private/DB/MissingPrimaryKeyInformation.php',
'OC\\DB\\MySQLMigrator' => $baseDir . '/lib/private/DB/MySQLMigrator.php',
'OC\\DB\\MySqlTools' => $baseDir . '/lib/private/DB/MySqlTools.php',
'OC\\DB\\OCSqlitePlatform' => $baseDir . '/lib/private/DB/OCSqlitePlatform.php',

5
lib/composer/composer/autoload_static.php
View File

@ -825,6 +825,7 @@ class ComposerStaticInit53792487c5a8370acc0b06b1a864ff4c
'OC\\Core\\Command\\Config\\System\\SetConfig' => __DIR__ . '/../../..' . '/core/Command/Config/System/SetConfig.php',
'OC\\Core\\Command\\Db\\AddMissingColumns' => __DIR__ . '/../../..' . '/core/Command/Db/AddMissingColumns.php',
'OC\\Core\\Command\\Db\\AddMissingIndices' => __DIR__ . '/../../..' . '/core/Command/Db/AddMissingIndices.php',
'OC\\Core\\Command\\Db\\AddMissingPrimaryKeys' => __DIR__ . '/../../..' . '/core/Command/Db/AddMissingPrimaryKeys.php',
'OC\\Core\\Command\\Db\\ConvertFilecacheBigInt' => __DIR__ . '/../../..' . '/core/Command/Db/ConvertFilecacheBigInt.php',
'OC\\Core\\Command\\Db\\ConvertMysqlToMB4' => __DIR__ . '/../../..' . '/core/Command/Db/ConvertMysqlToMB4.php',
'OC\\Core\\Command\\Db\\ConvertType' => __DIR__ . '/../../..' . '/core/Command/Db/ConvertType.php',
@ -948,6 +949,9 @@ class ComposerStaticInit53792487c5a8370acc0b06b1a864ff4c
'OC\\Core\\Migrations\\Version18000Date20191014105105' => __DIR__ . '/../../..' . '/core/Migrations/Version18000Date20191014105105.php',
'OC\\Core\\Migrations\\Version18000Date20191204114856' => __DIR__ . '/../../..' . '/core/Migrations/Version18000Date20191204114856.php',
'OC\\Core\\Migrations\\Version19000Date20200211083441' => __DIR__ . '/../../..' . '/core/Migrations/Version19000Date20200211083441.php',
'OC\\Core\\Migrations\\Version20000Date20201109081915' => __DIR__ . '/../../..' . '/core/Migrations/Version20000Date20201109081915.php',
'OC\\Core\\Migrations\\Version20000Date20201109081918' => __DIR__ . '/../../..' . '/core/Migrations/Version20000Date20201109081918.php',
'OC\\Core\\Migrations\\Version20000Date20201109081919' => __DIR__ . '/../../..' . '/core/Migrations/Version20000Date20201109081919.php',
'OC\\Core\\Notification\\RemoveLinkSharesNotifier' => __DIR__ . '/../../..' . '/core/Notification/RemoveLinkSharesNotifier.php',
'OC\\Core\\Service\\LoginFlowV2Service' => __DIR__ . '/../../..' . '/core/Service/LoginFlowV2Service.php',
'OC\\DB\\Adapter' => __DIR__ . '/../../..' . '/lib/private/DB/Adapter.php',
@ -965,6 +969,7 @@ class ComposerStaticInit53792487c5a8370acc0b06b1a864ff4c
'OC\\DB\\Migrator' => __DIR__ . '/../../..' . '/lib/private/DB/Migrator.php',
'OC\\DB\\MissingColumnInformation' => __DIR__ . '/../../..' . '/lib/private/DB/MissingColumnInformation.php',
'OC\\DB\\MissingIndexInformation' => __DIR__ . '/../../..' . '/lib/private/DB/MissingIndexInformation.php',
'OC\\DB\\MissingPrimaryKeyInformation' => __DIR__ . '/../../..' . '/lib/private/DB/MissingPrimaryKeyInformation.php',
'OC\\DB\\MySQLMigrator' => __DIR__ . '/../../..' . '/lib/private/DB/MySQLMigrator.php',
'OC\\DB\\MySqlTools' => __DIR__ . '/../../..' . '/lib/private/DB/MySqlTools.php',
'OC\\DB\\OCSqlitePlatform' => __DIR__ . '/../../..' . '/lib/private/DB/OCSqlitePlatform.php',

9
lib/private/DB/Connection.php
View File

@ -39,6 +39,7 @@ use Doctrine\DBAL\Configuration;
use Doctrine\DBAL\DBALException;
use Doctrine\DBAL\Driver;
use Doctrine\DBAL\Exception\ConstraintViolationException;
use Doctrine\DBAL\Exception\NotNullConstraintViolationException;
use Doctrine\DBAL\Platforms\MySqlPlatform;
use Doctrine\DBAL\Schema\Schema;
use OC\DB\QueryBuilder\QueryBuilder;
@ -297,6 +298,8 @@ class Connection extends ReconnectWrapper implements IDBConnection {
}, array_merge($keys, $values))
);
return $insertQb->execute();
} catch (NotNullConstraintViolationException $e) {
throw $e;
} catch (ConstraintViolationException $e) {
// value already exists, try update
$updateQb = $this->getQueryBuilder();
@ -307,12 +310,18 @@ class Connection extends ReconnectWrapper implements IDBConnection {
$where = $updateQb->expr()->andX();
$whereValues = array_merge($keys, $updatePreconditionValues);
foreach ($whereValues as $name => $value) {
if ($value === '') {
$where->add($updateQb->expr()->emptyString(
$name
));
} else {
$where->add($updateQb->expr()->eq(
$name,
$updateQb->createNamedParameter($value, $this->getType($value)),
$this->getType($value)
));
}
}
$updateQb->where($where);
$affected = $updateQb->execute();

42
lib/private/DB/MissingPrimaryKeyInformation.php Normal file
View File

@ -0,0 +1,42 @@
<?php
declare(strict_types=1);
/**
* @copyright Copyright (c) 2018 Morris Jobke <hey@morrisjobke.de>
*
* @author Morris Jobke <hey@morrisjobke.de>
* @author Roeland Jago Douma <roeland@famdouma.nl>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OC\DB;
class MissingPrimaryKeyInformation {
private $listOfMissingPrimaryKeys = [];
public function addHintForMissingSubject(string $tableName) {
$this->listOfMissingPrimaryKeys[] = [
'tableName' => $tableName,
];
}
public function getListOfMissingPrimaryKeys(): array {
return $this->listOfMissingPrimaryKeys;
}
}

6
lib/private/Files/Cache/Cache.php
View File

@ -998,7 +998,11 @@ class Cache implements ICache {
$path = $result->fetchColumn();
$result->closeCursor();
return $path === false ? null : $path;
if ($path === false) {
return null;
}
return (string) $path;
}
/**

23
lib/private/Security/CredentialsManager.php
View File

@ -35,7 +35,7 @@ use OCP\Security\ICrypto;
* @package OC\Security
*/
class CredentialsManager implements ICredentialsManager {
public const DB_TABLE = 'credentials';
public const DB_TABLE = 'storages_credentials';
/** @var ICrypto */
protected $crypto;
@ -81,9 +81,13 @@ class CredentialsManager implements ICredentialsManager {
$qb = $this->dbConnection->getQueryBuilder();
$qb->select('credentials')
->from(self::DB_TABLE)
->where($qb->expr()->eq('user', $qb->createNamedParameter((string)$userId)))
->andWhere($qb->expr()->eq('identifier', $qb->createNamedParameter($identifier)))
;
->where($qb->expr()->eq('identifier', $qb->createNamedParameter($identifier)));
if ($userId === '') {
$qb->andWhere($qb->expr()->emptyString('user'));
} else {
$qb->andWhere($qb->expr()->eq('user', $qb->createNamedParameter((string)$userId)));
}
$qResult = $qb->execute();
$result = $qResult->fetch();
@ -107,9 +111,14 @@ class CredentialsManager implements ICredentialsManager {
public function delete($userId, $identifier) {
$qb = $this->dbConnection->getQueryBuilder();
$qb->delete(self::DB_TABLE)
->where($qb->expr()->eq('user', $qb->createNamedParameter((string)$userId)))
->andWhere($qb->expr()->eq('identifier', $qb->createNamedParameter($identifier)))
;
->where($qb->expr()->eq('identifier', $qb->createNamedParameter($identifier)));
if ($userId === '') {
$qb->andWhere($qb->expr()->emptyString('user'));
} else {
$qb->andWhere($qb->expr()->eq('user', $qb->createNamedParameter((string)$userId)));
}
return $qb->execute();
}

7
lib/private/Share20/DefaultShareProvider.php
View File

@ -874,6 +874,11 @@ class DefaultShareProvider implements IShareProvider {
$cursor = $qb->execute();
while ($data = $cursor->fetch()) {
if ($data['fileid'] && $data['path'] === null) {
$data['path'] = (string) $data['path'];
$data['name'] = (string) $data['name'];
$data['checksum'] = (string) $data['checksum'];
}
if ($this->isAccessibleResult($data)) {
$shares[] = $this->createShare($data);
}
@ -1004,7 +1009,7 @@ class DefaultShareProvider implements IShareProvider {
->setShareType((int)$data['share_type'])
->setPermissions((int)$data['permissions'])
->setTarget($data['file_target'])
->setNote($data['note'])
->setNote((string)$data['note'])
->setMailSend((bool)$data['mail_send'])
->setStatus((int)$data['accepted'])
->setLabel($data['label']);

2
lib/public/IDBConnection.php
View File

@ -50,6 +50,8 @@ use OCP\DB\QueryBuilder\IQueryBuilder;
interface IDBConnection {
public const ADD_MISSING_INDEXES_EVENT = self::class . '::ADD_MISSING_INDEXES';
public const CHECK_MISSING_INDEXES_EVENT = self::class . '::CHECK_MISSING_INDEXES';
public const ADD_MISSING_PRIMARY_KEYS_EVENT = self::class . '::ADD_MISSING_PRIMARY_KEYS';
public const CHECK_MISSING_PRIMARY_KEYS_EVENT = self::class . '::CHECK_MISSING_PRIMARY_KEYS';
public const ADD_MISSING_COLUMNS_EVENT = self::class . '::ADD_MISSING_COLUMNS';
public const CHECK_MISSING_COLUMNS_EVENT = self::class . '::CHECK_MISSING_COLUMNS';

5
tests/lib/AppConfigTest.php
View File

@ -9,6 +9,7 @@
namespace Test;
use OC\AppConfig;
use OCP\IConfig;
/**
@ -42,7 +43,7 @@ class AppConfigTest extends TestCase {
$sql->delete('appconfig');
$sql->execute();
$this->overwriteService('AppConfig', new \OC\AppConfig($this->connection));
$this->overwriteService(AppConfig::class, new \OC\AppConfig($this->connection));
$sql = $this->connection->getQueryBuilder();
$sql->insert('appconfig')
@ -132,7 +133,7 @@ class AppConfigTest extends TestCase {
$sql->execute();
}
$this->restoreService('AppConfig');
$this->restoreService(AppConfig::class);
parent::tearDown();
}

47
tests/lib/DB/ConnectionTest.php
View File

@ -97,14 +97,17 @@ class ConnectionTest extends \Test\TestCase {
$this->assertTableNotExist('table');
}
private function getTextValueByIntergerField($integerField) {
private function getTextValueByIntegerField($integerField) {
$builder = $this->connection->getQueryBuilder();
$query = $builder->select('textfield')
$query = $builder->select('*')
->from('table')
->where($builder->expr()->eq('integerfield', $builder->createNamedParameter($integerField, IQueryBuilder::PARAM_INT)));
$result = $query->execute();
return $result->fetchColumn();
$row = $result->fetch();
$result->closeCursor();
return $row['textfield'] ?? null;
}
public function testSetValues() {
@ -116,7 +119,7 @@ class ConnectionTest extends \Test\TestCase {
'clobfield' => 'not_null'
]);
$this->assertEquals('foo', $this->getTextValueByIntergerField(1));
$this->assertEquals('foo', $this->getTextValueByIntegerField(1));
}
public function testSetValuesOverWrite() {
@ -133,7 +136,7 @@ class ConnectionTest extends \Test\TestCase {
'textfield' => 'bar'
]);
$this->assertEquals('bar', $this->getTextValueByIntergerField(1));
$this->assertEquals('bar', $this->getTextValueByIntegerField(1));
}
public function testSetValuesOverWritePrecondition() {
@ -154,7 +157,7 @@ class ConnectionTest extends \Test\TestCase {
'booleanfield' => true
]);
$this->assertEquals('bar', $this->getTextValueByIntergerField(1));
$this->assertEquals('bar', $this->getTextValueByIntegerField(1));
}
@ -200,6 +203,10 @@ class ConnectionTest extends \Test\TestCase {
}
public function testInsertIfNotExist() {
if (\OC::$server->getConfig()->getSystemValue('dbtype', 'sqlite') === 'oci') {
self::markTestSkipped('Insert if not exist does not work with clob on oracle');
}
$this->makeTestTable();
$categoryEntries = [
['user' => 'test', 'category' => 'Family', 'expectedResult' => 1],
@ -213,13 +220,15 @@ class ConnectionTest extends \Test\TestCase {
['user' => 'test2', 'category' => 'Coworkers', 'expectedResult' => 1],
];
$row = 0;
foreach ($categoryEntries as $entry) {
$result = $this->connection->insertIfNotExist('*PREFIX*table',
[
'textfield' => $entry['user'],
'clobfield' => $entry['category'],
]);
$this->assertEquals($entry['expectedResult'], $result);
'integerfield' => $row++,
], ['textfield', 'clobfield']);
$this->assertEquals($entry['expectedResult'], $result, json_encode($entry));
}
$query = $this->connection->prepare('SELECT * FROM `*PREFIX*table`');
@ -229,6 +238,10 @@ class ConnectionTest extends \Test\TestCase {
}
public function testInsertIfNotExistNull() {
if (\OC::$server->getConfig()->getSystemValue('dbtype', 'sqlite') === 'oci') {
self::markTestSkipped('Insert if not exist does not work with clob on oracle');
}
$this->makeTestTable();
$categoryEntries = [
['addressbookid' => 123, 'fullname' => null, 'expectedResult' => 1],
@ -236,13 +249,15 @@ class ConnectionTest extends \Test\TestCase {
['addressbookid' => 123, 'fullname' => 'test', 'expectedResult' => 1],
];
$row = 0;
foreach ($categoryEntries as $entry) {
$result = $this->connection->insertIfNotExist('*PREFIX*table',
[
'integerfield_default' => $entry['addressbookid'],
'clobfield' => $entry['fullname'],
]);
$this->assertEquals($entry['expectedResult'], $result);
'integerfield' => $row++,
], ['integerfield_default', 'clobfield']);
$this->assertEquals($entry['expectedResult'], $result, json_encode($entry));
}
$query = $this->connection->prepare('SELECT * FROM `*PREFIX*table`');
@ -252,6 +267,10 @@ class ConnectionTest extends \Test\TestCase {
}
public function testInsertIfNotExistDonTOverwrite() {
if (\OC::$server->getConfig()->getSystemValue('dbtype', 'sqlite') === 'oci') {
self::markTestSkipped('Insert if not exist does not work with clob on oracle');
}
$this->makeTestTable();
$fullName = 'fullname test';
$uri = 'uri_1';
@ -288,6 +307,10 @@ class ConnectionTest extends \Test\TestCase {
}
public function testInsertIfNotExistsViolating() {
if (\OC::$server->getConfig()->getSystemValue('dbtype', 'sqlite') === 'oci') {
self::markTestSkipped('Insert if not exist does not work with clob on oracle');
}
$this->makeTestTable();
$result = $this->connection->insertIfNotExist('*PREFIX*table',
[
@ -318,6 +341,10 @@ class ConnectionTest extends \Test\TestCase {
* @param array $compareKeys
*/
public function testInsertIfNotExistsViolatingUnique($compareKeys) {
if (\OC::$server->getConfig()->getSystemValue('dbtype', 'sqlite') === 'oci') {
self::markTestSkipped('Insert if not exist does not work with clob on oracle');
}
$this->makeTestTable();
$result = $this->connection->insertIfNotExist('*PREFIX*table',
[

30
tests/lib/DB/MDB2SchemaReaderTest.php
View File

@ -14,6 +14,10 @@ use Doctrine\DBAL\Schema\Schema;
use OC\DB\MDB2SchemaReader;
use OCP\IConfig;
use Test\TestCase;
use Doctrine\DBAL\Types\IntegerType;
use Doctrine\DBAL\Types\TextType;
use Doctrine\DBAL\Types\StringType;
use Doctrine\DBAL\Types\BooleanType;
/**
* Class MDB2SchemaReaderTest
@ -51,13 +55,13 @@ class MDB2SchemaReaderTest extends TestCase {
$this->assertCount(1, $schema->getTables());
$table = $schema->getTable('test_table');
$this->assertCount(8, $table->getColumns());
$this->assertCount(9, $table->getColumns());
$this->assertEquals(4, $table->getColumn('integerfield')->getLength());
$this->assertTrue($table->getColumn('integerfield')->getAutoincrement());
$this->assertEquals(0, $table->getColumn('integerfield')->getDefault());
$this->assertTrue($table->getColumn('integerfield')->getNotnull());
$this->assertInstanceOf('Doctrine\DBAL\Types\IntegerType', $table->getColumn('integerfield')->getType());
$this->assertEquals(4, $table->getColumn('id')->getLength());
$this->assertTrue($table->getColumn('id')->getAutoincrement());
$this->assertEquals(0, $table->getColumn('id')->getDefault());
$this->assertTrue($table->getColumn('id')->getNotnull());
$this->assertInstanceOf(IntegerType::class, $table->getColumn('id')->getType());
$this->assertSame(10, $table->getColumn('integerfield_default')->getDefault());
@ -65,18 +69,19 @@ class MDB2SchemaReaderTest extends TestCase {
$this->assertFalse($table->getColumn('textfield')->getAutoincrement());
$this->assertSame('foo', $table->getColumn('textfield')->getDefault());
$this->assertTrue($table->getColumn('textfield')->getNotnull());
$this->assertInstanceOf('Doctrine\DBAL\Types\StringType', $table->getColumn('textfield')->getType());
$this->assertInstanceOf(StringType::class, $table->getColumn('textfield')->getType());
$this->assertNull($table->getColumn('clobfield')->getLength());
$this->assertFalse($table->getColumn('clobfield')->getAutoincrement());
$this->assertNull($table->getColumn('clobfield')->getDefault());
$this->assertFalse($table->getColumn('clobfield')->getNotnull());
$this->assertInstanceOf('Doctrine\DBAL\Types\TextType', $table->getColumn('clobfield')->getType());
$this->assertInstanceOf(StringType::class, $table->getColumn('clobfield')->getType());
// $this->assertInstanceOf(TextType::class, $table->getColumn('clobfield')->getType());
$this->assertNull($table->getColumn('booleanfield')->getLength());
$this->assertFalse($table->getColumn('booleanfield')->getAutoincrement());
$this->assertNull($table->getColumn('booleanfield')->getDefault());
$this->assertInstanceOf('Doctrine\DBAL\Types\BooleanType', $table->getColumn('booleanfield')->getType());
$this->assertInstanceOf(BooleanType::class, $table->getColumn('booleanfield')->getType());
$this->assertTrue($table->getColumn('booleanfield_true')->getDefault());
$this->assertFalse($table->getColumn('booleanfield_false')->getDefault());
@ -84,10 +89,13 @@ class MDB2SchemaReaderTest extends TestCase {
$this->assertEquals(12, $table->getColumn('decimalfield_precision_scale')->getPrecision());
$this->assertEquals(2, $table->getColumn('decimalfield_precision_scale')->getScale());
$this->assertCount(2, $table->getIndexes());
$this->assertEquals(['integerfield'], $table->getIndex('primary')->getUnquotedColumns());
$this->assertCount(3, $table->getIndexes());
$this->assertEquals(['id'], $table->getIndex('primary')->getUnquotedColumns());
$this->assertTrue($table->getIndex('primary')->isPrimary());
$this->assertTrue($table->getIndex('primary')->isUnique());
$this->assertEquals(['integerfield'], $table->getIndex('index_integerfield')->getUnquotedColumns());
$this->assertFalse($table->getIndex('index_integerfield')->isPrimary());
$this->assertTrue($table->getIndex('index_integerfield')->isUnique());
$this->assertEquals(['booleanfield'], $table->getIndex('index_boolean')->getUnquotedColumns());
$this->assertFalse($table->getIndex('index_boolean')->isPrimary());
$this->assertFalse($table->getIndex('index_boolean')->isUnique());

20
tests/lib/DB/testschema.xml
View File

@ -13,7 +13,7 @@
<declaration>
<field>
<name>integerfield</name>
<name>id</name>
<type>integer</type>
<default>0</default>
<notnull>true</notnull>
@ -21,6 +21,13 @@
<length>4</length>
<autoincrement>1</autoincrement>
</field>
<field>
<name>integerfield</name>
<type>integer</type>
<default>0</default>
<notnull>true</notnull>
<length>4</length>
</field>
<field>
<name>integerfield_default</name>
<type>integer</type>
@ -37,7 +44,7 @@
</field>
<field>
<name>clobfield</name>
<type>clob</type>
<type>text</type>
</field>
<field>
<name>booleanfield</name>
@ -64,6 +71,15 @@
<name>index_primary</name>
<primary>true</primary>
<unique>true</unique>
<field>
<name>id</name>
<sorting>ascending</sorting>
</field>
</index>
<index>
<name>index_integerfield</name>
<unique>true</unique>
<field>
<name>integerfield</name>
<sorting>ascending</sorting>

2
version.php
View File

@ -29,7 +29,7 @@
// between betas, final and RCs. This is _not_ the public version number. Reset minor/patchlevel
// when updating major/minor version number.
$OC_Version = [20, 0, 1, 1];
$OC_Version = [20, 0, 1, 6];
// The human readable string
$OC_VersionString = '20.0.1';