Completed
Push — develop ( cc6530...59407f )
by Mathias
08:03
created
module/Applications/src/Applications/Auth/Dependency/ListListener.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
         foreach ($this->repository->getUserApplications($user->getId(), $limit) as $application) /* @var $application \Applications\Entity\Application */
64 64
         {
65 65
             $title = $application->getJob()->getTitle();
66
-            $title .= ' ('. $view->dateFormat($application->getDateCreated()) . ')';
66
+            $title .= ' ('.$view->dateFormat($application->getDateCreated()).')';
67 67
             $url = $view->url('lang/applications/detail', ['id' => $application->getId()]);
68 68
             $items[] = new ListItem($title, $url);
69 69
         }
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,9 +60,11 @@
 block discarded – undo
60 60
     {
61 61
         $items = [];
62 62
         
63
-        foreach ($this->repository->getUserApplications($user->getId(), $limit) as $application) /* @var $application \Applications\Entity\Application */
63
+        foreach ($this->repository->getUserApplications($user->getId(), $limit) as $application) {
64
+            /* @var $application \Applications\Entity\Application */
64 65
         {
65 66
             $title = $application->getJob()->getTitle();
67
+        }
66 68
             $title .= ' ('. $view->dateFormat($application->getDateCreated()) . ')';
67 69
             $url = $view->url('lang/applications/detail', ['id' => $application->getId()]);
68 70
             $items[] = new ListItem($title, $url);
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/Auth/Dependency/ListListener.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
         foreach ($this->repository->getUserJobs($user->getId(), $limit) as $job) /* @var $job \Jobs\Entity\Job */
64 64
         {
65 65
             $title = $job->getTitle() ?: $view->translate('untitled');
66
-            $title .= ' ('. $view->dateFormat($job->getDateCreated(), 'short', 'none') . ')';
66
+            $title .= ' ('.$view->dateFormat($job->getDateCreated(), 'short', 'none').')';
67 67
             $url = $view->url('lang/jobs/manage', ['action' => 'edit'], [
68 68
                 'query' => [
69 69
                     'id' => $job->getId()
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,9 +60,11 @@
 block discarded – undo
60 60
     {
61 61
         $items = [];
62 62
         
63
-        foreach ($this->repository->getUserJobs($user->getId(), $limit) as $job) /* @var $job \Jobs\Entity\Job */
63
+        foreach ($this->repository->getUserJobs($user->getId(), $limit) as $job) {
64
+            /* @var $job \Jobs\Entity\Job */
64 65
         {
65 66
             $title = $job->getTitle() ?: $view->translate('untitled');
67
+        }
66 68
             $title .= ' ('. $view->dateFormat($job->getDateCreated(), 'short', 'none') . ')';
67 69
             $url = $view->url('lang/jobs/manage', ['action' => 'edit'], [
68 70
                 'query' => [
Please login to merge, or discard this patch.
module/Auth/src/Auth/Dependency/Manager.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@
 block discarded – undo
56 56
 
57 57
     protected function attachDefaultListeners()
58 58
     {
59
-        $this->getEventManager()->attach(static::EVENT_REMOVE_ITEMS, function (EventInterface $event) {
59
+        $this->getEventManager()->attach(static::EVENT_REMOVE_ITEMS, function(EventInterface $event) {
60 60
             $user = $event->getParam('user');
61 61
             foreach ($this->getLists() as $list) {
62 62
                 foreach ($list->getEntities($user) as $entity) {
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -5 removed lines patch added patch discarded remove patch
@@ -41,9 +41,7 @@  discard block
 block discarded – undo
41 41
     }
42 42
     
43 43
     /**
44
-     * @param User $user
45
-     * @param Router $router
46
-     * @return ListInterface[]
44
+     * @return \Zend\EventManager\ResponseCollection
47 45
      */
48 46
     public function getLists()
49 47
     {
@@ -52,8 +50,7 @@  discard block
 block discarded – undo
52 50
  
53 51
     /**
54 52
      * @param User $user
55
-     * @param Router $router
56
-     * @return \Zend\EventManager\ResponseCollection
53
+     * @return boolean
57 54
      */
58 55
     public function removeItems(User $user)
59 56
     {
Please login to merge, or discard this patch.
module/Core/src/Core/Form/View/Helper/FormFileUpload.php 2 patches
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -98,12 +98,12 @@  discard block
 block discarded – undo
98 98
     </a>
99 99
     <div class="fu-errors input-error">
100 100
         <ul class="errors">
101
-            <li class="fu-error-size">' . $translator->translate('The file is too big', $textDomain) . '</li>
102
-            <li class="fu-error-type">' . $translator->translate('The file type is not supported', $textDomain) . '</li>
101
+            <li class="fu-error-size">' . $translator->translate('The file is too big', $textDomain).'</li>
102
+            <li class="fu-error-type">' . $translator->translate('The file type is not supported', $textDomain).'</li>
103 103
             <li class="fu-error-count">' . sprintf(
104 104
     $translator->translate('You may only upload %d files', $textDomain),
105 105
     $element->getAttribute('data-maxfilecount')
106
-) . '</li><li class="fu-error-general">' . $translator->translate('An unknown error occured.') . '</li>
106
+).'</li><li class="fu-error-general">'.$translator->translate('An unknown error occured.').'</li>
107 107
         </ul>
108 108
    </div>
109 109
 </li>';
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
         /* @var $basepath \Zend\View\Helper\BasePath */
112 112
         $renderer          = $this->getView();
113 113
         $basepath          = $renderer->plugin('basepath');
114
-        $createFileDisplay = function ($file) use ($template, $basepath) {
114
+        $createFileDisplay = function($file) use ($template, $basepath) {
115 115
             /* @var $file \Core\Entity\FileInterface */
116 116
             $uri  = $basepath($file->getUri());
117 117
             $name = $file->getName();
@@ -146,11 +146,11 @@  discard block
 block discarded – undo
146 146
         }
147 147
 
148 148
         $nonemptynotice =
149
-            '<div class="fu-nonempty-notice"' . ('' == trim($preview) ? ' style="display:none;"' : '') . '>'
150
-            . $this->getNonEmptyNotice() . '</div>';
149
+            '<div class="fu-nonempty-notice"'.('' == trim($preview) ? ' style="display:none;"' : '').'>'
150
+            . $this->getNonEmptyNotice().'</div>';
151 151
         $emptynotice    = '<div class="fu-empty-notice"'
152
-                          . ('' == trim($preview) ? '' : ' style="display: none;"') . '>
153
-                       ' . $this->getEmptyNotice() . '
152
+                          . ('' == trim($preview) ? '' : ' style="display: none;"').'>
153
+                       ' . $this->getEmptyNotice().'
154 154
                   </div>';
155 155
 
156 156
         $markup = '
@@ -298,7 +298,7 @@  discard block
 block discarded – undo
298 298
 	 */
299 299
 	protected function getDefaultNotice()
300 300
 	{
301
-		return '<small>' . $this->getTranslator()->translate('Click here to add files or use drag and drop.') . '</small>';
301
+		return '<small>'.$this->getTranslator()->translate('Click here to add files or use drag and drop.').'</small>';
302 302
 	}
303 303
 	
304 304
     /**
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
 	 */
309 309
 	public function setAllowRemove($allowRemove)
310 310
 	{
311
-		$this->allowRemove = (bool)$allowRemove;
311
+		$this->allowRemove = (bool) $allowRemove;
312 312
 		
313 313
 		return $this;
314 314
 	}
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
 	 */
321 321
 	public function setAllowClickableDropZone($allowClickableDropZone)
322 322
 	{
323
-		$this->allowClickableDropZone = (bool)$allowClickableDropZone;
323
+		$this->allowClickableDropZone = (bool) $allowClickableDropZone;
324 324
 		
325 325
 		return $this;
326 326
 	}
Please login to merge, or discard this patch.
Indentation   +78 added lines, -78 removed lines patch added patch discarded remove patch
@@ -113,11 +113,11 @@  discard block
 block discarded – undo
113 113
 
114 114
             return str_replace(
115 115
                 array('#abort',
116
-                      '__file-uri__',
117
-                      '__file-name__',
118
-                      '__file-size__',
119
-                      'fu-working',
120
-                      'fa-file-o'
116
+                        '__file-uri__',
117
+                        '__file-name__',
118
+                        '__file-size__',
119
+                        'fu-working',
120
+                        'fa-file-o'
121 121
                 ),
122 122
                 array("$uri?do=delete", $uri, $name, $size, '', $icon),
123 123
                 $template
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
             '<div class="fu-nonempty-notice"' . ('' == trim($preview) ? ' style="display:none;"' : '') . '>'
142 142
             . $this->getNonEmptyNotice() . '</div>';
143 143
         $emptynotice    = '<div class="fu-empty-notice"'
144
-                          . ('' == trim($preview) ? '' : ' style="display: none;"') . '>
144
+                            . ('' == trim($preview) ? '' : ' style="display: none;"') . '>
145 145
                        ' . $this->getEmptyNotice() . '
146 146
                   </div>';
147 147
 
@@ -230,99 +230,99 @@  discard block
 block discarded – undo
230 230
     }
231 231
     
232 232
     /**
233
-	 * @param string $emptyNotice
234
-	 * @return FormFileUpload
235
-	 * @since 0.27
236
-	 */
237
-	public function setEmptyNotice($emptyNotice)
238
-	{
239
-		$this->emptyNotice = $emptyNotice;
233
+     * @param string $emptyNotice
234
+     * @return FormFileUpload
235
+     * @since 0.27
236
+     */
237
+    public function setEmptyNotice($emptyNotice)
238
+    {
239
+        $this->emptyNotice = $emptyNotice;
240 240
 		
241
-		return $this;
242
-	}
241
+        return $this;
242
+    }
243 243
     
244 244
     /**
245
-	 * @return string
246
-	 * @since 0.27
247
-	 */
248
-	protected function getEmptyNotice()
249
-	{
250
-	    if (!isset($this->emptyNotice))
251
-	    {
252
-	        $this->emptyNotice = '
245
+     * @return string
246
+     * @since 0.27
247
+     */
248
+    protected function getEmptyNotice()
249
+    {
250
+        if (!isset($this->emptyNotice))
251
+        {
252
+            $this->emptyNotice = '
253 253
 	            <div class="pull-left">
254 254
                     <span class="yk-icon fa-files-o fa-5x"></span>
255 255
                 </div>' . $this->getDefaultNotice();
256
-	    }
256
+        }
257 257
 	    
258
-		return $this->emptyNotice;
259
-	}
258
+        return $this->emptyNotice;
259
+    }
260 260
 
261 261
     /**
262
-	 * @param string $nonEmptyNotice
263
-	 * @return FormFileUpload
264
-	 * @since 0.27
265
-	 */
266
-	public function setNonEmptyNotice($nonEmptyNotice)
267
-	{
268
-		$this->nonEmptyNotice = $nonEmptyNotice;
262
+     * @param string $nonEmptyNotice
263
+     * @return FormFileUpload
264
+     * @since 0.27
265
+     */
266
+    public function setNonEmptyNotice($nonEmptyNotice)
267
+    {
268
+        $this->nonEmptyNotice = $nonEmptyNotice;
269 269
 		
270
-		return $this;
271
-	}
270
+        return $this;
271
+    }
272 272
 
273 273
     /**
274
-	 * @return string
275
-	 * @since 0.27
276
-	 */
277
-	protected function getNonEmptyNotice()
278
-	{
279
-	    if (!isset($this->nonEmptyNotice))
280
-	    {
281
-	        $this->nonEmptyNotice = $this->getDefaultNotice();
282
-	    }
274
+     * @return string
275
+     * @since 0.27
276
+     */
277
+    protected function getNonEmptyNotice()
278
+    {
279
+        if (!isset($this->nonEmptyNotice))
280
+        {
281
+            $this->nonEmptyNotice = $this->getDefaultNotice();
282
+        }
283 283
 	    
284
-		return $this->nonEmptyNotice;
285
-	}
284
+        return $this->nonEmptyNotice;
285
+    }
286 286
 
287 287
     /**
288
-	 * @return string
289
-	 * @since 0.27
290
-	 */
291
-	protected function getDefaultNotice()
292
-	{
293
-		return '<small>' . $this->getTranslator()->translate('Click here to add files or use drag and drop.') . '</small>';
294
-	}
288
+     * @return string
289
+     * @since 0.27
290
+     */
291
+    protected function getDefaultNotice()
292
+    {
293
+        return '<small>' . $this->getTranslator()->translate('Click here to add files or use drag and drop.') . '</small>';
294
+    }
295 295
 	
296 296
     /**
297
-	 * @param boolean $allowRemove
298
-	 * @return FormFileUpload
299
-	 * @since 0.27
300
-	 */
301
-	public function setAllowRemove($allowRemove)
302
-	{
303
-		$this->allowRemove = (bool)$allowRemove;
297
+     * @param boolean $allowRemove
298
+     * @return FormFileUpload
299
+     * @since 0.27
300
+     */
301
+    public function setAllowRemove($allowRemove)
302
+    {
303
+        $this->allowRemove = (bool)$allowRemove;
304 304
 		
305
-		return $this;
306
-	}
305
+        return $this;
306
+    }
307 307
 	
308 308
     /**
309
-	 * @param boolean $allowClickableDropZone
310
-	 * @return FormFileUpload
311
-	 * @since 0.27
312
-	 */
313
-	public function setAllowClickableDropZone($allowClickableDropZone)
314
-	{
315
-		$this->allowClickableDropZone = (bool)$allowClickableDropZone;
309
+     * @param boolean $allowClickableDropZone
310
+     * @return FormFileUpload
311
+     * @since 0.27
312
+     */
313
+    public function setAllowClickableDropZone($allowClickableDropZone)
314
+    {
315
+        $this->allowClickableDropZone = (bool)$allowClickableDropZone;
316 316
 		
317
-		return $this;
318
-	}
317
+        return $this;
318
+    }
319 319
 	
320
-	/**
321
-	 * @since 0.27
322
-	 */
323
-	protected function setupAssets()
324
-	{
325
-	    /* @var $renderer \Zend\View\Renderer\PhpRenderer */
320
+    /**
321
+     * @since 0.27
322
+     */
323
+    protected function setupAssets()
324
+    {
325
+        /* @var $renderer \Zend\View\Renderer\PhpRenderer */
326 326
         /* @var $basepath \Zend\View\Helper\BasePath */
327 327
         $renderer = $this->getView();
328 328
         $basepath = $renderer->plugin('basepath');
@@ -331,5 +331,5 @@  discard block
 block discarded – undo
331 331
             ->appendFile($basepath('assets/blueimp-file-upload/js/jquery.iframe-transport.js'))
332 332
             ->appendFile($basepath('assets/blueimp-file-upload/js/jquery.fileupload.js'))
333 333
             ->appendFile($basepath($this->scriptFile));
334
-	}
334
+    }
335 335
 }
Please login to merge, or discard this patch.
module/Core/src/Core/Entity/FileEntity.php 3 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -297,9 +297,9 @@
 block discarded – undo
297 297
     }
298 298
     
299 299
     /**
300
-	 * @see \Core\Entity\FileInterface::getUri()
301
-	 * @since 0.27
302
-	 */
303
-	public function getUri()
304
-	{}
300
+     * @see \Core\Entity\FileInterface::getUri()
301
+     * @since 0.27
302
+     */
303
+    public function getUri()
304
+    {}
305 305
 }
306 306
\ No newline at end of file
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -146,18 +146,18 @@
 block discarded – undo
146 146
         $size = $this->getLength();
147 147
         
148 148
         if ($size >= 1073741824) {
149
-            return round($size / 1073741824, 2) . ' GB';
149
+            return round($size / 1073741824, 2).' GB';
150 150
         }
151 151
         
152 152
         if ($size >= 1048576) {
153
-            return round($size / 1048576, 2) . ' MB';
153
+            return round($size / 1048576, 2).' MB';
154 154
         }
155 155
         
156 156
         if ($size >= 1024) {
157
-            return round($size / 1024, 2) . ' kB';
157
+            return round($size / 1024, 2).' kB';
158 158
         }
159 159
         
160
-        return (string)$size;
160
+        return (string) $size;
161 161
     }
162 162
 
163 163
     /**
Please login to merge, or discard this patch.
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
     /**
130 130
      * Gets the name of a file
131 131
      *
132
-     * @return mixed
132
+     * @return string
133 133
      */
134 134
     public function getName()
135 135
     {
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
     /**
198 198
      * Gets the uploaded date of a file
199 199
      *
200
-     * @return mixed
200
+     * @return null|\DateTime
201 201
      */
202 202
     public function getDateUploaded()
203 203
     {
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
     /**
234 234
      * Gets the length of the file
235 235
      *
236
-     * @return mixed
236
+     * @return string|null
237 237
      */
238 238
     public function getLength()
239 239
     {
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/Entity/History.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -41,6 +41,9 @@
 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) {
Please login to merge, or discard this patch.
Organizations/src/Organizations/ImageFileCache/ApplicationListener.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@
 block discarded – undo
66 66
         // try get image from repository
67 67
         $image = $this->repository->find($id);
68 68
 
69
-        if (! $image) {
69
+        if (!$image) {
70 70
             // abort if image does not exist
71 71
             return;
72 72
         }
Please login to merge, or discard this patch.
module/Organizations/src/Organizations/ImageFileCache/Manager.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
      */
85 85
     public function matchUri($uri)
86 86
     {
87
-        $pattern = '#^' . preg_quote($this->options->getUriPath(), '#') . '/[0-9a-z]/[0-9a-z]/([0-9a-z]+)\.[a-zA-Z]{3,4}$#';
87
+        $pattern = '#^'.preg_quote($this->options->getUriPath(), '#').'/[0-9a-z]/[0-9a-z]/([0-9a-z]+)\.[a-zA-Z]{3,4}$#';
88 88
         $matches = [];
89 89
         preg_match($pattern, $uri, $matches);
90 90
         
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     {
121 121
         $dir = rtrim($dir, '/\\');
122 122
         
123
-        if (! is_dir($dir)) {
123
+        if (!is_dir($dir)) {
124 124
             $this->createDirectoryRecursively(dirname($dir));
125 125
             
126 126
             $oldUmask = umask(0);
Please login to merge, or discard this patch.
module/Organizations/src/Organizations/ImageFileCache/ODMListener.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
      */
49 49
     public function getSubscribedEvents()
50 50
     {
51
-        if (! $this->manager->isEnabled()) {
51
+        if (!$this->manager->isEnabled()) {
52 52
             return [];
53 53
         }
54 54
         
@@ -66,19 +66,19 @@  discard block
 block discarded – undo
66 66
         $organization = $eventArgs->getDocument();
67 67
         
68 68
         // check for a organization instance
69
-        if (! $organization instanceof Organization) {
69
+        if (!$organization instanceof Organization) {
70 70
             return;
71 71
         }
72 72
         
73 73
         // check if the image has been changed
74
-        if (! $eventArgs->hasChangedField('image')) {
74
+        if (!$eventArgs->hasChangedField('image')) {
75 75
             return;
76 76
         }
77 77
         
78 78
         $image = $eventArgs->getOldValue('image');
79 79
         
80 80
         // check if any image existed
81
-        if (! $image instanceof OrganizationImage) {
81
+        if (!$image instanceof OrganizationImage) {
82 82
             return;
83 83
         }
84 84
         
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
     }
88 88
 
89 89
     /**
90
-     * @param LifecycleEventArgs $eventArgs
90
+     * @param PostFlushEventArgs $eventArgs
91 91
      */
92 92
     public function postFlush(PostFlushEventArgs $eventArgs)
93 93
     {
Please login to merge, or discard this patch.