cultuurnet / udb3-php

src/Media/Events/MediaObjectCreated.php

Doc Comments +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
     }
 
     /**
-     * @return StringLiteral
+     * @return string
      */
     public function getDescription()
     {
@@ -91,7 +91,7 @@ 
     }
 
     /**
-     * @return StringLiteral
+     * @return string
      */
     public function getCopyrightHolder()
     {

Spacing +2 added lines, -2 removed lines

@@ -124,8 +124,8 @@
             'mime_type' => $this->getMimeType()->toNative(),
             'description' => $this->getDescription()->toNative(),
             'copyright_holder' => $this->getCopyrightHolder()->toNative(),
-            'source_location' => (string) $this->getSourceLocation(),
-            'language' => (string) $this->getLanguage(),
+            'source_location' => (string)$this->getSourceLocation(),
+            'language' => (string)$this->getLanguage(),
         );
     }
 

src/Media/ImageUploaderService.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Broadway\CommandHandling\CommandBusInterface;
 use Broadway\UuidGenerator\UuidGeneratorInterface;
-use CultuurNet\Entry\Number;
 use CultuurNet\UDB3\Language;
 use CultuurNet\UDB3\Media\Commands\UploadImage;
 use CultuurNet\UDB3\Media\Properties\MIMEType;

src/Media/Serialization/MediaObjectSerializer.php

Spacing +8 added lines, -8 removed lines

@@ -41,11 +41,11 @@ 
         $normalizedData = [
             '@id' => $this->iriGenerator->iri($mediaObject->getMediaObjectId()),
             '@type' => $this->serializeMimeType($mediaObject->getMimeType()),
-            'contentUrl' => (string) $mediaObject->getSourceLocation(),
-            'thumbnailUrl' => (string) $mediaObject->getSourceLocation(),
-            'description' => (string) $mediaObject->getDescription(),
-            'copyrightHolder' => (string) $mediaObject->getCopyrightHolder(),
-            'inLanguage' => (string) $mediaObject->getLanguage(),
+            'contentUrl' => (string)$mediaObject->getSourceLocation(),
+            'thumbnailUrl' => (string)$mediaObject->getSourceLocation(),
+            'description' => (string)$mediaObject->getDescription(),
+            'copyrightHolder' => (string)$mediaObject->getCopyrightHolder(),
+            'inLanguage' => (string)$mediaObject->getLanguage(),
         ];
 
         return $normalizedData;
