Merge branch 'release/0.0.4'

0.0.3

Jenkinsfile

@@ -1 +1 @@
-apiPipeline(name: 'dites-le-en-chanson-api')
+apiPipeline(name: 'dites-le-en-chanson-api', phpVersion: '8.3')

src/app/Http/Controllers/Api/OrderConfirmationController.php

@@ -31,6 +31,7 @@ class OrderConfirmationController extends Controller
         $validator = Validator::make($request->all(), [
             'orderData' => 'required|array',
             'sessionId' => 'nullable|string',
+            'orderId' => 'nullable|string',
         ]);
 
         if ($validator->fails()) {
@@ -41,6 +42,7 @@ class OrderConfirmationController extends Controller
 
         $orderData = $request->input('orderData');
         $sessionId = $request->input('sessionId');
+        $orderId = $request->input('orderId');
 
         // 1. Récupérer l'email vérifié depuis la session Stripe
         $customerEmail = null;
@@ -57,7 +59,20 @@ class OrderConfirmationController extends Controller
             $customerEmail = $orderData['email'] ?? $orderData['customer_email'] ?? null;
         }
 
-        // 2. Enregistrer la commande dans la base de données
+        // 2. Chercher une commande existante ou en créer une nouvelle
+        $order = null;
+        
+        // Chercher d'abord par orderId si fourni
+        if ($orderId) {
+            $order = Order::find($orderId);
+        }
+        
+        // Sinon, chercher par sessionId
+        if (!$order && $sessionId) {
+            $order = Order::where('session_id', $sessionId)->first();
+        }
+
+        // Préparer les données de mise à jour
         $finalOrderData = [
             ...$orderData,
             'customer_email' => $customerEmail,
@@ -66,7 +81,13 @@ class OrderConfirmationController extends Controller
         ];
 
         try {
+            if ($order) {
+                // Mettre à jour la commande existante
+                $order->update($finalOrderData);
+            } else {
+                // Créer une nouvelle commande si aucune n'existe
                 $order = Order::create($finalOrderData);
+            }
         } catch (\Exception $e) {
             \Log::error('Error saving order: ' . $e->getMessage());
             $order = null;

src/app/Http/Controllers/Api/StripeController.php

@@ -30,7 +30,13 @@ class StripeController extends Controller
      */
     public function createCheckoutSession(Request $request)
     {
-        $validator = Validator::make($request->all(), [
+        // Normaliser les chaînes vides en null pour customerEmail
+        $requestData = $request->all();
+        if (isset($requestData['customerEmail']) && $requestData['customerEmail'] === '') {
+            $requestData['customerEmail'] = null;
+        }
+        
+        $validator = Validator::make($requestData, [
             'priceId' => 'required|string',
             'orderData' => 'nullable|array',
             'successUrl' => 'required|url',
@@ -47,12 +53,12 @@ class StripeController extends Controller
 
         try {
             // 1. Sauvegarder la commande dans la base de données
-            $orderData = $request->input('orderData', []);
+            $orderData = $requestData['orderData'] ?? [];
             
             // Si orderData n'est pas fourni, créer un objet minimal
             if (empty($orderData)) {
                 $orderData = [
-                    'customer_email' => $request->input('customerEmail'),
+                    'customer_email' => $requestData['customerEmail'] ?? null,
                 ];
             }
             
@@ -62,23 +68,44 @@ class StripeController extends Controller
             ]);
 
             // 2. Créer la session Stripe
-            $session = $this->stripe->checkout->sessions->create([
+            $successUrl = $requestData['successUrl'];
+            // Ajouter les paramètres à l'URL (utiliser ? ou & selon si l'URL contient déjà des paramètres)
+            $separator = strpos($successUrl, '?') !== false ? '&' : '?';
+            $successUrlWithParams = $successUrl . $separator . 'session_id={CHECKOUT_SESSION_ID}&order_id=' . $order->id;
+            
+            // Récupérer l'email (peut être dans customerEmail, orderData.email ou orderData.customer_email)
+            $customerEmail = $requestData['customerEmail'] ?? $orderData['email'] ?? $orderData['customer_email'] ?? null;
+            
+            // Ne passer customer_email à Stripe que si l'email est valide et non vide
+            // Sinon, Stripe demandera l'email au client lors du checkout
+            $validEmail = null;
+            if ($customerEmail && is_string($customerEmail) && trim($customerEmail) !== '' && filter_var(trim($customerEmail), FILTER_VALIDATE_EMAIL)) {
+                $validEmail = trim($customerEmail);
+            }
+            
+            $sessionParams = [
                 'payment_method_types' => ['card'],
                 'line_items' => [
                     [
-                        'price' => $request->input('priceId'),
+                        'price' => $requestData['priceId'],
                         'quantity' => 1,
                     ],
                 ],
                 'mode' => 'payment',
-                'success_url' => $request->input('successUrl') . '&session_id={CHECKOUT_SESSION_ID}&order_id=' . $order->id,
+                'success_url' => $successUrlWithParams,
-                'cancel_url' => $request->input('cancelUrl'),
+                'cancel_url' => $requestData['cancelUrl'],
                 'metadata' => [
                     'order_id' => $order->id,
                     'product_name' => $orderData['product_name'] ?? '',
                 ],
-                'customer_email' => $request->input('customerEmail') ?? $orderData['email'] ?? $orderData['customer_email'] ?? null,
+            ];
-            ]);
+            
+            // Ajouter customer_email seulement si valide
+            if ($validEmail) {
+                $sessionParams['customer_email'] = $validEmail;
+            }
+            
+            $session = $this->stripe->checkout->sessions->create($sessionParams);
 
             return response()->json([
                 'sessionId' => $session->id,

src/composer.json

@@ -2,6 +2,7 @@
     "$schema": "https://getcomposer.org/schema.json",
     "name": "laravel/laravel",
     "type": "project",
+    "version": "0.0.4",
     "description": "The skeleton application for the Laravel framework.",
     "keywords": ["laravel", "framework"],
     "license": "MIT",