chore: remove debug dump statements from views

Remove @dump($errors) debug statements across multiple view files to clean up codebase before production deployment

app/Http/Controllers/OrderController.php

@@ -33,6 +33,7 @@ use Illuminate\Http\Request;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Storage;
 use Symfony\Component\HttpFoundation\StreamedResponse;
+use Throwable;
 use ZipArchive;
 
 class OrderController extends Controller
@@ -468,13 +469,20 @@ class OrderController extends Controller
         $data = $request->validate([
             'photo.*' => 'mimes:jpeg,jpg,png',
         ]);
-        $f = [];
-        foreach ($data['photo'] as $photo) {
-            $f[] = $fileService->saveUploadedFile( 'orders/' . $order->id . '/photo', $photo);
+
+        try {
+            $f = [];
+            foreach ($data['photo'] as $photo) {
+                $f[] = $fileService->saveUploadedFile('orders/' . $order->id . '/photo', $photo);
+            }
+            $order->photos()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('order.show', $order)
+                ->with(['error' => 'Ошибка загрузки фотографий. Проверьте имя файла и повторите попытку.']);
         }
-        $order->photos()->syncWithoutDetaching($f);
 
-        return redirect()->route('order.show', $order);
+        return redirect()->route('order.show', $order)->with(['success' => 'Фотографии успешно загружены!']);
     }
 
     public function deletePhoto(Order $order, File $file, FileService $fileService)
@@ -499,17 +507,24 @@ class OrderController extends Controller
     public function uploadDocument(Request $request, Order $order, FileService $fileService)
     {
         $data = $request->validate([
-            'document.*'    => 'file',
+            'document.*' => 'file',
         ]);
-        $f = [];
-        $i = 0;
-        foreach ($data['document'] as $document) {
-            if($i++ >= 5) break;
-            $f[] = $fileService->saveUploadedFile( 'orders/' . $order->id . '/document', $document);
+
+        try {
+            $f = [];
+            $i = 0;
+            foreach ($data['document'] as $document) {
+                if ($i++ >= 5) break;
+                $f[] = $fileService->saveUploadedFile('orders/' . $order->id . '/document', $document);
+            }
+            $order->documents()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('order.show', $order)
+                ->with(['error' => 'Ошибка загрузки документов. Проверьте имя файла и повторите попытку.']);
         }
-        $order->documents()->syncWithoutDetaching($f);
 
-        return redirect()->route('order.show', $order);
+        return redirect()->route('order.show', $order)->with(['success' => 'Документы успешно загружены!']);
     }
 
     public function deleteDocument(Order $order, File $file)
@@ -534,17 +549,24 @@ class OrderController extends Controller
     public function uploadStatement(Request $request, Order $order, FileService $fileService)
     {
         $data = $request->validate([
-            'statement.*'   => 'file',
+            'statement.*' => 'file',
         ]);
-        $f = [];
-        $i = 0;
-        foreach ($data['statement'] as $statement) {
-            if($i++ >= 5) break;
-            $f[] = $fileService->saveUploadedFile( 'orders/' . $order->id . '/statement', $statement);
+
+        try {
+            $f = [];
+            $i = 0;
+            foreach ($data['statement'] as $statement) {
+                if ($i++ >= 5) break;
+                $f[] = $fileService->saveUploadedFile('orders/' . $order->id . '/statement', $statement);
+            }
+            $order->statements()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('order.show', $order)
+                ->with(['error' => 'Ошибка загрузки ведомостей. Проверьте имя файла и повторите попытку.']);
         }
-        $order->statements()->syncWithoutDetaching($f);
 
-        return redirect()->route('order.show', $order);
+        return redirect()->route('order.show', $order)->with(['success' => 'Ведомости успешно загружены!']);
     }
 
     public function deleteStatement(Order $order, File $file)

app/Http/Controllers/ProductController.php

@@ -11,6 +11,7 @@ use Illuminate\Http\RedirectResponse;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Facades\Storage;
+use Throwable;
 
 class ProductController extends Controller
 {
@@ -176,9 +177,17 @@ class ProductController extends Controller
             'certificate' => 'file',
         ]);
 
-        $f = $fileService->saveUploadedFile('products/' . $product->id . '/certificate', $data['certificate']);
-        $product->update(['certificate_id' => $f->id]);
-        return redirect()->route('catalog.show', ['product' => $product, 'previous_url' => $request->get('previous_url')]);
+        try {
+            $f = $fileService->saveUploadedFile('products/' . $product->id . '/certificate', $data['certificate']);
+            $product->update(['certificate_id' => $f->id]);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('catalog.show', ['product' => $product, 'previous_url' => $request->get('previous_url')])
+                ->with(['error' => 'Ошибка загрузки сертификата. Проверьте имя файла и повторите попытку.']);
+        }
+
+        return redirect()->route('catalog.show', ['product' => $product, 'previous_url' => $request->get('previous_url')])
+            ->with(['success' => 'Сертификат успешно загружен!']);
     }
 
     public function deleteCertificate(Request $request, Product $product, File $file)

