@@ -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) { |
@@ -42,7 +42,9 @@ discard block |
||
| 42 | 42 | public function addType(TypeInterface $type) |
| 43 | 43 | { |
| 44 | 44 | $typeName = $this->getTypeName($type); |
| 45 | - if ($this->isTypeNameRegistered($typeName)) return $this; |
|
| 45 | + if ($this->isTypeNameRegistered($typeName)) { |
|
| 46 | + return $this; |
|
| 47 | + } |
|
| 46 | 48 | |
| 47 | 49 | $this->typesList[$typeName] = $type; |
| 48 | 50 | return $this; |
@@ -54,7 +56,9 @@ discard block |
||
| 54 | 56 | } |
| 55 | 57 | |
| 56 | 58 | private function getTypeName($type) { |
| 57 | - if (is_string($type)) return $type; |
|
| 59 | + if (is_string($type)) { |
|
| 60 | + return $type; |
|
| 61 | + } |
|
| 58 | 62 | if (is_object($type) && $type instanceof AbstractType) { |
| 59 | 63 | return $type->getName(); |
| 60 | 64 | } |
@@ -48,7 +48,9 @@ |
||
| 48 | 48 | */ |
| 49 | 49 | public function isValidValue($value) |
| 50 | 50 | { |
| 51 | - if (is_null($value)) return true; |
|
| 51 | + if (is_null($value)) { |
|
| 52 | + return true; |
|
| 53 | + } |
|
| 52 | 54 | foreach ($this->getConfig()->get('values') as $item) { |
| 53 | 55 | if ($value === $item['name'] || $value === $item['value']) { |
| 54 | 56 | return true; |
@@ -53,7 +53,9 @@ |
||
| 53 | 53 | // code for creating a new post goes here |
| 54 | 54 | // we simple use our DataProvider for now |
| 55 | 55 | $post = DataProvider::getPost(10); |
| 56 | - if (!empty($args['post']['title'])) $post['title'] = $args['post']['title']; |
|
| 56 | + if (!empty($args['post']['title'])) { |
|
| 57 | + $post['title'] = $args['post']['title']; |
|
| 58 | + } |
|
| 57 | 59 | |
| 58 | 60 | return $post; |
| 59 | 61 | } |
@@ -20,7 +20,9 @@ |
||
| 20 | 20 | |
| 21 | 21 | protected function collectTypes(AbstractType $type) |
| 22 | 22 | { |
| 23 | - if (is_object($type) && array_key_exists($type->getName(), $this->types)) return; |
|
| 23 | + if (is_object($type) && array_key_exists($type->getName(), $this->types)) { |
|
| 24 | + return; |
|
| 25 | + } |
|
| 24 | 26 | |
| 25 | 27 | switch ($type->getKind()) { |
| 26 | 28 | case TypeMap::KIND_INTERFACE: |
@@ -15,7 +15,9 @@ discard block |
||
| 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 : (string) self::keyToCursor($index); |
@@ -81,8 +83,7 @@ discard block |
||
| 81 | 83 | $key = self::cursorToKey($cursor); |
| 82 | 84 | if (empty($array)) { |
| 83 | 85 | $offset = $key; |
| 84 | - } |
|
| 85 | - else { |
|
| 86 | + } else { |
|
| 86 | 87 | $offset = array_search($key, array_keys($array)); |
| 87 | 88 | } |
| 88 | 89 | |
@@ -86,7 +86,9 @@ |
||
| 86 | 86 | |
| 87 | 87 | public function parseValue($value) |
| 88 | 88 | { |
| 89 | - if (is_null($value)) return null; |
|
| 89 | + if (is_null($value)) { |
|
| 90 | + return null; |
|
| 91 | + } |
|
| 90 | 92 | if($value instanceof InputObject) { |
| 91 | 93 | $value = $value->getValue(); |
| 92 | 94 | } |
@@ -219,11 +219,15 @@ |
||
| 219 | 219 | $this->variables = $variables; |
| 220 | 220 | foreach ($this->variableReferences as $reference) { |
| 221 | 221 | /** invalid request with no variable */ |
| 222 | - if (!$reference->getVariable()) continue; |
|
| 222 | + if (!$reference->getVariable()) { |
|
| 223 | + continue; |
|
| 224 | + } |
|
| 223 | 225 | $variableName = $reference->getVariable()->getName(); |
| 224 | 226 | |
| 225 | 227 | /** no variable was set at the time */ |
| 226 | - if (!array_key_exists($variableName, $variables)) continue; |
|
| 228 | + if (!array_key_exists($variableName, $variables)) { |
|
| 229 | + continue; |
|
| 230 | + } |
|
| 227 | 231 | |
| 228 | 232 | $reference->getVariable()->setValue($variables[$variableName]); |
| 229 | 233 | $reference->setValue($variables[$variableName]); |