Completed
Push — develop ( fd8298...e653e2 )
by Carsten
14s
created
module/Core/src/Core/Form/Hydrator/Strategy/TreeSelectStrategy.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@
 block discarded – undo
97 97
     /**
98 98
      * Set the allow multiple selections flag.
99 99
      *
100
-     * @param Callable|bool $flagOrCallback When a Callable is passed, it must return bool.
100
+     * @param \Closure $flagOrCallback When a Callable is passed, it must return bool.
101 101
      *
102 102
      * @return self
103 103
      */
Please login to merge, or discard this patch.
module/Core/src/Core/Form/Hydrator/TreeHydrator.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@
 block discarded – undo
93 93
      * @param  array  $data Form values
94 94
      * @param  Collection $object
95 95
      *
96
-     * @return object
96
+     * @return NodeInterface
97 97
      */
98 98
     public function hydrate(array $data, $object)
99 99
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
         $data = [];
54 54
         $this->flattenTree($object, $data);
55 55
 
56
-        return ['items' => $data ];
56
+        return ['items' => $data];
57 57
 
58 58
     }
59 59
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 
80 80
         if ($tree->hasChildren()) {
81 81
             foreach ($tree->getChildren() as $i => $child) {
82
-                $this->flattenTree($child, $data, $curId . '-' . ($i + 1) );
82
+                $this->flattenTree($child, $data, $curId.'-'.($i + 1));
83 83
             }
84 84
         }
85 85
     }
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
          * unflatten tree
117 117
          */
118 118
         $items = $data['items'];
119
-        $tree = [ '__root__' => array_shift($items) ];
119
+        $tree = ['__root__' => array_shift($items)];
120 120
 
121 121
         foreach ($items as $item) {
122 122
             $parent = substr($item['current'], 0, strrpos($item['current'], '-'));
Please login to merge, or discard this patch.
module/Core/src/Core/Repository/RepositoryService.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
             $name = "\\$namespace\\Entity\\$entityName";
39 39
         }
40 40
         
41
-        $repository  = $this->dm->getRepository($name);
41
+        $repository = $this->dm->getRepository($name);
42 42
         return $repository;
43 43
     }
44 44
     
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -24,6 +24,9 @@
 block discarded – undo
24 24
         $this->dm = $dm;
25 25
     }
26 26
     
27
+    /**
28
+     * @param string $name
29
+     */
27 30
     public function get($name)
