From eaf33de13e05e11f19db68fbad5864c4e9738735 Mon Sep 17 00:00:00 2001 From: Lee Rowlands <lee.rowlands@previousnext.com.au> Date: Mon, 12 Oct 2020 17:13:52 +1000 Subject: [PATCH] Issue #3174848 by alexpott: Fix typo in Connection::prepare deprecation message --- core/lib/Drupal/Core/Database/Connection.php | 4 ++-- core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php | 2 +- .../Core/Database/DatabaseExceptionWrapperTest.php | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/core/lib/Drupal/Core/Database/Connection.php b/core/lib/Drupal/Core/Database/Connection.php index 94a07d3599dc..aad323229f06 100644 --- a/core/lib/Drupal/Core/Database/Connection.php +++ b/core/lib/Drupal/Core/Database/Connection.php @@ -1736,13 +1736,13 @@ abstract public function nextId($existing_id = 0); * * @deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database * drivers should instantiate \PDOStatement objects by calling - * \PDO::prepare in their Collection::prepareStatement method instead. + * \PDO::prepare in their Connection::prepareStatement method instead. * \PDO::prepare should not be called outside of driver code. * * @see https://www.drupal.org/node/3137786 */ public function prepare($statement, array $driver_options = []) { - @trigger_error('Connection::prepare() is deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database drivers should instantiate \PDOStatement objects by calling \PDO::prepare in their Collection::prepareStatement method instead. \PDO::prepare should not be called outside of driver code. See https://www.drupal.org/node/3137786', E_USER_DEPRECATED); + @trigger_error('Connection::prepare() is deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database drivers should instantiate \PDOStatement objects by calling \PDO::prepare in their Connection::prepareStatement method instead. \PDO::prepare should not be called outside of driver code. See https://www.drupal.org/node/3137786', E_USER_DEPRECATED); return $this->connection->prepare($statement, $driver_options); } diff --git a/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php b/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php index 817cebd05581..d59e2d07bc06 100644 --- a/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php +++ b/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php @@ -349,7 +349,7 @@ public static function sqlFunctionLikeBinary($pattern, $subject) { * {@inheritdoc} */ public function prepare($statement, array $driver_options = []) { - @trigger_error('Connection::prepare() is deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database drivers should instantiate \PDOStatement objects by calling \PDO::prepare in their Collection::prepareStatement method instead. \PDO::prepare should not be called outside of driver code. See https://www.drupal.org/node/3137786', E_USER_DEPRECATED); + @trigger_error('Connection::prepare() is deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database drivers should instantiate \PDOStatement objects by calling \PDO::prepare in their Connection::prepareStatement method instead. \PDO::prepare should not be called outside of driver code. See https://www.drupal.org/node/3137786', E_USER_DEPRECATED); return new Statement($this->connection, $this, $statement, $driver_options); } diff --git a/core/tests/Drupal/KernelTests/Core/Database/DatabaseExceptionWrapperTest.php b/core/tests/Drupal/KernelTests/Core/Database/DatabaseExceptionWrapperTest.php index 0d2ea9fdc423..1b2984ed4f5d 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/DatabaseExceptionWrapperTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/DatabaseExceptionWrapperTest.php @@ -17,7 +17,7 @@ class DatabaseExceptionWrapperTest extends KernelTestBase { * Tests deprecation of Connection::prepare. * * @group legacy - * @expectedDeprecation Connection::prepare() is deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database drivers should instantiate \PDOStatement objects by calling \PDO::prepare in their Collection::prepareStatement method instead. \PDO::prepare should not be called outside of driver code. See https://www.drupal.org/node/3137786 + * @expectedDeprecation Connection::prepare() is deprecated in drupal:9.1.0 and is removed from drupal:10.0.0. Database drivers should instantiate \PDOStatement objects by calling \PDO::prepare in their Connection::prepareStatement method instead. \PDO::prepare should not be called outside of driver code. See https://www.drupal.org/node/3137786 */ public function testPrepare() { $connection = Database::getConnection(); -- GitLab