Passed
Pull Request — master (#368)
by Valentin
03:58
created
src/DataGrid/src/InputMapper.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -32,28 +32,28 @@  discard block
 block discarded – undo
32 32
     public function hasOption(string $option): bool
33 33
     {
34 34
         $mapped = $this->map($option);
35
-        if (!isset($mapped[$option])) {
35
+        if (!isset($mapped[$option])){
36 36
             return $this->input->hasValue($option);
37 37
         }
38 38
 
39 39
         $paths = (array)$mapped[$option][0];
40 40
 
41 41
         $firstPath = array_shift($paths);
42
-        if (!$this->input->hasValue($firstPath)) {
42
+        if (!$this->input->hasValue($firstPath)){
43 43
             return false;
44 44
         }
45 45
 
46
-        if (empty($paths)) {
46
+        if (empty($paths)){
47 47
             return true;
48 48
         }
49 49
 
50 50
         $data = $this->input->getValue($firstPath);
51
-        foreach ($paths as $path) {
52
-            if (!is_array($data)) {
51
+        foreach ($paths as $path){
52
+            if (!is_array($data)){
53 53
                 return false;
54 54
             }
55 55
 
56
-            if (!isset($data[$path])) {
56
+            if (!isset($data[$path])){
57 57
                 return false;
58 58
             }
59 59
 
@@ -67,37 +67,37 @@  discard block
 block discarded – undo
67 67
     {
68 68
         $mappedOptions = $this->map($option);
69 69
 
70
-        if (!isset($mappedOptions)) {
70
+        if (!isset($mappedOptions)){
71 71
             return $this->input->getValue($option);
72 72
         }
73 73
 
74 74
         $output = [];
75
-        foreach ($mappedOptions as $mappedOption) {
75
+        foreach ($mappedOptions as $mappedOption){
76 76
             $paths = (array)$mappedOption[0];
77 77
 
78 78
             $firstPath = array_shift($paths);
79
-            if (!$this->input->getValue($firstPath)) {
79
+            if (!$this->input->getValue($firstPath)){
80 80
                 return false;
81 81
             }
82 82
 
83 83
             $data = $this->input->getValue($firstPath);
84 84
             $unmappedData = $data;
85
-            if (!empty($paths) && is_array($unmappedData)) {
85
+            if (!empty($paths) && is_array($unmappedData)){
86 86
                 dump(compact('unmappedData', 'paths'));
87 87
                 unset($unmappedData[$paths[0]]);
88 88
                 dump(compact('unmappedData'));
89 89
             }
90 90
 
91
-            foreach ($paths as $path) {
91
+            foreach ($paths as $path){
92 92
                 $data = $data[$path];
93 93
             }
94 94
 
95 95
             $result = $data;
96
-            foreach ((array)$mappedOption[1] as $path) {
96
+            foreach ((array)$mappedOption[1] as $path){
97 97
                 $result = [$path => $result];
98 98
             }
99 99
             $output[] = $result;
100
-            if (is_array($unmappedData)) {
100
+            if (is_array($unmappedData)){
101 101
                 $output[] = $unmappedData;
102 102
             }
103 103
         }
@@ -107,14 +107,14 @@  discard block
 block discarded – undo
107 107
 
108 108
     protected function map(string $option): array
109 109
     {
110
-        if (isset($this->mapped[$option])) {
110
+        if (isset($this->mapped[$option])){
111 111
             return $this->mapped[$option];
112 112
         }
113 113
 
114 114
         $result = [];
115
-        foreach ($this->mapping as $from => $to) {
115
+        foreach ($this->mapping as $from => $to){
116 116
             $to = explode('.', $to);
117
-            if (isset($to[0]) && $to[0] === $option) {
117
+            if (isset($to[0]) && $to[0] === $option){
118 118
                 array_shift($to);
119 119
                 $result[] = [explode('.', $from), array_reverse($to)];
120 120
             }
Please login to merge, or discard this patch.
src/DataGrid/src/GridFactory.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -111,11 +111,11 @@  discard block
 block discarded – undo
111 111
         $view = clone $this->view;
112 112
 
113 113
         $filters = [];
114
-        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value) {
115
-            if ($schema->hasFilter($name)) {
114
+        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value){
115
+            if ($schema->hasFilter($name)){
116 116
                 $filter = $schema->getFilter($name)->withValue($value);
117 117
 
118
-                if ($filter !== null) {
118
+                if ($filter !== null){
119 119
                     $source = $this->compiler->compile($source, $filter);
120 120
                     $filters[$name] = $filter->getValue();
121 121
                 }
@@ -123,16 +123,16 @@  discard block
 block discarded – undo
123 123
         }
124 124
         $view = $view->withOption(GridInterface::FILTERS, $filters);
125 125
 
126
-        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)) {
126
+        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)){
127 127
             $view = $view->withOption(GridInterface::COUNT, ($this->count)($source));
128 128
         }
129 129
 
130 130
         $sorters = [];
131
-        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value) {
132
-            if ($schema->hasSorter($name)) {
131
+        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value){
132
+            if ($schema->hasSorter($name)){
133 133
                 $sorter = $schema->getSorter($name)->withDirection($value);
134 134
 
135
-                if ($sorter !== null) {
135
+                if ($sorter !== null){
136 136
                     $source = $this->compiler->compile($source, $sorter);
137 137
                     $sorters[$name] = $sorter->getValue();
138 138
                 }
@@ -140,9 +140,9 @@  discard block
 block discarded – undo
140 140
         }
141 141
         $view = $view->withOption(GridInterface::SORTERS, $sorters);
142 142
 
143
-        if ($schema->getPaginator() !== null) {
143
+        if ($schema->getPaginator() !== null){
144 144
             $paginator = $schema->getPaginator()->withValue($this->getOption(static::KEY_PAGINATE));
145
-            if ($paginator === null) {
145
+            if ($paginator === null){
146 146
                 throw new CompilerException('The paginator can not be null');
147 147
             }
148 148
 
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
             $view = $view->withOption(GridInterface::PAGINATOR, $paginator->getValue());
151 151
         }
152 152
 
153
-        if (!is_iterable($source)) {
153
+        if (!is_iterable($source)){
154 154
             throw new GridViewException('GridView expects the source to be iterable after all.');
155 155
         }
156 156
 
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
     private function getOptionArray(string $option): array
167 167
     {
168 168
         $result = $this->getOption($option);
169
-        if (!is_array($result)) {
169
+        if (!is_array($result)){
170 170
             return [];
171 171
         }
172 172
 
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
181 181
      */
182 182
     private function getOption(string $option)
183 183
     {
184
-        if ($this->mapper->hasOption($option)) {
184
+        if ($this->mapper->hasOption($option)){
185 185
             return $this->mapper->getOption($option);
186 186
         }
187 187
 
Please login to merge, or discard this patch.