cultuurnet / udb3-php

src/Role/ReadModel/Constraints/Doctrine/UserConstraintsReadRepository.php

Spacing +7 added lines, -7 removed lines

@@ -63,34 +63,34 @@
         $userRolesSubQuery = $this->connection->createQueryBuilder()
             ->select(PermissionsSchemaConfigurator::ROLE_ID_COLUMN)
             ->from($this->userRolesTableName->toNative())
-            ->where(PermissionsSchemaConfigurator::USER_ID_COLUMN . ' = :userId');
+            ->where(PermissionsSchemaConfigurator::USER_ID_COLUMN.' = :userId');
 
         $queryBuilder = $this->connection->createQueryBuilder();
         $userConstraintsQuery = $queryBuilder
-            ->select('rs.' . SearchSchemaConfigurator::CONSTRAINT_COLUMN)
+            ->select('rs.'.SearchSchemaConfigurator::CONSTRAINT_COLUMN)
             ->from($this->rolesSearchTableName, 'rs')
             ->innerJoin(
                 'rs',
                 sprintf('(%s)', $userRolesSubQuery->getSQL()),
                 'ur',
-                'rs.' . SearchSchemaConfigurator::UUID_COLUMN . ' = ur.' . PermissionsSchemaConfigurator::ROLE_ID_COLUMN
+                'rs.'.SearchSchemaConfigurator::UUID_COLUMN.' = ur.'.PermissionsSchemaConfigurator::ROLE_ID_COLUMN
             )
             ->innerJoin(
                 'rs',
                 $this->rolePermissionsTableName->toNative(),
                 'rp',
-                'rs.' . SearchSchemaConfigurator::UUID_COLUMN . ' = rp.' . PermissionsSchemaConfigurator::ROLE_ID_COLUMN
+                'rs.'.SearchSchemaConfigurator::UUID_COLUMN.' = rp.'.PermissionsSchemaConfigurator::ROLE_ID_COLUMN
             )
-            ->where(PermissionsSchemaConfigurator::PERMISSION_COLUMN . ' = :permission')
+            ->where(PermissionsSchemaConfigurator::PERMISSION_COLUMN.' = :permission')
             ->andWhere($queryBuilder->expr()->isNotNull(
-                'rs.' . SearchSchemaConfigurator::CONSTRAINT_COLUMN
+                'rs.'.SearchSchemaConfigurator::CONSTRAINT_COLUMN
             ))
             ->setParameter('userId', $userId->toNative())
             ->setParameter('permission', $permission->toNative());
 
         $results = $userConstraintsQuery->execute()->fetchAll(\PDO::FETCH_COLUMN);
 
-        return array_map(function ($constraint) {
+        return array_map(function($constraint) {
             return new StringLiteral($constraint);
         }, $results);
     }

src/Offer/Commands/AbstractTranslateTitle.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     }
 
     /**
-     * @return StringLiteral
+     * @return string
      */
     public function getTitle()
     {

src/Offer/Commands/AbstractTranslateDescription.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     }
 
     /**
-     * @return StringLiteral
+     * @return string
      */
     public function getTitle()
     {

src/EventExport/Notification/DefaultHTMLBodyFactory.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
         EventExportResult $eventExportResult
     ) {
         $url = $eventExportResult->getUrl();
-        return '<p>Beste, <br /><br />Hierbij vind je de link naar de door jou geëxporteerde documenten uit UiTdatabank: <a href="' . $url . '">' . $url . '</a><br /><br />
+        return '<p>Beste, <br /><br />Hierbij vind je de link naar de door jou geëxporteerde documenten uit UiTdatabank: <a href="'.$url.'">'.$url.'</a><br /><br />
         Mocht je vragen hebben, of meer informatie wensen over onze diensten, kan je terecht bij vragen@uitdatabank.be.<br /><br />
         Met vriendelijke groeten,<br />Het UiTdatabank team</p>';
     }

src/EventExport/Notification/DefaultPlainTextBodyFactory.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
         return 'Beste,
 
-        Hierbij vind je de link naar de door jou geëxporteerde documenten uit UiTdatabank: ' . $eventExportResult->getUrl() . '
+        Hierbij vind je de link naar de door jou geëxporteerde documenten uit UiTdatabank: ' . $eventExportResult->getUrl().'
 
         Mocht je vragen hebben, of meer informatie wensen over onze diensten, kan je terecht bij vragen@uitdatabank.be.
 

