Passed
Push — master ( a0694d...b099f7 )
by Valentin
05:07
created
src/Bridge/DataGrid/src/Writer/QueryWriter.php 2 patches
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -52,23 +52,23 @@  discard block
 block discarded – undo
52 52
      */
53 53
     public function write($source, SpecificationInterface $specification, Compiler $compiler)
54 54
     {
55
-        if (!$this->targetAcceptable($source)) {
55
+        if (!$this->targetAcceptable($source)){
56 56
             return null;
57 57
         }
58 58
 
59
-        if ($specification instanceof Specification\FilterInterface) {
59
+        if ($specification instanceof Specification\FilterInterface){
60 60
             return $this->writeFilter($source, $specification, $compiler);
61 61
         }
62 62
 
63
-        if ($specification instanceof Specification\SorterInterface) {
63
+        if ($specification instanceof Specification\SorterInterface){
64 64
             return $this->writeSorter($source, $specification, $compiler);
65 65
         }
66 66
 
67
-        if ($specification instanceof Specification\Pagination\Limit) {
67
+        if ($specification instanceof Specification\Pagination\Limit){
68 68
             return $source->limit($specification->getValue());
69 69
         }
70 70
 
71
-        if ($specification instanceof Specification\Pagination\Offset) {
71
+        if ($specification instanceof Specification\Pagination\Offset){
72 72
             return $source->offset($specification->getValue());
73 73
         }
74 74
 
@@ -83,15 +83,15 @@  discard block
 block discarded – undo
83 83
      */
84 84
     protected function writeFilter($source, Specification\FilterInterface $filter, Compiler $compiler)
85 85
     {
86
-        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map) {
86
+        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map){
87 87
             return $source->where(static function () use ($compiler, $filter, $source): void {
88 88
                 $compiler->compile($source, ...$filter->getFilters());
89 89
             });
90 90
         }
91 91
 
92
-        if ($filter instanceof Specification\Filter\Any) {
92
+        if ($filter instanceof Specification\Filter\Any){
93 93
             return $source->where(static function () use ($compiler, $filter, $source): void {
94
-                foreach ($filter->getFilters() as $subFilter) {
94
+                foreach ($filter->getFilters() as $subFilter){
95 95
                     $source->orWhere(static function () use ($compiler, $subFilter, $source): void {
96 96
                         $compiler->compile($source, $subFilter);
97 97
                     });
@@ -99,9 +99,9 @@  discard block
 block discarded – undo
99 99
             });
100 100
         }
101 101
 
102
-        if ($filter instanceof Specification\Filter\InjectionFilter) {
102
+        if ($filter instanceof Specification\Filter\InjectionFilter){
103 103
             $expression = $filter->getFilter();
104
-            if ($expression instanceof Specification\Filter\Expression) {
104
+            if ($expression instanceof Specification\Filter\Expression){
105 105
                 return $source->where(
106 106
                     $filter->getInjection(),
107 107
                     $this->getExpressionOperator($expression),
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
             }
111 111
         }
112 112
 
113
-        if ($filter instanceof Specification\Filter\Expression) {
113
+        if ($filter instanceof Specification\Filter\Expression){
114 114
             return $source->where(
115 115
                 $filter->getExpression(),
116 116
                 $this->getExpressionOperator($filter),
@@ -127,11 +127,11 @@  discard block
 block discarded – undo
127 127
      */
128 128
     protected function getExpressionOperator(Specification\Filter\Expression $filter): string
129 129
     {
130
-        if ($filter instanceof Specification\Filter\Like) {
130
+        if ($filter instanceof Specification\Filter\Like){
131 131
             return 'LIKE';
132 132
         }
133 133
 
134
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
134
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray){
135 135
             return static::ARRAY_OPERATORS[get_class($filter)];
136 136
         }
137 137
 
@@ -144,11 +144,11 @@  discard block
 block discarded – undo
144 144
      */
145 145
     protected function getExpressionArgs(Specification\Filter\Expression $filter): array
146 146
     {
147
-        if ($filter instanceof Specification\Filter\Like) {
147
+        if ($filter instanceof Specification\Filter\Like){
148 148
             return [sprintf($filter->getPattern(), $this->fetchValue($filter->getValue()))];
149 149
         }
150 150
 
151
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
151
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray){
152 152
             return [new Parameter($this->fetchValue($filter->getValue()))];
153 153
         }
154 154
 
@@ -163,8 +163,8 @@  discard block
 block discarded – undo
163 163
      */
164 164
     protected function writeSorter($source, Specification\SorterInterface $sorter, Compiler $compiler)
165 165
     {
166
-        if ($sorter instanceof Specification\Sorter\SorterSet) {
167
-            foreach ($sorter->getSorters() as $subSorter) {
166
+        if ($sorter instanceof Specification\Sorter\SorterSet){
167
+            foreach ($sorter->getSorters() as $subSorter){
168 168
                 $source = $compiler->compile($source, $subSorter);
169 169
             }
170 170
 
@@ -174,9 +174,9 @@  discard block
 block discarded – undo
174 174
         if (
175 175
             $sorter instanceof Specification\Sorter\AscSorter
176 176
             || $sorter instanceof Specification\Sorter\DescSorter
177
-        ) {
177
+        ){
178 178
             $direction = static::SORTER_DIRECTIONS[get_class($sorter)];
179
-            foreach ($sorter->getExpressions() as $expression) {
179
+            foreach ($sorter->getExpressions() as $expression){
180 180
                 $source = $source->orderBy($expression, $direction);
181 181
             }
182 182
 
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
      */
195 195
     protected function fetchValue($value)
196 196
     {
197
-        if ($value instanceof Specification\ValueInterface) {
197
+        if ($value instanceof Specification\ValueInterface){
198 198
             throw new CompilerException('Value expects user input, none given');
199 199
         }
200 200
 
@@ -207,11 +207,11 @@  discard block
 block discarded – undo
207 207
      */
208 208
     protected function targetAcceptable($target): bool
209 209
     {
210
-        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery) {
210
+        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery){
211 211
             return true;
212 212
         }
213 213
 
214
-        if (class_exists(Select::class) && $target instanceof Select) {
214
+        if (class_exists(Select::class) && $target instanceof Select){
215 215
             return true;
216 216
         }
217 217
 
Please login to merge, or discard this patch.
Braces   +42 added lines, -21 removed lines patch added patch discarded remove patch
@@ -52,23 +52,28 @@  discard block
 block discarded – undo
52 52
      */
53 53
     public function write($source, SpecificationInterface $specification, Compiler $compiler)
54 54
     {
55
-        if (!$this->targetAcceptable($source)) {
55
+        if (!$this->targetAcceptable($source))
56
+        {
56 57
             return null;
57 58
         }
58 59
 
59
-        if ($specification instanceof Specification\FilterInterface) {
60
+        if ($specification instanceof Specification\FilterInterface)
61
+        {
60 62
             return $this->writeFilter($source, $specification, $compiler);
61 63
         }
62 64
 
63
-        if ($specification instanceof Specification\SorterInterface) {
65
+        if ($specification instanceof Specification\SorterInterface)
66
+        {
64 67
             return $this->writeSorter($source, $specification, $compiler);
65 68
         }
66 69
 
67
-        if ($specification instanceof Specification\Pagination\Limit) {
70
+        if ($specification instanceof Specification\Pagination\Limit)
71
+        {
68 72
             return $source->limit($specification->getValue());
69 73
         }
70 74
 
71
-        if ($specification instanceof Specification\Pagination\Offset) {
75
+        if ($specification instanceof Specification\Pagination\Offset)
76
+        {
72 77
             return $source->offset($specification->getValue());
73 78
         }
74 79
 
@@ -83,15 +88,18 @@  discard block
 block discarded – undo
83 88
      */
84 89
     protected function writeFilter($source, Specification\FilterInterface $filter, Compiler $compiler)
85 90
     {
86
-        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map) {
91
+        if ($filter instanceof Specification\Filter\All || $filter instanceof Specification\Filter\Map)
92
+        {
87 93
             return $source->where(static function () use ($compiler, $filter, $source): void {
88 94
                 $compiler->compile($source, ...$filter->getFilters());
89 95
             });
90 96
         }
91 97
 
92
-        if ($filter instanceof Specification\Filter\Any) {
98
+        if ($filter instanceof Specification\Filter\Any)
99
+        {
93 100
             return $source->where(static function () use ($compiler, $filter, $source): void {
94
-                foreach ($filter->getFilters() as $subFilter) {
101
+                foreach ($filter->getFilters() as $subFilter)
102
+                {
95 103
                     $source->orWhere(static function () use ($compiler, $subFilter, $source): void {
96 104
                         $compiler->compile($source, $subFilter);
97 105
                     });
@@ -99,9 +107,11 @@  discard block
 block discarded – undo
99 107
             });
100 108
         }
101 109
 
102
-        if ($filter instanceof Specification\Filter\InjectionFilter) {
110
+        if ($filter instanceof Specification\Filter\InjectionFilter)
111
+        {
103 112
             $expression = $filter->getFilter();
104
-            if ($expression instanceof Specification\Filter\Expression) {
113
+            if ($expression instanceof Specification\Filter\Expression)
114
+            {
105 115
                 return $source->where(
106 116
                     $filter->getInjection(),
107 117
                     $this->getExpressionOperator($expression),
@@ -110,7 +120,8 @@  discard block
 block discarded – undo
110 120
             }
111 121
         }
112 122
 
113
-        if ($filter instanceof Specification\Filter\Expression) {
123
+        if ($filter instanceof Specification\Filter\Expression)
124
+        {
114 125
             return $source->where(
115 126
                 $filter->getExpression(),
116 127
                 $this->getExpressionOperator($filter),
@@ -127,11 +138,13 @@  discard block
 block discarded – undo
127 138
      */
128 139
     protected function getExpressionOperator(Specification\Filter\Expression $filter): string
129 140
     {
130
-        if ($filter instanceof Specification\Filter\Like) {
141
+        if ($filter instanceof Specification\Filter\Like)
142
+        {
131 143
             return 'LIKE';
132 144
         }
133 145
 
134
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
146
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray)
147
+        {
135 148
             return static::ARRAY_OPERATORS[get_class($filter)];
136 149
         }
137 150
 
@@ -144,11 +157,13 @@  discard block
 block discarded – undo
144 157
      */
145 158
     protected function getExpressionArgs(Specification\Filter\Expression $filter): array
146 159
     {
147
-        if ($filter instanceof Specification\Filter\Like) {
160
+        if ($filter instanceof Specification\Filter\Like)
161
+        {
148 162
             return [sprintf($filter->getPattern(), $this->fetchValue($filter->getValue()))];
149 163
         }
150 164
 
151
-        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray) {
165
+        if ($filter instanceof Specification\Filter\InArray || $filter instanceof Specification\Filter\NotInArray)
166
+        {
152 167
             return [new Parameter($this->fetchValue($filter->getValue()))];
153 168
         }
154 169
 
@@ -163,8 +178,10 @@  discard block
 block discarded – undo
163 178
      */
164 179
     protected function writeSorter($source, Specification\SorterInterface $sorter, Compiler $compiler)
165 180
     {
166
-        if ($sorter instanceof Specification\Sorter\SorterSet) {
167
-            foreach ($sorter->getSorters() as $subSorter) {
181
+        if ($sorter instanceof Specification\Sorter\SorterSet)
182
+        {
183
+            foreach ($sorter->getSorters() as $subSorter)
184
+            {
168 185
                 $source = $compiler->compile($source, $subSorter);
169 186
             }
170 187
 
@@ -176,7 +193,8 @@  discard block
 block discarded – undo
176 193
             || $sorter instanceof Specification\Sorter\DescSorter
177 194
         ) {
178 195
             $direction = static::SORTER_DIRECTIONS[get_class($sorter)];
179
-            foreach ($sorter->getExpressions() as $expression) {
196
+            foreach ($sorter->getExpressions() as $expression)
197
+            {
180 198
                 $source = $source->orderBy($expression, $direction);
181 199
             }
182 200
 
@@ -194,7 +212,8 @@  discard block
 block discarded – undo
194 212
      */
195 213
     protected function fetchValue($value)
196 214
     {
197
-        if ($value instanceof Specification\ValueInterface) {
215
+        if ($value instanceof Specification\ValueInterface)
216
+        {
198 217
             throw new CompilerException('Value expects user input, none given');
199 218
         }
200 219
 
@@ -207,11 +226,13 @@  discard block
 block discarded – undo
207 226
      */
208 227
     protected function targetAcceptable($target): bool
209 228
     {
210
-        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery) {
229
+        if (class_exists(SelectQuery::class) && $target instanceof SelectQuery)
230
+        {
211 231
             return true;
212 232
         }
213 233
 
214
-        if (class_exists(Select::class) && $target instanceof Select) {
234
+        if (class_exists(Select::class) && $target instanceof Select)
235
+        {
215 236
             return true;
216 237
         }
217 238
 
Please login to merge, or discard this patch.