app/Http/Controllers/ProductSKUController.php

@@ -12,6 +12,7 @@ use App\Services\FileService;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Storage;
 use Illuminate\Support\Str;
+use Throwable;
 
 // Склад - МАФ
 class ProductSKUController extends Controller
@@ -106,8 +107,21 @@ class ProductSKUController extends Controller
             'passport' => 'file',
         ]);
 
-        $f = $fileService->saveUploadedFile('maf/' . $product_sku->id . '/passport', $data['passport']);
-        $product_sku->update(['passport_id' => $f->id]);
+        try {
+            $f = $fileService->saveUploadedFile('maf/' . $product_sku->id . '/passport', $data['passport']);
+            $product_sku->update(['passport_id' => $f->id]);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('product_sku.show', [
+                'product_sku' => $product_sku,
+                'previous_url' => $this->previousUrlForRedirect(
+                    $request,
+                    'previous_url_product_sku',
+                    route('product_sku.index', session('gp_product_sku'))
+                ),
+            ])->with(['error' => 'Ошибка загрузки паспорта. Проверьте имя файла и повторите попытку.']);
+        }
+
         return redirect()->route('product_sku.show', [
             'product_sku' => $product_sku,
             'previous_url' => $this->previousUrlForRedirect(
@@ -115,7 +129,7 @@ class ProductSKUController extends Controller
                 'previous_url_product_sku',
                 route('product_sku.index', session('gp_product_sku'))
             ),
-        ]);
+        ])->with(['success' => 'Паспорт успешно загружен!']);
     }
 
     public function deletePassport(ProductSKU $product_sku, File $file)

app/Http/Controllers/ReclamationController.php

