Skip to content

Commit

Permalink
#1246 DDC-3487 - removed unused assignment, direct return instead
Browse files Browse the repository at this point in the history
  • Loading branch information
Ocramius committed Jan 13, 2015
1 parent 17a865e commit c436612
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions lib/Doctrine/ORM/Persisters/ManyToManyPersister.php
Original file line number Diff line number Diff line change
Expand Up @@ -289,8 +289,9 @@ protected function generateFilterConditionSQL(ClassMetadata $targetEntity, $targ
}
}

$sql = implode(' AND ', $filterClauses);
return $sql ? '(' . $sql . ')' : '';
return $filterClauses
? '(' . implode(' AND ', $filterClauses) . ')'
: '';
}

/**
Expand Down

0 comments on commit c436612

Please sign in to comment.