diff --git a/lib/API.class.php b/lib/API.class.php index 44dd657..5d87f9f 100755 --- a/lib/API.class.php +++ b/lib/API.class.php @@ -26,6 +26,9 @@ class WPStrava_API { $response = wp_remote_post( $url . $uri, $args ); + if ( is_wp_error( $response ) ) + return $response; + if ( $response['response']['code'] != 200 ) { //see if there's useful info in the body $body = json_decode( $response['body'] ); diff --git a/lib/LatestMapWidget.class.php b/lib/LatestMapWidget.class.php index ec40b16..3534807 100644 --- a/lib/LatestMapWidget.class.php +++ b/lib/LatestMapWidget.class.php @@ -3,26 +3,31 @@ class WPStrava_LatestMapWidget extends WP_Widget { private $som; - + public function __construct() { $this->som = WPStrava_SOM::get_som(); - + parent::__construct( false, 'Strava Latest Map', // Name array( 'description' => __( 'Strava latest ride using static google map image', 'wp-strava' ), ) // Args ); } - + public function form( $instance ) { // outputs the options form on admin + $title = isset( $instance['title'] ) ? esc_attr( $instance['title'] ) : __( 'Latest Activity', 'wp-strava' ); $distance_min = isset( $instance['distance_min'] ) ? esc_attr( $instance['distance_min'] ) : ''; $strava_club_id = isset( $instance['strava_club_id'] ) ? esc_attr( $instance['strava_club_id'] ) : ''; //provide some defaults - //$ride_index_params = $ride_index_params ? $ride_index_params : 'athleteId=21'; + //$ride_index_params = $ride_index_params ?: 'athleteId=21'; ?> +

+ + +

@@ -31,12 +36,13 @@ class WPStrava_LatestMapWidget extends WP_Widget {

- rides; @@ -73,37 +80,38 @@ class WPStrava_LatestMapWidget extends WP_Widget { } if ( ! empty( $rides ) ) { - + if ( ! empty( $distance_min ) ) $rides = $strava_rides->getRidesLongerThan( $rides, $distance_min ); $ride = current( $rides ); //update transients & options - if ( $ride->id != $ride_option->id ) { + if ( empty( $ride_option->id ) || $ride->id != $ride_option->id ) { $build_new = true; update_option( 'strava_latest_map_ride', $ride ); } - if ( $ride->id != $ride_transient->id ) - set_transient( 'strava_latest_map_ride', $ride, 60 * 60 ); //one hour + if ( empty( $ride_transient->id ) || $ride->id != $ride_transient->id ) { + set_transient( 'strava_latest_map_ride', $ride, HOUR_IN_SECONDS ); + } } } - if ( $ride ): + if ( $ride ) { echo $before_widget; - ?>

Latest Ride

- getStaticImage( $ride->id, $build_new ); ?>rides->getRide( $ride_id ); $img = WPStrava_StaticMap::get_image_tag( $ride ); @@ -112,5 +120,4 @@ class WPStrava_LatestMapWidget extends WP_Widget { return $img; } - } \ No newline at end of file diff --git a/lib/LatestRidesWidget.class.php b/lib/LatestRidesWidget.class.php index a4f50b7..c112169 100644 --- a/lib/LatestRidesWidget.class.php +++ b/lib/LatestRidesWidget.class.php @@ -30,7 +30,7 @@ class WPStrava_LatestRidesWidget extends WP_Widget { ?> - strava_request_handler( $strava_club_id, $strava_som_option, $quantity ); ?> + strava_request_handler( $strava_club_id, $quantity ); ?> rides; @@ -85,7 +80,7 @@ class WPStrava_LatestRidesWidget extends WP_Widget { $response = "