Completed
Push — develop ( 62f68e...872778 )
by Carsten
13s
created
module/Install/config/module.config.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 	    'factories' => [
26 26
 		    'mvctranslator' => \Zend\Mvc\I18n\TranslatorFactory::class,
27 27
 		    'FilterManager' => \Zend\Filter\FilterPluginManagerFactory::class,
28
-            'Tracy' => [\Core\Service\Tracy::class,'factory'],
28
+            'Tracy' => [\Core\Service\Tracy::class, 'factory'],
29 29
             'Core/Options' => 'Core\Factory\ModuleOptionsFactory',
30 30
         ],
31 31
         'abstract_factories' => [
@@ -95,12 +95,12 @@  discard block
 block discarded – undo
95 95
         'exception_template' => 'error/index',
96 96
         // Map template to files. Speeds up the lookup through the template stack.
97 97
         'template_map' => [
98
-            'layout/layout' => __DIR__ . '/../view/layout/layout.phtml',
99
-            'error/index' => __DIR__ . '/../view/error/index.phtml',
98
+            'layout/layout' => __DIR__.'/../view/layout/layout.phtml',
99
+            'error/index' => __DIR__.'/../view/error/index.phtml',
100 100
         ],
101 101
         // Where to look for view templates not mapped above
102 102
         'template_path_stack' => [
103
-            __DIR__ . '/../view',
103
+            __DIR__.'/../view',
104 104
         ],
105 105
     ],
106 106
 
@@ -114,13 +114,13 @@  discard block
 block discarded – undo
