@@ -124,7 +124,7 @@ discard block |
||
124 | 124 | * @param string $geoService |
125 | 125 | * @param boolean $checkForCoords |
126 | 126 | * |
127 | - * @return LatLongValue|false |
|
127 | + * @return LatLongValue |
|
128 | 128 | */ |
129 | 129 | public static function attemptToGeocode( $coordsOrAddress, $geoService = '', $checkForCoords = true ) { |
130 | 130 | if ( $checkForCoords ) { |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | * @param string $address |
151 | 151 | * @param string $geoService |
152 | 152 | * |
153 | - * @return LatLongValue|false |
|
153 | + * @return LatLongValue |
|
154 | 154 | * @throws MWException |
155 | 155 | */ |
156 | 156 | public static function geocode( $address, $geoService = '' ) { |
@@ -190,6 +190,9 @@ discard block |
||
190 | 190 | return $coordinates; |
191 | 191 | } |
192 | 192 | |
193 | + /** |
|
194 | + * @param string $address |
|
195 | + */ |
|
193 | 196 | private static function getGeocoded( Geocoder $geocoder, $address ) { |
194 | 197 | $coordinates = self::getGeocodedAsArray( $geocoder, $address ); |
195 | 198 | |
@@ -260,7 +263,7 @@ discard block |
||
260 | 263 | * @since 0.7 |
261 | 264 | * |
262 | 265 | * @param string $geocoderIdentifier |
263 | - * @param string|\Maps\Geocoders\Geocoder $geocoder |
|
266 | + * @param string $geocoder |
|
264 | 267 | */ |
265 | 268 | public static function registerGeocoder( $geocoderIdentifier, $geocoder ) { |
266 | 269 | self::$registeredGeocoders[$geocoderIdentifier] = $geocoder; |
@@ -331,7 +334,7 @@ discard block |
||
331 | 334 | * |
332 | 335 | * @param string $geocoderIdentifier |
333 | 336 | * |
334 | - * @return string|bool |
|
337 | + * @return string |
|
335 | 338 | * @throws MWException |
336 | 339 | */ |
337 | 340 | private static function getValidGeocoderIdentifier( $geocoderIdentifier ) { |
@@ -128,6 +128,9 @@ discard block |
||
128 | 128 | } |
129 | 129 | } |
130 | 130 | |
131 | + /** |
|
132 | + * @param string $link |
|
133 | + */ |
|
131 | 134 | protected function setLinkFromParameter( Line &$line , $link ) { |
132 | 135 | if ( filter_var( $link , FILTER_VALIDATE_URL , FILTER_FLAG_SCHEME_REQUIRED ) ) { |
133 | 136 | $line->setLink( $link ); |
@@ -141,7 +144,7 @@ discard block |
||
141 | 144 | * Checks if a string is prefixed with link: |
142 | 145 | * @static |
143 | 146 | * @param $link |
144 | - * @return bool|string |
|
147 | + * @return string|false |
|
145 | 148 | * @since 2.0 |
146 | 149 | */ |
147 | 150 | private function isLinkParameter( $link ) { |
@@ -17,6 +17,9 @@ discard block |
||
17 | 17 | private $fileFetcher; |
18 | 18 | private $geoNamesUser; |
19 | 19 | |
20 | + /** |
|
21 | + * @param string $geoNamesUser |
|
22 | + */ |
|
20 | 23 | public function __construct( FileFetcher $fileFetcher, $geoNamesUser ) { |
21 | 24 | $this->fileFetcher = $fileFetcher; |
22 | 25 | $this->geoNamesUser = $geoNamesUser; |
@@ -57,6 +60,10 @@ discard block |
||
57 | 60 | . urlencode( $this->geoNamesUser ); |
58 | 61 | } |
59 | 62 | |
63 | + /** |
|
64 | + * @param string $xml |
|
65 | + * @param string $tagName |
|
66 | + */ |
|
60 | 67 | private function getXmlElementValue( $xml, $tagName ) { |
61 | 68 | $match = []; |
62 | 69 | preg_match( "/<$tagName>(.*?)<\/$tagName>/", $xml, $match ); |