@@ -23,6 +23,7 @@ use App\Services\SparePartReservationService;
 use Illuminate\Http\Request;
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Facades\Storage;
+use Throwable;
 
 class ReclamationController extends Controller
 {
@@ -173,13 +174,21 @@ class ReclamationController extends Controller
         $data = $request->validate([
             'photo.*' => 'mimes:jpeg,jpg,png|max:8192',
         ]);
-        $f = [];
-        foreach ($data['photo'] as $photo) {
-            $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/photo_before', $photo);
+
+        try {
+            $f = [];
+            foreach ($data['photo'] as $photo) {
+                $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/photo_before', $photo);
+            }
+            $reclamation->photos_before()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+                ->with(['error' => 'Ошибка загрузки фотографий проблемы. Проверьте имя файла и повторите попытку.']);
         }
-        $reclamation->photos_before()->syncWithoutDetaching($f);
 
-        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')]);
+        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+            ->with(['success' => 'Фотографии проблемы успешно загружены!']);
     }
 
     public function uploadPhotoAfter(Request $request, Reclamation $reclamation, FileService $fileService)
@@ -187,13 +196,21 @@ class ReclamationController extends Controller
         $data = $request->validate([
             'photo.*' => 'mimes:jpeg,jpg,png|max:8192',
         ]);
-        $f = [];
-        foreach ($data['photo'] as $photo) {
-            $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/photo_after', $photo);
+
+        try {
+            $f = [];
+            foreach ($data['photo'] as $photo) {
+                $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/photo_after', $photo);
+            }
+            $reclamation->photos_after()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+                ->with(['error' => 'Ошибка загрузки фотографий после устранения. Проверьте имя файла и повторите попытку.']);
         }
-        $reclamation->photos_after()->syncWithoutDetaching($f);
 
-        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')]);
+        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+            ->with(['success' => 'Фотографии после устранения успешно загружены!']);
     }
 
     public function deletePhotoBefore(Request $request, Reclamation $reclamation, File $file, FileService $fileService)
@@ -217,15 +234,23 @@ class ReclamationController extends Controller
         $data = $request->validate([
             'document.*' => 'file',
         ]);
-        $f = [];
-        $i = 0;
-        foreach ($data['document'] as $document) {
-            if($i++ >= 5) break;
-            $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/document', $document);
+
+        try {
+            $f = [];
+            $i = 0;
+            foreach ($data['document'] as $document) {
+                if ($i++ >= 5) break;
+                $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/document', $document);
+            }
+            $reclamation->documents()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+                ->with(['error' => 'Ошибка загрузки документов рекламации. Проверьте имя файла и повторите попытку.']);
         }
-        $reclamation->documents()->syncWithoutDetaching($f);
 
-        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')]);
+        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+            ->with(['success' => 'Документы рекламации успешно загружены!']);
     }
 
     public function deleteDocument(Request $request, Reclamation $reclamation, File $file)
@@ -241,15 +266,23 @@ class ReclamationController extends Controller
         $data = $request->validate([
             'acts.*' => 'file',
         ]);
-        $f = [];
-        $i = 0;
-        foreach ($data['acts'] as $document) {
-            if($i++ >= 5) break;
-            $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/act', $document);
+
+        try {
+            $f = [];
+            $i = 0;
+            foreach ($data['acts'] as $document) {
+                if ($i++ >= 5) break;
+                $f[] = $fileService->saveUploadedFile('reclamations/' . $reclamation->id . '/act', $document);
+            }
+            $reclamation->acts()->syncWithoutDetaching($f);
+        } catch (Throwable $e) {
+            report($e);
+            return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+                ->with(['error' => 'Ошибка загрузки актов. Проверьте имя файла и повторите попытку.']);
         }
-        $reclamation->acts()->syncWithoutDetaching($f);
 
-        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')]);
+        return redirect()->route('reclamations.show', ['reclamation' => $reclamation, 'previous_url' => $request->get('previous_url')])
+            ->with(['success' => 'Акты успешно загружены!']);
     }
 
     public function deleteAct(Request $request, Reclamation $reclamation, File $file)

app/Services/FileService.php

@@ -7,6 +7,7 @@ use Exception;
 use Illuminate\Support\Facades\Storage;
 use Illuminate\Support\Str;
 use RecursiveIteratorIterator;
+use RuntimeException;
 use ZipArchive;
 
 class FileService
@@ -21,8 +22,11 @@ class FileService
         $originalName = $this->sanitizeOriginalName((string)$file->getClientOriginalName());
         $storedFilename = $this->buildStoredFilename($originalName);
         $relativePath = $this->buildUniquePath($path, $storedFilename);
-
-        Storage::disk('public')->put($relativePath, $file->getContent());
+        try {
+            Storage::disk('public')->put($relativePath, $file->getContent());
+        } catch (\Throwable $e) {
+            throw new RuntimeException('Не удалось сохранить файл. Проверьте имя файла и повторите попытку.', 0, $e);
+        }
 
         return File::query()->create([
             'link' => url('/storage/' . $relativePath),
@@ -36,7 +40,16 @@ class FileService
     private function sanitizeOriginalName(string $originalName): string
     {
         $name = basename(str_replace('\\', '/', $originalName));
-        $name = preg_replace('/[\x00-\x1F\x7F]+/u', '', $name) ?? '';
+        if (class_exists('\Normalizer')) {
+            $normalized = \Normalizer::normalize($name, \Normalizer::FORM_C);
+            if (is_string($normalized)) {
+                $name = $normalized;
+            }
+        }
+
+        // Remove control/format unicode symbols (e.g. zero-width, LRM/RLM), then normalize spaces.
+        $name = preg_replace('/\p{C}+/u', '', $name) ?? '';
+        $name = preg_replace('/\p{Z}+/u', ' ', $name) ?? '';
         $name = trim($name);
 
         return $name === '' ? 'file' : $name;

resources/views/admin/areas/index.blade.php

@@ -122,8 +122,4 @@
             </div>
         </div>
     </div>
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
 @endsection

resources/views/admin/districts/index.blade.php

@@ -103,8 +103,4 @@
             </div>
         </div>
     </div>
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
 @endsection

resources/views/catalog/edit.blade.php

@@ -96,7 +96,4 @@
             @endif
         </div>
     </div>
-    @if($errors->any())
-        @dump($errors)
-    @endif
 @endsection

resources/views/catalog/index.blade.php

@@ -21,8 +21,6 @@
             @endif
         </div>
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -33,8 +31,6 @@
 
     @include('partials.pagination', ['items' => $products])
 
-
-
     <!-- Модальное окно импорта-->
     <div class="modal fade" id="importModal" tabindex="-1" aria-labelledby="exampleModalLabel" aria-hidden="true">
         <div class="modal-dialog modal-fullscreen-sm-down">
@@ -83,8 +79,4 @@
             </div>
         </div>
     </div>
-
-    @if($errors->count())
-        @dump($errors)
-    @endif
 @endsection

resources/views/contracts/index.blade.php

@@ -9,8 +9,6 @@
             <a href="{{ route('contract.create') }}" class="btn btn-sm btn-primary">Добавить</a>
         </div>
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -20,11 +18,4 @@
     ])
 
     @include('partials.pagination', ['items' => $contracts])
-
-
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
-
 @endsection

resources/views/import/index.blade.php

@@ -13,8 +13,6 @@
             </div>
         @endif
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -23,8 +21,6 @@
     ])
 
     @include('partials.pagination', ['items' => $imports])
