Enhance product creation process in ProductController with improved handling for wholesale prices, variations, and image management

This commit refines the product creation logic in the ProductController by adding detailed handling for wholesale prices, including the use of polymorphic relationships for better data management. It also improves the management of product variations and images, ensuring that attributes are correctly processed and associated with their respective variations. Additionally, logging has been enhanced for better traceability during the product creation process.

Author: Level237

app/Http/Controllers/Seller/ProductController.php

@@ -53,14 +53,14 @@ public function store(Request $request)
             Log::info('Product store request received', ['request_all' => $request->all()]);
             Log::info('Files in request', ['request_files' => $request->files->all()]);
 
-
             $user = Auth::guard('api')->user();
             $shop = Shop::where('user_id', $user->id)->first();
 
             if (!$shop) {
                 throw new \Exception("Shop not found for the authenticated user.");
             }
 
+            // Mise à jour du niveau du shop si c'est le premier produit
             if ($shop->products()->count() === 0) {
                 $shop->shop_level = "3";
                 $shop->save();
@@ -70,50 +70,53 @@ public function store(Request $request)
             $product->product_name = $request->product_name;
             $product->product_description = $request->product_description;
             $product->shop_id = $shop->id;
-            $product->type = $request->type == 'simple' ? 0 : 1;
+            $product->type = $request->type == 'simple' ? 0 : 1; // 0 pour simple, 1 pour variable
             $product->product_gender = $request->product_gender;
             $product->whatsapp_number = $request->whatsapp_number;
             $product->product_residence = $request->product_residence;
-            $product->status = 0; // Default status
-
+            $product->status = 0; // Statut par défaut
 
-            if ($product->type == 0) { // Simple product
+            // Gestion du produit simple
+            if ($product->type == 0) {
                 $product->product_price = $request->product_price;
                 $product->product_quantity = $request->product_quantity;
             }
 
+            // Gestion de l'image principale
             if ($request->hasFile('product_profile')) {
                 $product->product_profile = $request->file('product_profile')->store('product/profile', 'public');
                 Log::info('Product profile image stored', ['path' => $product->product_profile]);
             }
 
-            $product->save();
+            $product->save(); // Sauvegarde du produit pour obtenir l'ID
             Log::info('Product saved', ['product_id' => $product->id]);
 
             GenerateProductUrlJob::dispatch($product->id);
 
-
+            // Gestion des prix de gros au niveau du produit (pour les produits simples ou les variations 'couleur uniquement' avec prix de gros global)
             if ($request->is_wholesale == "1") {
                 $product->is_wholesale = true;
                 if ($request->is_only_wholesale == "1") {
                     $product->is_only_wholesale = true;
                 }
-                $product->save();
+                $product->save(); // Sauvegarde pour mettre à jour is_wholesale et is_only_wholesale
+
                 if ($request->has('wholesale_prices')) {
-                    $wholesalePrices = json_decode($request->wholesale_prices);
-                    if ($wholesalePrices) {
-                        foreach ($wholesalePrices as $wholeSalePrice) {
-                            $newWholeSalePrice = new WholeSalePrice;
-                            $newWholeSalePrice->min_quantity = $wholeSalePrice->min_quantity;
-                            $newWholeSalePrice->wholesale_price = $wholeSalePrice->wholesale_price;
-                            $newWholeSalePrice->product_id = $product->id;
-                            $newWholeSalePrice->save();
-                            Log::info('Product wholesale price saved', ['price_id' => $newWholeSalePrice->id]);
+                    $wholesalePricesData = json_decode($request->wholesale_prices, true);
+                    if ($wholesalePricesData) {
+                        foreach ($wholesalePricesData as $wpData) {
+                            // Lie les prix de gros au produit directement via la relation polymorphique
+                            $product->wholesalePrices()->create([
+                                'min_quantity' => $wpData['min_quantity'],
+                                'wholesale_price' => $wpData['wholesale_price'],
+                            ]);
+                            Log::info('Product global wholesale price saved', ['product_id' => $product->id, 'min_quantity' => $wpData['min_quantity']]);
                         }
                     }
                 }
             }
 
+            // Gestion des images du produit simple (non liées aux variations)
             if ($request->hasFile('images')) {
                 foreach ($request->file('images') as $image) {
                     $imagePath = $image->store('product/images', 'public');
@@ -122,45 +125,62 @@ public function store(Request $request)
                 }
             }
 
+            // Gestion des variations pour produit variable
             if ($product->type == 1 && $request->filled('variations')) {
                 $variationsData = json_decode($request->variations, true);
                 Log::info('Decoded variations data', ['variations' => $variationsData]);
 
                 foreach ($variationsData as $colorGroup) {
+                    // Vérifie si c'est une variation "couleur uniquement" ou "couleur et attribut"
+                    // En se basant sur la présence des tableaux 'sizes' ou 'shoeSizes'
+                    $isColorAndAttribute = (isset($colorGroup['sizes']) && is_array($colorGroup['sizes']) && count($colorGroup['sizes']) > 0) ||
+                                           (isset($colorGroup['shoeSizes']) && is_array($colorGroup['shoeSizes']) && count($colorGroup['shoeSizes']) > 0);
+
                     // Création de la variation principale (couleur)
                     $variation = $product->variations()->create([
                         'color_id' => $colorGroup['color']['id'],
-                        'price' => $colorGroup['price'] ?? 0,
-                        'quantity' => $colorGroup['quantity'] ?? null,
+                        // Pour 'Couleur uniquement', le prix et la quantité sont directement sur la variation de couleur
+                        'price' => !$isColorAndAttribute && isset($colorGroup['price']) ? $colorGroup['price'] : 0,
+                        'quantity' => !$isColorAndAttribute && isset($colorGroup['quantity']) ? $colorGroup['quantity'] : null,
                     ]);
                     Log::info('ProductVariation (color) created', ['variation_id' => $variation->id, 'color_id' => $colorGroup['color']['id']]);
 
-                    // Gestion des images pour cette couleur
+                    // Gestion des images pour cette couleur (envoyées sous 'color_{id}_image_{index}')
                     $colorImageKeyPrefix = "color_" . $colorGroup['color']['id'] . "_image_";
                     $imageIndex = 0;
                     while ($request->hasFile($colorImageKeyPrefix . $imageIndex)) {
                         $imageFile = $request->file($colorImageKeyPrefix . $imageIndex);
                         $imagePath = $imageFile->store('product/variations', 'public');
-                        $variation->images()->create(['image_path' => $imagePath]);
+                        $variation->images()->create(['image_path' => $imagePath]); // Assurez-vous que ProductVariation a une relation images() morphMany
                         Log::info('Variation image stored', ['color_id' => $colorGroup['color']['id'], 'image_index' => $imageIndex, 'path' => $imagePath]);
                         $imageIndex++;
                     }
 
-                    // Gestion des sous-variations (tailles/pointures)
-                    if (isset($colorGroup['sizes']) && is_array($colorGroup['sizes'])) {
-                        Log::info('Processing sizes for color group', ['color_id' => $colorGroup['color']['id'], 'sizes_data' => $colorGroup['sizes']]);
-                        foreach ($colorGroup['sizes'] as $attributeValue) {
-                            if (!$variation->attributesVariation()->where('attribute_value_id', $attributeValue['id'])->exists()) {
-                                $attrVariation = $variation->attributesVariation()->create([
-                                    'attribute_value_id' => $attributeValue['id'],
-                                    'quantity' => $attributeValue['quantity'],
-                                    'price' => $attributeValue['price'],
-                                ]);
-                                Log::info('ProductAttributeVariation (size) created', ['id' => $attrVariation->id, 'attribute_value_id' => $attributeValue['id']]);
-
-                                if (isset($attributeValue['is_wholesale']) && $attributeValue['is_wholesale'] && isset($attributeValue['wholesale_prices']) && is_array($attributeValue['wholesale_prices'])) {
+                    // Gestion des sous-variations (tailles/pointures/autres attributs)
+                    if ($isColorAndAttribute) {
+                        // Traitement des tailles
+                        if (isset($colorGroup['sizes']) && is_array($colorGroup['sizes'])) {
+                            Log::info('Processing sizes for color group', ['color_id' => $colorGroup['color']['id'], 'sizes_data' => $colorGroup['sizes']]);
+                            foreach ($colorGroup['sizes'] as $attributeValue) {
+                                // Crée ou met à jour la VariationAttribute
+                                $attrVariation = $variation->attributesVariation()->updateOrCreate(
+                                    ['attribute_value_id' => $attributeValue['id']],
+                                    [
+                                        'quantity' => $attributeValue['quantity'],
+                                        'price' => $attributeValue['price'],
+                                    ]
+                                );
+                                Log::info('VariationAttribute (size) processed', ['id' => $attrVariation->id, 'attribute_value_id' => $attributeValue['id']]);
+
+                                // Gestion des prix de gros pour cette VariationAttribute spécifique
+                                if (isset($attributeValue['is_wholesale']) && $attributeValue['is_wholesale'] &&
+                                    isset($attributeValue['wholesale_prices']) && is_array($attributeValue['wholesale_prices'])) {
+
+                                    // Supprime les anciens prix de gros pour éviter les doublons si vous utilisez updateOrCreate
+                                    $attrVariation->wholesalePrices()->delete();
+
                                     foreach ($attributeValue['wholesale_prices'] as $wholesalePriceData) {
-                                        $attrVariation->wholesalePrices()->create([
+                                        $attrVariation->wholesalePrices()->create([ // Lie les prix de gros à VariationAttribute
                                             'min_quantity' => $wholesalePriceData['min_quantity'],
                                             'wholesale_price' => $wholesalePriceData['wholesale_price'],
                                         ]);
@@ -169,22 +189,30 @@ public function store(Request $request)
                                 }
                             }
                         }
-                    }
 
-                    if (isset($colorGroup['shoeSizes']) && is_array($colorGroup['shoeSizes'])) {
-                        Log::info('Processing shoe sizes for color group', ['color_id' => $colorGroup['color']['id'], 'shoe_sizes_data' => $colorGroup['shoeSizes']]);
-                        foreach ($colorGroup['shoeSizes'] as $attributeValue) {
-                            if (!$variation->attributesVariation()->where('attribute_value_id', $attributeValue['id'])->exists()) {
-                                $attrVariation = $variation->attributesVariation()->create([
-                                    'attribute_value_id' => $attributeValue['id'],
-                                    'quantity' => $attributeValue['quantity'],
-                                    'price' => $attributeValue['price'],
-                                ]);
-                                Log::info('ProductAttributeVariation (shoe size) created', ['id' => $attrVariation->id, 'attribute_value_id' => $attributeValue['id']]);
-
-                                if (isset($attributeValue['is_wholesale']) && $attributeValue['is_wholesale'] && isset($attributeValue['wholesale_prices']) && is_array($attributeValue['wholesale_prices'])) {
+                        // Traitement des pointures
+                        if (isset($colorGroup['shoeSizes']) && is_array($colorGroup['shoeSizes'])) {
+                            Log::info('Processing shoe sizes for color group', ['color_id' => $colorGroup['color']['id'], 'shoe_sizes_data' => $colorGroup['shoeSizes']]);
+                            foreach ($colorGroup['shoeSizes'] as $attributeValue) {
+                                // Crée ou met à jour la VariationAttribute
+                                $attrVariation = $variation->attributesVariation()->updateOrCreate(
+                                    ['attribute_value_id' => $attributeValue['id']],
+                                    [
+                                        'quantity' => $attributeValue['quantity'],
+                                        'price' => $attributeValue['price'],
+                                    ]
+                                );
+                                Log::info('VariationAttribute (shoe size) processed', ['id' => $attrVariation->id, 'attribute_value_id' => $attributeValue['id']]);
+
+                                // Gestion des prix de gros pour cette VariationAttribute spécifique
+                                if (isset($attributeValue['is_wholesale']) && $attributeValue['is_wholesale'] &&
+                                    isset($attributeValue['wholesale_prices']) && is_array($attributeValue['wholesale_prices'])) {
+
+                                    // Supprime les anciens prix de gros pour éviter les doublons si vous utilisez updateOrCreate
+                                    $attrVariation->wholesalePrices()->delete();
+
                                     foreach ($attributeValue['wholesale_prices'] as $wholesalePriceData) {
-                                        $attrVariation->wholesalePrices()->create([
+                                        $attrVariation->wholesalePrices()->create([ // Lie les prix de gros à VariationAttribute
                                             'min_quantity' => $wholesalePriceData['min_quantity'],
                                             'wholesale_price' => $wholesalePriceData['wholesale_price'],
                                         ]);
@@ -197,6 +225,7 @@ public function store(Request $request)
                 }
             }
 
+            // Gestion des catégories et sous-catégories
             if ($request->has('categories') && is_array($request->categories)) {
                 $product->categories()->attach(array_map('intval', $request->categories));
                 Log::info('Categories attached', ['categories' => $request->categories]);
@@ -217,8 +246,8 @@ public function store(Request $request)
             Log::error('Product creation failed', [
                 'error' => $e->getMessage(),
                 'trace' => $e->getTraceAsString(),
-                'request_all' => $request->all(), // Log all request data on failure
-                'request_files' => $request->files->all(), // Log all file data on failure
+                'request_all' => $request->all(), // Log toutes les données de la requête en cas d'échec
+                'request_files' => $request->files->all(), // Log tous les fichiers en cas d'échec
             ]);
             return response()->json([
                 'success' => false,

database/migrations/2025_09_05_182104_create_whole_sale_prices_table.php

@@ -22,7 +22,7 @@ public function up(): void
 
             $table->timestamps();
 
-            $table->index(['priceable_id', 'priceable_type']);
+            
         });
     }