@@ -51,7 +51,7 @@ |
||
| 51 | 51 | * @param IParamDefinition[] $definitions |
| 52 | 52 | * @param IParam[] $params |
| 53 | 53 | * |
| 54 | - * @return mixed |
|
| 54 | + * @return string |
|
| 55 | 55 | */ |
| 56 | 56 | protected function formatValue( $value, IParam $param, array &$definitions, array $params ) { |
| 57 | 57 | // Make sure the service is valid. |
@@ -232,7 +232,7 @@ |
||
| 232 | 232 | * |
| 233 | 233 | * @since 2.0 |
| 234 | 234 | * |
| 235 | - * @return string |
|
| 235 | + * @return boolean |
|
| 236 | 236 | */ |
| 237 | 237 | public function hasGroup() { |
| 238 | 238 | return $this->group !== ''; |
@@ -91,7 +91,7 @@ |
||
| 91 | 91 | * |
| 92 | 92 | * @param $type |
| 93 | 93 | * |
| 94 | - * @return array |
|
| 94 | + * @return string[] |
|
| 95 | 95 | */ |
| 96 | 96 | protected function getDefaultParameters( $type ) { |
| 97 | 97 | return array( 'location1', 'location2', 'unit', 'decimals' ); |
@@ -91,7 +91,7 @@ |
||
| 91 | 91 | * |
| 92 | 92 | * @param $type |
| 93 | 93 | * |
| 94 | - * @return array |
|
| 94 | + * @return string[] |
|
| 95 | 95 | */ |
| 96 | 96 | protected function getDefaultParameters( $type ) { |
| 97 | 97 | return array( 'location1', 'location2', 'unit', 'decimals' ); |
@@ -91,7 +91,7 @@ |
||
| 91 | 91 | * |
| 92 | 92 | * @param $type |
| 93 | 93 | * |
| 94 | - * @return array |
|
| 94 | + * @return string[] |
|
| 95 | 95 | */ |
| 96 | 96 | protected function getDefaultParameters( $type ) { |
| 97 | 97 | return array( 'location1', 'location2', 'unit', 'decimals' ); |
@@ -91,7 +91,7 @@ |
||
| 91 | 91 | * |
| 92 | 92 | * @param $type |
| 93 | 93 | * |
| 94 | - * @return array |
|
| 94 | + * @return string[] |
|
| 95 | 95 | */ |
| 96 | 96 | protected function getDefaultParameters( $type ) { |
| 97 | 97 | return array( 'location1', 'location2', 'unit', 'decimals' ); |
@@ -100,7 +100,7 @@ discard block |
||
| 100 | 100 | * |
| 101 | 101 | * @param DatabaseUpdater $updater |
| 102 | 102 | * |
| 103 | - * @return true |
|
| 103 | + * @return boolean |
|
| 104 | 104 | */ |
| 105 | 105 | public static function onLoadExtensionSchemaUpdates( DatabaseUpdater $updater ) { |
| 106 | 106 | switch( $GLOBALS['wgDBtype'] ) { |
@@ -122,7 +122,7 @@ discard block |
||
| 122 | 122 | * @since 3.0 |
| 123 | 123 | * |
| 124 | 124 | * @param $article WikiPage|Article (depending on MW version, WikiPage in 1.18+) |
| 125 | - * @return type |
|
| 125 | + * @return boolean |
|
| 126 | 126 | */ |
| 127 | 127 | public static function onArticlePurge( &$article ) { |
| 128 | 128 | self::$purgedBeforeStore = true; |
@@ -137,7 +137,7 @@ discard block |
||
| 137 | 137 | * @param Parser &$parser |
| 138 | 138 | * @param string &$text |
| 139 | 139 | * |
| 140 | - * @return true |
|
| 140 | + * @return boolean |
|
| 141 | 141 | */ |
| 142 | 142 | public static function onParserAfterTidy( Parser &$parser, &$text ) { |
| 143 | 143 | |
@@ -165,7 +165,7 @@ discard block |
||
| 165 | 165 | * |
| 166 | 166 | * @param LinksUpdate &$linksUpdate |
| 167 | 167 | * |
| 168 | - * @return true |
|
| 168 | + * @return boolean |
|
| 169 | 169 | */ |
| 170 | 170 | public static function onLinksUpdateConstructed( LinksUpdate &$linksUpdate ) { |
| 171 | 171 | $title = $linksUpdate->getTitle(); |
@@ -209,7 +209,7 @@ discard block |
||
| 209 | 209 | * |
| 210 | 210 | * @param Parser $parser |
| 211 | 211 | * |
| 212 | - * @return true |
|
| 212 | + * @return boolean |
|
| 213 | 213 | */ |
| 214 | 214 | public static function onParserClearState( Parser &$parser ) { |
| 215 | 215 | $parser->getOutput()->mExtMapsLayers = null; |
@@ -91,7 +91,7 @@ |
||
| 91 | 91 | * |
| 92 | 92 | * @param $type |
| 93 | 93 | * |
| 94 | - * @return array |
|
| 94 | + * @return string[] |
|
| 95 | 95 | */ |
| 96 | 96 | protected function getDefaultParameters( $type ) { |
| 97 | 97 | return array( 'location1', 'location2', 'unit', 'decimals' ); |
@@ -124,7 +124,7 @@ discard block |
||
| 124 | 124 | * @param string|false $mappingService |
| 125 | 125 | * @param boolean $checkForCoords |
| 126 | 126 | * |
| 127 | - * @return LatLongValue|false |
|
| 127 | + * @return LatLongValue |
|
| 128 | 128 | */ |
| 129 | 129 | public static function attemptToGeocode( $coordsOrAddress, $geoservice = '', $mappingService = false, $checkForCoords = true ) { |
| 130 | 130 | if ( $checkForCoords ) { |
@@ -200,7 +200,7 @@ discard block |
||
| 200 | 200 | * @param string $geoService |
| 201 | 201 | * @param string|false $mappingService |
| 202 | 202 | * |
| 203 | - * @return LatLongValue|false |
|
| 203 | + * @return LatLongValue |
|
| 204 | 204 | * @throws MWException |
| 205 | 205 | */ |
| 206 | 206 | public static function geocode( $address, $geoService = '', $mappingService = false ) { |
@@ -240,6 +240,9 @@ discard block |
||
| 240 | 240 | return $coordinates; |
| 241 | 241 | } |
| 242 | 242 | |
| 243 | + /** |
|
| 244 | + * @param string $address |
|
| 245 | + */ |
|
| 243 | 246 | private static function getGeocoded( Geocoder $geocoder, $address ) { |
| 244 | 247 | $coordinates = self::getGeocodedAsArray( $geocoder, $address ); |
| 245 | 248 | |
@@ -373,7 +376,7 @@ discard block |
||
| 373 | 376 | * |
| 374 | 377 | * @param string $geocoderIdentifier |
| 375 | 378 | * |
| 376 | - * @return string|bool |
|
| 379 | + * @return string |
|
| 377 | 380 | * @throws MWException |
| 378 | 381 | */ |
| 379 | 382 | protected static function getValidGeocoderIdentifier( $geocoderIdentifier ) { |