@@ -24,8 +24,8 @@ discard block |
||
| 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 |
||
| 84 | 84 | } |
| 85 | 85 | |
| 86 | 86 | /** |
| 87 | - * @return Field[]|Query[] |
|
| 87 | + * @return Query[] |
|
| 88 | 88 | */ |
| 89 | 89 | public function getFields() |
| 90 | 90 | { |
@@ -23,6 +23,9 @@ discard block |
||
| 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 |
||
| 61 | 64 | } |
| 62 | 65 | |
| 63 | 66 | /** |
| 64 | - * @return Field[]|Query[] |
|
| 67 | + * @return Query[] |
|
| 65 | 68 | */ |
| 66 | 69 | public function getFields() |
| 67 | 70 | { |
@@ -73,7 +73,7 @@ |
||
| 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 | /** |
@@ -1,9 +1,9 @@ |
||
| 1 | 1 | <?php |
| 2 | 2 | /** |
| 3 | - * Date: 23.11.15 |
|
| 4 | - * |
|
| 5 | - * @author Portey Vasil <[email protected]> |
|
| 6 | - */ |
|
| 3 | + * Date: 23.11.15 |
|
| 4 | + * |
|
| 5 | + * @author Portey Vasil <[email protected]> |
|
| 6 | + */ |
|
| 7 | 7 | |
| 8 | 8 | namespace Youshido\GraphQL; |
| 9 | 9 | |
@@ -45,6 +45,9 @@ |
||
| 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); |
@@ -16,7 +16,6 @@ |
||
| 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 | |
@@ -88,20 +88,28 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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(); |
@@ -32,7 +32,7 @@ |
||
| 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' |
@@ -45,7 +45,7 @@ |
||
| 45 | 45 | 'type' => 'int' |
| 46 | 46 | ] |
| 47 | 47 | ], |
| 48 | - 'resolve' => function ($object, $args = []) { |
|
| 48 | + 'resolve' => function($object, $args = []) { |
|
| 49 | 49 | |
| 50 | 50 | } |
| 51 | 51 | ]); |
@@ -18,7 +18,7 @@ discard block |
||
| 18 | 18 | |
| 19 | 19 | public function testTypeName() |
| 20 | 20 | { |
| 21 | - foreach(TestScalarDataProvider::getTypesList() as $typeName) { |
|
| 21 | + foreach (TestScalarDataProvider::getTypesList() as $typeName) { |
|
| 22 | 22 | $className = 'Youshido\GraphQL\Type\Scalar\\' . $typeName; |
| 23 | 23 | /** @var TypeInterface $object */ |
| 24 | 24 | $object = new $className(); |
@@ -28,14 +28,14 @@ discard block |
||
| 28 | 28 | |
| 29 | 29 | public function testScalarPrimitives() |
| 30 | 30 | { |
| 31 | - foreach(TestScalarDataProvider::getTypesList() as $typeName) { |
|
| 31 | + foreach (TestScalarDataProvider::getTypesList() as $typeName) { |
|
| 32 | 32 | $className = 'Youshido\GraphQL\Type\Scalar\\' . $typeName; |
| 33 | 33 | /** @var TypeInterface $object */ |
| 34 | 34 | $object = new $className(); |
| 35 | 35 | $testDataMethod = 'get' . $typeName . 'TestData'; |
| 36 | 36 | $this->checkDescription($object); |
| 37 | 37 | |
| 38 | - foreach(call_user_func(array('Youshido\Tests\DataProvider\TestScalarDataProvider', $testDataMethod)) as list($data, $serialized, $isValid)) { |
|
| 38 | + foreach (call_user_func(array('Youshido\Tests\DataProvider\TestScalarDataProvider', $testDataMethod)) as list($data, $serialized, $isValid)) { |
|
| 39 | 39 | |
| 40 | 40 | $this->checkSerialization($object, $data, $serialized); |
| 41 | 41 | |
@@ -1,9 +1,9 @@ |
||
| 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 | |
@@ -141,7 +141,7 @@ |
||
| 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 | |