-
-
     <!-- Модальное окно импорта-->
     <div class="modal fade" id="importModal" tabindex="-1" aria-labelledby="exampleModalLabel" aria-hidden="true">
         <div class="modal-dialog modal-fullscreen-sm-down">
@@ -44,9 +40,4 @@
             </div>
         </div>
     </div>
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
-
 @endsection

resources/views/import/show.blade.php

@@ -30,14 +30,7 @@
                 @include('partials.input',  ['name' => 'updated_at', 'title' => 'Изменен', 'value' => $import->updated_at, 'disabled' => true])
 
                 @include('partials.textarea', ['name' => 'result', 'title' => 'Результат', 'value' => $import->result , 'disabled' => true, 'size' => 30])
-
-
             </div>
         </div>
     </div>
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
-
 @endsection

resources/views/layouts/app.blade.php

@@ -47,6 +47,17 @@
             @endforeach
         @endif
 
+        @if($message = session('error'))
+            @php
+                if(!is_array($message)) $message = [$message];
+            @endphp
+            @foreach($message as $m)
+                <div class="main-alert alert alert-danger" role="alert">
+                    {{ $m }}
+                </div>
+            @endforeach
+        @endif
+
         @if($message = session('warning'))
             @php
                 if(!is_array($message)) $message = [$message];
@@ -57,6 +68,14 @@
                 </div>
             @endforeach
         @endif
+
+        @if($errors->any())
+            @foreach($errors->all() as $m)
+                <div class="main-alert alert alert-danger" role="alert">
+                    {{ $m }}
+                </div>
+            @endforeach
+        @endif
     </div>
 
     <div id="app">

resources/views/maf_orders/index.blade.php

@@ -16,8 +16,6 @@
 
         </div>
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -26,8 +24,6 @@
     ])
 
     @include('partials.pagination', ['items' => $maf_orders])
-
-
     <!-- Модальное окно добавления-->
     <div class="modal fade" id="addModal" tabindex="-1" aria-labelledby="exampleModalLabel" aria-hidden="true">
         <div class="modal-dialog modal-fullscreen-sm-down modal-lg">
@@ -56,13 +52,7 @@
             </div>
         </div>
     </div>
-
-    @if($errors->count())
-        @dump($errors)
-    @endif
 @endsection
-
-
 @push('scripts')
     <script type="module">
         $('#select_maf').on('change', function () {

resources/views/orders/edit.blade.php

@@ -92,11 +92,6 @@
         </form>
 
     </div>
-
-    @if($errors->any())
-        @dump($errors->all())
-    @endif
-
 @endsection
 
 @push('scripts')

resources/views/orders/index.blade.php

@@ -20,8 +20,6 @@
             </form>
         @endif
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -31,9 +29,4 @@
     ])
 
     @include('partials.pagination', ['items' => $orders])
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
-
 @endsection

