Skip to main content
update heading
Source Link

Resource Controller could be used

Resource Controller

Resource Controller could be used

update headers
Source Link

Update method

Update method is a bit long

Pass fields to update to model method update()

Pass fields to update to model method update()

Make a form request class for handling the validation

Make a form request class for handling the validation

Middleware

Middleware can be added to a group of routes

Resource Controller

Resource Controller

Testing

Remember to add testing

JavaScript

JavaScript can be simplified slightly with jQuery shortcuts

Update method

Pass fields to update to model method update()

Make a form request class for handling the validation

Middleware

Resource Controller

Testing

JavaScript

Update method is a bit long

Pass fields to update to model method update()

Make a form request class for handling the validation

Middleware can be added to a group of routes

Resource Controller

Remember to add testing

JavaScript can be simplified slightly with jQuery shortcuts

Bounty Awarded with 50 reputation awarded by Razvan Zamfir
added 115 characters in body
Source Link

Also, for the sake of readability (e.g. see section 2.3section 2.3 of PSR-12) it would be wise to alias the profile controller using a use statement.

Also, for the sake of readability (e.g. see section 2.3 of PSR-12) it would be wise to alias the profile controller using a use statement.

Also, for the sake of readability (e.g. see section 2.3 of PSR-12) it would be wise to alias the profile controller using a use statement.

added 415 characters in body
Source Link
Loading
Add section about testing
Source Link
Loading
Add section about testing
Source Link
Loading
fix reference in UserUpdateRequest, since $current_user was undefined
Source Link
Loading
deleted 22 characters in body
Source Link
Mast
  • 13.8k
  • 12
  • 57
  • 127
Loading
Source Link
Loading