@@ -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( |