@@ -103,7 +103,7 @@ |
||
| 103 | 103 | abstract protected function applyChanges( Item $item ); |
| 104 | 104 | |
| 105 | 105 | /** |
| 106 | - * @return Item|null |
|
| 106 | + * @return null|\Wikibase\DataModel\Entity\EntityDocument |
|
| 107 | 107 | */ |
| 108 | 108 | private function getItem() { |
| 109 | 109 | $params = $this->getParams(); |
@@ -176,6 +176,9 @@ discard block |
||
| 176 | 176 | $changeOp->apply( $item ); |
| 177 | 177 | } |
| 178 | 178 | |
| 179 | + /** |
|
| 180 | + * @param string $itemIdString |
|
| 181 | + */ |
|
| 179 | 182 | private function makeNewItemWithClaim( $itemIdString, $snak ) { |
| 180 | 183 | $item = new Item( new ItemId( $itemIdString ) ); |
| 181 | 184 | |
@@ -189,6 +192,9 @@ discard block |
||
| 189 | 192 | return $item; |
| 190 | 193 | } |
| 191 | 194 | |
| 195 | + /** |
|
| 196 | + * @param string $propertyId |
|
| 197 | + */ |
|
| 192 | 198 | private function makeSnak( $propertyId, $value ) { |
| 193 | 199 | if ( is_string( $propertyId ) ) { |
| 194 | 200 | $propertyId = new PropertyId( $propertyId ); |
@@ -76,6 +76,10 @@ |
||
| 76 | 76 | $this->assertEquals( $rank, $expectedRank, "No reference with expected hash" ); |
| 77 | 77 | } |
| 78 | 78 | |
| 79 | + /** |
|
| 80 | + * @param string $itemIdString |
|
| 81 | + * @param PropertyValueSnak $mainSnak |
|
| 82 | + */ |
|
| 79 | 83 | private function newItemWithClaim( $itemIdString, $mainSnak ) { |
| 80 | 84 | $item = new Item( new ItemId( $itemIdString ) ); |
| 81 | 85 | |
@@ -37,6 +37,9 @@ |
||
| 37 | 37 | */ |
| 38 | 38 | class EditEntityTest extends MediaWikiTestCase { |
| 39 | 39 | |
| 40 | + /** |
|
| 41 | + * @param string $name |
|
| 42 | + */ |
|
| 40 | 43 | private function getUser( $name ) { |
| 41 | 44 | $user = User::newFromName( $name ); |
| 42 | 45 | |
@@ -190,6 +190,9 @@ |
||
| 190 | 190 | return $mockDataTypeLookup; |
| 191 | 191 | } |
| 192 | 192 | |
| 193 | + /** |
|
| 194 | + * @param string $string |
|
| 195 | + */ |
|
| 193 | 196 | private function fixLineEndings( $string ) { |
| 194 | 197 | return preg_replace( '~(*BSR_ANYCRLF)\R~', "\n", $string ); |
| 195 | 198 | } |
@@ -142,6 +142,9 @@ |
||
| 142 | 142 | return $this->wrapInListview( $referencesHtml ); |
| 143 | 143 | } |
| 144 | 144 | |
| 145 | + /** |
|
| 146 | + * @param string $listviewContent |
|
| 147 | + */ |
|
| 145 | 148 | private function wrapInListview( $listviewContent ) { |
| 146 | 149 | if ( $listviewContent !== '' ) { |
| 147 | 150 | return $this->templateFactory->render( 'wikibase-listview', $listviewContent ); |
@@ -754,7 +754,7 @@ |
||
| 754 | 754 | * @param Title|null $title use for further information |
| 755 | 755 | * @param bool $local shall links be generated locally or globally |
| 756 | 756 | * |
| 757 | - * @return bool |
|
| 757 | + * @return boolean|null |
|
| 758 | 758 | */ |
| 759 | 759 | public static function onFormat( $data, &$comment, $pre, $auto, $post, $title, $local ) { |
| 760 | 760 | global $wgLang, $wgTitle; |
@@ -74,6 +74,9 @@ |
||
| 74 | 74 | ); |
| 75 | 75 | } |
| 76 | 76 | |
| 77 | + /** |
|
| 78 | + * @param EntityUsage[] $actualUsages |
|
| 79 | + */ |
|
| 77 | 80 | private function hasUsage( $actualUsages, EntityId $entityId, $aspect ) { |
| 78 | 81 | $usage = new EntityUsage( $entityId, $aspect ); |
| 79 | 82 | $key = $usage->getIdentityString(); |
@@ -754,7 +754,7 @@ |
||
| 754 | 754 | * @param Title|null $title use for further information |
| 755 | 755 | * @param bool $local shall links be generated locally or globally |
| 756 | 756 | * |
| 757 | - * @return bool |
|
| 757 | + * @return boolean|null |
|
| 758 | 758 | */ |
| 759 | 759 | public static function onFormat( $data, &$comment, $pre, $auto, $post, $title, $local ) { |
| 760 | 760 | global $wgLang, $wgTitle; |