@@ -27,7 +27,9 @@ |
||
27 | 27 | if (!$type) { |
28 | 28 | return; |
29 | 29 | } |
30 | - if (is_object($type) && array_key_exists($type->getName(), $this->types)) return; |
|
30 | + if (is_object($type) && array_key_exists($type->getName(), $this->types)) { |
|
31 | + return; |
|
32 | + } |
|
31 | 33 | |
32 | 34 | switch ($type->getKind()) { |
33 | 35 | case TypeMap::KIND_INTERFACE: |
@@ -47,7 +47,9 @@ |
||
47 | 47 | // code for creating a new post goes here |
48 | 48 | // we simple use our DataProvider for now |
49 | 49 | $post = DataProvider::getPost(10); |
50 | - if (!empty($args['post']['title'])) $post['title'] = $args['post']['title']; |
|
50 | + if (!empty($args['post']['title'])) { |
|
51 | + $post['title'] = $args['post']['title']; |
|
52 | + } |
|
51 | 53 | return $post; |
52 | 54 | } |
53 | 55 | ] |
@@ -59,7 +59,9 @@ |
||
59 | 59 | { |
60 | 60 | if ($value instanceof AbstractObjectType) { |
61 | 61 | foreach($value->getInterfaces() as $interface) { |
62 | - if ($interface instanceof $this) return true; |
|
62 | + if ($interface instanceof $this) { |
|
63 | + return true; |
|
64 | + } |
|
63 | 65 | } |
64 | 66 | } |
65 | 67 | return false; |
@@ -26,7 +26,9 @@ |
||
26 | 26 | { |
27 | 27 | if (is_object($typeName)) { |
28 | 28 | /** freeing memory */ |
29 | - if (!($typeName instanceof AbstractScalarType)) throw new ConfigurationException('Configuration problem with type ' . $typeName); |
|
29 | + if (!($typeName instanceof AbstractScalarType)) { |
|
30 | + throw new ConfigurationException('Configuration problem with type ' . $typeName); |
|
31 | + } |
|
30 | 32 | $typeName = $typeName->getName(); |
31 | 33 | } |
32 | 34 | if (TypeService::isScalarType($typeName)) { |
@@ -18,7 +18,9 @@ |
||
18 | 18 | |
19 | 19 | public function buildArguments() |
20 | 20 | { |
21 | - if ($this->_isArgumentsBuilt) return true; |
|
21 | + if ($this->_isArgumentsBuilt) { |
|
22 | + return true; |
|
23 | + } |
|
22 | 24 | |
23 | 25 | if (!empty($this->data['args'])) { |
24 | 26 | $this->addArguments($this->data['args']); |
@@ -28,7 +28,9 @@ discard block |
||
28 | 28 | |
29 | 29 | public function validate($data, $ruleInfo) |
30 | 30 | { |
31 | - if (!is_string($ruleInfo)) return false; |
|
31 | + if (!is_string($ruleInfo)) { |
|
32 | + return false; |
|
33 | + } |
|
32 | 34 | |
33 | 35 | switch ($ruleInfo) { |
34 | 36 | case TypeService::TYPE_ANY: |
@@ -98,7 +100,9 @@ discard block |
||
98 | 100 | |
99 | 101 | private function isEnumValues($data) |
100 | 102 | { |
101 | - if (!is_array($data) || empty($data)) return false; |
|
103 | + if (!is_array($data) || empty($data)) { |
|
104 | + return false; |
|
105 | + } |
|
102 | 106 | |
103 | 107 | foreach ($data as $item) { |
104 | 108 | if (!is_array($item) || !array_key_exists('name', $item) || !is_string($item['name']) || !preg_match('/^[_a-zA-Z][_a-zA-Z0-9]*$/', $item['name'])) { |
@@ -115,7 +119,9 @@ discard block |
||
115 | 119 | |
116 | 120 | private static function isArrayOfInterfaces($data) |
117 | 121 | { |
118 | - if (!is_array($data)) return false; |
|
122 | + if (!is_array($data)) { |
|
123 | + return false; |
|
124 | + } |
|
119 | 125 | |
120 | 126 | foreach ($data as $item) { |
121 | 127 | if (!TypeService::isInterface($item)) { |
@@ -128,10 +134,14 @@ discard block |
||
128 | 134 | |
129 | 135 | private function isArrayOfFields($data) |
130 | 136 | { |
131 | - if (!is_array($data) || empty($data)) return false; |
|
137 | + if (!is_array($data) || empty($data)) { |
|
138 | + return false; |
|
139 | + } |
|
132 | 140 | |
133 | 141 | foreach ($data as $name => $item) { |
134 | - if (!$this->isField($item, $name)) return false; |
|
142 | + if (!$this->isField($item, $name)) { |
|
143 | + return false; |
|
144 | + } |
|
135 | 145 | } |
136 | 146 | |
137 | 147 | return true; |
@@ -157,10 +167,14 @@ discard block |
||
157 | 167 | |
158 | 168 | private function isArrayOfInputFields($data) |
159 | 169 | { |
160 | - if (!is_array($data)) return false; |
|
170 | + if (!is_array($data)) { |
|
171 | + return false; |
|
172 | + } |
|
161 | 173 | |
162 | 174 | foreach ($data as $name => $item) { |
163 | - if (!$this->isInputField($item)) return false; |
|
175 | + if (!$this->isInputField($item)) { |
|
176 | + return false; |
|
177 | + } |
|
164 | 178 | } |
165 | 179 | |
166 | 180 | return true; |
@@ -36,7 +36,9 @@ discard block |
||
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) { |
@@ -54,7 +56,9 @@ discard block |
||
54 | 56 | } elseif (!array_key_exists($fieldName, $data)) { |
55 | 57 | continue; |
56 | 58 | } |
57 | - if (!empty($fieldRules['final'])) unset($fieldRules['final']); |
|
59 | + if (!empty($fieldRules['final'])) { |
|
60 | + unset($fieldRules['final']); |
|
61 | + } |
|
58 | 62 | |
59 | 63 | /** Validation of all other rules*/ |
60 | 64 | foreach ($fieldRules as $ruleName => $ruleInfo) { |
@@ -59,7 +59,9 @@ discard block |
||
59 | 59 | */ |
60 | 60 | public function validateArguments(AbstractField $field, $query, Request $request) |
61 | 61 | { |
62 | - if (!count($field->getArguments())) return true; |
|
62 | + if (!count($field->getArguments())) { |
|
63 | + return true; |
|
64 | + } |
|
63 | 65 | |
64 | 66 | $requiredArguments = array_filter($field->getArguments(), function (InputField $argument) { |
65 | 67 | return $argument->getType()->getKind() == TypeMap::KIND_NON_NULL; |
@@ -141,7 +143,9 @@ discard block |
||
141 | 143 | { |
142 | 144 | $unionTypes = $unionType->getTypes(); |
143 | 145 | $valid = false; |
144 | - if (empty($unionTypes)) return false; |
|
146 | + if (empty($unionTypes)) { |
|
147 | + return false; |
|
148 | + } |
|
145 | 149 | |
146 | 150 | foreach ($unionTypes as $unionType) { |
147 | 151 | if ($unionType->getName() == $type->getName()) { |
@@ -15,7 +15,9 @@ |
||
15 | 15 | |
16 | 16 | public static function cursorForObjectInConnection($data, $object) |
17 | 17 | { |
18 | - if (!is_array($data)) return null; |
|
18 | + if (!is_array($data)) { |
|
19 | + return null; |
|
20 | + } |
|
19 | 21 | |
20 | 22 | $index = array_search($object, $data); |
21 | 23 | return $index === false ? null : self::offsetToCursor($index); |