@@ -66,7 +66,7 @@ |
||
| 66 | 66 | /** |
| 67 | 67 | * @since 1.9 |
| 68 | 68 | * |
| 69 | - * @return true |
|
| 69 | + * @return boolean |
|
| 70 | 70 | */ |
| 71 | 71 | public function process() { |
| 72 | 72 | |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | * @param Parser $parser |
| 23 | 23 | * @param ProcessingResult $result |
| 24 | 24 | * |
| 25 | - * @return mixed |
|
| 25 | + * @return string |
|
| 26 | 26 | */ |
| 27 | 27 | public function handle( Parser $parser, ProcessingResult $result ) { |
| 28 | 28 | $parameters = $result->getParameters(); |
@@ -40,7 +40,7 @@ |
||
| 40 | 40 | * |
| 41 | 41 | * @param Store $store |
| 42 | 42 | * @param PropertyStatisticsStore $propertyStatisticsStore |
| 43 | - * @param RequestOptions $requestOptions|null |
|
| 43 | + * @param RequestOptions $requestOptions |
|
| 44 | 44 | */ |
| 45 | 45 | public function __construct( Store $store, PropertyStatisticsStore $propertyStatisticsStore, RequestOptions $requestOptions = null ) { |
| 46 | 46 | $this->store = $store; |
@@ -177,7 +177,7 @@ |
||
| 177 | 177 | /** |
| 178 | 178 | * @since 2.2 |
| 179 | 179 | * |
| 180 | - * @return array |
|
| 180 | + * @return string[] |
|
| 181 | 181 | */ |
| 182 | 182 | public function getErrors() { |
| 183 | 183 | return $this->errors; |
@@ -134,6 +134,7 @@ discard block |
||
| 134 | 134 | * but just controls query generation. For general effect, the default namespaces |
| 135 | 135 | * should be set to NULL. |
| 136 | 136 | * |
| 137 | + * @param boolean $setNS |
|
| 137 | 138 | * @return Description|null |
| 138 | 139 | */ |
| 139 | 140 | private function getSubqueryDescription( &$setNS ) { |
@@ -242,6 +243,7 @@ discard block |
||
| 242 | 243 | * be the content of "[[ ... ]]". Returns NULL upon error. |
| 243 | 244 | * |
| 244 | 245 | * Parameters $setNS has the same use as in getSubqueryDescription(). |
| 246 | + * @param boolean $setNS |
|
| 245 | 247 | */ |
| 246 | 248 | private function getLinkDescription( &$setNS ) { |
| 247 | 249 | // This method is called when we encountered an opening '[['. The following |
@@ -424,6 +426,7 @@ discard block |
||
| 424 | 426 | * a category or property) and create a suitable description. |
| 425 | 427 | * The first chunk behind the "[[" has already been read and is |
| 426 | 428 | * passed as a parameter. |
| 429 | + * @param string $firstChunk |
|
| 427 | 430 | */ |
| 428 | 431 | private function getArticleDescription( $firstChunk, &$setNS ) { |
| 429 | 432 | $chunk = $firstChunk; |
@@ -469,6 +472,10 @@ discard block |
||
| 469 | 472 | return $this->finishLinkDescription( $chunk, true, $result, $setNS ); |
| 470 | 473 | } |
| 471 | 474 | |
| 475 | + /** |
|
| 476 | + * @param boolean $hasNamespaces |
|
| 477 | + * @param Description|null $result |
|
| 478 | + */ |
|
| 472 | 479 | private function finishLinkDescription( $chunk, $hasNamespaces, $result, &$setNS ) { |
| 473 | 480 | if ( is_null( $result ) ) { // no useful information or concrete error found |
| 474 | 481 | $this->descriptionProcessor->addErrorForMsgCode( 'smw_unexpectedpart', $chunk ); // was smw_badqueryatom |
@@ -565,6 +572,7 @@ discard block |
||
| 565 | 572 | /** |
| 566 | 573 | * Enter a new subblock in the query, which must at some time be terminated by the |
| 567 | 574 | * given $endstring delimiter calling popDelimiter(); |
| 575 | + * @param string $endstring |
|
| 568 | 576 | */ |
| 569 | 577 | private function pushDelimiter( $endstring ) { |
| 570 | 578 | array_push( $this->separatorStack, $endstring ); |
@@ -574,6 +582,7 @@ discard block |
||
| 574 | 582 | * Exit a subblock in the query ending with the given delimiter. |
| 575 | 583 | * If the delimiter does not match the top-most open block, false |
| 576 | 584 | * will be returned. Otherwise return true. |
| 585 | + * @param string $endstring |
|
| 577 | 586 | */ |
| 578 | 587 | private function popDelimiter( $endstring ) { |
| 579 | 588 | $topdelim = array_pop( $this->separatorStack ); |
@@ -166,6 +166,7 @@ discard block |
||
| 166 | 166 | * |
| 167 | 167 | * The return value is the expected combined description. The object $currentDescription will |
| 168 | 168 | * also be changed (if it was non-NULL). |
| 169 | + * @param integer $compoundType |
|
| 169 | 170 | */ |
| 170 | 171 | private function getCompoundDescription( Description $currentDescription = null, Description $newDescription = null, $compoundType = SMW_CONJUNCTION_QUERY ) { |
| 171 | 172 | |
@@ -201,6 +202,10 @@ discard block |
||
| 201 | 202 | } |
| 202 | 203 | } |
| 203 | 204 | |
| 205 | + /** |
|
| 206 | + * @param Description $currentDescription |
|
| 207 | + * @param Description $newDescription |
|
| 208 | + */ |
|
| 204 | 209 | private function newCompoundDescriptionFor( $compoundType, $currentDescription, $newDescription ) { |
| 205 | 210 | |
| 206 | 211 | if ( ( ( $compoundType & SMW_CONJUNCTION_QUERY ) != 0 && ( $currentDescription instanceof Conjunction ) ) || |
@@ -177,6 +177,10 @@ discard block |
||
| 177 | 177 | return $resource; |
| 178 | 178 | } |
| 179 | 179 | |
| 180 | + /** |
|
| 181 | + * @param DIWikiPage $diWikiPage |
|
| 182 | + * @param string $modifier |
|
| 183 | + */ |
|
| 180 | 184 | private function newExpNsResource( $diWikiPage, $modifier ) { |
| 181 | 185 | |
| 182 | 186 | $importDataItem = $this->tryToFindImportDataItem( $diWikiPage, $modifier ); |
@@ -254,6 +258,9 @@ discard block |
||
| 254 | 258 | ); |
| 255 | 259 | } |
| 256 | 260 | |
| 261 | + /** |
|
| 262 | + * @return DataItem |
|
| 263 | + */ |
|
| 257 | 264 | private function tryToFindImportDataItem( DIWikiPage $diWikiPage, $modifier ) { |
| 258 | 265 | |
| 259 | 266 | $importDataItems = null; |
@@ -3,7 +3,6 @@ |
||
| 3 | 3 | namespace SMW\Tests\SQLStore; |
| 4 | 4 | |
| 5 | 5 | use SMW\SQLStore\PropertyTableOutdatedReferenceDisposer; |
| 6 | -use Title; |
|
| 7 | 6 | |
| 8 | 7 | /** |
| 9 | 8 | * @covers \SMW\SQLStore\PropertyTableOutdatedReferenceDisposer |
@@ -60,7 +60,8 @@ discard block |
||
| 60 | 60 | * |
| 61 | 61 | * @since 1.7 |
| 62 | 62 | * |
| 63 | - * @param SMWDataItem $dataItem |
|
| 63 | + * @param DataItem $dataItem |
|
| 64 | + * @param PrintRequest $printRequest |
|
| 64 | 65 | * |
| 65 | 66 | * @return mixed |
| 66 | 67 | */ |
@@ -143,7 +144,7 @@ discard block |
||
| 143 | 144 | * |
| 144 | 145 | * @since 1.7 |
| 145 | 146 | * |
| 146 | - * @param SMWQueryResult $result |
|
| 147 | + * @param SMWQueryResult $queryResult |
|
| 147 | 148 | * |
| 148 | 149 | * @return array |
| 149 | 150 | */ |
@@ -194,6 +195,9 @@ discard block |
||
| 194 | 195 | return array( 'printrequests' => $printRequests, 'results' => $results); |
| 195 | 196 | } |
| 196 | 197 | |
| 198 | + /** |
|
| 199 | + * @param PrintRequest $printRequest |
|
| 200 | + */ |
|
| 197 | 201 | private static function getSerializedPrintRequestFormat( $printRequest ) { |
| 198 | 202 | |
| 199 | 203 | $serialized = array( |
@@ -91,6 +91,9 @@ |
||
| 91 | 91 | return new Conjunction( $subdescriptions ); |
| 92 | 92 | } |
| 93 | 93 | |
| 94 | + /** |
|
| 95 | + * @param \SMWDataValue $dataValue |
|
| 96 | + */ |
|
| 94 | 97 | private function newSubdescription( $dataValue, $comparator ) { |
| 95 | 98 | |
| 96 | 99 | $description = new ValueDescription( |