Completed
Push — master ( 50615c...cea04e )
by Alexandr
02:45
created
src/Parser/Ast/Fragment.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -24,8 +24,8 @@  discard block
 block discarded – undo
24 24
     /**
25 25
      * Fragment constructor.
26 26
      *
27
-     * @param                 $name
28
-     * @param                 $model
27
+     * @param                 boolean|null $name
28
+     * @param                 boolean|null $model
29 29
      * @param Field[]|Query[] $children
30 30
      */
31 31
     public function __construct($name, $model, $children)
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
     }
85 85
 
86 86
     /**
87
-     * @return Field[]|Query[]
87
+     * @return Query[]
88 88
      */
89 89
     public function getFields()
90 90
     {
Please login to merge, or discard this patch.
src/Parser/Ast/Query.php 2 patches
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,6 +23,9 @@  discard block
 block discarded – undo
23 23
     /** @var Field[]|Query[] */
24 24
     protected $fields;
25 25
 
26
+    /**
27
+     * @param boolean|string $alias
28
+     */
26 29
     public function __construct($name, $alias = null, $arguments = [], $fields = [])
27 30
     {
28 31
         $this->name      = $name;
@@ -61,7 +64,7 @@  discard block
 block discarded – undo
61 64
     }
62 65
 
63 66
     /**
64
-     * @return Field[]|Query[]
67
+     * @return Query[]
65 68
      */
66 69
     public function getFields()
67 70
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@
 block discarded – undo
73 73
      */
74 74
     public function hasFields()
75 75
     {
76
-        return (bool) count($this->fields);
76
+        return (bool)count($this->fields);
77 77
     }
78 78
 
79 79
     /**
Please login to merge, or discard this patch.
src/Schema.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -45,6 +45,9 @@
 block discarded – undo
45 45
     {
46 46
     }
47 47
 
48
+    /**
49
+     * @param string $name
50
+     */
48 51
     public function addQuery($name, AbstractObjectType $query)
49 52
     {
50 53
         $this->getQueryType()->getConfig()->addField($name, $query);
Please login to merge, or discard this patch.
src/Validator/ConfigValidator/Rules/TypeValidationRule.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,6 @@
 block discarded – undo
16 16
 use Youshido\GraphQL\Type\Field\InputField;
17 17
 use Youshido\GraphQL\Type\Object\AbstractInputObjectType;
18 18
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
19
-use Youshido\GraphQL\Type\Object\ObjectType;
20 19
 use Youshido\GraphQL\Type\TypeMap;
21 20
 use Youshido\GraphQL\Validator\Exception\ConfigurationException;
22 21
 
Please login to merge, or discard this patch.
Braces   +18 added lines, -6 removed lines patch added patch discarded remove patch
@@ -88,20 +88,28 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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();
Please login to merge, or discard this patch.
Tests/ProcessorTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
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'
Please login to merge, or discard this patch.
Tests/Type/ObjectTypeTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
                         'type' => 'int'
46 46
                     ]
47 47
                 ],
48
-                'resolve'   => function ($object, $args = []) {
48
+                'resolve'   => function($object, $args = []) {
49 49
 
50 50
                 }
51 51
             ]);
Please login to merge, or discard this patch.
src/Definition/TypeDefinitionType.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,9 +1,9 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
- * Date: 03.12.15
4
- *
5
- * @author Portey Vasil <[email protected]>
6
- */
3
+     * Date: 03.12.15
4
+     *
5
+     * @author Portey Vasil <[email protected]>
6
+     */
7 7
 
8 8
 namespace Youshido\GraphQL\Definition;
9 9
 
Please login to merge, or discard this patch.
src/Parser/Parser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@
 block discarded – undo
141 141
                 return new Mutation($name, $alias, $arguments, $fields);
142 142
             }
143 143
         } else {
144
-            if($highLevel && $type == Token::TYPE_MUTATION){
144
+            if ($highLevel && $type == Token::TYPE_MUTATION) {
145 145
                 return new Mutation($name, $alias, $arguments);
146 146
             }
147 147
 
Please login to merge, or discard this patch.
src/Parser/Value/ValueInterface.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,9 +1,9 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
- * Date: 01.12.15
4
- *
5
- * @author Portey Vasil <[email protected]>
6
- */
3
+     * Date: 01.12.15
4
+     *
5
+     * @author Portey Vasil <[email protected]>
6
+     */
7 7
 
8 8
 namespace Youshido\GraphQL\Parser\Value;
9 9
 
Please login to merge, or discard this patch.