Merge pull request #2 from php-api-clients/improvement-replace-getter…

…s-n-setters-with-a-commandbus

Replace getters/setters with command bus

composer.json

@@ -11,7 +11,8 @@
   ],
   "require": {
     "php": "^7.0",
-    "api-clients/hydrator": "dev-master"
+    "api-clients/hydrator": "dev-master",
+    "league/tactician": "^1.0"
   },
   "require-dev": {
     "phpunit/phpunit": "^5.2.3",

src/AbstractResource.php

@@ -2,26 +2,25 @@
 
 namespace ApiClients\Foundation\Resource;
 
-use ApiClients\Foundation\Hydrator\HydrateTrait;
+use League\Tactician\CommandBus;
 
 abstract class AbstractResource implements ResourceInterface
 {
-    use HydrateTrait;
+    /**
+     * @var CommandBus
+     */
+    private $commandBus;
 
-    public function setExtraProperties(array $properties)
+    final public function __construct(CommandBus $commandBus)
     {
-        foreach ($properties as $key => $value) {
-            $this->setPropertyValue($key, $value);
-        }
+        $this->commandBus = $commandBus;
     }
 
-    private function setPropertyValue(string $key, $value)
+    /**
+     * @return CommandBus
+     */
+    protected function getCommandBus()
     {
-        $methodName = $key . 'Setter';
-        if (!method_exists($this, $methodName)) {
-            return;
-        }
-
-        $this->$methodName($value);
+        return $this->commandBus;
     }
 }

src/ResourceInterface.php

@@ -2,6 +2,9 @@
 
 namespace ApiClients\Foundation\Resource;
 
+use League\Tactician\CommandBus;
+
 interface ResourceInterface
 {
+    public function __construct(CommandBus $commandBus);
 }