Kaydet (Commit) 22b1a724 authored tarafından Baran Sekin's avatar Baran Sekin

Refactoring validations to provide straight error message

üst 641f677b
......@@ -12,7 +12,6 @@ use Illuminate\Http\JsonResponse;
use Illuminate\Http\Response;
use Illuminate\Support\Str;
use Illuminate\View\View;
use Illuminate\Support\Facades\Validator;
use App\Jobs\ExtensionUpdaterJob;
use Illuminate\Contracts\Bus\Dispatcher;
use App\System\Command;
......@@ -88,14 +87,10 @@ class MainController extends Controller
"request" => request()->all(),
]);
$flag = Validator::make(request()->all(), [
'extension' => 'required | max:5000000',
validate([
'extension' => 'required|max:5000000',
]);
try {
$flag->validate();
} catch (\Exception $exception) {
return respond("Lütfen geçerli bir eklenti giriniz.", 201);
}
$verify = false;
$zipFile = request()->file('extension');
if (
......
......@@ -10,7 +10,6 @@ use App\Models\Permission;
use App\Models\Extension;
use App\Models\RoleMapping;
use App\Models\Server;
use Illuminate\Support\Facades\Validator;
class RoleController extends Controller
{
......@@ -89,16 +88,10 @@ class RoleController extends Controller
"request" => request()->all(),
]);
$flag = Validator::make(request()->all(), [
validate([
'name' => ['required', 'string', 'max:255', 'unique:roles'],
]);
try {
$flag->validate();
} catch (\Exception $exception) {
return respond("Lütfen geçerli veri giriniz.", 201);
}
$role = Role::create([
"name" => request('name'),
]);
......
......@@ -7,7 +7,6 @@ use App\Models\LdapRestriction;
use Illuminate\Http\Request;
use App\Models\Role;
use App\Models\RoleMapping;
use Illuminate\Support\Facades\Validator;
class RoleMappingController extends Controller
{
......@@ -23,17 +22,11 @@ class RoleMappingController extends Controller
*/
public function add()
{
$flag = Validator::make(request()->all(), [
validate([
'dn' => ['required', 'string'],
'role_id' => ['required', 'exists:roles,id'],
]);
try {
$flag->validate();
} catch (\Exception $exception) {
return respond("Lütfen geçerli veri giriniz.", 201);
}
RoleMapping::create([
"dn" => request('dn'),
"role_id" => request('role_id'),
......@@ -54,16 +47,10 @@ class RoleMappingController extends Controller
*/
public function delete()
{
$flag = Validator::make(request()->all(), [
validate([
'role_mapping_id' => ['required', 'exists:role_mappings,id'],
]);
try {
$flag->validate();
} catch (\Exception $exception) {
return respond("Lütfen geçerli veri giriniz.", 201);
}
RoleMapping::find(request('role_mapping_id'))->delete();
return respond("Rol eşleştirmesi başarıyla silindi.");
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment