@@ -45,7 +45,7 @@ |
||
45 | 45 | 'type' => 'int' |
46 | 46 | ] |
47 | 47 | ], |
48 | - 'resolve' => function ($object, $args = []) { |
|
48 | + 'resolve' => function($object, $args = []) { |
|
49 | 49 | |
50 | 50 | } |
51 | 51 | ]); |
@@ -36,7 +36,9 @@ |
||
36 | 36 | |
37 | 37 | public function validate($data, $rules = [], $extraFieldsAllowed = null) |
38 | 38 | { |
39 | - if ($extraFieldsAllowed !== null) $this->setExtraFieldsAllowed($extraFieldsAllowed); |
|
39 | + if ($extraFieldsAllowed !== null) { |
|
40 | + $this->setExtraFieldsAllowed($extraFieldsAllowed); |
|
41 | + } |
|
40 | 42 | |
41 | 43 | $processedFields = []; |
42 | 44 | foreach ($rules as $fieldName => $fieldRules) { |
@@ -88,20 +88,28 @@ discard block |
||
88 | 88 | |
89 | 89 | private function isArrayOfFields($data) |
90 | 90 | { |
91 | - if (!is_array($data)) return false; |
|
91 | + if (!is_array($data)) { |
|
92 | + return false; |
|
93 | + } |
|
92 | 94 | |
93 | 95 | foreach ($data as $name => $item) { |
94 | - if (!$this->isField($item, $name)) return false; |
|
96 | + if (!$this->isField($item, $name)) { |
|
97 | + return false; |
|
98 | + } |
|
95 | 99 | } |
96 | 100 | return true; |
97 | 101 | } |
98 | 102 | |
99 | 103 | private function isArrayOfInputs($data) |
100 | 104 | { |
101 | - if (!is_array($data)) return false; |
|
105 | + if (!is_array($data)) { |
|
106 | + return false; |
|
107 | + } |
|
102 | 108 | |
103 | 109 | foreach ($data as $name => $item) { |
104 | - if (!$this->isInputField($item, $name)) return false; |
|
110 | + if (!$this->isInputField($item, $name)) { |
|
111 | + return false; |
|
112 | + } |
|
105 | 113 | } |
106 | 114 | return true; |
107 | 115 | } |
@@ -114,7 +122,9 @@ discard block |
||
114 | 122 | |
115 | 123 | try { |
116 | 124 | /** @todo need to change it to optimize performance */ |
117 | - if (empty($data['name'])) $data['name'] = $name; |
|
125 | + if (empty($data['name'])) { |
|
126 | + $data['name'] = $name; |
|
127 | + } |
|
118 | 128 | |
119 | 129 | $config = new FieldConfig($data); |
120 | 130 | return $config->isValid(); |
@@ -131,7 +141,9 @@ discard block |
||
131 | 141 | } |
132 | 142 | try { |
133 | 143 | /** @todo need to change it to optimize performance */ |
134 | - if (empty($data['name'])) $data['name'] = $name; |
|
144 | + if (empty($data['name'])) { |
|
145 | + $data['name'] = $name; |
|
146 | + } |
|
135 | 147 | |
136 | 148 | $config = new InputFieldConfig($data); |
137 | 149 | return $config->isValid(); |
@@ -19,7 +19,7 @@ discard block |
||
19 | 19 | |
20 | 20 | public function testTypeName() |
21 | 21 | { |
22 | - foreach(TypeMap::getScalarTypes() as $typeName) { |
|
22 | + foreach (TypeMap::getScalarTypes() as $typeName) { |
|
23 | 23 | $className = 'Youshido\GraphQL\Type\Scalar\\' . $typeName . 'Type'; |
24 | 24 | /** @var TypeInterface $object */ |
25 | 25 | $object = new $className(); |
@@ -29,14 +29,14 @@ discard block |
||
29 | 29 | |
30 | 30 | public function testScalarPrimitives() |
31 | 31 | { |
32 | - foreach(TypeMap::getScalarTypes() as $typeName) { |
|
32 | + foreach (TypeMap::getScalarTypes() as $typeName) { |
|
33 | 33 | $className = 'Youshido\GraphQL\Type\Scalar\\' . $typeName . 'Type'; |
34 | 34 | /** @var TypeInterface $object */ |
35 | 35 | $object = new $className(); |
36 | 36 | $testDataMethod = 'get' . $typeName . 'TestData'; |
37 | 37 | $this->checkDescription($object); |
38 | 38 | |
39 | - foreach(call_user_func(array('Youshido\Tests\DataProvider\TestScalarDataProvider', $testDataMethod)) as list($data, $serialized, $isValid)) { |
|
39 | + foreach (call_user_func(array('Youshido\Tests\DataProvider\TestScalarDataProvider', $testDataMethod)) as list($data, $serialized, $isValid)) { |
|
40 | 40 | |
41 | 41 | $this->checkSerialization($object, $data, $serialized); |
42 | 42 |
@@ -21,14 +21,14 @@ |
||
21 | 21 | |
22 | 22 | public function testScalarTypeCheck() |
23 | 23 | { |
24 | - foreach($this->getScalarTypes() as $type) { |
|
24 | + foreach ($this->getScalarTypes() as $type) { |
|
25 | 25 | $this->assertTrue(TypeMap::isInputType($type)); |
26 | 26 | } |
27 | 27 | } |
28 | 28 | |
29 | 29 | public function testScalarTypeObjectCreation() |
30 | 30 | { |
31 | - foreach($this->getScalarTypes() as $type) { |
|
31 | + foreach ($this->getScalarTypes() as $type) { |
|
32 | 32 | $object = TypeMap::getScalarTypeObject($type); |
33 | 33 | $this->assertEquals($object->getKind(), TypeMap::KIND_SCALAR); |
34 | 34 | $this->assertEquals($object->getName(), $type); |
@@ -32,7 +32,7 @@ |
||
32 | 32 | 'id' => ['type' => 'int'], |
33 | 33 | 'name' => ['type' => 'string'] |
34 | 34 | ], |
35 | - 'resolve' => function () { |
|
35 | + 'resolve' => function() { |
|
36 | 36 | return [ |
37 | 37 | 'id' => 1, |
38 | 38 | 'name' => 'Alex' |
@@ -37,7 +37,7 @@ |
||
37 | 37 | return false; |
38 | 38 | } |
39 | 39 | |
40 | - $requiredFields = array_filter($this->getConfig()->getFields(), function (Field $field) { |
|
40 | + $requiredFields = array_filter($this->getConfig()->getFields(), function(Field $field) { |
|
41 | 41 | return $field->getConfig()->isRequired(); |
42 | 42 | }); |
43 | 43 |
@@ -25,7 +25,7 @@ |
||
25 | 25 | */ |
26 | 26 | public function validateArguments($queryType, $query, $request) |
27 | 27 | { |
28 | - $requiredArguments = array_filter($queryType->getConfig()->getArguments(), function (Field $argument) { |
|
28 | + $requiredArguments = array_filter($queryType->getConfig()->getArguments(), function(Field $argument) { |
|
29 | 29 | return $argument->getConfig()->isRequired(); |
30 | 30 | }); |
31 | 31 |