diff --git a/app/Http/Controllers/Accessories/AccessoriesFilesController.php b/app/Http/Controllers/Accessories/AccessoriesFilesController.php index ebc1e4b8e0..c251166ce2 100644 --- a/app/Http/Controllers/Accessories/AccessoriesFilesController.php +++ b/app/Http/Controllers/Accessories/AccessoriesFilesController.php @@ -51,7 +51,7 @@ class AccessoriesFilesController extends Controller } - return redirect()->route('accessories.show', $accessory->id)->with('success', trans('general.file_upload_success')); + return redirect()->route('accessories.show', $accessory->id)->withFragment('files')->with('success', trans('general.file_upload_success')); } @@ -90,8 +90,7 @@ class AccessoriesFilesController extends Controller $log->delete(); - return redirect()->back() - ->with('success', trans('admin/hardware/message.deletefile.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/hardware/message.deletefile.success')); } // Redirect to the licence management page diff --git a/app/Http/Controllers/AssetModelsFilesController.php b/app/Http/Controllers/AssetModelsFilesController.php index c905282cc5..11ede43c64 100644 --- a/app/Http/Controllers/AssetModelsFilesController.php +++ b/app/Http/Controllers/AssetModelsFilesController.php @@ -44,10 +44,10 @@ class AssetModelsFilesController extends Controller $model->logUpload($file_name, $request->get('notes')); } - return redirect()->back()->with('success', trans('general.file_upload_success')); + return redirect()->back()->withFragment('files')->with('success', trans('general.file_upload_success')); } - return redirect()->back()->with('error', trans('admin/hardware/message.upload.nofiles')); + return redirect()->back()->withFragment('files')->with('error', trans('admin/hardware/message.upload.nofiles')); } /** @@ -119,11 +119,10 @@ class AssetModelsFilesController extends Controller } $log->delete(); - return redirect()->back()->with('success', trans('admin/hardware/message.deletefile.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/hardware/message.deletefile.success')); } - return redirect()->back() - ->with('success', trans('admin/hardware/message.deletefile.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/hardware/message.deletefile.success')); } // Redirect to the hardware management page diff --git a/app/Http/Controllers/Assets/AssetFilesController.php b/app/Http/Controllers/Assets/AssetFilesController.php index d15055c4b2..96011728f9 100644 --- a/app/Http/Controllers/Assets/AssetFilesController.php +++ b/app/Http/Controllers/Assets/AssetFilesController.php @@ -45,7 +45,7 @@ class AssetFilesController extends Controller $asset->logUpload($file_name, $request->get('notes')); } - return redirect()->back()->with('success', trans('admin/hardware/message.upload.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/hardware/message.upload.success')); } return redirect()->back()->with('error', trans('admin/hardware/message.upload.nofiles')); @@ -97,25 +97,19 @@ class AssetFilesController extends Controller */ public function destroy($assetId = null, $fileId = null) : RedirectResponse { - $asset = Asset::find($assetId); - $this->authorize('update', $asset); - $rel_path = 'private_uploads/assets'; - - // the asset is valid - if (isset($asset->id)) { + if ($asset = Asset::find($assetId)) { $this->authorize('update', $asset); - $log = Actionlog::find($fileId); - if ($log) { + $rel_path = 'private_uploads/assets'; + + if ($log = Actionlog::find($fileId)) { if (Storage::exists($rel_path.'/'.$log->filename)) { Storage::delete($rel_path.'/'.$log->filename); } $log->delete(); - - return redirect()->back()->with('success', trans('admin/hardware/message.deletefile.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/hardware/message.deletefile.success')); } - return redirect()->back() - ->with('success', trans('admin/hardware/message.deletefile.success')); + return redirect()->route('hardware.show', ['hardware' => $asset])->with('error', trans('general.log_record_not_found')); } return redirect()->route('hardware.index')->with('error', trans('admin/hardware/message.does_not_exist')); diff --git a/app/Http/Controllers/Components/ComponentsFilesController.php b/app/Http/Controllers/Components/ComponentsFilesController.php index 83468a0b10..b5e30aa694 100644 --- a/app/Http/Controllers/Components/ComponentsFilesController.php +++ b/app/Http/Controllers/Components/ComponentsFilesController.php @@ -50,7 +50,7 @@ class ComponentsFilesController extends Controller } - return redirect()->route('components.show', $component->id)->with('success', trans('general.file_upload_success')); + return redirect()->route('components.show', $component->id)->withFragment('files')->with('success', trans('general.file_upload_success')); } @@ -91,7 +91,7 @@ class ComponentsFilesController extends Controller $log->delete(); - return redirect()->back() + return redirect()->back()->withFragment('files') ->with('success', trans('admin/hardware/message.deletefile.success')); } diff --git a/app/Http/Controllers/Consumables/ConsumablesFilesController.php b/app/Http/Controllers/Consumables/ConsumablesFilesController.php index 054fdc0b81..545b008dc0 100644 --- a/app/Http/Controllers/Consumables/ConsumablesFilesController.php +++ b/app/Http/Controllers/Consumables/ConsumablesFilesController.php @@ -48,7 +48,7 @@ class ConsumablesFilesController extends Controller } - return redirect()->route('consumables.show', $consumable->id)->with('success', trans('general.file_upload_success')); + return redirect()->route('consumables.show', $consumable->id)->withFragment('files')->with('success', trans('general.file_upload_success')); } @@ -89,7 +89,7 @@ class ConsumablesFilesController extends Controller $log->delete(); - return redirect()->back() + return redirect()->back()->withFragment('files') ->with('success', trans('admin/hardware/message.deletefile.success')); } diff --git a/app/Http/Controllers/Users/UserFilesController.php b/app/Http/Controllers/Users/UserFilesController.php index e99bfe298f..cae4074dd5 100644 --- a/app/Http/Controllers/Users/UserFilesController.php +++ b/app/Http/Controllers/Users/UserFilesController.php @@ -56,7 +56,7 @@ class UserFilesController extends Controller $logActions[] = $logAction; } // dd($logActions); - return redirect()->back()->with('success', trans('admin/users/message.upload.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/users/message.upload.success')); } return redirect()->back()->with('error', trans('admin/users/message.upload.nofiles')); @@ -87,7 +87,7 @@ class UserFilesController extends Controller if (Storage::exists($rel_path.'/'.$filename)) { Storage::delete($rel_path.'/'.$filename); - return redirect()->back()->with('success', trans('admin/users/message.deletefile.success')); + return redirect()->back()->withFragment('files')->with('success', trans('admin/users/message.deletefile.success')); } } diff --git a/app/Http/Requests/UploadFileRequest.php b/app/Http/Requests/UploadFileRequest.php index 4762e52b75..e58f1a1be3 100644 --- a/app/Http/Requests/UploadFileRequest.php +++ b/app/Http/Requests/UploadFileRequest.php @@ -46,8 +46,6 @@ class UploadFileRequest extends Request $extension = $file->getClientOriginalExtension(); $file_name = $name_prefix.'-'.str_random(8).'-'.str_slug(basename($file->getClientOriginalName(), '.'.$extension)).'.'.$file->guessExtension(); - - Log::debug("Your filetype IS: ".$file->getMimeType()); // Check for SVG and sanitize it if ($file->getMimeType() === 'image/svg+xml') { Log::debug('This is an SVG'); @@ -66,7 +64,6 @@ class UploadFileRequest extends Request } else { $put_results = Storage::put($dirname.$file_name, file_get_contents($file)); - Log::debug("Here are the '$put_results' (should be 0 or 1 or true or false or something?)"); } return $file_name; } diff --git a/resources/views/models/view.blade.php b/resources/views/models/view.blade.php index f26b9babd4..15ce82d67f 100755 --- a/resources/views/models/view.blade.php +++ b/resources/views/models/view.blade.php @@ -49,7 +49,7 @@
  • - +