Parcourir la source

Remove TriggerEvent setter and adding arguments to the constructor

François Pluchino il y a 13 ans
Parent
commit
c7b898d10d

+ 1 - 5
src/Composer/Trigger/TriggerDispatcher.php

@@ -56,11 +56,7 @@ class TriggerDispatcher
      */
     public function dispatch($eventName)
     {
-        $event = new TriggerEvent();
-
-        $event->setName($eventName);
-        $event->setComposer($this->composer);
-        $event->setIO($this->io);
+        $event = new TriggerEvent($eventName, $this->composer, $this->io);
 
         $this->doDispatch($event);
     }

+ 12 - 28
src/Composer/Trigger/TriggerEvent.php

@@ -38,23 +38,27 @@ class TriggerEvent
     private $io;
 
     /**
-     * Returns the event's name.
+     * Constructor.
      *
-     * @return string The event name
+     * @param string      $name     The event name
+     * @param Composer    $composer The composer objet
+     * @param IOInterface $io       The IOInterface object
      */
-    public function getName()
+    public function __construct($name, Composer $composer, IOInterface $io)
     {
-        return $this->name;
+        $this->name = $name;
+        $this->composer = $composer;
+        $this->io = $io;
     }
 
     /**
-     * Stores the event's name.
+     * Returns the event's name.
      *
-     * @param string $name The event name
+     * @return string The event name
      */
-    public function setName($name)
+    public function getName()
     {
-        $this->name = $name;
+        return $this->name;
     }
 
     /**
@@ -67,16 +71,6 @@ class TriggerEvent
         return $this->composer;
     }
 
-    /**
-     * Stores the composer instance.
-     *
-     * @param Composer $composer
-     */
-    public function setComposer(Composer $composer)
-    {
-        $this->composer = $composer;
-    }
-
     /**
      * Returns the IO instance.
      *
@@ -86,14 +80,4 @@ class TriggerEvent
     {
         return $this->io;
     }
-
-    /**
-     * Stores the IO instance.
-     *
-     * @param IOInterface $io
-     */
-    public function setIO(IOInterface $io)
-    {
-        $this->io = $io;
-    }
 }