Clean Integration configuration

Also avoid the extra Util\Common class and just stuff it into the global Integration.

src/DDTrace/Integrations/AMQP/AMQPIntegration.php

@@ -22,14 +22,6 @@ class AMQPIntegration extends Integration
     const SYSTEM = 'rabbitmq';
     protected $protocolVersion;
 
-    /**
-     * @return string The integration name.
-     */
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     // Source: https://magp.ie/2015/09/30/convert-large-integer-to-hexadecimal-without-php-math-extension/
     public static function largeBaseConvert($numString, $fromBase, $toBase)
     {
@@ -454,7 +446,7 @@ public function setGenericTags(
         $span->meta[Tag::MQ_PROTOCOL_VERSION] = $this->protocolVersion;
 
         if ($exception) {
-            $this->setError($span, $exception);
+            $span->exception = $exception;
         }
     }
 

src/DDTrace/Integrations/CakePHP/CakePHPIntegration.php

@@ -17,14 +17,6 @@ class CakePHPIntegration extends Integration
     public $appName;
     public $rootSpan;
 
-    /**
-     * @return string The integration name.
-     */
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     // CakePHP v2.x - we don't need to check for v3 since it does not have \Dispatcher or \ShellDispatcher
     public function init(): int
     {
@@ -102,7 +94,7 @@ function (SpanData $span, array $args) use ($integration) {
         \DDTrace\trace_method('ExceptionRenderer', '__construct', [
             'instrument_when_limited' => 1,
             'posthook' => function (SpanData $span, array $args) use ($integration) {
-                $integration->setError($integration->rootSpan, $args[0]);
+                $integration->rootSpan->exception = $args[0];
                 $span->meta[Tag::COMPONENT] = CakePHPIntegration::NAME;
                 return false;
             },

src/DDTrace/Integrations/CodeIgniter/V2/CodeIgniterIntegration.php

@@ -12,14 +12,6 @@ class CodeIgniterIntegration extends Integration
 {
     const NAME = 'codeigniter';
 
-    /**
-     * @return string The integration name.
-     */
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     /**
      * Add instrumentation to CodeIgniter requests
      */

src/DDTrace/Integrations/Curl/CurlIntegration.php

@@ -29,11 +29,6 @@ final class CurlIntegration extends Integration
 {
     const NAME = 'curl';
 
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     public function init(): int
     {
         if (!extension_loaded('curl')) {

src/DDTrace/Integrations/Drupal/DrupalIntegration.php

@@ -23,9 +23,9 @@ class DrupalIntegration extends Integration
     /**
      * {@inheritdoc}
      */
-    public function getName()
+    public function isForcingTraceAnalytics(): bool
     {
-        return self::NAME;
+        return true;
     }
 
     public function init(): int

src/DDTrace/Integrations/ElasticSearch/V1/ElasticSearchIntegration.php

@@ -11,14 +11,6 @@ class ElasticSearchIntegration extends Integration
 {
     const NAME = 'elasticsearch';
 
-    /**
-     * @return string The integration name.
-     */
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     /**
      * Add instrumentation to Elasticsearch requests
      */

src/DDTrace/Integrations/ElasticSearch/V8/ElasticSearchIntegration.php

@@ -15,14 +15,6 @@ class ElasticSearchIntegration extends Integration
 
     public $logNextBody = false;
 
-    /**
-     * @return string The integration name.
-     */
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     /**
      * Add instrumentation to Elasticsearch requests
      */

src/DDTrace/Integrations/Eloquent/EloquentIntegration.php

@@ -16,14 +16,6 @@ class EloquentIntegration extends Integration
      */
     private $appName;
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     /**
      * {@inheritDoc}
      */

src/DDTrace/Integrations/Exec/ExecIntegration.php

@@ -24,11 +24,6 @@ class ExecIntegration extends Integration
     const REDACTED_BINARIES = array('md5' => null);
     const UNREDACTED_ENV_VARS = array('LD_PRELOAD' => null, 'LD_LIBRARY_PATH' => null, 'PATH' => null);
 
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     public function init(): int
     {
         \DDTrace\install_hook(

src/DDTrace/Integrations/Frankenphp/FrankenphpIntegration.php

@@ -13,11 +13,6 @@ class FrankenphpIntegration extends Integration
 {
     const NAME = 'frankenphp';
 
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     /**
      * {@inheritdoc}
      */

src/DDTrace/Integrations/Guzzle/GuzzleIntegration.php

@@ -14,11 +14,6 @@ class GuzzleIntegration extends Integration
 {
     const NAME = 'guzzle';
 
-    public function getName()
-    {
-        return self::NAME;
-    }
-
     public function init(): int
     {
         $integration = $this;