Merge pull request #6 from inextensodigital/simplify-handler-declaration

Simplify command handlers declaration

Author: Azoog

DependencyInjection/Compiler/CommandHandlerPass.php

@@ -23,11 +23,13 @@ public function process(ContainerBuilder $container)
 
         foreach ($container->findTaggedServiceIds('rezzza_command_bus.command_handler') as $serviceId => $handlers) {
             foreach ($handlers as $data) {
-                if (!isset($data['command'])) {
+                $command = $this->getCommand($serviceId, $handlers, $data);
+
+                if (null === $command) {
                     throw new \LogicException('Please provide a command with "rezzza_command.command_handler" tag');
                 }
 
-                $services[$data['command']] = new Definition('Rezzza\CommandBusBundle\Handler\HandlerServiceDefinition', [
+                $services[$command] = new Definition('Rezzza\CommandBusBundle\Handler\HandlerServiceDefinition', [
                     $serviceId,
                     isset($data['method']) ? $data['method'] : null
                 ]);
@@ -36,4 +38,28 @@ public function process(ContainerBuilder $container)
 
         $container->getDefinition('rezzza_command_bus.command_handler_locator.container')->addArgument($services);
     }
+
+    private function getCommand(string $serviceId, array $handlers, array $data): ?string
+    {
+        if (isset($data['command'])) {
+            return $data['command'];
+        }
+
+        // if no command is defined the handler should manage only one command
+        if (1 < \count($handlers)) {
+            return null;
+        }
+
+        $command = $this->guessCommandFromHandlerName($serviceId);
+
+        return class_exists($command) ? $command : null;
+    }
+
+    private function guessCommandFromHandlerName(string $handler): string
+    {
+        $parts = preg_split('/(?=[A-Z])/',$handler);
+        array_pop($parts);
+
+        return implode('', $parts);
+    }
 }