* Format a coordinate value, convert numbers from floating point
* into degree minute second representation.
*
- * @param int $coord Degrees, minutes and seconds
- * @param string $type Latitude or longitude (for if its a NWS or E)
- * @return mixed A floating point number or whatever we were fed
+ * @param float|string $coord Expected to be a number or numeric string in degrees
+ * @param string $type "latitude" or "longitude"
+ * @return string
*/
- private function formatCoords( $coord, $type ) {
+ private function formatCoords( $coord, string $type ) {
+ if ( !is_numeric( $coord ) ) {
+ wfDebugLog( 'exif', __METHOD__ . ": \"$coord\" is not a number" );
+ return (string)$coord;
+ }
+
$ref = '';
if ( $coord < 0 ) {
$nCoord = -$coord;
$ref = 'W';
}
} else {
- $nCoord = $coord;
+ $nCoord = (float)$coord;
if ( $type === 'latitude' ) {
$ref = 'N';
} elseif ( $type === 'longitude' ) {
}
$deg = floor( $nCoord );
- $min = floor( ( $nCoord - $deg ) * 60.0 );
- $sec = round( ( ( $nCoord - $deg ) - $min / 60 ) * 3600, 2 );
+ $min = floor( ( $nCoord - $deg ) * 60 );
+ $sec = round( ( ( $nCoord - $deg ) * 60 - $min ) * 60, 2 );
$deg = $this->formatNum( $deg );
$min = $this->formatNum( $min );
$sec = $this->formatNum( $sec );
+ // Note the default message "$1° $2′ $3″ $4" ignores the 5th parameter
return $this->msg( 'exif-coordinate-format', $deg, $min, $sec, $ref, $coord )->text();
}