Changeset 94f05dc for app/Http/Controllers/Dashboard/DepartmentsController.php

Timestamp:

10/24/21 23:23:54 (3 years ago)

Author:

beratkjufliju <kufliju@…>

Branches:

develop, master

Children:

df6e9ec

Parents:

fa80fbe

Message:

bug fixes

File:

• app/Http/Controllers/Dashboard/DepartmentsController.php (modified) (4 diffs)

app/Http/Controllers/Dashboard/DepartmentsController.php

@@ -47 +47 @@
 
         $users = User::all();
-        Notification::send($users, new NewDepartmentCreated("New department created"));
 
         $department->save();
@@ -127 +126 @@
         try {
             $files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($path));
-            $flag = false;
-
             foreach ($files as $file) {
 
@@ -154 +151 @@
         $department = Department::find($id);
 
-        $FileSystem = new Filesystem();
-        $zip_file = Storage::disk('uploads')->path('Department.zip');
+        $zip_file=Storage::disk('uploads')->path('Department.zip');
         $zip = new \ZipArchive();
         $zip->open($zip_file, \ZipArchive::CREATE | \ZipArchive::OVERWRITE);
-        $path = Storage::disk('uploads')->path($department->location) . DIRECTORY_SEPARATOR;
+        $path = Storage::disk('uploads')->path($department->location);
 
-        try{
+        try {
             $files = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($path));
             foreach ($files as $file) {
+
+                // We're skipping all subfolders
                 if (!$file->isDir()) {
                     $filePath = $file->getRealPath();
@@ -169 +167 @@
                     $zip->addFile($filePath, $relativePath);
                 }
+            }
             $zip->close();
-            $headers = array('Content-Type' => 'application/octet-stream',);
-            $zip_new_name = Carbon::now()->format('d.m.Y - H:i') . $department->name . '.zip';
+            $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("This department has no files", "warning");
+        catch(\Exception $e){
+            Alert::flash("All departments are empty", "warning");
             return redirect()->back();
-    }
+        }
     }
 }