mirror of
https://github.com/snipe/snipe-it.git
synced 2024-12-26 14:09:43 -08:00
64d649be7f
* Extract a handlesimages trait to centralize logic for parsing/storing images on upload in create/edit methods. * Use same image upload/layout in accessories as consum+components. * Monster: Cleanup/Refactor http controllers. This cleans up docblocks, pulls most non-crudy actions into their own controllers, and does general cleanup/logic refactoring. There /should/ be no functional changes, but we all know how should works.. Extract checkin/checkout functions to a separate controller for accessories. Move controllers to subdirectory. Cleanup AssetModelsController Extract component checkin/checkout Assorted cleanups/doc/formatting in controllers. Refactor LicenseController. Refactor UsersController Update viewassetscontroller. * Codacy cleanups * More codacy cleanups. Extract a LicenseCheckout Form request as well. * A bit more refactor/cleaning of the license checkout method. * Review Related Cleanups * Fix most of the item_not_found translations. In many cases, the string being generated did not even use the id parameter. Where it does, pass it as id instead of as a different value. * Remove some old $data arrays from when we manually sent emails from the controllers. This has been superseeded by the notification system (yay!) * Bugfix: Only log the checkin of an accessory if the checkin completes sucessfully.
147 lines
5.2 KiB
PHP
147 lines
5.2 KiB
PHP
<?php
|
|
|
|
namespace App\Http\Controllers\Licenses;
|
|
|
|
use App\Http\Requests\AssetFileRequest;
|
|
use App\Models\Actionlog;
|
|
use App\Models\License;
|
|
use App\Http\Controllers\Controller;
|
|
use Illuminate\Support\Facades\Input;
|
|
use Illuminate\Support\Facades\Response;
|
|
use Symfony\Component\HttpFoundation\JsonResponse;
|
|
|
|
class LicenseFilesController extends Controller
|
|
{
|
|
|
|
/**
|
|
* Validates and stores files associated with a license.
|
|
*
|
|
* @todo Switch to using the AssetFileRequest form request validator.
|
|
* @author [A. Gianotto] [<snipe@snipe.net>]
|
|
* @since [v1.0]
|
|
* @param AssetFileRequest $request
|
|
* @param int $licenseId
|
|
* @return \Illuminate\Http\RedirectResponse
|
|
* @throws \Illuminate\Auth\Access\AuthorizationException
|
|
*/
|
|
public function store(AssetFileRequest $request, $licenseId = null)
|
|
{
|
|
$license = License::find($licenseId);
|
|
// the license is valid
|
|
$destinationPath = config('app.private_uploads').'/licenses';
|
|
|
|
if (isset($license->id)) {
|
|
$this->authorize('update', $license);
|
|
|
|
if (Input::hasFile('file')) {
|
|
$upload_success = false;
|
|
foreach (Input::file('file') as $file) {
|
|
$extension = $file->getClientOriginalExtension();
|
|
$filename = 'license-'.$license->id.'-'.str_random(8).'-'.str_slug(basename($file->getClientOriginalName(), '.'.$extension)).'.'.$extension;
|
|
$upload_success = $file->move($destinationPath, $filename);
|
|
|
|
//Log the upload to the log
|
|
$license->logUpload($filename, e($request->input('notes')));
|
|
}
|
|
// This being called from a modal seems to confuse redirect()->back()
|
|
// It thinks we should go to the dashboard. As this is only used
|
|
// from the modal at present, hardcode the redirect. Longterm
|
|
// maybe we evaluate something else.
|
|
if ($upload_success) {
|
|
return redirect()->route('licenses.show', $license->id)->with('success', trans('admin/licenses/message.upload.success'));
|
|
}
|
|
return redirect()->route('licenses.show', $license->id)->with('error', trans('admin/licenses/message.upload.error'));
|
|
}
|
|
return redirect()->route('licenses.show', $license->id)->with('error', trans('admin/licenses/message.upload.nofiles'));
|
|
}
|
|
// Prepare the error message
|
|
return redirect()->route('licenses.index')
|
|
->with('error', trans('admin/licenses/message.does_not_exist'));
|
|
}
|
|
|
|
|
|
/**
|
|
* Deletes the selected license file.
|
|
*
|
|
* @author [A. Gianotto] [<snipe@snipe.net>]
|
|
* @since [v1.0]
|
|
* @param int $licenseId
|
|
* @param int $fileId
|
|
* @return \Illuminate\Http\RedirectResponse
|
|
* @throws \Illuminate\Auth\Access\AuthorizationException
|
|
*/
|
|
public function destroy($licenseId = null, $fileId = null)
|
|
{
|
|
$license = License::find($licenseId);
|
|
$destinationPath = config('app.private_uploads').'/licenses';
|
|
|
|
// the license is valid
|
|
if (isset($license->id)) {
|
|
$this->authorize('edit', $license);
|
|
$log = Actionlog::find($fileId);
|
|
$full_filename = $destinationPath.'/'.$log->filename;
|
|
if (file_exists($full_filename)) {
|
|
unlink($destinationPath.'/'.$log->filename);
|
|
}
|
|
$log->delete();
|
|
return redirect()->back()->with('success', trans('admin/licenses/message.deletefile.success'));
|
|
}
|
|
|
|
// Redirect to the licence management page
|
|
return redirect()->route('licenses.index')->with('error', trans('admin/licenses/message.does_not_exist'));
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
* Allows the selected file to be viewed.
|
|
*
|
|
* @author [A. Gianotto] [<snipe@snipe.net>]
|
|
* @since [v1.4]
|
|
* @param int $licenseId
|
|
* @param int $fileId
|
|
* @return \Symfony\Component\HttpFoundation\Response
|
|
* @throws \Illuminate\Auth\Access\AuthorizationException
|
|
*/
|
|
public function show($licenseId = null, $fileId = null, $download = true)
|
|
{
|
|
|
|
$license = License::find($licenseId);
|
|
|
|
// the license is valid
|
|
if (isset($license->id)) {
|
|
$this->authorize('view', $license);
|
|
$log = Actionlog::find($fileId);
|
|
$file = $log->get_src('licenses');
|
|
|
|
|
|
if ($file =='') {
|
|
return response('File not found on server', 404)
|
|
->header('Content-Type', 'text/plain');
|
|
}
|
|
|
|
$mimetype = \File::mimeType($file);
|
|
|
|
|
|
if (!file_exists($file)) {
|
|
return response('File '.$file.' not found on server', 404)
|
|
->header('Content-Type', 'text/plain');
|
|
}
|
|
|
|
if ($download != 'true') {
|
|
if ($contents = file_get_contents($file)) {
|
|
return Response::make($contents)->header('Content-Type', $mimetype);
|
|
}
|
|
return JsonResponse::create(["error" => "Failed validation: "], 500);
|
|
}
|
|
return Response::download($file);
|
|
}
|
|
|
|
|
|
return redirect()->route('licenses.index')->with('error', trans('admin/licenses/message.does_not_exist'));
|
|
}
|
|
|
|
|
|
|
|
}
|