V2

app/Http/Controllers/HouseController.php

@@ -22,35 +22,33 @@ public function store(Request $request)
 
     public function update(Request $request, House $house)
     {
-        $fileOld=$house->photo;
         $filename = $request->file('photo')->store('houses');
 
         // TASK: Delete the old file from the storage
+        if($house->photo)
+        {
+            Storage::delete($house->photo);
+        }
 
         $house->update([
             'name' => $request->name,
             'photo' => $filename,
         ]);
 
-        if($fileOld && !$fileOld==$filename)
-        {
-            Storage::delete($fileOld);
-        }
         return 'Success';
     }
 
     public function download(House $house)
     {
         // TASK: Return the $house->photo file from "storage/app/houses" folder
         // for download in browser
-        $filePath=storage_path('app/houses' . $house->photo);
+        $filePath = storage_path("app/houses/{$house->photo}");
         if(file_exists($filePath))
         {
             return response()->download($filePath);
-
         }
-        else
-             abort(404, 'File not found');
-
+        else {
+            return response()->json(['error' => 'File not found'], 404);
+        }
     }
-}
+}