First and foremost, Laravel provides the most value when you write things the way Laravel intended you to write. It means better and more extensive use of native laravel features or Laravel “products” like Nova. Whenever you do something differently, make sure you have a justification for why you didn’t follow the defaults.
Secondary, Laravel often provides few ways/APIs to achieve your goals, in this document we try to list all our conventions to keep our code more consistent and use only one way.
[[toc]]
Facades SHOULD be used in PHP code, helpers SHOULD be used in Blade views. Don’t use Facade root aliases (it’s extra magic that’s easy to avoid).
We generally don’t use short and magic syntax for queries:
// GOOD
Member::query()->firstWhere('id', 42);
// BAD
Member::firstWhere('id', 42);
Mass assignment SHOULD not be used when it’s easily possible. When it's used in a wrong way, it can add security vulnerabilities, it also allows creating Models with a wrong state.
The preferred way to create or update models is to assign attributes line by line and call save()
at the end:
// PREFERRED WAY:
$member = new Member();
$member->name = $request->input('name');
$member->email = $request->input('email');
$member->save();
// AVOID THIS:
$member->forceFill([
'name' => $request->input('name'),
'email' => $request->input('email'),
])->save();
// NEVER DO THIS:
$member->forceFill($request->all())
->save();
Don’t use magic where{Something}
methods.
When you add a relationship or scope, add the appropriate PHPDoc block to the Model:
// Models/Member.php
/**
* @property-read \Illuminate\Database\Eloquent\Collection<int, \App\Modules\Permission\Models\Role> $roles Member’s Roles (added by a Member::roles() relationship)
* @method static \Illuminate\Database\Eloquent\Builder|\App\Modules\Member\Models\Member canceled() Cancelled Member state (added by a Member::scopeCanceled())
*/
Model’s attributes should not rely on DB’s default values.
Instead, we should duplicate defaults in the model by filling the $attributes
array.
It helps us to be more independent of the DB and simplifies Model’s Factories as well as testing.
User::query()->scopes(['trial'])->...
For scopes with parameters, we recommend to use tappable scopes:
$unverifiedUsers = User::query()
->tap(new Unverified())
->get();
To simplify models & enable better type-hint by IDE for big Models, we should extract a custom query builder class.
It's a recommendation for Models with 4+ query scopes and a requirement for Models with 10+ query scopes.
Here's how we can add the builder to the model class.
class User extends Model
{
/**
* @inheritDoc
* @param \Illuminate\Database\Query\Builder $query
* @return \App\Models\UserEloquentBuilder<self>
*/
public function newEloquentBuilder($query): UserEloquentBuilder
{
return new UserEloquentBuilder($query);
}
}
This is the builder with one custom query:
/** @extends \Illuminate\Database\Eloquent\Builder<\App\Models\User> */
final class UserEloquentBuilder extends Builder
{
public function confirmed(): self
{
return $this->whereNotNull('confirmed_at');
}
}
And here's how we can use it:
$confirmedUsers = User::query()->confirmed()->get();
Make sure to wrap orWhere
clauses inside another where clause to make a query safe for other "where" conditions:
final class UserEloquentBuilder extends Builder
{
public function publishedOrCanceled(): self
{
- return $this->withTrashed()->whereNotNull('published_at')->orWhereNotNull('deleted_at');
+ return $this->withTrashed()->where(static function (\Illuminate\Database\Eloquent\Builder $builder): void {
+ $builder->whereNotNull('published_at');
+ $builder->orWhereNotNull('deleted_at');
+ })
}
}
Read more:
It's always better to use for specific column names. Examples:
created_at
->registered_at
,issued_at
, etcupdated_at
->reviewed_at
, etcdeleted_at
->rejected_at
,canceled_at
, etc
Eloquent Factory classes SHOULD be used only for tests. In the application context, Model methods, Actions and Services SHOULD be used instead.
To even more separate "application" and "test" contexts, please:
- keep all factories at the
tests/Factories
directory (Tests\Factories
namespace) - do not use
HasFactory
trait in Model classes - in tests, call Factory classes directly:
$user = UserFactory::new()->create([...]);
For Models that have finite number of states
(Finite-state machine,
e.g. Article
can be one of draft
, published
, archiced
states),
the Factory::definition
method SHOULD NOT make a Model of any non-default state: the state should be set explicitly in the test.
For cases, when the state is not important, the recommendation is to create a method alias that underlines this (see ofAnyValidState
):
final class ArticleFactory extends Factory
{
/** @inheritDoc */
public function definition(): array
{
return [
'title' => $this->faker->sentence,
'body' => $this->faker->paragraph,
];
}
public function draft(): self
{
return $this->state(['published_at' => null]);
}
public function published(): self
{
return $this->state([
'published_at' => today(),
'meta_description' => $this->faker->sentence,
]);
}
public function ofAnyValidState(): self
{
return $this->draft(); // or even return a random valid state
}
}
The names given to artisan commands SHOULD all be kebab-cased.
# GOOD
php artisan delete-old-records
# BAD
php artisan deleteOldRecords
# BAD
php artisan delete_old_records
Inject any dependencies in the handle()
method instead of in the constructor.
Laravel initiates ALL console commands on every artisan
call, for this reason
console command class constructors should be fast and not contain any heavy logic.
Use different verbosity levels.
$this->info('Updating Articles...', 'v');
// ...
foreach($articles as $article) {
// ...
$this->info("\t Article #{$article->id} has been updated", 'vvv');
}
$this->info("{$articles->count()} Articles has been updated");
quiet
mode: only errors and important warnings.normal
mode: errors, all warnings and general feedback likeAll OK, processed XX records!
.v
,vv
,vvv
modes: errors, warnings and any additional info.
The idea behind it is to send email with console command outputs only when output is present (not empty).
Use non-zero exit codes if a command execution failed (alternatively throw an exception — this is the same as exit code 1).
This allows to use global on-error handlers, e.g. for automated reporting about failed console commands, please see
\Illuminate\Console\Scheduling\Event::emailOutputOnFailure
as an example.
Controllers that control a resource must use the singular resource name.
// GOOD
final class CourseController {}
// BAD
final class CoursesController {}
Try to keep controllers simple and stick to the default CRUD keywords (index
, create
, store
, show
, edit
, update
and destroy
).
Extract a new controller if you need other actions.
This is a loose guideline that doesn’t need to be enforced.
// GOOD
public function update(Request $request, Course $course)
{
$this->validate($request, ['email' => ['email']);
$name = $request->input('name');
}
// BAD
public function update(Course $course)
{
$this->validate(request(), ['email' => ['email']);
$name = request('name');
}
// GOOD
public function update(Team $team, Request $request, DetachFromTeamToIndividualGracePeriodAction $detachAction)
{
...
}
// BAD
public function update(Request $request, Team $team, DetachFromTeamToIndividualGracePeriodAction $detachAction)
{
...
}
The same for scalar GET params (good example: public function update(int $teamId, Request $request
).
For the sake of consistency, we use $request->input()
only.
// GOOD
public function store(Request $request)
{
$email = $request->input('email');
}
// BAD
public function store(Request $request)
{
$email = $request->get('email');
}
// GOOD
return redirect()->route('home');
return redirect()->to($url);
// BAD
return redirect(route('home')); // mixed return type (RedirectResponse|Redirector)
return redirect($url); // mixed return type (RedirectResponse|Redirector)
Limit the number of HTTP codes the app can return and process them in a consistent way.
Public-facing urls must use kebab-case.
https://www.interaction-design.org/about/people-behind
https://www.interaction-design.org/my-private-profile
Routes MUST have names, please use route()
helper to generate URLs from named routes. Route names MUST use camelCase.
Route::get('about', [AboutPageController::class, 'index'])->name('about.index');
<a href="{{ route('about.index') }}">About</a>
Route names SHOULD include the plural form of the resource name and the action: articles.show
, articles.delete
.
There are few valid options on how to use route()
helper for named routes:
// route: '/meetups/{meetupId}'
$meetup = \App\Modules\LocalGroup\Models\Meetup::query()->find($meetupId);
route('meetups.show', $meetup); // GOOD (RECOMMENDED) for routes with a single parameter
route('meetups.show', $meetupId); // GOOD for cases when you don’t have Meetup object but have an ID/key
route('meetups.show', [$meetup]); // BAD, please don’t use array syntax for a single param routes or use array keys
route('meetups.show', ['id' => $meetup]); // GOOD (RECOMMENDED)
route('meetups.show', ['id' => $meetupId]); // GOOD
route('meetups.show', ['meetupId' => $meetup->id]); // ERROR: Missing required parameter "id"
2+ required parameters:
// route: '/master-classes/{masterclass}/registrations/{registration}'
$registration = \App\Modules\Masterclass\Models\Registration::query()->first();
route('masterclasses.registrations.show', [$registration->masterclass, $registration]);
route('masterclasses.registrations.show', ['masterclass' => $registration->masterclass, 'registration' => $registration]); // GOOD, RECOMMENDED
route('masterclasses.registrations.show', ['registration' => $registration, 'masterclass' => $registration->masterclass]); // BAD, params mixed up (but still working as expected)
route('masterclasses.registrations.show', [$registration->masterclass, 'registration' => $registration]); // BAD: missing first key (Inconsistency)
route('masterclasses.registrations.show', ['masterclass' => $registration->masterclass, $registration]); // BAD: missing second key (Inconsistency)
When defining routes, use method chaining instead of array of params:
// GOOD:
Route::get('about', [AboutPageController::class, 'index'])->name('about.index')->middleware(['cache:1day']);
// BAD:
Route::get('about', ['as' => 'about.index', 'uses' => [AboutPageController::class, 'index']])->middleware(['cache:1day']);
// GOOD
Route::get('about', [AboutPageController::class, 'index'])->middleware(['cache:1day']);
Route::get('about', [AboutPageController::class, 'index'])->middleware(['cache:1day', 'CORS']);
// BAD
Route::get('about', [AboutPageController::class, 'index'])->middleware('cache:1day', 'CORS');
Route::get('about', [AboutPageController::class, 'index'])->middleware('cache:1day');
Tuple notation MUST be used to declare a route (when it’s possible):
// GOOD
Route::get('about', [AboutPageController::class, 'index']);
// BAD
Route::get('about', 'AboutPageController@index');
Route parameters SHOULD use camelCase.
Route::get('members/{memberId}', [MembersController::class, 'show']);
All routes have an HTTP verb; that’s why we put the verb first when defining a route. It makes a group of routes very readable. Any other route options MUST come after it.
// GOOD: all http verbs come first
Route::get('/', [HomeController::class, 'index'])->name('home');
// BAD: http verbs not easily scannable
Route::name('home')->get('/', [HomeController::class, 'index']);
- Policies MUST use camelCase. Example:
@can('editPost', $post)
(Laravel does it under the hood) - Try to name abilities using default CRUD words. One exception: replace
show
withview
. A server shows a resource, a user views it.
Avoid using |
as separator for validation rules, always use array notation.
Using an array notation will make it easier to apply custom rule classes to a field.
// GOOD
public function rules(): array
{
return [
'email' => ['required', 'email'],
];
}
// BAD
public function rules(): array
{
return [
'email' => 'required|email',
];
}
All custom validation rules must use snake_case:
Validator::extend('is_null', fn ($attribute, $value, $parameters, $validator) => $value === null);
View files and directories MUST use camelCase.
resources/
views/
pages/
localGroups/
show.blade.php
createDiscussion.blade.php
{{-- GOOD --}}
{{ $user->name }}
@include('welcome', ['user' => $user])
{{-- BAD --}}
{{ $user->name }}
@include('welcome')
You SHOULD create and maintain PHPDoc blocks at the top of every view file. You MUST create and maintain PHPDoc blocks for components.
Add PHP injection using <?php
and ?>
. The @php
and @endphp
Blade directives pair looks better, but the tools we use (Psalm, Rector, PHPCS, PHP-CS-Fixer) can’t parse Blade syntax.
Translations MUST be rendered with the __()
function.
We prefer using this over the @lang
directive in Blade views because __()
can be used in both Blade views and regular PHP code. Here’s an example:
{{-- GOOD --}}
{{ __('newsletter.form.title') }}
{{-- BAD --}}
@lang('newsletter.form.title')
// GOOD
__('newsletter.form.title')
// BAD
trans('newsletter.form.title')
__('app.message', ['firstName' => 'Peter', 'productName' => 'Bananas']);
// GOOD
abort(404, "The course with the ID $courseId could not be found.");
// BAD
abort(404);
There are few characteristics, our Jobs should follow:
- Reentrancy. If a task is interrupted, it can be restarted and completed successfully.
- Idempotence. A task can be called multiple times, without changing the side effects.
- Concurrence. More than one of a task can be run at the same time.
- Sequence Independence. The order of the tasks doesn’t matter.
You can find more details on awesome talk: Matt Stauffer - Patterns That Pay Off
You SHOULD use Bus::dispatch()
Facade or use \Illuminate\Contracts\Bus\Dispatcher
DI
instead of YourJobClass::dispatch()
magic to make code readable for static analyzers:
// GOOD
use Illuminate\Support\Facades\Bus;
Bus::dispatch(new YouJob($parameter));
// BAD
YouJob::dispatch($parameter)
By default, Laravel adds few traits to a new Event class, even if it’s not needed in your particular case. We fixed it in our custom stub file for Event, but it’s still better to control traits more explicitly.
-use Dispatchable, InteractsWithSockets, SerializesModels;
+use SerializesModels; // only if the Event will be used with Queued Event Listeners
Dispatchable
is to add static methods to simplify event dispatching, likeYourEvent::dispatch()
. We do not use this syntax, so we don’t need this trait. Please use\Illuminate\Support\Facades\Event
facade instead, e.g.Event::dispatch(new YourEvent())
.SerializesModels
is to gracefully serialize any Eloquent models if the event object contains Eloquent models and going to be serialized using PHP'sserialize
function, such as when utilizing queued listeners.InteractsWithSockets
is for broadcasting only, e.g. using Laravel Echo.
Best Practices:
- Tailor Event class traits based on specific needs rather than using the default set.
- Understand the implications of each trait to avoid unnecessary overhead or missing functionality.
- Event class should be
final readonly
Write down()
methods because:
- it should be possible to rollback failed releases
- developer experience: simplify switching between branches
We use ixdf_
prefix for our custom config files to separate our config vars from Laravel’s and 3rd party packages ones.
It also helps us to migrate to new Laravel versions: we have fewer conflicts.
Usually we have one config file per system.
Laravel provides a robust Query Builder and Eloquent ORM. And thanks to them most of the queries are protected in Laravel applications by default, so for example a query like
Product::query()->where('category_id', $request->input('categoryId'))->get();
will be automatically protected, since under the hood Laravel will translate the code into a prepared statement and execute.
But developers usually make mistakes by assuming Laravel protects from all SQL injections, while there are some attack vectors that Laravel can’t protect, here are the most common causes of SQL injections.
It’s not safe to pass user-controlled column names to the query builder. Here is a warning fromLaravel’s documentation.
So the following code will be vulnerable to an SQL injection:
$categoryId = $request->input('categoryId');
$orderBy = $request->input('orderBy');
Product::query()
->where('category_id', $categoryId)
->orderBy($orderBy)
->get();
This way, someone can use a query like http://example.com/users?orderBy=id->test"' ASC, IF((SELECT count (*) FROM users ) < 10, SLEEP(20), SLEEP(0)) DESC -- "'
Resume: Do not pass user-controlled column names to Query Builder without whitelisting.
Let’s look at the following simplified validation code
$userId = $request->input('id');
Validator::make($request->post(), [
'username' => ['required', "unique:users,name,$userId"],
]);
Since Laravel uses $userId
here to query that database and $userId
is not escaped, it will allow an attacker to perform an SQL injection.
The simplest thing that we can do here is to send a request with ID = 10|sometimes
,
which will alter the validation rule torequired|unique:users,username,10|sometimes
and will allow us to not skip the username in the request data,
depending on your application business logic, a bypass like this might create a security issue.
Another attack vector here could be to create an evil Regex validation, that is vulnerable to ReDoS attack and DDOS the app. For example, the following request would consume a lot of CPU and if multiple requests sent concurrently can cause a big CPU spike on the server.
PUT /api/users/1,id,name,444|regex:%23(.*a){100}%23
{
"username": "aaaaa.....ALOT_OF_REPETED_As_aaaaaaaaaa"
}
The simplest SQL injection here would be to just add an extra validation rule that is querying the database, for example
PUT /api/users/1,id,name,444|unique:users,secret_col_name_here
{
"username": "secret_value_to_check"
}
But important to mention, since using unique, we are able to provide both custom column name and values (values are not going through PDO parameter binding), the possibilities of SQL injection here could be not limited to just a simple attack vector mentioned above. For more details, check out Laravel Blog’s post "Unique Rule SQL Injection Warning".
Resume: The best prevention here is to not use user-provided data to create a validation rule.
DB::raw
function is dangerous when developers don’t escape passed data.
If you have to use DB::raw
function for some custom query, make sure you escape the passed data via DB::getPdo()->quote()
method.
Cross-Site Scripting can be very dangerous, for example an XSS attack in the admin panel can allow an attacker to inject a code like this:
Some text
<input onfocus='$.post("/admin/users", {name:"hacker", email:"[email protected]", password: "test123", });' autofocus />
test
Which will allow an attacker to create an admin user with his credentials and take over the admin panel.
Laravel Blade protects from most XSS attacks, so for example an attack like this will not work:
// $name = 'John Doe
<script>
alert('xss');
</script>
';
<div>{{ $name }}</div>
::: v-pre
Blade’s {{ }}
statement automatically encodes the output. So the server will send the following properly encoded code to the browser (which will prevent the XSS attack):
:::
<div>John Doe<script>alert("xss");</script></div>
But frameworks can’t handle all cases for developers.
Sometimes you need to output a text that contains HTML, and for it you will use {!! !!}
:
<div>{!! $htmlDescription !!}</div>
In this case Laravel can’t do anything for you and if the $htmlDescription
contains JavaScript code, it will be executed as-is and we will get an XSS attack.
Prevention tips:
- If you can, avoid outputting user supplied data without html encoding.
- If in some cases you know that the data can contain HTML, use HTML Purifier to clean the HTML from JS and unwanted tags before outputting the content.
If you are outputting user provided value as a link, here are some examples on how it can turn into an XSS attack.
// $user->website = "javascript:alert('Hacked!');";
<a href="{{ $user->website }}">My Website</a>
The alert(‘Hacked!’)
code will get executed when a user clicks on the link.
Prevention tips:
- Validate user provided links, in most cases, you need only to allow http/https schemas
- As an extra layer of security, before outputting you can replace any link that is not starting with http/https schema with some “#broken-link” value.
When you write a custom directive, don’t forget to use Laravel’s e
function to escape any code that is user provided.
An example of vulnerable code:
// Registering the directive code
Blade::directive('hello', function ($name) {
return "<?php echo 'Hello ' . $name; ?>";
});
// user.blade.php file
// $name = 'John Doe <script>alert("xss");</script>';
@hello($name);
Example: a User
model with:
protected $fillable = ['name', 'email', 'password', 'role'];
If in a Controller a developer uses something like $user->fill($request->all());
or $user->update($request->all());
,
a user/attacker can add an input with a role
name and submit the form and thus, set a role, e.g. to "administrator".
Prevention tips:
- Don’t use Mass Assignment
- Pass to Model only fields that have been validated:
$user->update($validator->validated());
- Use whitelisting instead of blacklisting (prefer
$fillable
over$guarded
, because it’s easy to forget to add a new column to$guarded
when you add it to a Model) - Use
$model->forceFill($data)
method with caution, make sure passed data cannot be manipulated by the user
🦄