Changeset 507ade0 for app/Http/Controllers

Timestamp:

10/25/21 12:59:43 (3 years ago)

Author:

beratkjufliju <kufliju@…>

Branches:

develop, master

Children:

13ff930

Parents:

bdda09f

Message:

bug fixes

File:

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

app/Http/Controllers/Dashboard/FoldersController.php

@@ -276 +276 @@
     public function downloadfolder(Request $request, $id)
     {
-        try {
             $folder = Folder::find($id);
             $FileSystem = new Filesystem();
-            $zip_file = Storage::disk('uploads')->path('Folder.zip');
-            $zip = new \ZipArchive();
-            $zip->open($zip_file, \ZipArchive::CREATE | \ZipArchive::OVERWRITE);
-
-            $path = Storage::disk('uploads')->path($folder->department->location . DIRECTORY_SEPARATOR . $folder->name);
-
-            $files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($path));
-
-            $filesInDoc = $FileSystem->allFiles($path);
-
-            foreach ($files as $file) {
-                if (!$file->isDir()) {
-                    $filePath = $file->getRealPath();
-                    // extracting filename with substr/strlen
-                    $relativePath = substr($filePath, strlen($path) + 1);
-                    $zip->addFile($filePath, $relativePath);
+
+            if($folder->no_of_files > 0) {
+                $zip_file = Storage::disk('uploads')->path('Folder.zip');
+                $zip = new \ZipArchive();
+                $zip->open($zip_file, \ZipArchive::CREATE | \ZipArchive::OVERWRITE);
+
+                $path = Storage::disk('uploads')->path($folder->department->location . DIRECTORY_SEPARATOR . $folder->name);
+
+                $files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($path));
+
+                $filesInDoc = $FileSystem->allFiles($path);
+
+                foreach ($files as $file) {
+                    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') . $folder->name . '.zip';
-            return response()->download($zip_file, $zip_new_name, $headers);
-        }
-        catch (\Exception $e) {
-            //abort(403, "No files found");
+                $zip->close();
+                $headers = array('Content-Type' => 'application/octet-stream',);
+                $zip_new_name = Carbon::now()->format('d.m.Y - H:i') . $folder->name . '.zip';
+                return response()->download($zip_file, $zip_new_name, $headers);
+            }
+        else {
             Alert::flash("This folder has no files", "warning");
             return redirect()->back();