cultuurnet / udb3-php

src/Offer/Commands/AbstractImportLabels.php

Spacing +3 added lines, -3 removed lines

@@ -48,14 +48,14 @@ 
     public function getLabelsToImport()
     {
         $labelNamesToKeep = array_map(
-            function (Label $label) {
+            function(Label $label) {
                 return $label->getName();
             },
             $this->labelsToKeepIfAlreadyOnOffer->toArray()
         );
 
         return $this->labels->filter(
-            function (Label $label) use ($labelNamesToKeep) {
+            function(Label $label) use ($labelNamesToKeep) {
                 return !in_array($label->getName(), $labelNamesToKeep);
             }
         );
@@ -67,7 +67,7 @@ 
     public function getNames()
     {
         return array_map(
-            function (Label $label) {
+            function(Label $label) {
                 return new StringLiteral($label->getName()->toString());
             },
             $this->getLabelsToImport()->toArray()

src/BackwardsCompatiblePayloadSerializerFactory.php

Spacing +29 added lines, -29 removed lines

@@ -69,7 +69,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\Events\EventCreated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 return self::removeLocationNameAndAddress(
                     self::addDefaultMainLanguage($serializedObject)
                 );
@@ -78,7 +78,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             MajorInfoUpdated::class,
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 return self::removeLocationNameAndAddress(
                     self::replaceEventIdWithItemId($serializedObject)
                 );
@@ -87,14 +87,14 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Place\Events\PlaceCreated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 return self::addDefaultMainLanguage($serializedObject);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Organizer\Events\OrganizerCreatedWithUniqueWebsite',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 return self::addDefaultMainLanguage($serializedObject);
             }
         );
@@ -105,7 +105,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\TitleTranslated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = TitleTranslated::class;
 
                 $serializedObject = self::replaceEventIdWithItemId($serializedObject);
@@ -116,7 +116,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\DescriptionTranslated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = DescriptionTranslated::class;
 
                 $serializedObject = self::replaceEventIdWithItemId($serializedObject);
@@ -131,49 +131,49 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Label\Events\MadeInvisible',
-            function (array $serializedObject) use ($labelRepository) {
+            function(array $serializedObject) use ($labelRepository) {
                 return self::addLabelName($serializedObject, $labelRepository);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Label\Events\MadeVisible',
-            function (array $serializedObject) use ($labelRepository) {
+            function(array $serializedObject) use ($labelRepository) {
                 return self::addLabelName($serializedObject, $labelRepository);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Label\Events\MadePrivate',
-            function (array $serializedObject) use ($labelRepository) {
+            function(array $serializedObject) use ($labelRepository) {
                 return self::addLabelName($serializedObject, $labelRepository);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Label\Events\MadePublic',
-            function (array $serializedObject) use ($labelRepository) {
+            function(array $serializedObject) use ($labelRepository) {
                 return self::addLabelName($serializedObject, $labelRepository);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Organizer\Events\LabelAdded',
-            function (array $serializedObject) use ($labelRepository) {
+            function(array $serializedObject) use ($labelRepository) {
                 return self::fixOrganizerLabelEvent($serializedObject, $labelRepository);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Organizer\Events\LabelRemoved',
-            function (array $serializedObject) use ($labelRepository) {
+            function(array $serializedObject) use ($labelRepository) {
                 return self::fixOrganizerLabelEvent($serializedObject, $labelRepository);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\Events\EventWasLabelled',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = LabelAdded::class;
 
                 $serializedObject = self::replaceEventIdWithItemId($serializedObject);
@@ -184,7 +184,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\EventWasTagged',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = LabelAdded::class;
 
                 $serializedObject = self::replaceEventIdWithItemId($serializedObject);
@@ -197,7 +197,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\TagErased',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = LabelRemoved::class;
 
                 $serializedObject = self::replaceEventIdWithItemId($serializedObject);
@@ -210,7 +210,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\Events\Unlabelled',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = LabelRemoved::class;
 
                 $serializedObject = self::replaceEventIdWithItemId($serializedObject);
@@ -225,7 +225,7 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Event\EventImportedFromUDB2',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = EventImportedFromUDB2::class;
 
                 return $serializedObject;
@@ -237,7 +237,7 @@ 
          */
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Place\Events\FacilitiesUpdated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject = self::replacePlaceIdWithItemId($serializedObject);
 
                 return $serializedObject;
@@ -249,7 +249,7 @@ 
          */
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Place\Events\GeoCoordinatesUpdated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject = self::replacePlaceIdWithItemId($serializedObject);
 
                 return $serializedObject;
@@ -259,7 +259,7 @@ 
         /**
          * BOOKING INFO EVENT
          */
-        $manipulateAvailability = function (array $serializedBookingInfo, $propertyName) {
+        $manipulateAvailability = function(array $serializedBookingInfo, $propertyName) {
             if (!isset($serializedBookingInfo[$propertyName]) || empty($serializedBookingInfo[$propertyName])) {
                 $serializedBookingInfo[$propertyName] = null;
                 return $serializedBookingInfo;
@@ -289,7 +289,7 @@ 
             return $serializedBookingInfo;
         };
 
-        $manipulateUrlLabel = function (array $serializedBookingInfo) {
+        $manipulateUrlLabel = function(array $serializedBookingInfo) {
             if (!isset($serializedBookingInfo['urlLabel'])) {
                 return $serializedBookingInfo;
             }
@@ -315,7 +315,7 @@ 
             return $serializedBookingInfo;
         };
 
-        $manipulateBookingInfoEvent = function (
+        $manipulateBookingInfoEvent = function(
             array $serializedEvent
         ) use (
             $manipulateAvailability,
@@ -359,7 +359,7 @@ 
         foreach ($refactoredEventEvents as $refactoredEventEvent) {
             $payloadManipulatingSerializer->manipulateEventsOfClass(
                 $refactoredEventEvent,
-                function (array $serializedObject) {
+                function(array $serializedObject) {
                     $serializedObject = self::replaceEventIdWithItemId($serializedObject);
                     return $serializedObject;
                 }
@@ -382,7 +382,7 @@ 
         foreach ($refactoredPlaceEvents as $refactoredPlaceEvent) {
             $payloadManipulatingSerializer->manipulateEventsOfClass(
                 $refactoredPlaceEvent,
-                function (array $serializedObject) {
+                function(array $serializedObject) {
                     $serializedObject = self::replacePlaceIdWithItemId($serializedObject);
                     return $serializedObject;
                 }
@@ -400,11 +400,11 @@ 
         foreach ($priceInfoEvents as $priceInfoEvent) {
             $payloadManipulatingSerializer->manipulateEventsOfClass(
                 $priceInfoEvent,
-                function (array $serializedObject) {
+                function(array $serializedObject) {
                     $payload = &$serializedObject['payload'];
                     $priceInfo = &$payload['price_info'];
                     $tariffs = array_map(
-                        function (array $tariff) {
+                        function(array $tariff) {
                             $name = $tariff['name'];
                             if (is_string($name)) {
                                 $name = ['nl' => $name];
@@ -425,7 +425,7 @@ 
          */
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             'CultuurNet\UDB3\Role\Events\ConstraintCreated',
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 $serializedObject['class'] = ConstraintAdded::class;
                 return self::addDefaultSapiVersion($serializedObject);
             }
@@ -433,14 +433,14 @@ 
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             ConstraintUpdated::class,
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 return self::addDefaultSapiVersion($serializedObject);
             }
         );
 
         $payloadManipulatingSerializer->manipulateEventsOfClass(
             ConstraintRemoved::class,
-            function (array $serializedObject) {
+            function(array $serializedObject) {
                 return self::addDefaultSapiVersion($serializedObject);
             }
         );

src/Place/PlaceTypeResolver.php

Spacing +3 added lines, -3 removed lines

@@ -40,9 +40,9 @@
 
     public function byId(StringLiteral $typeId): EventType
     {
-        if (!array_key_exists((string) $typeId, $this->types)) {
-            throw new Exception("Unknown place type id: " . $typeId);
+        if (!array_key_exists((string)$typeId, $this->types)) {
+            throw new Exception("Unknown place type id: ".$typeId);
         }
-        return $this->types[(string) $typeId];
+        return $this->types[(string)$typeId];
     }
 }

src/Place/CannotMarkPlaceAsDuplicate.php

Spacing +2 added lines, -2 removed lines

@@ -8,11 +8,11 @@
 {
     public static function becauseItIsDeleted(string $placeId): self
     {
-        return new static('Cannot mark place ' . $placeId . ' as duplicate because it is deleted');
+        return new static('Cannot mark place '.$placeId.' as duplicate because it is deleted');
     }
 
     public static function becauseItIsAlreadyADuplicate(string $placeId): self
     {
-        return new static('Cannot mark place ' . $placeId . ' as duplicate because it is already a duplicate');
+        return new static('Cannot mark place '.$placeId.' as duplicate because it is already a duplicate');
     }
 }

src/Place/CannotMarkPlaceAsCanonical.php

Spacing +2 added lines, -2 removed lines

@@ -8,11 +8,11 @@
 {
     public static function becauseItIsDeleted(string $placeId): self
     {
-        return new static('Cannot mark place ' . $placeId . ' as canonical because it is deleted');
+        return new static('Cannot mark place '.$placeId.' as canonical because it is deleted');
     }
 
     public static function becauseItIsAlreadyADuplicate(string $placeId): self
     {
-        return new static('Cannot mark place ' . $placeId . ' as canonical because it is a duplicate');
+        return new static('Cannot mark place '.$placeId.' as canonical because it is a duplicate');
     }
 }

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/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'}))
                 );