Merge pull request #45 from thephpleague/feature/remove-command-interface

Remove command interface

Author: rosstuck

CHANGELOG.md

@@ -0,0 +1,15 @@
+## 0.4.0 (2015-03-30)
+BC breaks:
+
+- Removed the `League\Tactician\Command` interface. Now any plain ol' PHP object can be a command! See #43 and #45.
+
+New features:
+
+- None
+
+Bug fixes:
+
+- None
+
+## 0.3.0 (2015-02-15)
+First public release!

examples/1-beginner-standard-usage.php

@@ -3,18 +3,17 @@
 
 use League\Tactician\Handler\MethodNameInflector\HandleClassNameInflector;
 use League\Tactician\Handler\Locator\InMemoryLocator;
-use League\Tactician\Command;
 
 // Our example Command and Handler. ///////////////////////////////////////////
-class RegisterUserCommand implements Command
+class RegisterUserCommand
 {
     public $emailAddress;
     public $password;
 }
 
 class RegisterUserHandler
 {
-    public function handleRegisterUserCommand(RegisterUserCommand $command)
+    public function handleRegisterUserphp (RegisterUserCommand $command)
     {
         // Do your core application logic here. Don't actually echo stuff. :)
         echo "User {$command->emailAddress} was registered!\n";

examples/2-intermediate-create-middleware.php

@@ -4,7 +4,6 @@
 
 use League\Tactician\Middleware;
 use League\Tactician\CommandBus;
-use League\Tactician\Command;
 
 /**
  * Let's say we want something to happen every time we execute a command.
@@ -37,7 +36,7 @@ public function __construct(Logger $logger)
         $this->logger = $logger;
     }
 
-    public function execute(Command $command, callable $next)
+    public function execute($command, callable $next)
     {
         $commandClass = get_class($command);
 

examples/3-intermediate-custom-naming-conventions.php

@@ -12,7 +12,6 @@
  *
  * We can write a custom MethodNameInflector for that:
  */
-use League\Tactician\Command;
 use League\Tactician\CommandBus;
 use League\Tactician\Handler\CommandHandlerMiddleware;
 use League\Tactician\Handler\MethodNameInflector\MethodNameInflector;
@@ -21,7 +20,7 @@ class MyCustomInflector implements MethodNameInflector
 {
     // You can use the command and commandHandler to generate any name you
     // prefer but here, we'll always return the same one.
-    public function inflect(Command $command, $commandHandler)
+    public function inflect($command, $commandHandler)
     {
         return 'handle';
     }

examples/4-advanced-custom-handler-loading.php

@@ -9,7 +9,6 @@
  *
  * We can create a custom HandlerLocator for that.
  */
-use League\Tactician\Command;
 use League\Tactician\CommandBus;
 use League\Tactician\Handler\CommandHandlerMiddleware;
 use League\Tactician\Handler\Locator\HandlerLocator;
@@ -24,7 +23,7 @@ public function __construct($container)
         $this->container = $container;
     }
 
-    public function getHandlerForCommand(Command $command)
+    public function getHandlerForCommand($command)
     {
         // This is a cheesy naming strategy but it's just an example
         $handlerId = 'app.handler.' . get_class($command);

examples/5-advanced-self-executing-commands.php

@@ -1,7 +1,6 @@
 <?php
 require __DIR__ . '/../vendor/autoload.php';
 
-use League\Tactician\Command;
 use League\Tactician\Middleware;
 use League\Tactician\CommandBus;
 
@@ -12,7 +11,7 @@
  *
  * Here's a Tactician version of the wikipedia Light Switch example.
  */
-interface SelfExecutingCommand extends Command
+interface SelfExecutingCommand
 {
     public function execute(Light $light);
 }
@@ -53,7 +52,7 @@ public function __construct($light)
         $this->light = $light;
     }
 
-    public function execute(Command $command, callable $next)
+    public function execute($command, callable $next)
     {
         if (!$command instanceof SelfExecutingCommand) {
             throw new InvalidArgumentException("Can not execute command");

examples/6-conditional-handlers.php

@@ -12,12 +12,11 @@
  * We can write a custom middleware for that.
  */
 
-use League\Tactician\Command;
 use League\Tactician\CommandBus;
 use League\Tactician\Middleware;
 
 // External command
-interface ExternalCommand extends Command
+interface ExternalCommand
 {
 }
 
@@ -32,7 +31,7 @@ public function __construct(CommandBus $commandBus)
         $this->commandBus = $commandBus;
     }
 
-    public function execute(Command $command, callable $next)
+    public function execute($command, callable $next)
     {
         if ($command instanceof ExternalCommand) {
             return $this->commandBus->handle($command);
@@ -44,7 +43,7 @@ public function execute(Command $command, callable $next)
 // and we'll create a custom command handler/middleware
 final class ExternalCommandHandler implements Middleware
 {
-    public function execute(Command $command, callable $next)
+    public function execute($command, callable $next)
     {
         echo sprintf("Dispatched %s!\n", get_class($command));
     }

examples/repeated-sample-code.php

@@ -5,9 +5,8 @@
 
 use League\Tactician\Handler\MethodNameInflector\HandleClassNameInflector;
 use League\Tactician\Handler\Locator\InMemoryLocator;
-use League\Tactician\Command;
 
-class RegisterUserCommand implements Command
+class RegisterUserCommand
 {
     public $emailAddress;
     public $password;

src/Command.php

@@ -3,6 +3,7 @@
 namespace League\Tactician;
 
 use Closure;
+use League\Tactician\Exception\InvalidCommandException;
 
 /**
  * Receives a command and sends it through a chain of middleware for processing.
@@ -25,11 +26,15 @@ public function __construct(array $middleware)
     /**
      * Executes the given command and optionally returns a value
      *
-     * @param Command $command
+     * @param object $command
      * @return mixed
      */
-    public function handle(Command $command)
+    public function handle($command)
     {
+        if (!is_object($command)) {
+            throw InvalidCommandException::forUnknownValue($command);
+        }
+
         $middlewareChain = $this->middlewareChain;
         return $middlewareChain($command);
     }
@@ -40,12 +45,12 @@ public function handle(Command $command)
      */
     private function createExecutionChain($middlewareList)
     {
-        $lastCallable = function (Command $command) {
+        $lastCallable = function ($command) {
             // the final callable is a no-op
         };
 
         while ($middleware = array_pop($middlewareList)) {
-            $lastCallable = function (Command $command) use ($middleware, $lastCallable) {
+            $lastCallable = function ($command) use ($middleware, $lastCallable) {
                 return $middleware->execute($command, $lastCallable);
             };
         }