Changeset 2740efc for app/Http/Controllers

Timestamp:

10/25/21 00:21:56 (3 years ago)

Author:

beratkjufliju <kufliju@…>

Branches:

develop, master

Children:

c08466c

Parents:

68694d4

Message:

bug fixes

File:

• app/Http/Controllers/Dashboard/DepartmentsController.php (modified) (1 diff)

app/Http/Controllers/Dashboard/DepartmentsController.php

@@ -119 +119 @@
     public function downloadAll()
     {
-        $zip_file = Storage::disk('uploads')->path('Departments.zip');
-        $zip = new \ZipArchive();
-        $zip->open($zip_file, \ZipArchive::CREATE | \ZipArchive::OVERWRITE);
+        try {
+            $zip_file = Storage::disk('uploads')->path('Departments.zip');
+            $zip = new \ZipArchive();
+            $zip->open($zip_file, \ZipArchive::CREATE | \ZipArchive::OVERWRITE);
 
-        $path = Storage::disk('uploads')->path('Departments');
+            $path = Storage::disk('uploads')->path('Departments');
 
-        try {
             $files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($path));
-        }
-        catch(\Exception $e){
-            Alert::flash("All departments are empty", "warning");
-            return redirect()->back();
-        }
 
             foreach ($files as $file) {
 
-            // We're skipping all subfolders
-            if (!$file->isDir()) {
-                $filePath = $file->getRealPath();
-                // extracting filename with substr/strlen
-                $relativePath = substr($filePath, strlen($path) + 1);
-                $zip->addFile($filePath, $relativePath);
+                // We're skipping all subfolders
+                if (!$file->isDir()) {
+                    $filePath = $file->getRealPath();
+                    // extracting filename with substr/strlen
+                    $relativePath = substr($filePath, strlen($path) + 1);
+                    $zip->addFile($filePath, $relativePath);
+                }
             }
+            $zip->close();
+            $headers = array('Content-Type' => 'application/octet-stream');
+            $zip_new_name = Carbon::now()->format('d.m.Y - H:i') . '- Departments.zip';
+            return response()->download($zip_file, $zip_new_name, $headers);
         }
-                $zip->close();
-                $headers = array('Content-Type' => 'application/octet-stream');
-                $zip_new_name = Carbon::now()->format('d.m.Y - H:i') . '- Departments.zip';
-                return response()->download($zip_file, $zip_new_name, $headers);
-
+        catch(\Exception $e){
+        Alert::flash("All departments are empty", "warning");
+        return redirect()->back();
+    }
     }