@@ -53,18 +53,18 @@ 
 
     public function serializeMimeType(MIMEType $mimeType)
     {
-        $typeParts = explode('/', (string) $mimeType);
+        $typeParts = explode('/', (string)$mimeType);
         $type = array_shift($typeParts);
 
         if ($type === 'image') {
             return 'schema:ImageObject';
         }
 
-        if ((string) $mimeType === 'application/octet-stream') {
+        if ((string)$mimeType === 'application/octet-stream') {
             return 'schema:mediaObject';
         }
 
-        throw new UnsupportedException('Unsupported MIME-type "'. $mimeType .'"');
+        throw new UnsupportedException('Unsupported MIME-type "'.$mimeType.'"');
     }
 
     public function deserialize($data, $type, $format, array $context = array())

src/Media/MediaManager.php

Spacing +7 added lines, -7 removed lines

@@ -78,11 +78,11 @@ 
     ) {
         try {
             $existingMediaObject = $this->repository->load($id);
-            $this->logger->info('Trying to create media with id: ' .$id . ' but it already exists. Using existing Media Object!');
+            $this->logger->info('Trying to create media with id: '.$id.' but it already exists. Using existing Media Object!');
 
             return $existingMediaObject;
         } catch (AggregateNotFoundException $exception) {
-            $this->logger->info('No existing media with id: ' .$id . ' found. Creating a new Media Object!');
+            $this->logger->info('No existing media with id: '.$id.' found. Creating a new Media Object!');
         }
 
         $mediaObject = MediaObject::create(
@@ -117,7 +117,7 @@ 
 
         $this->filesystem->rename(
             $uploadImage->getFilePath(),
-            $this->mediaDirectory . '/' . $destinationPath
+            $this->mediaDirectory.'/'.$destinationPath
         );
 
         $this->create(
@@ -129,7 +129,7 @@ 
             $uploadImage->getLanguage()
         );
 
-        $jobInfo = ['file_id' => (string) $uploadImage->getFileId()];
+        $jobInfo = ['file_id' => (string)$uploadImage->getFileId()];
         $this->logger->info('job_info', $jobInfo);
     }
 
@@ -139,7 +139,7 @@ 
     public function get(UUID $fileId)
     {
         try {
-            $mediaObject = $this->repository->load((string) $fileId);
+            $mediaObject = $this->repository->load((string)$fileId);
         } catch (AggregateNotFoundException $e) {
             throw new MediaObjectNotFoundException(
                 sprintf("Media object with id '%s' not found", $fileId),
@@ -163,8 +163,8 @@ 
         $image = new Image(
             $mediaObject->getMediaObjectId(),
             $mediaObject->getMimeType(),
-            new Description((string) $mediaObject->getDescription()),
-            new CopyrightHolder((string) $mediaObject->getCopyrightHolder()),
+            new Description((string)$mediaObject->getDescription()),
+            new CopyrightHolder((string)$mediaObject->getCopyrightHolder()),
             $mediaObject->getSourceLocation(),
             $mediaObject->getLanguage()
         );

src/Offer/ReadModel/JSONLD/OfferLDProjector.php

Spacing +11 added lines, -11 removed lines

@@ -151,7 +151,7 @@ 
 
             if (preg_match('/^apply(.+)$/', $method, $matches)) {
                 $event = $matches[1];
-                $classNameMethod = 'get' . $event . 'ClassName';
+                $classNameMethod = 'get'.$event.'ClassName';
 
                 if (method_exists($this, $classNameMethod)) {
                     $eventFullClassName = call_user_func(array($this, $classNameMethod));
@@ -292,7 +292,7 @@ 
         $labelsProperty = $labelAdded->getLabel()->isVisible() ? 'labels' : 'hiddenLabels';
 
         $labels = isset($offerLd->{$labelsProperty}) ? $offerLd->{$labelsProperty} : [];
-        $label = (string) $labelAdded->getLabel();
+        $label = (string)$labelAdded->getLabel();
 
         $labels[] = $label;
         $offerLd->{$labelsProperty} = array_unique($labels);
@@ -318,7 +318,7 @@ 
             if (isset($offerLd->{$labelsProperty}) && is_array($offerLd->{$labelsProperty})) {
                 $offerLd->{$labelsProperty} = array_filter(
                     $offerLd->{$labelsProperty},
-                    function ($label) use ($labelRemoved) {
+                    function($label) use ($labelRemoved) {
                         return !$labelRemoved->getLabel()->equals(
                             new Label($label)
                         );
@@ -418,7 +418,7 @@ 
             return;
         }
 
-        $imageId = (string) $imageRemoved->getImage()->getMediaObjectId();
+        $imageId = (string)$imageRemoved->getImage()->getMediaObjectId();
 
         /**
          * Matches any object that is not the removed image.
@@ -429,7 +429,7 @@ 
          * @return bool
          *  Returns true when the media object does not match the image to remove.
          */
-        $shouldNotBeRemoved = function ($mediaObject) use ($imageId) {
+        $shouldNotBeRemoved = function($mediaObject) use ($imageId) {
             $containsId = !!strpos($mediaObject->{'@id'}, $imageId);
             return !$containsId;
         };
@@ -468,7 +468,7 @@ 
         $document = $this->loadDocumentFromRepository($mainImageSelected);
         $offerLd = $document->getBody();
         $imageId = $mainImageSelected->getImage()->getMediaObjectId();
-        $mediaObjectMatcher = function ($matchingMediaObject, $currentMediaObject) use ($imageId) {
+        $mediaObjectMatcher = function($matchingMediaObject, $currentMediaObject) use ($imageId) {
             if (!$matchingMediaObject && $this->mediaObjectMatchesId($currentMediaObject, $imageId)) {
                 $matchingMediaObject = $currentMediaObject;
             }
@@ -493,7 +493,7 @@ 
      */
     protected function mediaObjectMatchesId($mediaObject, UUID $mediaObjectId)
     {
-        return strpos($mediaObject->{'@id'}, (string) $mediaObjectId) > 0;
+        return strpos($mediaObject->{'@id'}, (string)$mediaObjectId) > 0;
     }
 
     /**
@@ -657,7 +657,7 @@ 
         $document = $this->loadDocumentFromRepository($typicalAgeRangeUpdated);
 
         $offerLd = $document->getBody();
-        $offerLd->typicalAgeRange = (string) $typicalAgeRangeUpdated->getTypicalAgeRange();
+        $offerLd->typicalAgeRange = (string)$typicalAgeRangeUpdated->getTypicalAgeRange();
 
         return $document->withBody($offerLd);
     }
@@ -785,14 +785,14 @@ 
         $images = $imagesEvent->getImages();
         $currentMediaObjects = isset($offerLd->mediaObject) ? $offerLd->mediaObject : [];
         $dutchMediaObjects = array_map(
-            function (Image $image) {
+            function(Image $image) {
                 return $this->mediaObjectSerializer->serialize($image, 'json-ld');
             },
             $images->toArray()
         );
         $translatedMediaObjects = array_filter(
             $currentMediaObjects,
-            function ($image) {
+            function($image) {
                 return $image->inLanguage !== 'nl';
             }
         );
@@ -805,7 +805,7 @@ 
         }
 
         if (isset($mainImage)) {
-            $offerLd->image = (string) $mainImage->getSourceLocation();
+            $offerLd->image = (string)$mainImage->getSourceLocation();
         }
     }
 

src/Offer/Offer.php

Spacing +2 added lines, -2 removed lines

@@ -426,7 +426,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);
             }
         }
 
@@ -557,7 +557,7 @@ 
         $dutchImagesList = $imagesEvent->getImages()->toArray();
         $translatedImagesList = array_filter(
             $this->images->toArray(),
-            function (Image $image) {
+            function(Image $image) {
                 return $image->getLanguage()->getCode() !== 'nl';
             }
         );

src/CalendarFactory.php

Spacing +10 added lines, -10 removed lines

@@ -25,14 +25,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;
@@ -45,17 +45,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 = iterator_count($cdbCalendar) > 1 ? end($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;
@@ -73,12 +73,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(
@@ -95,7 +95,7 @@ 
             }
 
             $periods = array_map(
-                function (array $periodParts) {
+                function(array $periodParts) {
                     $firstPart = array_shift($periodParts);
                     $lastPart = array_pop($periodParts);
                     return new Timestamp(

src/Location/Location.php

Indentation +3 added lines, -3 removed lines

@@ -62,9 +62,9 @@
     public function serialize()
     {
         return [
-          'cdbid' => $this->cdbid,
-          'name' => $this->name->toNative(),
-          'address' => $this->address->serialize(),
+            'cdbid' => $this->cdbid,
+            'name' => $this->name->toNative(),
+            'address' => $this->address->serialize(),
         ];
     }
 

src/Label/ReadModels/JSON/Repository/Doctrine/DBALWriteRepository.php

Spacing +3 added lines, -3 removed lines

@@ -129,7 +129,7 @@ 
         $queryBuilder = $this->createQueryBuilder()
             ->update($this->getTableName()->toNative())
             ->set($column, '?')
-            ->where(SchemaConfigurator::UUID_COLUMN . ' = ?')
+            ->where(SchemaConfigurator::UUID_COLUMN.' = ?')
             ->setParameters([
                 $value ? 1 : 0,
                 $uuid->toNative(),
@@ -155,7 +155,7 @@ 
                 SchemaConfigurator::COUNT_COLUMN,
                 $newCount < 0 ? 0 : $newCount
             )
-            ->where(SchemaConfigurator::UUID_COLUMN . ' = ?')
+            ->where(SchemaConfigurator::UUID_COLUMN.' = ?')
             ->setParameters([$uuid->toNative()]);
 
         $queryBuilder->execute();
@@ -170,7 +170,7 @@ 
         $queryBuilder = $this->createQueryBuilder()
             ->select([SchemaConfigurator::COUNT_COLUMN])
             ->from($this->getTableName()->toNative())
-            ->where(SchemaConfigurator::UUID_COLUMN . ' = ?')
+            ->where(SchemaConfigurator::UUID_COLUMN.' = ?')
             ->setParameters([$uuid->toNative()]);
 
         $statement = $queryBuilder->execute();