@@ -63,34 +63,34 @@ |
||
63 | 63 | $userRolesSubQuery = $this->connection->createQueryBuilder() |
64 | 64 | ->select(PermissionsSchemaConfigurator::ROLE_ID_COLUMN) |
65 | 65 | ->from($this->userRolesTableName->toNative()) |
66 | - ->where(PermissionsSchemaConfigurator::USER_ID_COLUMN . ' = :userId'); |
|
66 | + ->where(PermissionsSchemaConfigurator::USER_ID_COLUMN.' = :userId'); |
|
67 | 67 | |
68 | 68 | $queryBuilder = $this->connection->createQueryBuilder(); |
69 | 69 | $userConstraintsQuery = $queryBuilder |
70 | - ->select('rs.' . SearchSchemaConfigurator::CONSTRAINT_COLUMN) |
|
70 | + ->select('rs.'.SearchSchemaConfigurator::CONSTRAINT_COLUMN) |
|
71 | 71 | ->from($this->rolesSearchTableName, 'rs') |
72 | 72 | ->innerJoin( |
73 | 73 | 'rs', |
74 | 74 | sprintf('(%s)', $userRolesSubQuery->getSQL()), |
75 | 75 | 'ur', |
76 | - 'rs.' . SearchSchemaConfigurator::UUID_COLUMN . ' = ur.' . PermissionsSchemaConfigurator::ROLE_ID_COLUMN |
|
76 | + 'rs.'.SearchSchemaConfigurator::UUID_COLUMN.' = ur.'.PermissionsSchemaConfigurator::ROLE_ID_COLUMN |
|
77 | 77 | ) |
78 | 78 | ->innerJoin( |
79 | 79 | 'rs', |
80 | 80 | $this->rolePermissionsTableName->toNative(), |
81 | 81 | 'rp', |
82 | - 'rs.' . SearchSchemaConfigurator::UUID_COLUMN . ' = rp.' . PermissionsSchemaConfigurator::ROLE_ID_COLUMN |
|
82 | + 'rs.'.SearchSchemaConfigurator::UUID_COLUMN.' = rp.'.PermissionsSchemaConfigurator::ROLE_ID_COLUMN |
|
83 | 83 | ) |
84 | - ->where(PermissionsSchemaConfigurator::PERMISSION_COLUMN . ' = :permission') |
|
84 | + ->where(PermissionsSchemaConfigurator::PERMISSION_COLUMN.' = :permission') |
|
85 | 85 | ->andWhere($queryBuilder->expr()->isNotNull( |
86 | - 'rs.' . SearchSchemaConfigurator::CONSTRAINT_COLUMN |
|
86 | + 'rs.'.SearchSchemaConfigurator::CONSTRAINT_COLUMN |
|
87 | 87 | )) |
88 | 88 | ->setParameter('userId', $userId->toNative()) |
89 | 89 | ->setParameter('permission', $permission->toNative()); |
90 | 90 | |
91 | 91 | $results = $userConstraintsQuery->execute()->fetchAll(\PDO::FETCH_COLUMN); |
92 | 92 | |
93 | - return array_map(function ($constraint) { |
|
93 | + return array_map(function($constraint) { |
|
94 | 94 | return new StringLiteral($constraint); |
95 | 95 | }, $results); |
96 | 96 | } |
@@ -24,7 +24,7 @@ |
||
24 | 24 | } |
25 | 25 | |
26 | 26 | /** |
27 | - * @return StringLiteral |
|
27 | + * @return string |
|
28 | 28 | */ |
29 | 29 | public function getTitle() |
30 | 30 | { |
@@ -24,7 +24,7 @@ |
||
24 | 24 | } |
25 | 25 | |
26 | 26 | /** |
27 | - * @return StringLiteral |
|
27 | + * @return string |
|
28 | 28 | */ |
29 | 29 | public function getTitle() |
30 | 30 | { |
@@ -624,7 +624,7 @@ |
||
624 | 624 | } |
625 | 625 | |
626 | 626 | /** |
627 | - * @return callable |
|
627 | + * @return \Closure |
|
628 | 628 | */ |
629 | 629 | private function reject() |
630 | 630 | { |
@@ -125,7 +125,7 @@ discard block |
||
125 | 125 | |
126 | 126 | if (preg_match('/^apply(.+)$/', $method, $matches)) { |
127 | 127 | $event = $matches[1]; |
128 | - $classNameMethod = 'get' . $event . 'ClassName'; |
|
128 | + $classNameMethod = 'get'.$event.'ClassName'; |
|
129 | 129 | |
130 | 130 | if (method_exists($this, $classNameMethod)) { |
131 | 131 | $eventFullClassName = call_user_func(array($this, $classNameMethod)); |
@@ -265,7 +265,7 @@ discard block |
||
265 | 265 | $labelsProperty = $labelAdded->getLabel()->isVisible() ? 'labels' : 'hiddenLabels'; |
266 | 266 | |
267 | 267 | $labels = isset($offerLd->{$labelsProperty}) ? $offerLd->{$labelsProperty} : []; |
268 | - $label = (string) $labelAdded->getLabel(); |
|
268 | + $label = (string)$labelAdded->getLabel(); |
|
269 | 269 | |
270 | 270 | $labels[] = $label; |
271 | 271 | $offerLd->{$labelsProperty} = array_unique($labels); |
@@ -290,7 +290,7 @@ discard block |
||
290 | 290 | if (isset($offerLd->{$labelsProperty}) && is_array($offerLd->{$labelsProperty})) { |
291 | 291 | $offerLd->{$labelsProperty} = array_filter( |
292 | 292 | $offerLd->{$labelsProperty}, |
293 | - function ($label) use ($labelRemoved) { |
|
293 | + function($label) use ($labelRemoved) { |
|
294 | 294 | return !$labelRemoved->getLabel()->equals( |
295 | 295 | new Label($label) |
296 | 296 | ); |
@@ -387,7 +387,7 @@ discard block |
||
387 | 387 | return; |
388 | 388 | } |
389 | 389 | |
390 | - $imageId = (string) $imageRemoved->getImage()->getMediaObjectId(); |
|
390 | + $imageId = (string)$imageRemoved->getImage()->getMediaObjectId(); |
|
391 | 391 | |
392 | 392 | /** |
393 | 393 | * Matches any object that is not the removed image. |
@@ -398,7 +398,7 @@ discard block |
||
398 | 398 | * @return bool |
399 | 399 | * Returns true when the media object does not match the image to remove. |
400 | 400 | */ |
401 | - $shouldNotBeRemoved = function ($mediaObject) use ($imageId) { |
|
401 | + $shouldNotBeRemoved = function($mediaObject) use ($imageId) { |
|
402 | 402 | $containsId = !!strpos($mediaObject->{'@id'}, $imageId); |
403 | 403 | return !$containsId; |
404 | 404 | }; |
@@ -436,7 +436,7 @@ discard block |
||
436 | 436 | $document = $this->loadDocumentFromRepository($mainImageSelected); |
437 | 437 | $offerLd = $document->getBody(); |
438 | 438 | $imageId = $mainImageSelected->getImage()->getMediaObjectId(); |
439 | - $mediaObjectMatcher = function ($matchingMediaObject, $currentMediaObject) use ($imageId) { |
|
439 | + $mediaObjectMatcher = function($matchingMediaObject, $currentMediaObject) use ($imageId) { |
|
440 | 440 | if (!$matchingMediaObject && $this->mediaObjectMatchesId($currentMediaObject, $imageId)) { |
441 | 441 | $matchingMediaObject = $currentMediaObject; |
442 | 442 | } |
@@ -461,7 +461,7 @@ discard block |
||
461 | 461 | */ |
462 | 462 | protected function mediaObjectMatchesId($mediaObject, UUID $mediaObjectId) |
463 | 463 | { |
464 | - return strpos($mediaObject->{'@id'}, (string) $mediaObjectId) > 0; |
|
464 | + return strpos($mediaObject->{'@id'}, (string)$mediaObjectId) > 0; |
|
465 | 465 | } |
466 | 466 | |
467 | 467 | /** |
@@ -642,7 +642,7 @@ discard block |
||
642 | 642 | */ |
643 | 643 | protected function applyPublished(AbstractPublished $published) |
644 | 644 | { |
645 | - $this->applyEventTransformation($published, function ($offerLd) use ($published) { |
|
645 | + $this->applyEventTransformation($published, function($offerLd) use ($published) { |
|
646 | 646 | $offerLd->workflowStatus = WorkflowStatus::READY_FOR_VALIDATION()->getName(); |
647 | 647 | |
648 | 648 | $publicationDate = $published->getPublicationDate(); |
@@ -655,7 +655,7 @@ discard block |
||
655 | 655 | */ |
656 | 656 | protected function applyApproved(AbstractApproved $approved) |
657 | 657 | { |
658 | - $this->applyEventTransformation($approved, function ($offerLd) { |
|
658 | + $this->applyEventTransformation($approved, function($offerLd) { |
|
659 | 659 | $offerLd->workflowStatus = WorkflowStatus::APPROVED()->getName(); |
660 | 660 | }); |
661 | 661 | } |
@@ -691,7 +691,7 @@ discard block |
||
691 | 691 | */ |
692 | 692 | private function reject() |
693 | 693 | { |
694 | - return function ($offerLd) { |
|
694 | + return function($offerLd) { |
|
695 | 695 | $offerLd->workflowStatus = WorkflowStatus::REJECTED()->getName(); |
696 | 696 | }; |
697 | 697 | } |
@@ -728,7 +728,7 @@ discard block |
||
728 | 728 | { |
729 | 729 | $images = $imagesEvent->getImages(); |
730 | 730 | $jsonMediaObjects = array_map( |
731 | - function (Image $image) { |
|
731 | + function(Image $image) { |
|
732 | 732 | return $this->mediaObjectSerializer->serialize($image, 'json-ld'); |
733 | 733 | }, |
734 | 734 | $images->toArray() |
@@ -742,7 +742,7 @@ discard block |
||
742 | 742 | } |
743 | 743 | |
744 | 744 | if (isset($mainImage)) { |
745 | - $offerLd->image = (string) $mainImage->getSourceLocation(); |
|
745 | + $offerLd->image = (string)$mainImage->getSourceLocation(); |
|
746 | 746 | } |
747 | 747 | } |
748 | 748 |
@@ -26,10 +26,10 @@ |
||
26 | 26 | StringLiteral $userId, |
27 | 27 | AuthorizableCommandInterface $command |
28 | 28 | ) { |
29 | - parent::__construct('User with id: ' . $userId->toNative() . |
|
30 | - ' has no permission: "' . $command->getPermission()->toNative() . |
|
31 | - '" on item: ' . $command->getItemId() . |
|
32 | - ' when executing command: ' . get_class($command)); |
|
29 | + parent::__construct('User with id: '.$userId->toNative(). |
|
30 | + ' has no permission: "'.$command->getPermission()->toNative(). |
|
31 | + '" on item: '.$command->getItemId(). |
|
32 | + ' when executing command: '.get_class($command)); |
|
33 | 33 | |
34 | 34 | $this->userId = $userId; |
35 | 35 | $this->command = $command; |
@@ -66,9 +66,9 @@ |
||
66 | 66 | public function serialize() |
67 | 67 | { |
68 | 68 | return [ |
69 | - 'cdbid' => $this->cdbid, |
|
70 | - 'name' => $this->name->toNative(), |
|
71 | - 'address' => $this->address->serialize() |
|
69 | + 'cdbid' => $this->cdbid, |
|
70 | + 'name' => $this->name->toNative(), |
|
71 | + 'address' => $this->address->serialize() |
|
72 | 72 | ]; |
73 | 73 | } |
74 | 74 |
@@ -80,7 +80,7 @@ |
||
80 | 80 | /** |
81 | 81 | * Factory method to create a new event. |
82 | 82 | * |
83 | - * @param $eventId |
|
83 | + * @param string $eventId |
|
84 | 84 | * @param Title $title |
85 | 85 | * @param EventType $eventType |
86 | 86 | * @param Location $location |
@@ -271,7 +271,7 @@ |
||
271 | 271 | */ |
272 | 272 | public function applyAudienceUpdated(AudienceUpdated $audienceUpdated) |
273 | 273 | { |
274 | - $this->audience= $audienceUpdated->getAudience(); |
|
274 | + $this->audience = $audienceUpdated->getAudience(); |
|
275 | 275 | } |
276 | 276 | |
277 | 277 | /** |
@@ -20,6 +20,7 @@ discard block |
||
20 | 20 | * |
21 | 21 | * @param string $id |
22 | 22 | * @param string $description |
23 | + * @return string |
|
23 | 24 | */ |
24 | 25 | public function updateDescription($id, $description); |
25 | 26 | |
@@ -28,6 +29,7 @@ discard block |
||
28 | 29 | * |
29 | 30 | * @param string $id |
30 | 31 | * @param string $ageRange |
32 | + * @return string |
|
31 | 33 | */ |
32 | 34 | public function updateTypicalAgeRange($id, $ageRange); |
33 | 35 | |
@@ -35,6 +37,7 @@ discard block |
||
35 | 37 | * Delete the typical age range of a place. |
36 | 38 | * |
37 | 39 | * @param string $id |
40 | + * @return string |
|
38 | 41 | */ |
39 | 42 | public function deleteTypicalAgeRange($id); |
40 | 43 | |
@@ -43,6 +46,7 @@ discard block |
||
43 | 46 | * |
44 | 47 | * @param string $id |
45 | 48 | * @param string $organizerId |
49 | + * @return string |
|
46 | 50 | */ |
47 | 51 | public function updateOrganizer($id, $organizerId); |
48 | 52 | |
@@ -51,6 +55,7 @@ discard block |
||
51 | 55 | * |
52 | 56 | * @param string $id |
53 | 57 | * @param string $organizerId |
58 | + * @return string |
|
54 | 59 | */ |
55 | 60 | public function deleteOrganizer($id, $organizerId); |
56 | 61 | |
@@ -59,6 +64,7 @@ discard block |
||
59 | 64 | * |
60 | 65 | * @param string $id |
61 | 66 | * @param ContactPoint $contactPoint |
67 | + * @return string |
|
62 | 68 | */ |
63 | 69 | public function updateContactPoint($id, ContactPoint $contactPoint); |
64 | 70 | |
@@ -67,6 +73,7 @@ discard block |
||
67 | 73 | * |
68 | 74 | * @param string $id |
69 | 75 | * @param Image $image |
76 | + * @return string |
|
70 | 77 | */ |
71 | 78 | public function addImage($id, Image $image); |
72 | 79 | |
@@ -93,6 +100,7 @@ discard block |
||
93 | 100 | * |
94 | 101 | * @param string $id |
95 | 102 | * @param Image $image |
103 | + * @return string |
|
96 | 104 | */ |
97 | 105 | public function removeImage($id, Image $image); |
98 | 106 |
@@ -148,8 +148,8 @@ |
||
148 | 148 | } |
149 | 149 | |
150 | 150 | return parent::serialize() + array( |
151 | - 'website' => (string) $this->getWebsite(), |
|
152 | - 'title' => (string) $this->getTitle(), |
|
151 | + 'website' => (string)$this->getWebsite(), |
|
152 | + 'title' => (string)$this->getTitle(), |
|
153 | 153 | 'addresses' => $addresses, |
154 | 154 | 'phones' => $this->getPhones(), |
155 | 155 | 'emails' => $this->getEmails(), |