Factory refactoring.

Author: pfkristaps

src/Factories/Sync/ParameterFactory.php

@@ -2,7 +2,6 @@
 
 namespace Printful;
 
-use Printful\Factories\Sync\ParameterFactory;
 use Printful\Structures\Sync\Requests\SyncVariantRequest;
 use Printful\Structures\Sync\Responses\SyncProductRequestResponse;
 use Printful\Structures\Sync\Responses\SyncProductResponse;
@@ -108,7 +107,7 @@ public function getVariant($id)
      */
     public function createProduct(SyncProductCreationParameters $request)
     {
-        $params = ParameterFactory::buildSyncProductPostParams($request);
+        $params = $request->toPostArray();
         $result = $this->printfulClient->post(PrintfulProducts::ENDPOINT_PRODUCTS, $params);
 
         $syncProduct = SyncProductResponse::fromArray($result);
@@ -128,7 +127,7 @@ public function createProduct(SyncProductCreationParameters $request)
      */
     public function createVariant($productId, SyncVariantRequest $syncVariantRequest)
     {
-        $params = ParameterFactory::buildSyncVariantPostParams($syncVariantRequest);
+        $params = $syncVariantRequest->toPostArray();
         $result = $this->printfulClient->post(PrintfulProducts::ENDPOINT_PRODUCTS . '/' . $productId . '/variants', $params);
 
         $syncVariant = SyncVariantResponse::fromArray($result);
@@ -148,7 +147,7 @@ public function createVariant($productId, SyncVariantRequest $syncVariantRequest
      */
     public function updateProduct($productId, SyncProductUpdateParameters $request)
     {
-        $params = ParameterFactory::buildSyncProductPutParams($request);
+        $params = $request->toPutArray();
         $result = $this->printfulClient->put(PrintfulProducts::ENDPOINT_PRODUCTS . '/' . $productId, $params);
 
         $syncProduct = SyncProductResponse::fromArray($result);
@@ -168,7 +167,7 @@ public function updateProduct($productId, SyncProductUpdateParameters $request)
      */
     public function updateVariant($variantId, SyncVariantRequest $request)
     {
-        $params = ParameterFactory::buildSyncVariantPutParams($request);
+        $params = $request->toPutArray();
         $result = $this->printfulClient->put(PrintfulProducts::ENDPOINT_VARIANTS . '/' . $variantId, $params);
 
         $syncVariant = SyncVariantResponse::fromArray($result);

src/PrintfulProducts.php

@@ -2,6 +2,8 @@
 
 namespace Printful\Structures\Sync\Requests;
 
+use Printful\Exceptions\PrintfulSdkException;
+
 class SyncVariantRequest
 {
     /** @var int|null */
@@ -98,4 +100,85 @@ public static function fromArray(array $array)
 
         return $syncVariantRequest;
     }
+
+    /**
+     * Builds POST request array
+     *
+     * @return array
+     * @throws PrintfulSdkException
+     */
+    public function toPostArray()
+    {
+        if (!$this->variantId) {
+            throw new PrintfulSdkException('Missing variant_id');
+        }
+
+        $files = $this->getFiles();
+        if (empty($files)) {
+            throw new PrintfulSdkException('Missing files');
+        }
+
+        $syncVariantParams = [];
+
+        $syncVariantParams['external_id'] = $this->externalId;
+        $syncVariantParams['retail_price'] = $this->retailPrice;
+        $syncVariantParams['variant_id'] = $this->variantId;
+
+        $syncVariantParams['files'] = [];
+        foreach ($files as $file) {
+            $syncVariantParams['files'][] = $file->toArray();
+        }
+
+        $syncVariantParams['options'] = [];
+        foreach ($this->getOptions() as $option) {
+            $syncVariantParams['options'][] = $option->toArray();
+        }
+
+        return $syncVariantParams;
+    }
+
+    /**
+     * Builds PUT request array
+     *
+     * @return array
+     * @throws PrintfulSdkException
+     */
+    public function toPutArray()
+    {
+        $syncVariantParams = [];
+
+        if (!is_null($this->externalId)) {
+            $syncVariantParams['external_id'] = $this->externalId;
+        }
+
+        if (!is_null($this->retailPrice)) {
+            $syncVariantParams['retail_price'] = $this->retailPrice;
+        }
+
+        if (!is_null($this->variantId)) {
+            if (!$this->variantId) {
+                throw new PrintfulSdkException('Variant id is required');
+            }
+
+            $syncVariantParams['variant_id'] = $this->variantId;
+        }
+
+        $files = $this->getFiles();
+        if (!is_null($files)) {
+            $syncVariantParams['files'] = [];
+            foreach ($files as $file){
+                $syncVariantParams['files'][] = $file->toArray();
+            }
+        }
+
+        $options = $this->getOptions();
+        if (!is_null($options)) {
+            $syncVariantParams['options'] = [];
+            foreach ($options as $option) {
+                $syncVariantParams['options'][] = $option->toArray();
+            }
+        }
+
+        return $syncVariantParams;
+    }
 }