Add grouping parenthesis on condition about social issue and social action visibility

This improve readability and avoid errors with boolean operator precedence
This commit is contained in:
Julien Fastré 2024-02-07 14:22:18 +01:00
parent 574ad42a76
commit 13854e59de
Signed by: julienfastre
GPG Key ID: BDE2190974723FCB

View File

@ -292,8 +292,10 @@ class ActivityType
{
if ($this->socialIssuesVisible !== $this->socialActionsVisible) {
// if social issues are invisible then social actions cannot be optional or required + if social issues are optional then social actions shouldn't be required
if (0 === $this->socialIssuesVisible && (1 === $this->socialActionsVisible || 2 === $this->socialActionsVisible)
|| (1 === $this->socialIssuesVisible && 2 === $this->socialActionsVisible)) {
if (
(0 === $this->socialIssuesVisible && (1 === $this->socialActionsVisible || 2 === $this->socialActionsVisible))
|| (1 === $this->socialIssuesVisible && 2 === $this->socialActionsVisible)
) {
$context
->buildViolation('The socialActionsVisible value is not compatible with the socialIssuesVisible value')
->atPath('socialActionsVisible')