Add name property to the stopwatchEvent

Ahmed Raafat · Ahmed Raafat · commit 928ab32ec4f9 · 2020-07-27T14:09:05.000+02:00
diff --git a/src/Symfony/Component/Stopwatch/Section.php b/src/Symfony/Component/Stopwatch/Section.php
@@ -113,7 +113,7 @@ public function setId(string $id)
     public function startEvent(string $name, ?string $category)
     {
         if (!isset($this->events[$name])) {
-            $this->events[$name] = new StopwatchEvent($this->origin ?: microtime(true) * 1000, $category, $this->morePrecision);
+            $this->events[$name] = new StopwatchEvent($this->origin ?: microtime(true) * 1000, $category, $this->morePrecision, $name);
         }
 
         return $this->events[$name]->start();
diff --git a/src/Symfony/Component/Stopwatch/StopwatchEvent.php b/src/Symfony/Component/Stopwatch/StopwatchEvent.php
@@ -43,18 +43,25 @@ class StopwatchEvent
      */
     private $started = [];
 
+    /**
+     * @var string
+     */
+    private $name;
+
     /**
      * @param float       $origin        The origin time in milliseconds
      * @param string|null $category      The event category or null to use the default
      * @param bool        $morePrecision If true, time is stored as float to keep the original microsecond precision
+     * @param string      $name          The event name or empty string as default value
      *
      * @throws \InvalidArgumentException When the raw time is not valid
      */
-    public function __construct(float $origin, string $category = null, bool $morePrecision = false)
+    public function __construct(float $origin, string $category = null, bool $morePrecision = false, string $name = '')
     {
         $this->origin = $this->formatTime($origin);
         $this->category = \is_string($category) ? $category : 'default';
         $this->morePrecision = $morePrecision;
+        $this->name = $name;
     }
 
     /**
@@ -236,11 +243,21 @@ private function formatTime(float $time): float
         return round($time, 1);
     }
 
+    /**
+     * Gets the event name.
+     *
+     * @return string The event name
+     */
+    public function getName()
+    {
+        return $this->name;
+    }
+
     /**
      * @return string
      */
     public function __toString()
     {
-        return sprintf('%s: %.2F MiB - %d ms', $this->getCategory(), $this->getMemory() / 1024 / 1024, $this->getDuration());
+        return sprintf('%s: %.2F MiB - %d ms', ($this->getName() ?? $this->getCategory()), $this->getMemory() / 1024 / 1024, $this->getDuration());
     }
 }
diff --git a/src/Symfony/Component/Stopwatch/Tests/StopwatchEventTest.php b/src/Symfony/Component/Stopwatch/Tests/StopwatchEventTest.php
@@ -200,5 +200,17 @@ public function testHumanRepresentation()
 
         $event = new StopwatchEvent(microtime(true) * 1000, 'foo');
         $this->assertEquals('foo: 0.00 MiB - 0 ms', (string) $event);
+
+        $event = new StopwatchEvent(microtime(true) * 1000, 'foo', false, 'name');
+        $this->assertEquals('name: 0.00 MiB - 0 ms', (string) $event);
+    }
+
+    public function testGetName()
+    {
+        $event = new StopwatchEvent(microtime(true) * 1000);
+        $this->assertEquals('', $event->getName());
+
+        $event = new StopwatchEvent(microtime(true) * 1000, 'cat', false, 'test');
+        $this->assertEquals('test', $event->getCategory());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ public function setId(string $id)`
`113`	`113`	`public function startEvent(string $name, ?string $category)`
`114`	`114`	`{`
`115`	`115`	`if (!isset($this->events[$name])) {`
`116`		`- $this->events[$name] = new StopwatchEvent($this->origin ?: microtime(true) * 1000, $category, $this->morePrecision);`
	`116`	`+ $this->events[$name] = new StopwatchEvent($this->origin ?: microtime(true) * 1000, $category, $this->morePrecision, $name);`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`return $this->events[$name]->start();`