cultuurnet / udb3-php

src/Offer/Offer.php

Spacing +6 added lines, -6 removed lines

@@ -221,7 +221,7 @@ 
         $sameFacilities = array_uintersect(
             $facilities1,
             $facilities2,
-            function (Facility $facility1, Facility $facility2) {
+            function(Facility $facility1, Facility $facility2) {
                 return strcmp($facility1->getId(), $facility2->getId());
             }
         );
@@ -270,7 +270,7 @@ 
      */
     public function importLabels(Labels $labels, Labels $labelsToKeepIfAlreadyOnOffer, Labels $labelsToRemoveWhenOnOffer)
     {
-        $convertLabelClass = function (\CultuurNet\UDB3\Model\ValueObject\Taxonomy\Label\Label $label) {
+        $convertLabelClass = function(\CultuurNet\UDB3\Model\ValueObject\Taxonomy\Label\Label $label) {
             return new Label(
                 $label->getName()->toString(),
                 $label->isVisible()
@@ -306,7 +306,7 @@ 
         foreach ($addedLabels->asArray() as $addedLabel) {
             $importLabels = $importLabels->with(
                 new \CultuurNet\UDB3\Model\ValueObject\Taxonomy\Label\Label(
-                    new LabelName((string) $addedLabel),
+                    new LabelName((string)$addedLabel),
                     $addedLabel->isVisible()
                 )
             );
@@ -707,7 +707,7 @@ 
         $importImages = $imageCollection->toArray();
         $currentImages = $currentImageCollection->toArray();
 
-        $compareImages = function (Image $a, Image $b) {
+        $compareImages = function(Image $a, Image $b) {
             $idA = $a->getMediaObjectId()->toNative();
             $idB = $b->getMediaObjectId()->toNative();
             return strcmp($idA, $idB);
@@ -854,7 +854,7 @@ 
             if ($this->rejectedReason && $reason->sameValueAs($this->rejectedReason)) {
                 return true; // nothing left to do if the offer has already been rejected for the same reason
             } else {
-                throw new Exception('The offer has already been rejected for another reason: ' . $this->rejectedReason);
+                throw new Exception('The offer has already been rejected for another reason: '.$this->rejectedReason);
             }
         }
 
@@ -1032,7 +1032,7 @@ 
         $dutchImagesList = $imagesEvent->getImages()->toArray();
         $translatedImagesList = array_filter(
             $this->images->toArray(),
-            function (Image $image) {
+            function(Image $image) {
                 return $image->getLanguage()->getCode() !== 'nl';
             }
         );

src/Search/ResultsGenerator.php

Spacing +2 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
 
             $total = $results->getTotalItems()->toNative();
 
-            $this->logger->info('Search API reported ' . $total . ' results');
+            $this->logger->info('Search API reported '.$total.' results');
 
             foreach ($results->getItems() as $item) {
                 $id = $item->getId();
@@ -123,7 +123,7 @@ 
                         'query_duplicate_event',
                         array(
                             'query' => $query,
-                            'error' => "Found duplicate offer {$id} on page {$currentPage}, " .
+                            'error' => "Found duplicate offer {$id} on page {$currentPage}, ".
                                 "occurred first time on page {$ids[$id]}.",
                         )
                     );

src/Event/LocationMarkedAsDuplicateProcessManager.php

Spacing +5 added lines, -5 removed lines

@@ -75,7 +75,7 @@ 
             if (!$result->getType()->sameValueAs(OfferType::EVENT())) {
                 $skipped[] = $result->getId();
                 $this->logger->warning(
-                    'Skipped result with id ' . $result->getId() . ' because it\'s not an event according to the @id parser.'
+                    'Skipped result with id '.$result->getId().' because it\'s not an event according to the @id parser.'
                 );
                 continue;
             }
@@ -89,17 +89,17 @@ 
             $this->commandBus->dispatch($command);
 
             $this->logger->info(
-                'Dispatched UpdateLocation for result with id ' . $command->getItemId()
+                'Dispatched UpdateLocation for result with id '.$command->getItemId()
             );
         }
 
         $updated = count($commands);
         $total = $updated + count($skipped);
 
-        $this->logger->info('Received ' . $total . ' results from the search api.');
-        $this->logger->info('Updated ' . $updated . ' events to the canonical location.');
+        $this->logger->info('Received '.$total.' results from the search api.');
+        $this->logger->info('Updated '.$updated.' events to the canonical location.');
         $this->logger->info(
-            'Skipped ' . count($skipped) . ' events:' . PHP_EOL . implode(PHP_EOL, $skipped)
+            'Skipped '.count($skipped).' events:'.PHP_EOL.implode(PHP_EOL, $skipped)
         );
     }
 }

src/Offer/OfferCommandHandler.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use CultuurNet\UDB3\Offer\Commands\AbstractAddLabel;
 use CultuurNet\UDB3\Offer\Commands\AbstractDeleteCurrentOrganizer;
 use CultuurNet\UDB3\Offer\Commands\AbstractImportLabels;
-use CultuurNet\UDB3\Offer\Commands\AbstractLabelCommand;
 use CultuurNet\UDB3\Offer\Commands\AbstractRemoveLabel;
 use CultuurNet\UDB3\Offer\Commands\AbstractDeleteOffer;
 use CultuurNet\UDB3\Offer\Commands\AbstractDeleteOrganizer;

Spacing +2 added lines, -2 removed lines

@@ -109,7 +109,7 @@ 
             $matches = [];
             if (preg_match('/^handle(.+)$/', $method, $matches)) {
                 $command = $matches[1];
-                $classNameMethod = 'get' . $command . 'ClassName';
+                $classNameMethod = 'get'.$command.'ClassName';
 
                 if (method_exists($this, $classNameMethod)) {
                     $commandFullClassName = call_user_func(array($this, $classNameMethod));
@@ -304,7 +304,7 @@ 
     {
         $offer = $this->load($addLabel->getItemId());
 
-        $labelName = (string) $addLabel->getLabel();
+        $labelName = (string)$addLabel->getLabel();
         $labelVisibility = $addLabel->getLabel()->isVisible();
 
         // Load the label read model so we can determine the correct visibility.

src/Search/Sapi3SearchService.php

Spacing +4 added lines, -4 removed lines

@@ -80,7 +80,7 @@ 
 
         $offerRequest = new Request(
             'GET',
-            (string) $offerQuery,
+            (string)$offerQuery,
             $headers
         );
 
@@ -89,14 +89,14 @@ 
             ->getBody()
             ->getContents();
 
-        $this->logger->debug('Send SAPI3 request with the following parameters: ' . json_encode($queryParameters));
-        $this->logger->debug('Response data: ' . $searchResponseData);
+        $this->logger->debug('Send SAPI3 request with the following parameters: '.json_encode($queryParameters));
+        $this->logger->debug('Response data: '.$searchResponseData);
 
         $searchResponseData = json_decode($searchResponseData);
 
         $offerIds = array_reduce(
             $searchResponseData->{'member'},
-            function (OfferIdentifierCollection $offerIds, $item) {
+            function(OfferIdentifierCollection $offerIds, $item) {
                 return $offerIds->with(
                     $this->offerIdentifier->fromIri(Url::fromNative($item->{'@id'}))
                 );

src/CalendarFactory.php

Spacing +10 added lines, -10 removed lines

@@ -26,14 +26,14 @@ 
         if ($cdbCalendar instanceof \CultureFeed_Cdb_Data_Calendar_PeriodList) {
             /** @var \CultureFeed_Cdb_Data_Calendar_Period $period */
             $period = $cdbCalendar->current();
-            $startDateString = $period->getDateFrom() . 'T00:00:00';
+            $startDateString = $period->getDateFrom().'T00:00:00';
         } elseif ($cdbCalendar instanceof \CultureFeed_Cdb_Data_Calendar_TimestampList) {
             /** @var \CultureFeed_Cdb_Data_Calendar_Timestamp $timestamp */
             $timestamp = $cdbCalendar->current();
             if ($timestamp->getStartTime()) {
-                $startDateString = $timestamp->getDate() . 'T' . substr($timestamp->getStartTime(), 0, 5) . ':00';
+                $startDateString = $timestamp->getDate().'T'.substr($timestamp->getStartTime(), 0, 5).':00';
             } else {
-                $startDateString = $timestamp->getDate() . 'T00:00:00';
+                $startDateString = $timestamp->getDate().'T00:00:00';
             }
         }
         $startDate = !empty($startDateString) ? DateTimeFactory::dateTimeFromDateString($startDateString) : null;
@@ -46,17 +46,17 @@ 
         if ($cdbCalendar instanceof \CultureFeed_Cdb_Data_Calendar_PeriodList) {
             /** @var \CultureFeed_Cdb_Data_Calendar_Period $period */
             $period = $cdbCalendar->current();
-            $endDateString = $period->getDateTo() . 'T00:00:00';
+            $endDateString = $period->getDateTo().'T00:00:00';
         } elseif ($cdbCalendar instanceof \CultureFeed_Cdb_Data_Calendar_TimestampList) {
             $firstTimestamp = $cdbCalendar->current();
             /** @var \CultureFeed_Cdb_Data_Calendar_Timestamp $timestamp */
             $cdbCalendarAsArray = iterator_to_array($cdbCalendar);
             $timestamp = $this->getLastTimestamp($cdbCalendarAsArray, $firstTimestamp);
             if ($timestamp->getEndTime()) {
-                $endDateString = $timestamp->getDate() . 'T' . $timestamp->getEndTime();
+                $endDateString = $timestamp->getDate().'T'.$timestamp->getEndTime();
             } else {
                 $endTime = $timestamp->getStartTime() ? $timestamp->getStartTime() : '00:00:00';
-                $endDateString = $timestamp->getDate() . 'T' . $endTime;
+                $endDateString = $timestamp->getDate().'T'.$endTime;
             }
         }
         $endDate = !empty($endDateString) ? DateTimeFactory::dateTimeFromDateString($endDateString) : null;
@@ -74,12 +74,12 @@ 
                 $cdbCalendar->next();
 
                 $startTime = $timestamp->getStartTime() ? $timestamp->getStartTime() : '00:00:00';
-                $startDateString = $timestamp->getDate() . 'T' . $startTime;
+                $startDateString = $timestamp->getDate().'T'.$startTime;
 
                 if ($timestamp->getEndTime()) {
-                    $endDateString = $timestamp->getDate() . 'T' . $timestamp->getEndTime();
+                    $endDateString = $timestamp->getDate().'T'.$timestamp->getEndTime();
                 } else {
-                    $endDateString = $timestamp->getDate() . 'T' . $startTime;
+                    $endDateString = $timestamp->getDate().'T'.$startTime;
                 }
 
                 $timestamp = $this->createTimestamp(
@@ -96,7 +96,7 @@ 
             }
 
             $periods = array_map(
-                function (array $periodParts) {
+                function(array $periodParts) {
                     $firstPart = array_shift($periodParts);
                     $lastPart = array_pop($periodParts);
                     return new Timestamp(