28 31
     {
29 32
         if (!class_exists($name)) {
Please login to merge, or discard this patch.
module/Core/src/Core/Form/View/Helper/FormWizardContainer.php 2 patches
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -90,24 +90,24 @@
 block discarded – undo
90 90
             $tabId = $containerId . '-' . strtolower($tabElement->getName());
91 91
             $tabsNav .= '<li><a data-toggle="tab" href="#' . $tabId . '">' . $translate($tabElement->getLabel()) . '</a></li>';
92 92
             $tabsContent .= '<div class="tab-pane" id="' . $tabId . '">'
93
-                          . $formContainer($tabElement, $layout, $parameter)
94
-                          . '</div>';
93
+                            . $formContainer($tabElement, $layout, $parameter)
94
+                            . '</div>';
95 95
         }
96 96
 
97 97
         $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="' . $containerId . '">'
98
-                  . '<ul>' . $tabsNav . '</ul>'
99
-                  . '<div class="tab-content">' . $tabsContent . '</div>';
98
+                    . '<ul>' . $tabsNav . '</ul>'
99
+                    . '<div class="tab-content">' . $tabsContent . '</div>';
100 100
         if ($containerParams['pager']) {
101 101
             $content .='<ul class="pager wizard">'
102
-                  . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
103
-                  . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
104
-                  . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
105
-                  . (false !== $containerParams['finish_label']
102
+                    . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
103
+                    . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
104
+                    . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
105
+                    . (false !== $containerParams['finish_label']
106 106
                      ? '<a class="pull-right" href="' . $containerParams['finish_href'] . '">'
107
-                       . $translate($containerParams['finish_label']) . ' &bull;</a>'
107
+                        . $translate($containerParams['finish_label']) . ' &bull;</a>'
108 108
                      : ''
109 109
                     )
110
-                  . '</li></ul>';
110
+                    . '</li></ul>';
111 111
         }
112 112
         $content .= '</div>';
113 113
 
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -83,28 +83,28 @@
 block discarded – undo
83 83
 
84 84
         $containerId = $container->getAttribute('id');
85 85
         if (!$containerId) {
86
-            $containerId = 'wizardcontainer-' . strtolower(str_replace('\\', '-', get_class($container)));
86
+            $containerId = 'wizardcontainer-'.strtolower(str_replace('\\', '-', get_class($container)));
87 87
         }
88 88
 
89 89
         foreach ($container as $tabElement) {
90
-            $tabId = $containerId . '-' . strtolower($tabElement->getName());
91
-            $tabsNav .= '<li><a data-toggle="tab" href="#' . $tabId . '">' . $translate($tabElement->getLabel()) . '</a></li>';
92
-            $tabsContent .= '<div class="tab-pane" id="' . $tabId . '">'
90
+            $tabId = $containerId.'-'.strtolower($tabElement->getName());
91
+            $tabsNav .= '<li><a data-toggle="tab" href="#'.$tabId.'">'.$translate($tabElement->getLabel()).'</a></li>';
92
+            $tabsContent .= '<div class="tab-pane" id="'.$tabId.'">'
93 93
                           . $formContainer($tabElement, $layout, $parameter)
94 94
                           . '</div>';
95 95
         }
96 96
 
97
-        $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="' . $containerId . '">'
98
-                  . '<ul>' . $tabsNav . '</ul>'
99
-                  . '<div class="tab-content">' . $tabsContent . '</div>';
97
+        $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="'.$containerId.'">'
98
+                  . '<ul>'.$tabsNav.'</ul>'
99
+                  . '<div class="tab-content">'.$tabsContent.'</div>';
100 100
         if ($containerParams['pager']) {
101
-            $content .='<ul class="pager wizard">'
102
-                  . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
103
-                  . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
104
-                  . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
101
+            $content .= '<ul class="pager wizard">'
102
+                  . '<li class="previous"><a href="javascript:;">&larr; '.$translate('previous').'</a></li>'
103
+                  . '<li class="next"><a href="javascript:;">'.$translate('Next').' &rarr;</a></li>'
104
+                  . '<li class="finish'.($containerParams['finish_enabled'] ? '' : ' disabled').'">'
105 105
                   . (false !== $containerParams['finish_label']
106
-                     ? '<a class="pull-right" href="' . $containerParams['finish_href'] . '">'
107
-                       . $translate($containerParams['finish_label']) . ' &bull;</a>'
106
+                     ? '<a class="pull-right" href="'.$containerParams['finish_href'].'">'
107
+                       . $translate($containerParams['finish_label']).' &bull;</a>'
108 108
                      : ''
109 109
                     )
110 110
                   . '</li></ul>';
Please login to merge, or discard this patch.
module/Core/src/Core/Form/Tree/AddItemFieldset.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -79,13 +79,13 @@
 block discarded – undo
79 79
             'name' => [
80 80
                 'required' => true,
81 81
                 'filters' => [
82
-                    [ 'name' => 'StringTrim' ],
82
+                    ['name' => 'StringTrim'],
83 83
                 ],
84 84
             ],
85 85
             'value' => [
86 86
                 'required' => false,
87 87
                 'filters' => [
88
-                    [ 'name' => 'StringTrim' ],
88
+                    ['name' => 'StringTrim'],
89 89
                 ],
90 90
             ],
91 91
         ];
Please login to merge, or discard this patch.
module/Core/src/Core/Form/Tree/ManagementForm.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
      *
36 36
      * @var array
37 37
      */
38
-    protected $scripts = [ '/js/html.sortable.min.js', 'Core/js/forms.tree-management.js' ];
38
+    protected $scripts = ['/js/html.sortable.min.js', 'Core/js/forms.tree-management.js'];
39 39
 
40 40
     /**
41 41
      * Attributes.
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/Factory/Repository/DefaultCategoriesBuilderFactory.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -38,14 +38,14 @@
 block discarded – undo
38 38
 
39 39
         $globalConfigPaths = [];
40 40
         foreach ($config['config_glob_paths'] as $path) {
41
-            $globalConfigPaths[] = substr($path, 0, strrpos($path, DIRECTORY_SEPARATOR)) . DIRECTORY_SEPARATOR;
41
+            $globalConfigPaths[] = substr($path, 0, strrpos($path, DIRECTORY_SEPARATOR)).DIRECTORY_SEPARATOR;
42 42
         }
43 43
 
44 44
         $moduleConfigPath = '.';
45 45
         foreach ($config['module_paths'] as $path) {
46
-            $path = rtrim($path, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;
47
-            if (file_exists($path . 'Jobs')) {
48
-                $moduleConfigPath = $path . 'Jobs' . DIRECTORY_SEPARATOR . 'config' . DIRECTORY_SEPARATOR;
46
+            $path = rtrim($path, DIRECTORY_SEPARATOR).DIRECTORY_SEPARATOR;
47
+            if (file_exists($path.'Jobs')) {
48
+                $moduleConfigPath = $path.'Jobs'.DIRECTORY_SEPARATOR.'config'.DIRECTORY_SEPARATOR;
49 49
                 break;
50 50
             }
51 51
         }
Please login to merge, or discard this patch.
module/Jobs/src/Jobs/Repository/DefaultCategoriesBuilder.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -67,13 +67,13 @@
 block discarded – undo
67 67
         $file = "jobs.categories.$type.php";
68 68
 
69 69
         foreach ($this->globalConfigPaths as $path) {
70
-            if (file_exists($path . $file)) {
71
-                return include $path . $file;
70
+            if (file_exists($path.$file)) {
71
+                return include $path.$file;
72 72
             }
73 73
         }
74 74
 
75
-        if (file_exists($this->moduleConfigPath . $file)) {
76
-            return include $this->moduleConfigPath . $file;
75
+        if (file_exists($this->moduleConfigPath.$file)) {
76
+            return include $this->moduleConfigPath.$file;
77 77
         }
78 78
 
79 79
         return false;
Please login to merge, or discard this patch.
module/Core/src/Core/Controller/Plugin/PaginationBuilder.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -110,7 +110,6 @@
 block discarded – undo
110 110
      * @see \Core\Controller\Plugin\SearchForm::get()
111 111
      *
112 112
      * @param        $elementsFieldset
113
-     * @param null   $buttonsFieldset
114 113
      * @param string $as The name of the key in the result array.
115 114
      *
116 115
      * @return self
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
         }
71 71
 
72 72
         if (!is_array($stack)) {
73
-            throw new \InvalidArgumentException('Expected argument to be of type array, but received ' . gettype($stack));
73
+            throw new \InvalidArgumentException('Expected argument to be of type array, but received '.gettype($stack));
74 74
         }
75 75
 
76 76
         $stack = array_intersect_key($stack, ['params' => true, 'form' => true, 'paginator' => true]);
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
      *
144 144
      * @return self
145 145
      */
146
-    public function params($namespace, $defaults = [ 'page' => 1 ])
146
+    public function params($namespace, $defaults = ['page' => 1])
147 147
     {
148 148
         $this->stack['params'] = [$namespace, $defaults];
149 149
         return $this;
Please login to merge, or discard this patch.