Merge pull request #845 from Majkl578/deprecations

doctrine/common deprecations (soft)
doctrine · Jun 24, 2018 · 6e2d622 · 6e2d622
2 parents 3666662 + 39bc396
commit 6e2d622
Show file tree

Hide file tree

Showing 18 changed files with 51 additions and 2 deletions.
diff --git a/composer.json b/composer.json
@@ -27,7 +27,8 @@
     "require-dev": {
         "phpunit/phpunit": "^6.3",
         "doctrine/coding-standard": "^1.0",
-        "squizlabs/php_codesniffer": "^3.0"
+        "squizlabs/php_codesniffer": "^3.0",
+        "symfony/phpunit-bridge": "^4.0.5"
     },
     "autoload": {
         "psr-4": {

diff --git a/lib/Doctrine/Common/ClassLoader.php b/lib/Doctrine/Common/ClassLoader.php
@@ -1,6 +1,11 @@
 <?php
 namespace Doctrine\Common;
 
+use function trigger_error;
+use const E_USER_DEPRECATED;
+
+@trigger_error(ClassLoader::class . ' is deprecated.', E_USER_DEPRECATED);
+
 /**
  * A <tt>ClassLoader</tt> is an autoloader for class files that can be
  * installed on the SPL autoload stack. It is a class loader that either loads only classes
@@ -13,7 +18,7 @@
  * @author Roman Borschel <[email protected]>
  * @since 2.0
  *
- * @deprecated the ClassLoader is deprecated and will be removed in version 3.0 of doctrine/common.
+ * @deprecated The ClassLoader is deprecated and will be removed in version 3.0 of doctrine/common.
  */
 class ClassLoader
 {

diff --git a/lib/Doctrine/Common/CommonException.php b/lib/Doctrine/Common/CommonException.php
@@ -5,6 +5,8 @@
  * Base exception class for package Doctrine\Common.
  *
  * @author heinrich
+ *
+ * @deprecated The doctrine/common package is deprecated, please use specific packages and their exceptions instead.
  */
 class CommonException extends \Exception
 {

diff --git a/lib/Doctrine/Common/Lexer.php b/lib/Doctrine/Common/Lexer.php
@@ -2,6 +2,10 @@
 namespace Doctrine\Common;
 
 use Doctrine\Common\Lexer\AbstractLexer;
+use function trigger_error;
+use const E_USER_DEPRECATED;
+
+@trigger_error(Lexer::class . ' is deprecated.', E_USER_DEPRECATED);
 
 /**
  * Base class for writing simple lexers, i.e. for creating small DSLs.
@@ -13,6 +17,8 @@
  * @author Guilherme Blanco <[email protected]>
  * @author Jonathan Wage <[email protected]>
  * @author Roman Borschel <[email protected]>
+ *
+ * @deprecated Use Doctrine\Common\Lexer\AbstractLexer from doctrine/lexer package instead.
  */
 abstract class Lexer extends AbstractLexer
 {

diff --git a/lib/Doctrine/Common/Proxy/AbstractProxyFactory.php b/lib/Doctrine/Common/Proxy/AbstractProxyFactory.php
@@ -11,6 +11,8 @@
  * Abstract factory for proxy objects.
  *
  * @author Benjamin Eberlei <[email protected]>
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 abstract class AbstractProxyFactory
 {

diff --git a/lib/Doctrine/Common/Proxy/Autoloader.php b/lib/Doctrine/Common/Proxy/Autoloader.php
@@ -7,6 +7,10 @@
  * Special Autoloader for Proxy classes, which are not PSR-0 compliant.
  *
  * @author Benjamin Eberlei <[email protected]>
+ *
+ * @internal
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 class Autoloader
 {

diff --git a/lib/Doctrine/Common/Proxy/Exception/InvalidArgumentException.php b/lib/Doctrine/Common/Proxy/Exception/InvalidArgumentException.php
@@ -10,6 +10,8 @@
  * @link   www.doctrine-project.org
  * @since  2.4
  * @author Marco Pivetta <[email protected]>
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 class InvalidArgumentException extends BaseInvalidArgumentException implements ProxyException
 {

diff --git a/lib/Doctrine/Common/Proxy/Exception/OutOfBoundsException.php b/lib/Doctrine/Common/Proxy/Exception/OutOfBoundsException.php
@@ -8,6 +8,8 @@
  *
  * @link   www.doctrine-project.org
  * @author Fredrik Wendel <[email protected]>
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 class OutOfBoundsException extends BaseOutOfBoundsException implements ProxyException
 {

diff --git a/lib/Doctrine/Common/Proxy/Exception/ProxyException.php b/lib/Doctrine/Common/Proxy/Exception/ProxyException.php
@@ -7,6 +7,8 @@
  * @link   www.doctrine-project.org
  * @since  2.4
  * @author Marco Pivetta <[email protected]>
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 interface ProxyException
 {

diff --git a/lib/Doctrine/Common/Proxy/Exception/UnexpectedValueException.php b/lib/Doctrine/Common/Proxy/Exception/UnexpectedValueException.php
@@ -9,6 +9,8 @@
  * @link   www.doctrine-project.org
  * @since  2.4
  * @author Marco Pivetta <[email protected]>
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 class UnexpectedValueException extends BaseUnexpectedValueException implements ProxyException
 {

diff --git a/lib/Doctrine/Common/Proxy/Proxy.php b/lib/Doctrine/Common/Proxy/Proxy.php
@@ -10,6 +10,8 @@
  * @author Roman Borschel <[email protected]>
  * @author Marco Pivetta  <[email protected]>
  * @since  2.4
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 interface Proxy extends BaseProxy
 {

diff --git a/lib/Doctrine/Common/Proxy/ProxyDefinition.php b/lib/Doctrine/Common/Proxy/ProxyDefinition.php
@@ -5,6 +5,8 @@
  * Definition structure how to create a proxy.
  *
  * @author Benjamin Eberlei <[email protected]>
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 class ProxyDefinition
 {

diff --git a/lib/Doctrine/Common/Proxy/ProxyGenerator.php b/lib/Doctrine/Common/Proxy/ProxyGenerator.php
@@ -12,6 +12,8 @@
  *
  * @author Marco Pivetta <[email protected]>
  * @since  2.4
+ *
+ * @deprecated The Doctrine\Common\Proxy component is deprecated, please use ocramius/proxy-manager instead.
  */
 class ProxyGenerator
 {

diff --git a/lib/Doctrine/Common/Util/ClassUtils.php b/lib/Doctrine/Common/Util/ClassUtils.php
@@ -9,6 +9,8 @@
  *
  * @author Benjamin Eberlei <[email protected]>
  * @author Johannes Schmitt <[email protected]>
+ *
+ * @deprecated The ClassUtils class is deprecated.
  */
 class ClassUtils
 {

diff --git a/lib/Doctrine/Common/Util/Debug.php b/lib/Doctrine/Common/Util/Debug.php
@@ -13,6 +13,8 @@
  * @author Jonathan Wage <[email protected]>
  * @author Roman Borschel <[email protected]>
  * @author Giorgio Sironi <[email protected]>
+ *
+ * @deprecated The Debug class is deprecated, please use symfony/var-dumper instead.
  */
 final class Debug
 {

diff --git a/lib/Doctrine/Common/Util/Inflector.php b/lib/Doctrine/Common/Util/Inflector.php
@@ -2,11 +2,17 @@
 namespace Doctrine\Common\Util;
 
 use Doctrine\Common\Inflector\Inflector as BaseInflector;
+use function trigger_error;
+use const E_USER_DEPRECATED;
+
+@trigger_error(Inflector::class . ' is deprecated.', E_USER_DEPRECATED);
 
 /**
  * Doctrine inflector has static methods for inflecting text.
  *
  * Kept for backwards compatibility reasons, was moved to its own component.
+ *
+ * @deprecated The Inflector class is deprecated, use Doctrine\Common\Inflector\Inflector from doctrine/inflector package instead,
  */
 class Inflector extends BaseInflector
 {

diff --git a/lib/Doctrine/Common/Version.php b/lib/Doctrine/Common/Version.php
@@ -10,6 +10,8 @@
  * @author Guilherme Blanco <[email protected]>
  * @author Jonathan Wage <[email protected]>
  * @author Roman Borschel <[email protected]>
+ *
+ * @deprecated The Version class is deprecated, please refrain from checking the version of doctrine/common.
  */
 class Version
 {

diff --git a/tests/Doctrine/Tests/Common/ClassLoaderTest.php b/tests/Doctrine/Tests/Common/ClassLoaderTest.php
@@ -4,6 +4,9 @@
 
 use Doctrine\Common\ClassLoader;
 
+/**
+ * @group legacy
+ */
 class ClassLoaderTest extends \Doctrine\Tests\DoctrineTestCase
 {
     public function testClassLoader()