wip

src/Controller/ProductSkusController.php

@@ -96,8 +96,25 @@ class ProductSkusController extends AppController
         if ($this->request->is('post')) {
             $postedSkus = $this->request->getData();
             $saveOptions = [
+                'fields' => [
+                    'product_id',
+                    'sku',
+                    'barcode',
+                    'price',
+                    'cost',
+                    'product_sku_variant_values',
+                    'created',
+                    'modified',
+                    'enabled',
+                ],
                 'associated' => [
-                    'ProductSkuVariantValues',
+                    'ProductSkuVariantValues' => [
+                        'validate' => false,
+                        'fields' => [
+                            'product_category_variant_id',
+                            'product_category_variant_option_id',
+                        ],
+                    ],
                 ],
             ];
             $finalPostData = [];
@@ -117,11 +134,12 @@ class ProductSkusController extends AppController
                 return;
             }
 
-
             $productSkus = $table->patchEntities($productSkus, $finalPostData, $saveOptions);
             $errors = [];
             $successes = [];
             foreach ($productSkus as $productSkuToSave) {
+//                dd($productSkuToSave);
+
                 if (!$table->save($productSkuToSave, $saveOptions)) {
                     Log::debug(print_r('$productSkuToSave->getErrors()', true));
                     Log::debug(print_r($productSkuToSave->getErrors(), true));

src/Model/Entity/ProductSku.php

@@ -45,6 +45,6 @@ class ProductSku extends Entity
 
         // entities
         'product' => false,
-        'product_sku_variant_values' => false,
+        'product_sku_variant_values' => true,
     ];
 }

src/Model/Table/ProductSkuVariantValuesTable.php

@@ -101,8 +101,10 @@ class ProductSkuVariantValuesTable extends Table
     public function buildRules(RulesChecker $rules): RulesChecker
     {
         $rules->add($rules->existsIn(['product_sku_id'], 'ProductSkus'), ['errorField' => 'product_sku_id']);
-        $rules->add($rules->existsIn(['product_category_variant_id'], 'ProductCategoryVariants'), ['errorField' => 'product_category_variant_id']);
-        $rules->add($rules->existsIn(['product_category_variant_option_id'], 'ProductCategoryVariantOptions'), ['errorField' => 'product_category_variant_option_id']);
+
+        // @TODO why not working?? causing tests to fail / associated variant values not saving on product-skus/add
+//        $rules->add($rules->existsIn(['product_category_variant_id'], 'ProductCategoryVariants'), ['errorField' => 'product_category_variant_id']);
+//        $rules->add($rules->existsIn(['product_category_variant_option_id'], 'ProductCategoryVariantOptions'), ['errorField' => 'product_category_variant_option_id']);
 
         return $rules;
     }

tests/TestCase/Controller/ProductSkusControllerTest.php

@@ -8,6 +8,7 @@ use Cake\TestSuite\IntegrationTestTrait;
 use Cake\TestSuite\TestCase;
 use CakeProducts\Controller\ProductSkusController;
 use CakeProducts\Model\Table\ProductSkusTable;
+use CakeProducts\Model\Table\ProductSkuVariantValuesTable;
 use CakeProducts\Model\Table\ProductsTable;
 use PHPUnit\Exception;
 
@@ -25,6 +26,13 @@ class ProductSkusControllerTest extends BaseControllerTest
      */
     protected $ProductSkus;
 
+    /**
+     * Test subject table
+     *
+     * @var ProductSkuVariantValuesTable|Table
+     */
+    protected $ProductSkuVariantValues;
+
     /**
      * Fixtures
      *
@@ -50,6 +58,9 @@ class ProductSkusControllerTest extends BaseControllerTest
 
         $config = $this->getTableLocator()->exists('ProductSkus') ? [] : ['className' => ProductSkusTable::class];
         $this->ProductSkus = $this->getTableLocator()->get('ProductSkus', $config);
+
+        $config = $this->getTableLocator()->exists('ProductSkuVariantValues') ? [] : ['className' => ProductSkuVariantValuesTable::class];
+        $this->ProductSkuVariantValues = $this->getTableLocator()->get('ProductSkuVariantValues', $config);
     }
 
     /**
@@ -151,6 +162,7 @@ class ProductSkusControllerTest extends BaseControllerTest
     public function testAddPostSuccess(): void
     {
         $cntBefore = $this->ProductSkus->find()->count();
+        $cntVariantValuesBefore = $this->ProductSkuVariantValues->find()->count();
 
         $this->loginUserByRole('admin');
         $url = [
@@ -162,10 +174,16 @@ class ProductSkusControllerTest extends BaseControllerTest
         $data = [
             0 => [
                 'add' => '1',
-                'sku' => 'cfc98a9a-29b2-44c8-b587-8156adc05317',
-                'barcode' => 'cfc98a9a-29b2-44c8-b587-8156adc05317',
+                'sku' => 'cfc98a9a-29b2-44c8-b587-8156a',
+                'barcode' => 'cfc98a9a-29b2-44c8-b587-8156a',
                 'price' => 1.5,
                 'cost' => 1.5,
+                'product_sku_variant_values' => [
+                    0 => [
+                        'product_category_variant_id' => '5a386e9f-6e7a-4ae7-9360-c8e529f78d94',
+                        'product_category_variant_option_id' => '5a386e9f-6e7a-4ae7-9360-c8e529f78d26',
+                    ],
+                ],
             ],
         ];
         $this->post($url, $data);
@@ -173,7 +191,9 @@ class ProductSkusControllerTest extends BaseControllerTest
         $this->assertRedirectContains('product-skus');
 
         $cntAfter = $this->ProductSkus->find()->count();
+        $cntVariantValuesAfter = $this->ProductSkuVariantValues->find()->count();
         $this->assertEquals($cntBefore + 1, $cntAfter);
+        $this->assertEquals($cntVariantValuesBefore + 1, $cntVariantValuesAfter);
     }
 
     /**