Merge branch 'develop' into trunk

app/Http/Controllers/PlanningController.php

@@ -6,7 +6,9 @@ use App\Http\Requests\ImportPlanningXlsFile;
 use App\Imports\PlanningImport;
 use App\Models\Planning;
 use App\Models\Site;
+use Exception;
 use Illuminate\Http\UploadedFile;
+use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Facades\View;
 use Kwaadpepper\CrudPolicies\Traits\CrudController;
 use Maatwebsite\Excel\Facades\Excel;
@@ -84,15 +86,22 @@ class PlanningController extends Controller
                 // phpcs:ignore Generic.Files.LineLength.TooLong
                 'Echec lors de l\'import le fichier doit être conforme à celui fourni lors du developement de l\'application'
             );
+        } catch (Exception $e) {
+            Log::error($e->getMessage());
         } finally {
-            return redirect()->route('bo.plannings.import')
-            ->with('success', sprintf(
-                "%d plannings ont été importés \n Les plannings suivant n'ont pu être importés \n %s",
-                $planningImport->importedCount,
-                collect($planningImport->importedFailed)->map(function ($d) {
-                    return $d->implode(',');
-                })->implode("\n")
-            ));
+            $r = redirect()->route('bo.plannings.import');
+            if ($planningImport) {
+                $r = $r->with('success', sprintf(
+                    "%d plannings ont été importés \n Les plannings suivant n'ont pu être importés \n %s",
+                    $planningImport->importedCount,
+                    collect($planningImport->importedFailed)->map(function ($d) {
+                        return $d->implode(',');
+                    })->implode("\n")
+                ));
+            } else {
+                $r = $r->with('error', __('Aucun planning importé, erreur inconnue'));
+            }
+            return $r;
         }
     }