@@ -84,7 +84,7 @@ |
||
| 84 | 84 | /** |
| 85 | 85 | * Return list of data source fields available for the data source |
| 86 | 86 | * |
| 87 | - * @return FieldInterface |
|
| 87 | + * @return Field[] |
|
| 88 | 88 | */ |
| 89 | 89 | public function getFields() |
| 90 | 90 | { |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | * @param DataSourceInterface $dataSource |
| 67 | 67 | * @param QueryInterface $query |
| 68 | 68 | * |
| 69 | - * @return \Doctrine\DBAL\Query\QueryBuilder|null |
|
| 69 | + * @return QueryBuilder |
|
| 70 | 70 | */ |
| 71 | 71 | public function getQueryBuilder(DataSourceInterface $dataSource, QueryInterface $query) |
| 72 | 72 | { |
@@ -122,6 +122,7 @@ discard block |
||
| 122 | 122 | * |
| 123 | 123 | * @param array $dataRow Plain array of data for a single row, from the database |
| 124 | 124 | * @param \Netdudes\DataSourceryBundle\DataSource\Configuration\Field $dataSourceField |
| 125 | + * @param \Netdudes\DataSourceryBundle\DataSource\Configuration\FieldInterface[] $fields |
|
| 125 | 126 | * |
| 126 | 127 | * @throws \Exception |
| 127 | 128 | * |
@@ -33,8 +33,8 @@ discard block |
||
| 33 | 33 | private $joinCache = []; |
| 34 | 34 | |
| 35 | 35 | /** |
| 36 | - * @param array $queryBuilderDataSourceFields |
|
| 37 | - * @param $fromAlias |
|
| 36 | + * @param \Netdudes\DataSourceryBundle\DataSource\Configuration\FieldInterface[] $queryBuilderDataSourceFields |
|
| 37 | + * @param string $fromAlias |
|
| 38 | 38 | * @param RequiredFieldsExtractor $requiredFieldsExtractor |
| 39 | 39 | */ |
| 40 | 40 | public function __construct(array $queryBuilderDataSourceFields, $fromAlias, RequiredFieldsExtractor $requiredFieldsExtractor) |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | * Walks a node of the dependency tree, recursively generating an ordered list on Joins |
| 123 | 123 | * that is stored in the $this->joins cache. |
| 124 | 124 | * |
| 125 | - * @param $parentUniqueIdentifier |
|
| 125 | + * @param string $parentUniqueIdentifier |
|
| 126 | 126 | * @param $node |
| 127 | 127 | * @param $descendants |
| 128 | 128 | * @param array $completePath |
@@ -32,7 +32,7 @@ |
||
| 32 | 32 | } |
| 33 | 33 | |
| 34 | 34 | /** |
| 35 | - * @return \string[] |
|
| 35 | + * @return string[] |
|
| 36 | 36 | */ |
| 37 | 37 | public function getExpectedTokenCategories() |
| 38 | 38 | { |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | /** |
| 48 | 48 | * Lex, initialise and return the AST. |
| 49 | 49 | * |
| 50 | - * @param $string |
|
| 50 | + * @param string $string |
|
| 51 | 51 | * |
| 52 | 52 | * @return bool|ASTAssertion|ASTGroup |
| 53 | 53 | */ |
@@ -178,7 +178,7 @@ discard block |
||
| 178 | 178 | * Tries to match the following tokens to an <assertion>. |
| 179 | 179 | * |
| 180 | 180 | * @throws Exception\UQLSyntaxError |
| 181 | - * @return bool|ASTAssertion |
|
| 181 | + * @return boolean |
|
| 182 | 182 | */ |
| 183 | 183 | public function matchAssertion() |
| 184 | 184 | { |
@@ -224,7 +224,7 @@ discard block |
||
| 224 | 224 | /** |
| 225 | 225 | * Tries to match the next token to an <operator>. |
| 226 | 226 | * |
| 227 | - * @return bool |
|
| 227 | + * @return boolean|null |
|
| 228 | 228 | */ |
| 229 | 229 | public function matchOperator() |
| 230 | 230 | { |
@@ -325,7 +325,7 @@ discard block |
||
| 325 | 325 | } |
| 326 | 326 | |
| 327 | 327 | /** |
| 328 | - * @param mixed $tokenIndex |
|
| 328 | + * @param integer $tokenIndex |
|
| 329 | 329 | */ |
| 330 | 330 | public function setTokenIndex($tokenIndex) |
| 331 | 331 | { |
@@ -365,7 +365,7 @@ discard block |
||
| 365 | 365 | /** |
| 366 | 366 | * Helper method. Throws an Exception representing a Syntax Error. |
| 367 | 367 | * |
| 368 | - * @param $message |
|
| 368 | + * @param string $message |
|
| 369 | 369 | * |
| 370 | 370 | * @throws \Exception |
| 371 | 371 | */ |
@@ -387,6 +387,9 @@ discard block |
||
| 387 | 387 | ); |
| 388 | 388 | } |
| 389 | 389 | |
| 390 | + /** |
|
| 391 | + * @param string $message |
|
| 392 | + */ |
|
| 390 | 393 | private function throwSyntaxError($message) |
| 391 | 394 | { |
| 392 | 395 | throw new UQLSyntaxError('Syntax error: ' . $message); |
@@ -396,7 +399,7 @@ discard block |
||
| 396 | 399 | * Transforms a literal subtype (e.g. T_LITERAL_FALSE) into a plain |
| 397 | 400 | * literal match. Plain literals are unchanged. |
| 398 | 401 | * |
| 399 | - * @param $literal |
|
| 402 | + * @param boolean $literal |
|
| 400 | 403 | * |
| 401 | 404 | * @return array |
| 402 | 405 | */ |
@@ -94,6 +94,7 @@ |
||
| 94 | 94 | * This function is cached, and will only be generated once per execution. |
| 95 | 95 | * |
| 96 | 96 | * @param Query $query |
| 97 | + * @param string $entityClass |
|
| 97 | 98 | * |
| 98 | 99 | * @return QueryBuilder |
| 99 | 100 | */ |
@@ -276,7 +276,7 @@ |
||
| 276 | 276 | } |
| 277 | 277 | |
| 278 | 278 | /** |
| 279 | - * @param array $extraMethods |
|
| 279 | + * @param string[] $extraMethods |
|
| 280 | 280 | * |
| 281 | 281 | * @return \PHPUnit_Framework_MockObject_MockObject |
| 282 | 282 | */ |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | } |
| 51 | 51 | |
| 52 | 52 | /** |
| 53 | - * @return mixed |
|
| 53 | + * @return string |
|
| 54 | 54 | */ |
| 55 | 55 | public function getMethod() |
| 56 | 56 | { |
@@ -58,7 +58,7 @@ discard block |
||
| 58 | 58 | } |
| 59 | 59 | |
| 60 | 60 | /** |
| 61 | - * @return mixed |
|
| 61 | + * @return string |
|
| 62 | 62 | */ |
| 63 | 63 | public function getName() |
| 64 | 64 | { |
@@ -255,7 +255,7 @@ discard block |
||
| 255 | 255 | /** |
| 256 | 256 | * Translate Lexer <logic> tokens into Filter Condition Types. |
| 257 | 257 | * |
| 258 | - * @param $token |
|
| 258 | + * @param string $token |
|
| 259 | 259 | * |
| 260 | 260 | * @return string |
| 261 | 261 | * @throws \Exception |
@@ -280,7 +280,7 @@ discard block |
||
| 280 | 280 | * |
| 281 | 281 | * @param mixed $value |
| 282 | 282 | * |
| 283 | - * @return mixed |
|
| 283 | + * @return string |
|
| 284 | 284 | */ |
| 285 | 285 | private function parseValue($value) |
| 286 | 286 | { |