Completed
Push — develop ( 911e58...200791 )
by Carsten
13s
created
module/Jobs/src/Jobs/Listener/JobEntityDependencyListener.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
  */
26 26
 class JobEntityDependencyListener extends AbstractDependenciesListener
27 27
 {
28
-    protected $entityClasses = [ Job::class ];
28
+    protected $entityClasses = [Job::class];
29 29
 
30 30
     protected function dependencyCheck(DependencyResultEvent $event)
31 31
     {
Please login to merge, or discard this patch.
module/Applications/src/Entity/Application.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -404,7 +404,7 @@
 block discarded – undo
404 404
     /**
405 405
      * {@inheritDoc}
406 406
      * @see \Applications\Entity\ApplicationInterface::getSummary()
407
-     * @return Application
407
+     * @return string
408 408
      */
409 409
     public function getSummary()
410 410
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -446,7 +446,7 @@
 block discarded – undo
446 446
     public function getCv()
447 447
     {
448 448
         if (is_null($this->cv)) {
449
-            $this->cv= new Cv();
449
+            $this->cv = new Cv();
450 450
         }
451 451
         return $this->cv;
452 452
     }
Please login to merge, or discard this patch.
module/Applications/src/Entity/History.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,6 +41,9 @@  discard block
 block discarded – undo
41 41
      */
42 42
     protected $message;
43 43
     
44
+    /**
45
+     * @param StatusInterface|null $status
46
+     */
44 47
     public function __construct($status, $message = '[System]')
45 48
     {
46 49
         if (!$status instanceof StatusInterface) {
@@ -116,7 +119,7 @@  discard block
 block discarded – undo
116 119
     /**
117 120
      * Sets the history message
118 121
      *
119
-     * @param $message
122
+     * @param string $message
120 123
      *
121 124
      * @return $this
122 125
      */
Please login to merge, or discard this patch.
module/Applications/src/Options/ModuleOptions.php 3 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
     /**
122 122
      * Gets the the maximum number of allowed attachments
123 123
      *
124
-     * @return string
124
+     * @return integer
125 125
      */
126 126
     public function getAttachmentsCount()
127 127
     {
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
     /**
143 143
      * Gets the the maximum size of contact images in bytes
144 144
      *
145
-     * @return string
145
+     * @return integer
146 146
      */
147 147
     public function getContactImageMaxSize()
148 148
     {
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
 
80 80
     protected $workflow = [
81 81
 
82
-      'recruiter',
82
+        'recruiter',
83 83
     ];
84 84
 
85 85
     /**
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -237,7 +237,7 @@
 block discarded – undo
237 237
      */
238 238
     public function setAllowSubsequentAttachmentUpload($allowSubsequentAttachmentUpload)
239 239
     {
240
-        $this->allowSubsequentAttachmentUpload = (bool)$allowSubsequentAttachmentUpload;
240
+        $this->allowSubsequentAttachmentUpload = (bool) $allowSubsequentAttachmentUpload;
241 241
         
242 242
         return $this;
243 243
     }
Please login to merge, or discard this patch.
Applications/src/Repository/Event/UpdateFilesPermissionsSubscriber.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
      * Updates fiile permissions on Flush
36 36
      *
37 37
      * @param OnFlushEventArgs $eventArgs
38
-     * @return boolean
38
+     * @return boolean|null
39 39
      */
40 40
     public function onFlush(OnFlushEventArgs $eventArgs)
41 41
     {
Please login to merge, or discard this patch.
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,8 +56,8 @@  discard block
 block discarded – undo
56 56
                
57 57
                 foreach ($document->getAttachments() as $attachment) {  /* @var \Applications\Entity\Attachment $attachment */
58 58
                     $attachment->getPermissions()
59
-                               ->clear()
60
-                               ->inherit($permissions);
59
+                                ->clear()
60
+                                ->inherit($permissions);
61 61
                     if ($isUpdate) {
62 62
                         $uow->computeChangeSet(
63 63
                             $dm->getClassMetadata(get_class($attachment)),
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
68 68
                 
69 69
                 if ($image = $document->getContact()->getImage()) {
70 70
                     $image->getPermissions()
71
-                          ->clear()
72
-                          ->inherit($permissions);
71
+                            ->clear()
72
+                            ->inherit($permissions);
73 73
                     if ($isUpdate) {
74 74
                         $uow->computeChangeSet(
75 75
                             $dm->getClassMetadata(get_class($image)),
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
         $dm  = $eventArgs->getDocumentManager();
43 43
         $uow = $dm->getUnitOfWork();
44 44
         
45
-        $filter = function ($element) {
45
+        $filter = function($element) {
46 46
             return $element instanceof ApplicationInterface
47 47
                    && $element->getPermissions()->hasChanged();
48 48
         };
Please login to merge, or discard this patch.
module/Core/src/Collection/IdentityWrapper.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -374,7 +374,6 @@
 block discarded – undo
374 374
     }
375 375
 
376 376
     /**
377
-     * @param mixed $element
378 377
      * @return mixed
379 378
      */
380 379
     protected function getElement($key)
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
      */
117 117
     public function getKeys()
118 118
     {
119
-        return $this->collection->map(function ($element) {
119
+        return $this->collection->map(function($element) {
120 120
             return $this->getKey($element);
121 121
         })->toArray();
122 122
     }
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
      */
303 303
     public function offsetExists($offset)
304 304
     {
305
-        return (bool)$this->getElement($offset);
305
+        return (bool) $this->getElement($offset);
306 306
     }
307 307
     
308 308
     /**
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
     {
353 353
         if (!isset($this->identityExtractor)) {
354 354
             // default identity extractor
355
-            $this->identityExtractor = function ($element) {
355
+            $this->identityExtractor = function($element) {
356 356
                 if (!is_callable([$element, 'getId'])) {
357 357
                     throw new \LogicException('$element must have getId() method');
358 358
                 }
@@ -379,7 +379,7 @@  discard block
 block discarded – undo
379 379
      */
380 380
     protected function getElement($key)
381 381
     {
382
-        return $this->collection->filter(function ($element) use ($key) {
382
+        return $this->collection->filter(function($element) use ($key) {
383 383
             return $this->getKey($element) == $key;
384 384
         })->first();
385 385
     }
Please login to merge, or discard this patch.
module/Core/src/Controller/Console/AssetsInstallController.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -201,6 +201,7 @@
 block discarded – undo
201 201
      * Try to create relative symlink.
202 202
      *
203 203
      * Falling back to absolute symlink and finally hard copy.
204
+     * @param string $targetDir
204 205
      */
205 206
     private function relativeSymlinkWithFallback($originDir, $targetDir)
206 207
     {
Please login to merge, or discard this patch.
module/Core/src/Controller/Console/SubsplitController.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -153,6 +153,9 @@
 block discarded – undo
153 153
         $this->runCommand($command);
154 154
     }
155 155
 
156
+    /**
157
+     * @param string $command
158
+     */
156 159
     private function runCommand($command)
157 160
     {
158 161
         if ($this->io->getVerbosity() === OutputInterface::VERBOSITY_DEBUG) {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -156,7 +156,7 @@
 block discarded – undo
156 156
     private function runCommand($command)
157 157
     {
158 158
         if ($this->io->getVerbosity() === OutputInterface::VERBOSITY_DEBUG) {
159
-            $command.=' --debug';
159
+            $command .= ' --debug';
160 160
             $this->io->writeln("Executing: <info>$command</info>");
161 161
         }
162 162
         if ($this->dryRun) {
Please login to merge, or discard this patch.
module/Core/src/Controller/Plugin/CreatePaginator.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -128,7 +128,6 @@
 block discarded – undo
128 128
     }
129 129
     
130 130
     /**
131
-     * @param ControllerManager $controllerManager
132 131
      * @return CreatePaginator
133 132
      * @codeCoverageIgnore
134 133
      */
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -121,8 +121,8 @@
 block discarded – undo
121 121
             $paginator = $paginators->get($paginatorName, $params);
122 122
         }
123 123
         $paginator->setCurrentPageNumber(isset($params['page']) ? $params['page'] : 1)
124
-                  ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)
125
-                  ->setPageRange(isset($params['range']) ? $params['range'] : 5);
124
+                    ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)
125
+                    ->setPageRange(isset($params['range']) ? $params['range'] : 5);
126 126
 
127 127
         return $paginator;
128 128
     }
Please login to merge, or discard this patch.