Commit message for this issue:
Issue #1992950 by xjm, Pancho: Stop use-ing Exception.

Our coding standards state that we should not use classes in the global namespace, but I noticed today this is still all over the place for the Exception class. See http://drupal.org/node/1353118 and #1614186: Coding standards for using "native" PHP classes like stdClass in namespaced code.

Attached corrects this.

[tesla:drupal | Sun 07:19:25] $ grep -rl "use Exception" *
core/lib/Drupal/Component/Archiver/ArchiverException.php
core/lib/Drupal/Component/Archiver/ArchiveTar.php
core/lib/Drupal/Component/Plugin/Exception/ContextException.php
core/lib/Drupal/Component/Plugin/Exception/PluginException.php
core/lib/Drupal/Core/Config/DatabaseStorage.php
core/lib/Drupal/Core/Database/Driver/mysql/Schema.php
core/lib/Drupal/Core/Database/Driver/pgsql/Install/Tasks.php
core/lib/Drupal/Core/Database/Driver/pgsql/Schema.php
core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php
core/lib/Drupal/Core/Database/Driver/sqlite/Install/Tasks.php
core/lib/Drupal/Core/Database/Driver/sqlite/Schema.php
core/lib/Drupal/Core/Database/Install/Tasks.php
core/lib/Drupal/Core/Database/Query/Insert.php
core/lib/Drupal/Core/Entity/EntityMalformedException.php
core/lib/Drupal/Core/Entity/EntityStorageException.php
core/lib/Drupal/Core/Executable/ExecutableException.php
core/lib/Drupal/Core/TypedData/MissingContextException.php
core/lib/Drupal/Core/TypedData/ReadOnlyException.php
core/lib/Drupal/Core/Updater/UpdaterException.php
core/lib/Drupal/Core/Utility/UpdateException.php
core/modules/field/lib/Drupal/field/Tests/CrudTest.php
core/modules/field_sql_storage/lib/Drupal/field_sql_storage/Tests/FieldSqlStorageTest.php
core/modules/locale/lib/Drupal/locale/Gettext.php
core/modules/locale/lib/Drupal/locale/StringStorageException.php
core/modules/node/lib/Drupal/node/Tests/NodeCreationTest.php
core/modules/node/lib/Drupal/node/Tests/NodeQueryAlterTest.php
core/modules/simpletest/lib/Drupal/simpletest/TestBase.php
core/modules/system/lib/Drupal/system/Tests/Database/InvalidDataTest.php
core/modules/system/lib/Drupal/system/Tests/Database/SchemaTest.php
core/modules/system/lib/Drupal/system/Tests/Database/TransactionTest.php
core/modules/system/lib/Drupal/system/Tests/Entity/EntityTranslationFormTest.php
core/modules/system/lib/Drupal/system/Tests/Plugin/AlterDecoratorTest.php
core/modules/system/lib/Drupal/system/Tests/Plugin/FactoryTest.php
core/modules/system/lib/Drupal/system/Tests/Routing/RouteProviderTest.php
core/modules/system/lib/Drupal/system/Tests/Upgrade/UpgradePathTestBase.php
CommentFileSizeAuthor
un-use-exception.patch32.04 KBxjm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm’s picture

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community
+++ b/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.phpundefined
@@ -16,7 +16,6 @@
 use PDO;
-use Exception;
 use SplFileInfo;

We need to do the same for PDO and SplFileInfo

xjm’s picture

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Updated issue summary.