Browse Source

Refactor CustomerController@update

Add CustomerUpdateRequest class
pull/24/head
Nafies Luthfi 7 years ago
parent
commit
b24a14561c
  1. 19
      app/Http/Controllers/Partners/CustomersController.php
  2. 39
      app/Http/Requests/Partners/CustomerUpdateRequest.php

19
app/Http/Controllers/Partners/CustomersController.php

@ -6,6 +6,7 @@ use Illuminate\Http\Request;
use App\Entities\Partners\Customer;
use App\Http\Controllers\Controller;
use App\Http\Requests\Partners\CustomerCreateRequest;
use App\Http\Requests\Partners\CustomerUpdateRequest;
class CustomersController extends Controller
{
@ -75,25 +76,13 @@ class CustomersController extends Controller
/**
* Update the specified customer in storage.
*
* @param \Illuminate\Http\Request $request
* @param \App\Http\Requests\Partners\CustomerUpdateRequest $customerUpdateForm
* @param \App\Entities\Partners\Customer $customer
* @return \Illuminate\Http\RedirectResponse
*/
public function update(Request $request, Customer $customer)
public function update(CustomerUpdateRequest $customerUpdateForm, Customer $customer)
{
$customerData = $this->validate($request, [
'name' => 'required|max:60',
'email' => 'nullable|email|unique:customers,email,'.$customer->id,
'phone' => 'nullable|max:255',
'pic' => 'nullable|max:255',
'address' => 'nullable|max:255',
'website' => 'nullable|url|max:255',
'notes' => 'nullable|max:255',
'is_active' => 'required|boolean',
]);
$customer->update($customerData);
$customer->update($customerUpdateForm->validated());
flash(__('customer.updated'), 'success');
return redirect()->route('customers.show', $customer->id);

39
app/Http/Requests/Partners/CustomerUpdateRequest.php

@ -0,0 +1,39 @@
<?php
namespace App\Http\Requests\Partners;
use Illuminate\Foundation\Http\FormRequest;
class CustomerUpdateRequest extends FormRequest
{
/**
* Determine if the user is authorized to make this request.
*
* @return bool
*/
public function authorize()
{
return $this->user()->can('update', $this->route('customer'));
}
/**
* Get the validation rules that apply to the request.
*
* @return array
*/
public function rules()
{
$customer = $this->route('customer');
return [
'name' => 'required|max:60',
'email' => 'nullable|email|unique:customers,email,'.$customer->id,
'phone' => 'nullable|max:255',
'pic' => 'nullable|max:255',
'address' => 'nullable|max:255',
'website' => 'nullable|url|max:255',
'notes' => 'nullable|max:255',
'is_active' => 'required|boolean',
];
}
}
Loading…
Cancel
Save