diff --git a/readme.txt b/readme.txt index 3ea91d3..944351e 100755 --- a/readme.txt +++ b/readme.txt @@ -112,6 +112,11 @@ On the WP-Strava settings page you cannot currently remove and add another athle == Changelog == + += Unreleased = + +Fix latest_map class replacing echo with return https://wordpress.org/support/topic/latest-activity-map-shortcode-wrong-placed-on-page/ + = 2.5.0 = Fix missing translation domain on "Save Changes" in settings. https://wordpress.org/support/topic/small-fix-in-settings-php-function-print_clear_input diff --git a/src/WPStrava/LatestMap.php b/src/WPStrava/LatestMap.php index 67edac4..a28815d 100644 --- a/src/WPStrava/LatestMap.php +++ b/src/WPStrava/LatestMap.php @@ -1,7 +1,20 @@ + * @since 2.1.0 + */ public static function get_map_html( $args ) { $defaults = array( 'client_id' => WPStrava::get_instance()->settings->get_default_id(), @@ -35,7 +48,7 @@ class WPStrava_LatestMap { $activity = current( $activities ); - echo empty( $activity->map ) ? + return empty( $activity->map ) ? // Translators: Text with activity name shown in place of image if not available. esc_html( sprintf( __( 'Map not available for activity "%s"', 'wp-strava' ), $activity->name ) ) : // phpcs:disable WordPress.Security.EscapeOutput.OutputNotEscaped -- Image OK.