In PHP-7.4, nested ternary operators are to be bracketed
(as their default association is counter-intuitive) https://wiki.php.net/rfc/ternary_associativity
This commit is contained in:
committed by
Frédéric Guillot
parent
12d55408fe
commit
dcdbd3758b
@@ -156,7 +156,7 @@ class ActionParameterModel extends Base
|
|||||||
case 'dst_column_id':
|
case 'dst_column_id':
|
||||||
case 'column_id':
|
case 'column_id':
|
||||||
$column = $this->columnModel->getById($value);
|
$column = $this->columnModel->getById($value);
|
||||||
return empty($column) ? false : $this->columnModel->getColumnIdByTitle($project_id, $column['title']) ?: false;
|
return empty($column) ? false : ($this->columnModel->getColumnIdByTitle($project_id, $column['title']) ?: false);
|
||||||
case 'user_id':
|
case 'user_id':
|
||||||
case 'owner_id':
|
case 'owner_id':
|
||||||
if ($value == 0) {
|
if ($value == 0) {
|
||||||
@@ -165,7 +165,7 @@ class ActionParameterModel extends Base
|
|||||||
return $this->projectPermissionModel->isAssignable($project_id, $value) ? $value : false;
|
return $this->projectPermissionModel->isAssignable($project_id, $value) ? $value : false;
|
||||||
case 'swimlane_id':
|
case 'swimlane_id':
|
||||||
$column = $this->swimlaneModel->getById($value);
|
$column = $this->swimlaneModel->getById($value);
|
||||||
return empty($column) ? false : $this->swimlaneModel->getIdByName($project_id, $column['name']) ?: false;
|
return empty($column) ? false : ($this->swimlaneModel->getIdByName($project_id, $column['name']) ?: false);
|
||||||
default:
|
default:
|
||||||
return $value;
|
return $value;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user