@@ -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) { |
@@ -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); |
@@ -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 | |
@@ -21,7 +21,7 @@ |
||
| 21 | 21 | ->addField('kind', 'string') |
| 22 | 22 | ->addField('description', 'string') |
| 23 | 23 | ->addField('ofType', new QueryListType(), [ |
| 24 | - 'resolve' => function () { |
|
| 24 | + 'resolve' => function() { |
|
| 25 | 25 | return []; |
| 26 | 26 | } |
| 27 | 27 | ]) |
@@ -226,10 +226,10 @@ |
||
| 226 | 226 | |
| 227 | 227 | $value = substr($this->source, $start, $this->pos - $start); |
| 228 | 228 | |
| 229 | - if(strpos($value, '.') === false){ |
|
| 230 | - $value = (int) $value; |
|
| 229 | + if (strpos($value, '.') === false) { |
|
| 230 | + $value = (int)$value; |
|
| 231 | 231 | } else { |
| 232 | - $value = (float) $value; |
|
| 232 | + $value = (float)$value; |
|
| 233 | 233 | } |
| 234 | 234 | |
| 235 | 235 | return new Token(Token::TYPE_NUMBER, $value); |
@@ -40,7 +40,7 @@ |
||
| 40 | 40 | */ |
| 41 | 41 | public function addFields($fieldsArray) |
| 42 | 42 | { |
| 43 | - foreach($fieldsArray as $fieldName => $fieldConfig) { |
|
| 43 | + foreach ($fieldsArray as $fieldName => $fieldConfig) { |
|
| 44 | 44 | if (is_object($fieldConfig)) { |
| 45 | 45 | $this->addField($fieldName, $fieldConfig); |
| 46 | 46 | } else { |
@@ -79,7 +79,9 @@ discard block |
||
| 79 | 79 | |
| 80 | 80 | private function isArrayOfValues($data) |
| 81 | 81 | { |
| 82 | - if (!is_array($data)) return false; |
|
| 82 | + if (!is_array($data)) { |
|
| 83 | + return false; |
|
| 84 | + } |
|
| 83 | 85 | |
| 84 | 86 | foreach ($data as $item) { |
| 85 | 87 | if (!array_key_exists('value', $item)) { |
@@ -92,10 +94,14 @@ discard block |
||
| 92 | 94 | |
| 93 | 95 | private function isArrayOfFields($data) |
| 94 | 96 | { |
| 95 | - if (!is_array($data)) return false; |
|
| 97 | + if (!is_array($data)) { |
|
| 98 | + return false; |
|
| 99 | + } |
|
| 96 | 100 | |
| 97 | 101 | foreach ($data as $name => $item) { |
| 98 | - if (!$this->isField($item, $name)) return false; |
|
| 102 | + if (!$this->isField($item, $name)) { |
|
| 103 | + return false; |
|
| 104 | + } |
|
| 99 | 105 | } |
| 100 | 106 | |
| 101 | 107 | return true; |
@@ -109,7 +115,9 @@ discard block |
||
| 109 | 115 | |
| 110 | 116 | try { |
| 111 | 117 | /** @todo need to change it to optimize performance */ |
| 112 | - if (empty($data['name'])) $data['name'] = $name; |
|
| 118 | + if (empty($data['name'])) { |
|
| 119 | + $data['name'] = $name; |
|
| 120 | + } |
|
| 113 | 121 | |
| 114 | 122 | $config = new FieldConfig($data); |
| 115 | 123 | |
@@ -123,10 +131,14 @@ discard block |
||
| 123 | 131 | |
| 124 | 132 | private function isArrayOfInputs($data) |
| 125 | 133 | { |
| 126 | - if (!is_array($data)) return false; |
|
| 134 | + if (!is_array($data)) { |
|
| 135 | + return false; |
|
| 136 | + } |
|
| 127 | 137 | |
| 128 | 138 | foreach ($data as $name => $item) { |
| 129 | - if (!$this->isInputField($item, $name)) return false; |
|
| 139 | + if (!$this->isInputField($item, $name)) { |
|
| 140 | + return false; |
|
| 141 | + } |
|
| 130 | 142 | } |
| 131 | 143 | |
| 132 | 144 | return true; |
@@ -139,7 +151,9 @@ discard block |
||
| 139 | 151 | } |
| 140 | 152 | try { |
| 141 | 153 | /** @todo need to change it to optimize performance */ |
| 142 | - if (empty($data['name'])) $data['name'] = $name; |
|
| 154 | + if (empty($data['name'])) { |
|
| 155 | + $data['name'] = $name; |
|
| 156 | + } |
|
| 143 | 157 | |
| 144 | 158 | $config = new InputFieldConfig($data); |
| 145 | 159 | |
@@ -38,7 +38,7 @@ |
||
| 38 | 38 | { |
| 39 | 39 | $callable = $this->get('resolveType'); |
| 40 | 40 | |
| 41 | - if($callable && is_callable($callable)) { |
|
| 41 | + if ($callable && is_callable($callable)) { |
|
| 42 | 42 | return call_user_func_array($callable, [$object]); |
| 43 | 43 | } |
| 44 | 44 | |
@@ -43,7 +43,7 @@ discard block |
||
| 43 | 43 | 'id' => ['type' => TypeMap::TYPE_INT], |
| 44 | 44 | 'name' => ['type' => TypeMap::TYPE_STRING] |
| 45 | 45 | ], |
| 46 | - 'resolve' => function () { |
|
| 46 | + 'resolve' => function() { |
|
| 47 | 47 | return [ |
| 48 | 48 | 'id' => 1, |
| 49 | 49 | 'name' => 'Alex' |
@@ -109,8 +109,8 @@ discard block |
||
| 109 | 109 | ['name' => 'String', 'fields' => []], |
| 110 | 110 | ['name' => '__Schema', 'fields' => [['name' => 'queryType'], ['name' => 'mutationType'], ['name' => 'types']]], |
| 111 | 111 | ['name' => '__Type', 'fields' => [['name' => 'name'], ['name' => 'kind'], ['name' => 'description'], ['name' => 'ofType'], ['name' => 'inputFields'], ['name' => 'enumValues'], ['name' => 'fields'], ['name' => 'interfaces'], ['name' => 'possibleTypes']]], |
| 112 | - ['name' => '__InputValue', 'fields' => [['name' => 'name'],['name' => 'description'],['name' => 'type'],['name' => 'defaultValue'],]], |
|
| 113 | - ['name' => '__EnumValue', 'fields' => [['name' => 'name'],['name' => 'description'],['name' => 'deprecationReason'],['name' => 'isDeprecated'],]], |
|
| 112 | + ['name' => '__InputValue', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'type'], ['name' => 'defaultValue'], ]], |
|
| 113 | + ['name' => '__EnumValue', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'deprecationReason'], ['name' => 'isDeprecated'], ]], |
|
| 114 | 114 | ['name' => 'Boolean', 'fields' => []], |
| 115 | 115 | ['name' => '__Field', 'fields' => [['name' => 'name'], ['name' => 'description'], ['name' => 'isDeprecated'], ['name' => 'deprecationReason'], ['name' => 'type'], ['name' => 'args']]], |
| 116 | 116 | ['name' => '__Argument', 'fields' => [['name' => 'name'], ['name' => 'description']]], |
@@ -207,7 +207,7 @@ discard block |
||
| 207 | 207 | 'id' => ['type' => TypeMap::TYPE_INT], |
| 208 | 208 | 'name' => ['type' => TypeMap::TYPE_STRING] |
| 209 | 209 | ], |
| 210 | - 'resolve' => function () { |
|
| 210 | + 'resolve' => function() { |
|
| 211 | 211 | return [ |
| 212 | 212 | 'id' => 1, |
| 213 | 213 | 'name' => 'Alex' |