Code formatting & standards fixes

src/Bridge/Laravel/IlluminateEventDispatcher.php

@@ -7,6 +7,11 @@
 
 class IlluminateEventDispatcher implements EventDispatcher
 {
+    protected $dispatcher;
+
+    /**
+     * @param Dispatcher $dispatcher
+     */
     public function __construct(Dispatcher $dispatcher)
     {
         $this->dispatcher = $dispatcher;

src/Bridge/Laravel/IlluminateQueueHandler.php

@@ -6,6 +6,11 @@
 
 class IlluminateQueueHandler
 {
+    protected $bus;
+
+    /**
+     * @param SynchronousCommandBus $bus
+     */
     public function __construct(SynchronousCommandBus $bus)
     {
         $this->bus = $bus;
@@ -16,5 +21,7 @@ public function fire($job, $serializedCommand)
         $command = unserialize($serializedCommand);
 
         $this->bus->execute($command);
+
+        $job->delete();
     }
 }

src/Busses/QueueingCommandBus.php

@@ -15,6 +15,20 @@
  */
 class QueueingCommandBus implements CommandBus
 {
+    /**
+     * @var \Chief\CommandBus
+     */
+    protected $innerBus;
+
+    /**
+     * @var \Chief\CommandBusQueuer
+     */
+    protected $queuer;
+
+    /**
+     * @param CommandBusQueuer $queuer
+     * @param CommandBus $innerBus
+     */
     public function __construct(CommandBusQueuer $queuer, CommandBus $innerBus = null)
     {
         $this->queuer = $queuer;

src/Busses/TransactionalCommandBus.php

@@ -12,9 +12,23 @@
  */
 class TransactionalCommandBus implements CommandBus
 {
+    /**
+     * Whether or not a Command is in progress and the bus is locked
+     * @var bool
+     */
     protected $locked = false;
+
+    /**
+     * Queued Commands to be executed when the current command finishes
+     * @var array
+     */
     protected $queue = [];
 
+    /**
+     * @var CommandBus
+     */
+    protected $innerBus;
+
     public function __construct(CommandBus $innerBus = null)
     {
         $this->innerBus = $innerBus ?: new SynchronousCommandBus();

src/Chief.php

@@ -11,6 +11,11 @@
  */
 class Chief implements CommandBus
 {
+    /**
+     * @var CommandBus
+     */
+    protected $bus;
+
     /**
      * Constructor
      *

src/Decorators/EventDispatchingDecorator.php

@@ -5,9 +5,23 @@
 use Chief\Command;
 use Chief\CommandBus;
 use Chief\Decorator;
+use Exception;
 
 class EventDispatchingDecorator implements Decorator
 {
+    /**
+     * @var EventDispatcher
+     */
+    protected $dispatcher;
+
+    /**
+     * @var CommandBus
+     */
+    protected $innerCommandBus;
+
+    /**
+     * @param EventDispatcher $dispatcher
+     */
     public function __construct(EventDispatcher $dispatcher)
     {
         $this->dispatcher = $dispatcher;
@@ -23,9 +37,14 @@ public function setInnerBus(CommandBus $bus)
      *
      * @param Command $command
      * @return mixed
+     * @throws \Exception
      */
     public function execute(Command $command)
     {
+        if (!$this->innerCommandBus) {
+            throw new Exception('No inner bus defined for this decorator. Set an inner bus with setInnerBus()');
+        }
+
         $response = $this->innerCommandBus->execute($command);
 
         $eventName = $this->getEventName($command);

src/Handlers/LazyLoadingCommandHandler.php

@@ -14,10 +14,14 @@ class LazyLoadingCommandHandler implements CommandHandler
     protected $container;
 
     /**
-     * @var \Chief\CommandHandler
+     * @var string
      */
     protected $handler;
 
+    /**
+     * @param string $handlerName
+     * @param Container $container
+     */
     public function __construct($handlerName, Container $container)
     {
         $this->container = $container;