resources/views/orders/show.blade.php

@@ -424,14 +424,7 @@
             </div>
         </div>
     @endif
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
-
 @endsection
-
-
 @push('scripts')
     <script type="module">
         $('#tg').on('click', function () {
@@ -452,8 +445,6 @@
                 );
             }).trigger('keyup');
         @endif
-
-
         $('#check-all-maf').on('change', function () {
             $('input:checkbox.check-maf').not(this).prop('checked', this.checked);
         });

resources/views/products_sku/index.blade.php

@@ -20,8 +20,6 @@
             @endif
         </div>
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -31,8 +29,6 @@
     ])
 
     @include('partials.pagination', ['items' => $products_sku])
-
-
     <!-- Модальное окно импорта-->
     <div class="modal fade" id="importModal" tabindex="-1" aria-labelledby="exampleModalLabel" aria-hidden="true">
         <div class="modal-dialog modal-fullscreen-sm-down">
@@ -80,13 +76,7 @@
             </div>
         </div>
     </div>
-
-    @if($errors->count())
-        @dump($errors)
-    @endif
 @endsection
-
-
 @push('scripts')
     <script type="module">
         $('#select_maf').on('change', function () {

resources/views/reclamations/edit.blade.php

@@ -223,9 +223,6 @@
                                 <button class="btn btn-primary btn-sm" type="submit" @disabled(!hasRole('admin,manager'))>Сохранить запчасти</button>
                             </div>
                         </div>
-                        @if($errors->any())
-                            @dump($errors->all())
-                        @endif
                     </form>
 
                     {{-- Резервы и дефициты --}}

resources/views/reclamations/index.blade.php

@@ -29,9 +29,4 @@
     ])
 
     @include('partials.pagination', ['items' => $reclamations])
-
-
-    @if($errors->count())
-        @dump($errors)
-    @endif
 @endsection

resources/views/responsibles/index.blade.php

@@ -13,8 +13,6 @@
 
         </div>
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -24,8 +22,6 @@
     ])
 
     @include('partials.pagination', ['items' => $responsibles])
-
-
     <!-- Модальное окно добавления-->
     <div class="modal fade" id="addModal" tabindex="-1" aria-labelledby="exampleModalLabel" aria-hidden="true">
         <div class="modal-dialog modal-fullscreen-sm-down modal-lg">
@@ -50,13 +46,7 @@
             </div>
         </div>
     </div>
-
-    @if($errors->count())
-        @dump($errors)
-    @endif
 @endsection
-
-
 @push('scripts')
 {{--    <script type="module">--}}
 {{--        $('#select_maf').on('change', function () {--}}
@@ -64,8 +54,6 @@
 {{--            $('#product_name').val($('#select_maf option:selected').text()).slideDown();--}}
 {{--            $('#select_maf_form').slideUp();--}}
 {{--            $('#sku_form').slideDown();--}}
-
-
 {{--        });--}}
 {{--    </script>--}}
 @endpush

resources/views/schedule/index.blade.php

@@ -341,10 +341,6 @@
                 </div>
             </div>
         @endif
-
-        @if($errors->any())
-            @dump($errors)
-        @endif
     </div>
 @endsection
 
@@ -433,8 +429,6 @@
                 }
             );
         }).trigger('keyup');
-
-
         $('#district_id').on('change', function () {
             // load areas of selected district
             $.get('{{ route('area.ajax-get-areas-by-district') }}/' + $(this).val(),
@@ -446,8 +440,6 @@
                 }
             );
         });
-
-
     </script>
     <script type="text/javascript">
         function getWeekNumber(d) {

resources/views/users/index.blade.php

@@ -12,8 +12,6 @@
 
         </div>
     </div>
-
-
     @include('partials.table', [
         'id'        => $id,
         'header'    => $header,
@@ -49,9 +47,4 @@
             </div>
         </div>
     </div>
-
-    @if($errors->any())
-        @dump($errors)
-    @endif
-
 @endsection