Completed
Pull Request — master (#77)
by Sebastian
02:39
created
src/Introspection/Traits/TypeCollectorTrait.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,9 @@
 block discarded – undo
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:
Please login to merge, or discard this patch.
examples/02_blog/Schema/BlogSchema.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,9 @@
 block discarded – undo
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
             ]
Please login to merge, or discard this patch.
src/Type/InterfaceType/AbstractInterfaceType.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,9 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
src/Config/Traits/ArgumentsAwareConfigTrait.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,9 @@
 block discarded – undo
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']);
Please login to merge, or discard this patch.
src/Validator/ConfigValidator/Rules/TypeValidationRule.php 1 patch
Braces   +21 added lines, -7 removed lines patch added patch discarded remove patch
@@ -28,7 +28,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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;
Please login to merge, or discard this patch.
src/Validator/ConfigValidator/ConfigValidator.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@  discard block
 block discarded – undo
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
 block discarded – undo
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) {
Please login to merge, or discard this patch.
src/Relay/Connection/ArrayConnection.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,9 @@
 block discarded – undo
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);
Please login to merge, or discard this patch.
src/Execution/Processor.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -168,7 +168,9 @@
 block discarded – undo
168 168
         $value = [];
169 169
 
170 170
         if ($fieldType->getKind() == TypeMap::KIND_LIST) {
171
-            if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) return null;
171
+            if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) {
172
+                return null;
173
+            }
172 174
             foreach ($resolvedValue as $resolvedValueItem) {
173 175
                 $value[] = [];
174 176
                 $index   = count($value) - 1;
Please login to merge, or discard this patch.
src/Type/ListType/AbstractListType.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,9 @@
 block discarded – undo
37 37
         if ($isValid && $itemType->isInputType()) {
38 38
             foreach($value as $item) {
39 39
                 $isValid = $itemType->isValidValue($item);
40
-                if (!$isValid) return false;
40
+                if (!$isValid) {
41
+                    return false;
42
+                }
41 43
             }
42 44
         }
43 45
         return $isValid;
Please login to merge, or discard this patch.