src/Offer/ReadModel/JSONLD/OfferLDProjector.php

Doc Comments +1 added lines, -1 removed lines

@@ -624,7 +624,7 @@
     }
 
     /**
-     * @return callable
+     * @return \Closure
      */
     private function reject()
     {

Spacing +10 added lines, -10 removed lines

@@ -142,7 +142,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));
@@ -272,7 +272,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);
@@ -297,7 +297,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)
                         );
@@ -394,7 +394,7 @@ 
             return;
         }
 
-        $imageId = (string) $imageRemoved->getImage()->getMediaObjectId();
+        $imageId = (string)$imageRemoved->getImage()->getMediaObjectId();
 
         /**
          * Matches any object that is not the removed image.
@@ -405,7 +405,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;
         };
@@ -443,7 +443,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;
             }
@@ -468,7 +468,7 @@ 
      */
     protected function mediaObjectMatchesId($mediaObject, UUID $mediaObjectId)
     {
-        return strpos($mediaObject->{'@id'}, (string) $mediaObjectId) > 0;
+        return strpos($mediaObject->{'@id'}, (string)$mediaObjectId) > 0;
     }
 
     /**
@@ -649,7 +649,7 @@ 
      */
     protected function applyPublished(AbstractPublished $published)
     {
-        $this->applyEventTransformation($published, function ($offerLd) use ($published) {
+        $this->applyEventTransformation($published, function($offerLd) use ($published) {
             $offerLd->workflowStatus = WorkflowStatus::READY_FOR_VALIDATION()->getName();
 
             $publicationDate = $published->getPublicationDate();
@@ -662,7 +662,7 @@ 
      */
     protected function applyApproved(AbstractApproved $approved)
     {
-        $this->applyEventTransformation($approved, function ($offerLd) {
+        $this->applyEventTransformation($approved, function($offerLd) {
             $offerLd->workflowStatus = WorkflowStatus::APPROVED()->getName();
         });
     }
@@ -698,7 +698,7 @@ 
      */
     private function reject()
     {
-        return function ($offerLd) {
+        return function($offerLd) {
             $offerLd->workflowStatus = WorkflowStatus::REJECTED()->getName();
         };
     }

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use CultuurNet\UDB3\EntityServiceInterface;
 use CultuurNet\UDB3\Event\ReadModel\DocumentRepositoryInterface;
 use CultuurNet\UDB3\Event\ReadModel\JSONLD\CdbXMLImporter;
-use CultuurNet\UDB3\Event\ReadModel\JSONLD\CdbXMLLongDescriptionFilter;
 use CultuurNet\UDB3\Event\ReadModel\JSONLD\OrganizerServiceInterface;
 use CultuurNet\UDB3\EventHandling\DelegateEventHandlingToSpecificMethodTrait;
 use CultuurNet\UDB3\Iri\IriGeneratorInterface;

src/Offer/Offer.php

Spacing +2 added lines, -2 removed lines

@@ -257,7 +257,7 @@ 
     {
         $equalImages = array_filter(
             $this->mediaObjects,
-            function ($existingMediaObjectId) use ($image) {
+            function($existingMediaObjectId) use ($image) {
                 return $image
                     ->getMediaObjectId()
                     ->sameValueAs($existingMediaObjectId);
@@ -381,7 +381,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);
             }
         }
 

src/Security/CommandAuthorizationException.php

Spacing +4 added lines, -4 removed lines

@@ -26,10 +26,10 @@
         StringLiteral $userId,
         AuthorizableCommandInterface $command
     ) {
-        parent::__construct('User with id: ' . $userId->toNative() .
-            ' has no permission: "' . $command->getPermission()->toNative() .
-            '" on item: ' . $command->getItemId() .
-            ' when executing command: ' . get_class($command));
+        parent::__construct('User with id: '.$userId->toNative().
+            ' has no permission: "'.$command->getPermission()->toNative().
+            '" on item: '.$command->getItemId().
+            ' when executing command: '.get_class($command));
 
         $this->userId = $userId;
         $this->command = $command;

src/Location/Location.php

Indentation +3 added lines, -3 removed lines

@@ -66,9 +66,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()
         ];
     }