REFACTORING #1880 @40m: move creating new directory into interaction

This commit is contained in:
Michal Kváček
2016-11-23 01:30:28 +01:00
parent ca718a5efe
commit 675aec669b
6 changed files with 123 additions and 45 deletions
@@ -0,0 +1,6 @@
<?php
namespace App\CMS\Exceptions\FileManager;
class SubdirectoryExistsInDirectory extends \Exception
{
}
+1 -1
View File
@@ -24,7 +24,7 @@ class Interaction
* @var $interaction InteractionContract * @var $interaction InteractionContract
*/ */
$interaction = App::make($interactionClass); $interaction = app($interactionClass);
// validate interaction // validate interaction
$validator = Validator::make($data, $interaction->getValidationRules($extraParameters)); $validator = Validator::make($data, $interaction->getValidationRules($extraParameters));
@@ -0,0 +1,65 @@
<?php
namespace App\CMS\Interactions\Admin\FileManager;
use App\CMS\Contracts\Interaction;
use App\CMS\Exceptions\FileManager\SubdirectoryExistsInDirectory;
use App\CMS\Repositories\FileManager\Directory as DirectoryRepository;
use App\Models\FileManager\Directory;
class CreateDirectory implements Interaction
{
/**
* @var DirectoryRepository
*/
private $repository;
public function __construct(DirectoryRepository $repository)
{
$this->repository = $repository;
}
/**
* Perform given action
*
* @param $data array input data
* @param $parameters array parameters
*
* @return Directory
* @throws SubdirectoryExistsInDirectory
*/
public function handle($data, $parameters)
{
$data['parentDirectory'] = isset($data['parentDirectory']) ? $data['parentDirectory'] : null;
// todo this should probably go into validation, but we are not able to get $data in getValidationRules
// check if created folder does not exist in current directory
if ($this->repository->checkIfNameExists($data['name'], $data['parentDirectory'])) {
throw new SubdirectoryExistsInDirectory();
}
// todo use Directory::create() and rename columns/fields in request
// create new database entry
$directory = new Directory();
$directory->name = $data['name'];
$directory->parent_id = $data['parentDirectory'];
$directory->user_id = \Auth::user()->id;
$directory->description = $data['description'];
$directory->save();
return $directory;
}
/**
* @param $data array input data
*
* @return array
* @throws SubdirectoryExistsInDirectory
*/
public function getValidationRules($data = [])
{
return [
'name' => 'required|max:255',
'parentDirectory' => 'exists:directories,id'
];
}
}
@@ -0,0 +1,14 @@
<?php
namespace App\CMS\Repositories\FileManager;
use App\Models\FileManager\Directory as DirectoryModel;
class Directory
{
public function checkIfNameExists($name, $directoryId)
{
return DirectoryModel::where('name', $name)
->where('parent_id', $directoryId)
->exists();
}
}
@@ -2,12 +2,31 @@
namespace App\Http\Controllers\Admin\FileManager; namespace App\Http\Controllers\Admin\FileManager;
use App\CMS\Exceptions\FileManager\SubdirectoryExistsInDirectory;
use App\Cms\Handlers\Interactions\Interaction;
use App\CMS\Interactions\Admin\FileManager\CreateDirectory;
use App\CMS\Repositories\FileManager\Directory as DirectoryRepository;
use App\Http\Controllers\Admin\Controller; use App\Http\Controllers\Admin\Controller;
use App\Http\Requests; use App\Http\Requests;
use App\Models\FileManager\Directory; use Illuminate\Http\Request;
class CreateNewDirectory extends Controller class CreateNewDirectory extends Controller
{ {
/**
* @var DirectoryRepository
*/
private $repository;
/**
* CreateNewDirectory constructor.
*
* @param DirectoryRepository $repository
*/
public function __construct(DirectoryRepository $repository)
{
$this->repository = $repository;
}
/** /**
* Create new directory * Create new directory
* *
@@ -15,48 +34,21 @@ class CreateNewDirectory extends Controller
* *
* @return $this|\Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse * @return $this|\Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
*/ */
public function postNew(Requests\CreateNewDirectoryRequest $request) public function postNew(Request $request)
{ {
// check if created folder does not exist in current directory // create new directory
$exists = Directory::where('name', $request->name) try {
->where('parent_id', $request->parentDirectory) Interaction::handle(CreateDirectory::class, $request->all());
->exists(); } catch (SubdirectoryExistsInDirectory $e) {
if ($exists) { return redirect()
if ($request->ajax()) { ->back()
return response()->json([ ->withInput()
'status' => 'failed', ->with('error', trans('admin/fileManager.subdirectoryExists'));
'message' => trans('admin/fileManager.subdirectoryExists')
]);
} else {
return redirect()
->route('upload.overview', ['id' => $request->parentDirectory])
->withInput()
->with('error', trans('admin/fileManager.subdirectoryExists'));
}
} }
// create new database entry
$directory = new Directory();
$directory->name = $request->name;
$directory->parent_id = $request->parentDirectory;
$directory->user_id = \Auth::user()->id;
$directory->description = $request->description;
$directory->save();
$message = trans('admin/fileManager.newDirectorySuccess'); $message = trans('admin/fileManager.newDirectorySuccess');
if (!$request->ajax()) { return redirect()
return redirect() ->route('upload.overview', ['id' => $request->parentDirectory])
->route('upload.overview', ['id' => $request->parentDirectory]) ->with('success', $message);
->with('success', $message);
} else {
$payload = [
'status' => 'success',
'message' => $message,
'parentDirectory' => $request->parentDirectory
];
return response()->json($payload);
}
} }
} }
@@ -2,6 +2,8 @@
namespace App\Http\Requests; namespace App\Http\Requests;
use App\CMS\Interactions\Admin\FileManager\CreateDirectory;
class CreateNewDirectoryRequest extends Request class CreateNewDirectoryRequest extends Request
{ {
/** /**
@@ -11,9 +13,8 @@ class CreateNewDirectoryRequest extends Request
*/ */
public function rules() public function rules()
{ {
return [ $interaction = app(CreateDirectory::class);
'name' => 'required|max:255',
'parentDirectory' => 'exists:directories,id' return $interaction->getValidationRules();
];
} }
} }