Merge branch 'simplify-statement' into 'master'

fix: Simplify statements and fix syntax.

See merge request Chill-Projet/chill-bundles!20
This commit is contained in:
Julien Fastré 2021-04-26 11:33:08 +00:00
commit 8adb05c24f
2 changed files with 4 additions and 5 deletions

View File

@ -104,7 +104,7 @@ class Mailer
* @param \User $to
* @param array $subject Subject of the message [ 0 => $message (required), 1 => $parameters (optional), 3 => $domain (optional) ]
* @param array $bodies The bodies. An array where keys are the contentType and values the bodies
* @param \callable $callback a callback to customize the message (add attachment, etc.)
* @param callable $callback a callback to customize the message (add attachment, etc.)
*/
public function sendNotification(
$recipient,

View File

@ -152,7 +152,7 @@ class SingleTaskController extends AbstractController
$em = $this->getDoctrine()->getManager();
$task = $em->getRepository(SingleTask::class)->find($id);
if (!is_null($task->getPerson() === !null)) {
if ($task->getPerson() !== null) {
$personId = $task->getPerson()->getId();
if ($personId === null) {
@ -206,7 +206,7 @@ class SingleTaskController extends AbstractController
$em = $this->getDoctrine()->getManager();
$task = $em->getRepository(SingleTask::class)->find($id);
if (!is_null($task->getPerson() === !null)) {
if ($task->getPerson() !== null) {
$personId = $task->getPerson()->getId();
if ($personId === null) {
return new Response("You must provide a person_id", Response::HTTP_BAD_REQUEST);
@ -303,8 +303,7 @@ class SingleTaskController extends AbstractController
throw $this->createNotFoundException('Unable to find Task entity.');
}
if (!is_null($task->getPerson() === !null)) {
if ($task->getPerson() !== null) {
$personId = $task->getPerson()->getId();
if ($personId === null) {
return new Response("You must provide a person_id", Response::HTTP_BAD_REQUEST);