refactored category validation and removed unused module trait
This commit is contained in:
parent
e6c67ef504
commit
6f50a4e685
|
|
@ -12,11 +12,10 @@ use App\Jobs\Setting\DeleteCategory;
|
|||
use App\Jobs\Setting\UpdateCategory;
|
||||
use App\Models\Setting\Category;
|
||||
use App\Traits\Categories as Helper;
|
||||
use App\Traits\Modules;
|
||||
|
||||
class Categories extends Controller
|
||||
{
|
||||
use Helper, Modules;
|
||||
use Helper;
|
||||
|
||||
/**
|
||||
* Display a listing of the resource.
|
||||
|
|
@ -85,13 +84,16 @@ class Categories extends Controller
|
|||
$types = $this->getCategoryTypes(true, true);
|
||||
|
||||
$categories = [];
|
||||
$has_code = false;
|
||||
|
||||
foreach (config('type.category') as $type => $config) {
|
||||
if (empty($config['hide']) || ! in_array('code', $config['hide'])) {
|
||||
$has_code = true;
|
||||
}
|
||||
|
||||
$categories[$type] = [];
|
||||
}
|
||||
|
||||
$has_code = $this->moduleIsEnabled('double-entry');
|
||||
|
||||
Category::enabled()->orderBy('name')->get()->each(function ($category) use (&$categories) {
|
||||
$categories[$category->type][] = [
|
||||
'id' => $category->id,
|
||||
|
|
@ -171,8 +173,14 @@ class Categories extends Controller
|
|||
$edited_category_id = $category->id;
|
||||
|
||||
$categories = [];
|
||||
$has_code = false;
|
||||
|
||||
foreach (config('type.category') as $type => $config) {
|
||||
if (empty($config['hide']) || ! in_array('code', $config['hide'])) {
|
||||
$has_code = true;
|
||||
break;
|
||||
}
|
||||
|
||||
$categories[$type] = [];
|
||||
}
|
||||
|
||||
|
|
@ -205,8 +213,6 @@ class Categories extends Controller
|
|||
|
||||
$parent_categories = $categories[$category->type] ?? [];
|
||||
|
||||
$has_code = $this->moduleIsEnabled('double-entry');
|
||||
|
||||
return view('settings.categories.edit', compact('category', 'types', 'type_disabled', 'categories', 'parent_categories', 'has_code'));
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -3,11 +3,9 @@
|
|||
namespace App\Http\Requests\Setting;
|
||||
|
||||
use App\Abstracts\Http\FormRequest;
|
||||
use App\Traits\Modules;
|
||||
|
||||
class Category extends FormRequest
|
||||
{
|
||||
use Modules;
|
||||
/**
|
||||
* Get the validation rules that apply to the request.
|
||||
*
|
||||
|
|
@ -17,7 +15,10 @@ class Category extends FormRequest
|
|||
{
|
||||
$types = collect(config('type.category'))->keys();
|
||||
|
||||
$code = $this->moduleIsEnabled('double-entry') ? 'required|string' : 'nullable|string';
|
||||
$type = $this->request->get('type');
|
||||
$config = config('type.category.' . $type, []);
|
||||
$code_hidden = !empty($config['hide']) && in_array('code', $config['hide']);
|
||||
$code = $code_hidden ? 'nullable|string' : 'required|string';
|
||||
|
||||
return [
|
||||
'name' => 'required|string',
|
||||
|
|
|
|||
Loading…
Reference in New Issue