diff --git a/developer_manual/app_publishing_maintenance/upgrade-guide.rst b/developer_manual/app_publishing_maintenance/upgrade-guide.rst index f52a75a2920..0c9d7285cca 100644 --- a/developer_manual/app_publishing_maintenance/upgrade-guide.rst +++ b/developer_manual/app_publishing_maintenance/upgrade-guide.rst @@ -37,6 +37,7 @@ Optimistically speaking, the database connection and the query builder should mo Some (minor) breaking changes were inevitable. Here's the summary +* Many of the database connection methods have now a documented exception that can be thrown: ``\OCP\DB\Exception``. This also acts as a replacement for the removed ``\Doctrine\DBAL\DBALException``. * ``$queryBuilder->execute()->fetch()`` only has one argument now (there were three previously) * ``$queryBuilder->execute()->fetchColumn()`` has no more arguments and got also deprecated. Use ``fetchOne`` instead * ``$queryBuilder->execute()->bindParam()`` was removed because conceptually it does not make sense to bind a parameter *after* executing a query. Use ``bindParam`` on the ``IPeparedStatement`` instead. @@ -45,8 +46,8 @@ Some (minor) breaking changes were inevitable. Here's the summary * ``$queryBuilder->execute()->errorCode()`` was removed from Doctrine * ``$queryBuilder->execute()->errorInfo()`` was removed from Doctrine * ``$queryBuilder->execute()->setFetchMode()`` was removed from Doctrine -* ``$connection->prepare()->execute()`` previously returned ``false`` under some error conditions, it now always gives you an ``IResult`` or throws an exception -* ``\Doctrine\DBAL\Types\Type::*`` type constants were moved, which some apps used for column type constants in apps. Use ``\Doctrine\DBAL\Types\Types::*`` or inline the values. +* ``$connection->prepare()->execute()`` previously returned ``false`` under some error conditions, it now always gives you an ``IResult`` or throws a ``\OCP\DB\Exception``. +* ``\Doctrine\DBAL\Types\Type::*`` type constants were moved, which some apps used for column type constants in apps. Use the new ``\OCP\DB\Types::*`` as a replacement. The details of this change can also be seen in the `pull request on Github `__ and in the upstream `dbal 3.0.xx upgrade document `__. diff --git a/developer_manual/basics/storage/migrations.rst b/developer_manual/basics/storage/migrations.rst index b1b54fe5472..95112b1f038 100644 --- a/developer_manual/basics/storage/migrations.rst +++ b/developer_manual/basics/storage/migrations.rst @@ -75,7 +75,7 @@ With this step the new column gets created: $table = $schema->getTable('twofactor_backupcodes'); - $table->addColumn('user_id', Types::STRING, [ + $table->addColumn('user_id', \OCP\DB\Types::STRING, [ 'notnull' => true, 'length' => 64, ]);