Merge pull request #6 from Invertus/BRD-130/price-data

BRD-130: price data added v2

Author: tomas862

IMPLEMENTATION.md

@@ -63,7 +63,7 @@ SDK/php/
 ### 3. Data Validation
 
 - **Strict validation** against field configuration
-- **Type checking** for all field types (text, keyword, hierarchy, variants, URLs, etc.)
+- **Type checking** for all field types (text, keyword, hierarchy, variants, URLs, name-value lists, floats, etc.)
 - **Comprehensive error reporting** with detailed validation messages
 - **Variant validation** with attribute checking
 
@@ -98,8 +98,12 @@ All field types from the Go implementation:
 - `KEYWORD` - Exact keyword matching
 - `HIERARCHY` - Hierarchical categories
 - `VARIANTS` - Product variants with attributes
+- `NAME_VALUE_LIST` - List of name-value pairs (e.g., features, specifications)
 - `IMAGE_URL` - Image URLs object with size keys (`small`, `medium`)
 - `URL` - Regular URLs with validation
+- `FLOAT` - Numeric floating-point values
+- `INTEGER` - Integer numeric values
+- `DOUBLE` - Double precision floating-point values
 
 ## Usage Examples
 

src/Adapters/PrestaShopAdapter.php

@@ -41,6 +41,8 @@ private function transformProduct(array $product): array
         $result = [
             'id' => $this->getRequiredField($product, 'remoteId'),
             'sku' => $this->getRequiredField($product, 'sku'),
+            'price' => $this->getRequiredField($product, 'price'),
+            'formattedPrice' => $this->getRequiredField($product, 'formattedPrice'),
             'variants' => [],
         ];
 

src/Enums/FieldType.php

@@ -14,4 +14,7 @@ enum FieldType: string
     case NAME_VALUE_LIST = 'name_value_list';
     case IMAGE_URL = 'image_url';
     case URL = 'url';
-} 
+    case FLOAT = 'float';
+    case INTEGER = 'integer';
+    case DOUBLE = 'double';
+}

src/Models/FieldConfigBuilder.php

@@ -56,6 +56,30 @@ public static function imageUrl(): FieldConfig
         return new FieldConfig(FieldType::IMAGE_URL);
     }
 
+    /**
+     * Create a float field
+     */
+    public static function float(): FieldConfig
+    {
+        return new FieldConfig(FieldType::FLOAT);
+    }
+
+    /**
+     * Create an integer field
+     */
+    public static function integer(): FieldConfig
+    {
+        return new FieldConfig(FieldType::INTEGER);
+    }
+
+    /**
+     * Create a double field
+     */
+    public static function double(): FieldConfig
+    {
+        return new FieldConfig(FieldType::DOUBLE);
+    }
+
     /**
      * Create a variants field with attributes
      *
@@ -104,6 +128,12 @@ public static function ecommerceFields(array $locales): array
 
         $defaultFields = [
             'id' => self::keyword(),
+            'name' => self::textKeyword(),
+            'brand' => self::textKeyword(),
+            'price' => self::double(),
+            'formattedPrice' => self::keyword(),
+            'categoryDefault' => self::textKeyword(),
+            'categories' => self::hierarchy(),
             'sku' => self::keyword(),
             'imageUrl' => self::imageUrl(),
             'productUrl' => self::url(),

src/Validators/DataValidator.php

@@ -81,6 +81,32 @@ private function validateField(string $fieldName, mixed $value, FieldConfig $fie
                 }
                 break;
 
+            case FieldType::FLOAT:
+                if (!is_numeric($value)) {
+                    $errors[] = "Field '{$fieldName}' must be a numeric value";
+                } elseif (is_infinite((float)$value) || is_nan((float)$value)) {
+                    $errors[] = "Field '{$fieldName}' must be a finite numeric value";
+                }
+                break;
+
+            case FieldType::INTEGER:
+                if (!is_numeric($value)) {
+                    $errors[] = "Field '{$fieldName}' must be a numeric value";
+                } elseif (!is_int($value) && !ctype_digit((string)$value)) {
+                    $errors[] = "Field '{$fieldName}' must be an integer value";
+                } elseif (is_string($value) && (int)$value != $value) {
+                    $errors[] = "Field '{$fieldName}' must be an integer value";
+                }
+                break;
+
+            case FieldType::DOUBLE:
+                if (!is_numeric($value)) {
+                    $errors[] = "Field '{$fieldName}' must be a numeric value";
+                } elseif (is_infinite((float)$value) || is_nan((float)$value)) {
+                    $errors[] = "Field '{$fieldName}' must be a finite numeric value";
+                }
+                break;
+
             case FieldType::URL:
                 if (!is_string($value) || !filter_var($value, FILTER_VALIDATE_URL)) {
                     $errors[] = "Field '{$fieldName}' must be a valid URL";
@@ -115,6 +141,14 @@ private function validateField(string $fieldName, mixed $value, FieldConfig $fie
                     $errors = array_merge($errors, $this->validateVariants($fieldName, $value, $fieldConfig));
                 }
                 break;
+
+            case FieldType::NAME_VALUE_LIST:
+                if (!is_array($value)) {
+                    $errors[] = "Field '{$fieldName}' must be an array for name-value list type";
+                } else {
+                    $errors = array_merge($errors, $this->validateNameValueList($fieldName, $value));
+                }
+                break;
         }
 
         return $errors;
@@ -245,4 +279,36 @@ private function isValidImageUrl(string $url): bool
 
         return in_array($extension, $imageExtensions, true);
     }
+
+    /**
+     * Validate name-value list structure
+     *
+     * @return array<string>
+     */
+    private function validateNameValueList(string $fieldName, array $nameValueList): array
+    {
+        $errors = [];
+
+        foreach ($nameValueList as $index => $item) {
+            if (!is_array($item)) {
+                $errors[] = "Field '{$fieldName}' item at index {$index} must be an object with 'name' and 'value' fields";
+                continue;
+            }
+
+            // Validate required fields
+            if (!isset($item['name'])) {
+                $errors[] = "Field '{$fieldName}' item at index {$index} must have a 'name' field";
+            } elseif (!is_string($item['name']) || empty(trim($item['name']))) {
+                $errors[] = "Field '{$fieldName}' item at index {$index} 'name' must be a non-empty string";
+            }
+
+            if (!isset($item['value'])) {
+                $errors[] = "Field '{$fieldName}' item at index {$index} must have a 'value' field";
+            } elseif (!is_string($item['value'])) {
+                $errors[] = "Field '{$fieldName}' item at index {$index} 'value' must be a string";
+            }
+        }
+
+        return $errors;
+    }
 }