From d2704ad7e65dd723cb96d3da1252e76c576a8762 Mon Sep 17 00:00:00 2001 From: nobohan Date: Fri, 3 Dec 2021 14:34:21 +0100 Subject: [PATCH] activity: fix type hinting in activity form --- .../ChillActivityBundle/Form/ActivityType.php | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/src/Bundle/ChillActivityBundle/Form/ActivityType.php b/src/Bundle/ChillActivityBundle/Form/ActivityType.php index d332acdcb..0b9d10030 100644 --- a/src/Bundle/ChillActivityBundle/Form/ActivityType.php +++ b/src/Bundle/ChillActivityBundle/Form/ActivityType.php @@ -260,6 +260,11 @@ class ActivityType extends AbstractType return implode(',', $personIds); }, function (?string $personsAsString): array { + + if (null === $personsAsString) { + return []; + } + return array_map( fn (string $id): ?Person => $this->om->getRepository(Person::class)->findOneBy(['id' => (int) $id]), explode(',', $personsAsString) @@ -282,6 +287,10 @@ class ActivityType extends AbstractType return implode(',', $thirdpartyIds); }, function (?string $thirdpartyAsString): array { + if (null === $thirdpartyAsString) { + return []; + } + return array_map( fn (string $id): ?ThirdParty => $this->om->getRepository(ThirdParty::class)->findOneBy(['id' => (int) $id]), explode(',', $thirdpartyAsString) @@ -315,6 +324,9 @@ class ActivityType extends AbstractType return implode(',', $userIds); }, function (?string $usersAsString): array { + if (null === $socialActionsAsString) { + return []; + } return array_map( fn (string $id): ?User => $this->om->getRepository(User::class)->findOneBy(['id' => (int) $id]), explode(',', $usersAsString) @@ -379,7 +391,7 @@ class ActivityType extends AbstractType $timezoneUTC = new DateTimeZone('GMT'); /** @var DateTime $data */ $data = $formEvent->getData() === null ? - DateTime::createFromFormat('U', 300) : + DateTime::createFromFormat('U', '300') : $formEvent->getData(); $seconds = $data->getTimezone()->getOffset($data); $data->setTimeZone($timezoneUTC);