Fix estate/auto create: remove show.js, loosen validation rules
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -86,17 +86,14 @@ class AutoController extends Controller
|
||||
}
|
||||
|
||||
$request->validate([
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'ordered_customer_phone' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'car_category' => 'required',
|
||||
'car_mark' => 'required',
|
||||
'car_number' => 'required',
|
||||
'tech_passport' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'car_category' => 'required',
|
||||
'car_mark' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
]);
|
||||
|
||||
$id = DB::table('auto_orders')->insertGetId([
|
||||
@@ -158,17 +155,14 @@ class AutoController extends Controller
|
||||
}
|
||||
|
||||
$request->validate([
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'ordered_customer_phone' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'car_category' => 'required',
|
||||
'car_mark' => 'required',
|
||||
'car_number' => 'required',
|
||||
'tech_passport' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'car_category' => 'required',
|
||||
'car_mark' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
]);
|
||||
|
||||
DB::table('auto_orders')->where('id', $id)->update([
|
||||
|
||||
@@ -59,26 +59,21 @@ class EstateController extends Controller
|
||||
|
||||
public function store(Request $request)
|
||||
{
|
||||
// Sanitize cost before validation
|
||||
if ($request->has('cost')) {
|
||||
$request->merge([
|
||||
'cost' => str_replace(',', '', $request->cost)
|
||||
]);
|
||||
$request->merge(['cost' => preg_replace('/\D/', '', $request->cost)]);
|
||||
}
|
||||
|
||||
$request->validate([
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'ordered_customer_phone' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'name_of_object' => 'required',
|
||||
'region' => 'required',
|
||||
'district' => 'required',
|
||||
'address' => 'required',
|
||||
'tech_passport' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'name_of_object' => 'required',
|
||||
'region' => 'required',
|
||||
'district' => 'required',
|
||||
'address' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
]);
|
||||
|
||||
$id = DB::table('estate_orders')->insertGetId([
|
||||
@@ -134,26 +129,21 @@ class EstateController extends Controller
|
||||
|
||||
public function update(Request $request, $id)
|
||||
{
|
||||
// Sanitize cost before validation
|
||||
if ($request->has('cost')) {
|
||||
$request->merge([
|
||||
'cost' => str_replace(',', '', $request->cost)
|
||||
]);
|
||||
$request->merge(['cost' => preg_replace('/\D/', '', $request->cost)]);
|
||||
}
|
||||
|
||||
$request->validate([
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'ordered_customer_phone' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'name_of_object' => 'required',
|
||||
'region' => 'required',
|
||||
'district' => 'required',
|
||||
'address' => 'required',
|
||||
'tech_passport' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
'number' => 'required',
|
||||
'purpose_id' => 'required',
|
||||
'ordered_customer' => 'required',
|
||||
'customer_type' => 'required|in:physical,juridical',
|
||||
'owner_type' => 'required|in:physical,juridical',
|
||||
'name_of_object' => 'required',
|
||||
'region' => 'required',
|
||||
'district' => 'required',
|
||||
'address' => 'required',
|
||||
'cost' => 'required|numeric',
|
||||
]);
|
||||
|
||||
DB::table('estate_orders')->where('id', $id)->update([
|
||||
|
||||
@@ -576,6 +576,5 @@
|
||||
@section('script')
|
||||
<script src="{{ URL::asset('assets/libs/cleave.js/cleave.js.min.js') }}"></script>
|
||||
<script src="{{ URL::asset('/assets/js/app.min.js') }}"></script>
|
||||
<script src="{{ URL::asset('/assets/js/pages/orders/show.js') }}"></script>
|
||||
<script src="{{ URL::asset('/assets/js/pages/orders/create.js') }}"></script>
|
||||
@endsection
|
||||
|
||||
Reference in New Issue
Block a user