By murz on
Change record status:
Draft (View all draft change records)
Project:
Issue links:
Description:
In many cases, conditions, issued on separate lines, are actually more readable, than as a single line. You can see examples in the #3392321: Relax the "Conditions should not be wrapped into multiple lines" rule for more code readability.
To improve this, the rule about conditions is extended to aloow multi-lined conditions:
Previous text
Conditions should not be wrapped into multiple lines.
New text
Option A
Conditions should not be wrapped into multiple lines if they are simple. Long conditions can be formatted by separate lines per condition, with each condition on a separate line. Example:
if ( $this->entity->isPublished() && $this->entity->is_protected_by_access_code->value == TRUE && $this->requestStack->getCurrentRequest()->query->get('access-code') == $accessCode ) { // Do something. }
Impacts:
Module developers