@@ -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 | { |
@@ -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); |
@@ -36,7 +36,9 @@ |
||
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) { |
@@ -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(); |
@@ -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); |