[HttpKernel] Add Profiler::isEnabled() method

Bilge · fabpot · commit b305386c0303 · 2022-02-21T17:22:42.000+01:00
diff --git a/src/Symfony/Component/HttpKernel/CHANGELOG.md b/src/Symfony/Component/HttpKernel/CHANGELOG.md
@@ -6,6 +6,7 @@ CHANGELOG
 
  * Add `BackedEnumValueResolver` to resolve backed enum cases from request attributes in controller arguments
  * Deprecate StreamedResponseListener, it's not needed anymore
+ * Add `Profiler::isEnabled()` so collaborating collector services may elect to omit themselves.
 
 6.0
 ---
diff --git a/src/Symfony/Component/HttpKernel/Profiler/Profiler.php b/src/Symfony/Component/HttpKernel/Profiler/Profiler.php
@@ -60,6 +60,11 @@ public function enable()
         $this->enabled = true;
     }
 
+    public function isEnabled(): bool
+    {
+        return $this->enabled;
+    }
+
     /**
      * Loads the Profile for the given Response.
      */
diff --git a/src/Symfony/Component/HttpKernel/Tests/Profiler/ProfilerTest.php b/src/Symfony/Component/HttpKernel/Tests/Profiler/ProfilerTest.php
@@ -83,6 +83,28 @@ public function testFindWorksWithStatusCode()
         $this->assertCount(0, $profiler->find(null, null, null, null, null, null, '204'));
     }
 
+    public function testIsInitiallyEnabled()
+    {
+        self::assertTrue((new Profiler($this->storage))->isEnabled());
+    }
+
+    public function testDisable()
+    {
+        $profiler = new Profiler($this->storage);
+        $profiler->disable();
+
+        self::assertFalse($profiler->isEnabled());
+    }
+
+    public function testEnable()
+    {
+        $profiler = new Profiler($this->storage);
+        $profiler->disable();
+        $profiler->enable();
+
+        self::assertTrue($profiler->isEnabled());
+    }
+
     protected function setUp(): void
     {
         $this->tmp = tempnam(sys_get_temp_dir(), 'sf_profiler');