114 114
         'translation_file_patterns' => [
115 115
             [
116 116
                 'type' => 'gettext',
117
-                'base_dir' => __DIR__ . '/../language',
117
+                'base_dir' => __DIR__.'/../language',
118 118
                 'pattern' => '%s.mo',
119 119
                 'text_domain' => 'Install',
120 120
             ],
121 121
             [
122 122
                 'type'     => 'phparray',
123
-                'base_dir' => __DIR__ . '/../../Core/language',
123
+                'base_dir' => __DIR__.'/../../Core/language',
124 124
                 'pattern' => 'Zend_Validate.%s.php',
125 125
                 'text_domain' => 'default',
126 126
             ],
Please login to merge, or discard this patch.
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -8,15 +8,15 @@  discard block
 block discarded – undo
8 8
  */
9 9
 
10 10
 return [
11
-	'tracy' => [
12
-		'enabled' => true, // flag whether to load tracy at all
13
-		'mode' => true, // true = production|false = development|null = autodetect|IP address(es) csv/array
14
-		'bar' => false, // bool = enabled|Toggle nette diagnostics bar.
15
-		'strict' => true, // bool = cause immediate death|int = matched against error severity
16
-		'log' => __DIR__.'/../../../var/log/tracy', // path to log directory (this directory keeps error.log, snoozing mailsent file & html exception trace files)
17
-		'email' => null, // in production mode notifies the recipient
18
-		'email_snooze' => 900 // interval for sending email in seconds
19
-	],
11
+    'tracy' => [
12
+        'enabled' => true, // flag whether to load tracy at all
13
+        'mode' => true, // true = production|false = development|null = autodetect|IP address(es) csv/array
14
+        'bar' => false, // bool = enabled|Toggle nette diagnostics bar.
15
+        'strict' => true, // bool = cause immediate death|int = matched against error severity
16
+        'log' => __DIR__.'/../../../var/log/tracy', // path to log directory (this directory keeps error.log, snoozing mailsent file & html exception trace files)
17
+        'email' => null, // in production mode notifies the recipient
18
+        'email_snooze' => 900 // interval for sending email in seconds
19
+    ],
20 20
 
21 21
     'doctrine' => [
22 22
         'driver' => [
@@ -35,9 +35,9 @@  discard block
 block discarded – undo
35 35
         'invokables' => [
36 36
             'Install/Listener/LanguageSetter' => 'Install\Listener\LanguageSetter',
37 37
         ],
38
-	    'factories' => [
39
-		    'mvctranslator' => \Zend\Mvc\I18n\TranslatorFactory::class,
40
-		    'FilterManager' => \Zend\Filter\FilterPluginManagerFactory::class,
38
+        'factories' => [
39
+            'mvctranslator' => \Zend\Mvc\I18n\TranslatorFactory::class,
40
+            'FilterManager' => \Zend\Filter\FilterPluginManagerFactory::class,
41 41
             'Tracy' => [\Core\Service\Tracy::class,'factory'],
42 42
             'Core/Options' => 'Core\Factory\ModuleOptionsFactory',
43 43
         ],
@@ -63,9 +63,9 @@  discard block
 block discarded – undo
63 63
     ],
64 64
 
65 65
     'controllers' => [
66
-	    'abstract_factories' => [
67
-	    	\Install\Factory\Controller\LazyControllerFactory::class
68
-	    ],
66
+        'abstract_factories' => [
67
+            \Install\Factory\Controller\LazyControllerFactory::class
68
+        ],
69 69
     ],
70 70
 
71 71
     'controller_plugins' => [
Please login to merge, or discard this patch.
module/Auth/src/Auth/Controller/Plugin/UserSwitcher.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
     /**
120 120
      * Switch to another user.
121 121
      *
122
-     * @param string|UserInterface $id user id of the user to switch to.
122
+     * @param string $id user id of the user to switch to.
123 123
      * @param array $params Additional parameters to store in the session container.
124 124
      *
125 125
      * @return bool
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
     /**
223 223
      * Gets the session container.
224 224
      *
225
-     * @return Container
225
+     * @return string
226 226
      */
227 227
     private function getSessionContainer()
228 228
     {
Please login to merge, or discard this patch.
module/Auth/src/Auth/Service/Register.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
     }
155 155
 
156 156
     /**
157
-     * @param $user
157
+     * @param \Auth\Entity\UserInterface $user
158 158
      * @return $this
159 159
      */
160 160
     protected function setUser($user)
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
     /**
248 248
      * Email-Address
249 249
      * @param $email string
250
-     * @return mixed
250
+     * @return Register
251 251
      */
252 252
     protected function setEmail($email)
253 253
     {
Please login to merge, or discard this patch.
module/Behat/src/CoreContext.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -104,7 +104,6 @@  discard block
 block discarded – undo
104 104
     
105 105
     /**
106 106
      * @param $name
107
-     * @param array $params
108 107
      *
109 108
      * @return string
110 109
      */
@@ -115,6 +114,7 @@  discard block
 block discarded – undo
115 114
 
116 115
     /**
117 116
      * @When /^I hover over the element "([^"]*)"$/
117
+     * @param string $locator
118 118
      */
119 119
     public function iHoverOverTheElement($locator)
120 120
     {
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
     }
260 260
     
261 261
     /**
262
-     * @param $locator
262
+     * @param string $locator
263 263
      * @param string $selector
264 264
      *
265 265
      * @return \Behat\Mink\Element\NodeElement|mixed|null
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
     public function iSetMainWindowName()
168 168
     {
169 169
         $window_name = 'main_window';
170
-        $script = 'window.name = "' . $window_name . '"';
170
+        $script = 'window.name = "'.$window_name.'"';
171 171
         $this->getSession()->executeScript($script);
172 172
     }
173 173
     
@@ -226,14 +226,14 @@  discard block
 block discarded – undo
226 226
                 break;
227 227
             
228 228
             default:
229
-                throw new \Exception(__METHOD__ . ' Couldn\'t find selector: ' . $selector . ' - Allowed selectors: #id, .className, //xpath');
229
+                throw new \Exception(__METHOD__.' Couldn\'t find selector: '.$selector.' - Allowed selectors: #id, .className, //xpath');
230 230
                 break;
231 231
         }
232 232
         
233 233
         try {
234 234
             $this->getSession()->executeScript($function);
235 235
         } catch (\Exception $e) {
236
-            throw new \Exception(__METHOD__ . ' failed'. ' Message: for this locator:"'.$selector.'"');
236
+            throw new \Exception(__METHOD__.' failed'.' Message: for this locator:"'.$selector.'"');
237 237
         }
238 238
     }
239 239
     
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
      *
254 254
      * @return \Behat\Mink\Element\NodeElement|mixed|null
255 255
      */
256
-    public function getElement($locator, $selector='css')
256
+    public function getElement($locator, $selector = 'css')
257 257
     {
258 258
         $page = $this->minkContext->getSession()->getPage();
259 259
         $element = $page->find('css', $locator);
@@ -287,12 +287,12 @@  discard block
 block discarded – undo
287 287
         $session = $this->getSession();
288 288
         $element = $session->getPage()->find(
289 289
             'xpath',
290
-            $session->getSelectorsHandler()->selectorToXpath('xpath', '*//*[text()="'. $text .'"]')
290
+            $session->getSelectorsHandler()->selectorToXpath('xpath', '*//*[text()="'.$text.'"]')
291 291
         );
292 292
         if (null === $element) {
293 293
             $element = $session->getPage()->find(
294 294
                 'named',
295
-                array('id',$text)
295
+                array('id', $text)
296 296
             );
297 297
         }
298 298
         if (null === $element) {
Please login to merge, or discard this patch.
module/Core/src/Entity/Hydrator/FileCollectionUploadHydrator.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@
 block discarded – undo
21 21
     protected $elementName;
22 22
 
23 23
     /**
24
-     * @param $name
25
-     * @param $strategy
24
+     * @param string $name
25
+     * @param Strategy\FileUploadStrategy $strategy
26 26
      */
27 27
     public function __construct($name, $strategy)
28 28
     {
Please login to merge, or discard this patch.
module/Core/src/Entity/Hydrator/JsonEntityHydrator.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -29,6 +29,9 @@
 block discarded – undo
29 29
     {
30 30
     }
31 31
 
32
+    /**
33
+     * @param EntityHydrator $hydrator
34
+     */
32 35
     public function injectHydrator($hydrator)
33 36
     {
34 37
         if (empty($hydrator) || !$hydrator instanceof AbstractHydrator) {
Please login to merge, or discard this patch.
module/Jobs/src/Acl/WriteAssertion.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,8 +52,8 @@
 block discarded – undo
52 52
     /**
53 53
      * Returns true, if the user has write access to the job granted from the organization.
54 54
      *
55
-     * @param RoleInterface $role This must be a UserInterface instance
56
-     * @param ResourceInterface $resource This must be a JobInterface instance
55
+     * @param UserInterface $role This must be a UserInterface instance
56
+     * @param JobInterface $resource This must be a JobInterface instance
57 57
      *
58 58
      * @return bool
59 59
      */
Please login to merge, or discard this patch.
module/Settings/src/Form/FormAbstract.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -31,6 +31,11 @@
 block discarded – undo
31 31
         return $this->hydrator;
32 32
     }
33 33
     
34
+    /**
35
+     * @param string $name
36
+     *
37
+     * @return callable
38
+     */
34 39
     protected function getPlugin($name)
35 40
     {
36 41
         $plugin = null;
Please login to merge, or discard this patch.
module/Install/src/Factory/Controller/LazyControllerFactory.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
                             $parameter_instances[] = $container->get($cn);
55 55
                         } catch (\Exception $x) {
56 56
                             echo __CLASS__
57
-                                 . " couldn't create an instance of $cn to satisfy the constructor for $requestedName.";
57
+                                    . " couldn't create an instance of $cn to satisfy the constructor for $requestedName.";
58 58
                             exit;
59 59
                         }
60 60
                     } else {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@
 block discarded – undo
33 33
     
34 34
     public function canCreate(ContainerInterface $container, $requestedName)
35 35
     {
36
-        list($module, ) = explode('\\', __NAMESPACE__, 2);
37
-        return strstr($requestedName, $module . '\Controller') !== false;
36
+        list($module,) = explode('\\', __NAMESPACE__, 2);
37
+        return strstr($requestedName, $module.'\Controller') !== false;
38 38
     }
39 39
     
40 40
     public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
Please login to merge, or discard this patch.