Ditching type hinting for handlers and updating README

Author: nilportugues

README.md

@@ -140,25 +140,16 @@ final class RegisterUserHandler implements CommandHandler
         $this->userRepository = $userRepository;
     }
 
-    public function __invoke(Command $command)
-    {
-        $this->guard($command);
-        
+    public function __invoke(RegisterUser $command)
+    {        
         $user = new User(
             $command->getUsername(),
             $command->getPassword(),
             $command->getEmail(),
         );
 
         $this->userRepository->save($user);        
-    }
-    
-    private function guard(Command $command)
-    {
-        if (false === ($command instanceof RegisterUser)) {
-            throw new \InvalidArgumentException("Expected command of type: ".RegisterUser::class);
-        }
-    }
+    }    
 }
 ```
 
@@ -328,21 +319,13 @@ final class GetUserHandler implements QueryHandler
         $this->userRepository = $userRepository;
     }
 
-    public function __invoke(Query $query) : QueryResponse
-    {
-        $this->guard($query);        
+    public function __invoke(GetUser $query) : QueryResponse
+    {  
         $userId = $query->getUserId();        
         $user = $this->userRepository->find($userId);       
 
         return new UserQueryResponse($user);
     }
-    
-    private function guard(Query $query)
-    {
-        if (false === ($query instanceof GetUser)) {
-            throw new \InvalidArgumentException("Expected query of type: ".GetUser::class);
-        }
-    }
 }
 ```
 
@@ -543,19 +526,12 @@ final class SendWelcomeEmailHandler implements EventHandler
         $this->emailProvider = $emailProvider;
     }
 
-    public function __invoke(Event $event)
+    public function __invoke(UserRegistered $event)
     {
         $this->guard($event);        
         $this->emailProvider->send('welcome_email', $event->getEmail());        
     }
-    
-    private function guard(Event $event)
-    {
-        if (false === ($event instanceof UserRegistered)) {
-            throw new \InvalidArgumentException("Expected event of type: ".UserRegistered::class);
-        }
-    }
-        
+     
     public static function subscribedTo() : string
     {
         return UserRegistered::class;
@@ -583,19 +559,11 @@ final class SetupUserAccountHandler implements EventHandler
         $this->userCreditsRepository = $userCreditsRepository;
     }
 
-    public function __invoke(Event $event)
-    {
-        $this->guard($event);        
+    public function __invoke(UserRegistered $event)
+    {      
         $this->userFriendsRepository->add(new UserFriendsCollection($event->getUserId(), []));        
         $this->userCreditsRepository->add(new UserCredits($event->getUserId(), new Credits(0));        
-    }
-    
-    private function guard(Event $event)
-    {
-        if (false === ($event instanceof UserRegistered)) {
-            throw new \InvalidArgumentException("Expected event of type: ".UserRegistered::class);
-        }
-    }
+    }   
 
     public static function subscribedTo() : string
     {

src/CommandBus/CommandBus.php

@@ -36,6 +36,7 @@ public function __invoke(Command $command, callable $next = null)
 
         if (empty($middleware) && !empty($current)) {
             $current->__invoke($command);
+
             return;
         }
 

src/CommandBus/Contracts/CommandHandler.php

@@ -17,5 +17,5 @@ interface CommandHandler
     /**
      * @param Command $command
      */
-    public function __invoke(Command $command);
+    public function __invoke($command);
 }

src/EventBus/Contracts/EventHandler.php

@@ -20,9 +20,4 @@ interface EventHandler
      * @return string
      */
     public static function subscribedTo() : string;
-
-    /**
-     * @param Event $event
-     */
-    public function __invoke(Event $event);
 }

src/EventBus/EventBus.php

@@ -39,6 +39,7 @@ public function __invoke(Event $event, callable $next = null)
 
         if (empty($middleware) && !empty($current)) {
             $current->__invoke($event);
+
             return;
         }
 

src/QueryBus/Contracts/QueryHandler.php

@@ -14,10 +14,4 @@
  */
 interface QueryHandler
 {
-    /**
-     * @param Query $query
-     *
-     * @return QueryResponse
-     */
-    public function __invoke(Query $query) : QueryResponse;
 }

tests/CommandBus/DummyCommand.php

@@ -6,4 +6,7 @@
 
 class DummyCommand implements Command
 {
+    public function __invoke()
+    {
+    }
 }

tests/CommandBus/DummyCommandHandler.php

@@ -2,15 +2,14 @@
 
 namespace NilPortugues\Tests\MessageBus\CommandBus;
 
-use NilPortugues\MessageBus\CommandBus\Contracts\Command;
 use NilPortugues\MessageBus\CommandBus\Contracts\CommandHandler;
 
 class DummyCommandHandler implements CommandHandler
 {
     /**
-     * @param Command $command
+     * @param callable $command
      */
-    public function __invoke(Command $command)
+    public function __invoke($command)
     {
     }
 }

tests/EventBus/DummyEvent.php

@@ -6,4 +6,7 @@
 
 class DummyEvent implements Event
 {
+    public function __invoke()
+    {
+    }
 }

tests/EventBus/DummyEventHandler.php

@@ -11,7 +11,7 @@ class DummyEventHandler implements EventHandler, EventHandlerPriority
     /**
      * @param Event $event
      */
-    public function __invoke(Event $event)
+    public function __invoke($event)
     {
     }