From 414e2648bb96d4b8f04153e784ed15a5b0088d0b Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Sun, 31 Mar 2013 21:28:49 -0500
Subject: [PATCH 1/7] Renamed Util to API Added SOM classes for English &
 Metric Use WP HTTP API for requests

---
 lib/API.class.php             |  57 ++++++++++++++++
 lib/LatestMapWidget.class.php | 122 ++++++++++++++++++++++++++++++++++
 lib/Rides.class.php           |  39 +----------
 lib/SOM.class.php             |  25 +++++++
 lib/SOMEnglish.class.php      |  48 +++++++++++++
 lib/SOMMetric.class.php       |  46 +++++++++++++
 lib/Settings.class.php        |  30 +++++++--
 lib/Strava.class.php          |  16 +++++
 lib/Util.class.php            |  57 ----------------
 readme.txt                    |   2 +-
 10 files changed, 341 insertions(+), 101 deletions(-)
 create mode 100755 lib/API.class.php
 create mode 100644 lib/LatestMapWidget.class.php
 create mode 100644 lib/SOM.class.php
 create mode 100644 lib/SOMEnglish.class.php
 create mode 100644 lib/SOMMetric.class.php
 delete mode 100755 lib/Util.class.php

diff --git a/lib/API.class.php b/lib/API.class.php
new file mode 100755
index 0000000..d35f6c9
--- /dev/null
+++ b/lib/API.class.php
@@ -0,0 +1,57 @@
+<?php
+/*
+ * Util is a class with all the utility methods.
+ */
+class WPStrava_API {
+
+	const STRAVA_V1_API = 'http://www.strava.com/api/v1/'; //rides?athleteId=134698
+	const STRAVA_V2_API = 'http://www.strava.com/api/v2/'; //rides/:ride_id/map_details
+
+	/*
+	private $rideUrl = "http://www.strava.com/api/v1/rides/:id";
+	private $rideUrlV2 = "http://www.strava.com/api/v2/rides/:id";
+	private $ridesUrl = "http://www.strava.com/api/v1/rides";
+	private $authenticationUrl = "https://www.strava.com/api/v1/authentication/login";
+	private $authenticationUrlV2 = "https://www.strava.com/api/v2/authentication/login";
+	private $rideMapDetailsUrl = "http://www.strava.com/api/v1/rides/:id/map_details";
+	private $rideMapDetailsUrlV2 = "http://www.strava.com/api/v2/rides/:id/map_details";
+	*/
+	
+	public $ridesLinkUrl = "http://app.strava.com/rides/";
+	public $athletesLinkUrl = "http://app.strava.com/athletes/";
+	public $feedback = '';
+	
+	public function post( $uri, $data = NULL, $version = 2 ) {
+		$url = ( $version == 2 ) ? self::STRAVA_V2_API : self::STRAVA_V1_API;
+
+		$args = array(
+			'body' => http_build_query( $data ),
+		);
+
+		if ( $version == 2 )
+			$args['sslverify'] = false;
+
+		$response = wp_remote_post( $url . $uri, $args );
+
+		if ( empty( $response['response']['code'] ) || $response['response']['code'] != 200 ) {
+    		$this->feedback .= sprintf( __( 'ERROR - %s', 'wp-strava'), print_r( $response, true ) );
+			return false;
+		}
+
+		return json_decode( $response['body'] );
+	}
+
+	public function get( $uri, $version = 2 ) {
+		$url = ( $version == 2 ) ? self::STRAVA_V2_API : self::STRAVA_V1_API;
+
+		$response = wp_remote_get( $url . $uri );
+
+		if ( empty( $response['response']['code'] ) || $response['response']['code'] != 200 ) {
+    		$this->feedback .= sprintf( __( 'ERROR - %s', 'wp-strava'), print_r( $response, true ) );
+			return false;
+		}
+
+		return json_decode( $response['body'] );
+	}
+	
+} // class API
\ No newline at end of file
diff --git a/lib/LatestMapWidget.class.php b/lib/LatestMapWidget.class.php
new file mode 100644
index 0000000..d54aa64
--- /dev/null
+++ b/lib/LatestMapWidget.class.php
@@ -0,0 +1,122 @@
+<?php
+
+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
+        $distance_min = isset( $instance['distance_min'] ) ? esc_attr( $instance['distance_min'] ) : '';
+        $ride_index_params = isset( $instance['ride_index_params'] ) ? esc_attr( $instance['ride_index_params'] ) : '';
+
+		//provide some defaults
+        //$ride_index_params = $ride_index_params ? $ride_index_params : 'athleteId=21';
+
+		?>
+        <p>
+			<label for="<?php echo $this->get_field_id( 'distance_min' ); ?>"><?php echo sprintf( __( 'Min. Distance (%s):', 'wp-strava' ), $this->som->get_distance_label() ); ?></label> 
+        	<input class="widefat" id="<?php echo $this->get_field_id( 'distance_min' ); ?>" name="<?php echo $this->get_field_name( 'distance_min' ); ?>" type="text" value="<?php echo $distance_min; ?>" />
+        </p>
+        <p>
+			<label for="<?php echo $this->get_field_id( 'ride_index_params' ); ?>"><?php _e( 'Ride Search Parameters (one per line): ' ); ?>
+			<a href="https://stravasite-main.pbworks.com/w/page/51754146/Strava%20REST%20API%20Method%3A%20rides%20index" target="_blank"><?php _e( 'help' ); ?></a></label>
+			<textarea name="<?php echo $this->get_field_name( 'ride_index_params' ); ?>" id="<?php echo $this->get_field_id( 'ride_index_params' ); ?>" cols="10" rows="5" class="widefat"><?php echo $ride_index_params; ?></textarea>
+        </p>
+        <?php		
+	}
+	
+	public function update( $new_instance, $old_instance ) {
+		// processes widget options to be saved from the admin
+		$instance = $old_instance;
+		$instance['ride_index_params'] = strip_tags( $new_instance['ride_index_params'] );
+		$instance['distance_min'] = strip_tags( $new_instance['distance_min'] );
+
+		/*
+		if ( empty( $instance['ride_index_params'] ) ) {
+			$instance['ride_index_params'] = "athleteId={$auth->athlete->id}";
+		}
+		*/
+
+		//$instance['athlete_hash'] = strip_tags( $new_instance['athlete_hash'] );
+
+        return $instance;
+	}
+
+	public function widget( $args, $instance ) {
+		extract( $args );
+		$ride_index_params = $instance['ride_index_params'];
+		$distance_min = $instance['distance_min'];
+
+		$rides = $this->getRides( $ride_index_params );
+
+		if ( ! empty( $rides ) ):
+			
+			if ( ! empty( $distance_min ) )
+				$rides = $this->getRidesLongerThan( $rides, $distance_min );
+		
+			$ride = current( $rides );
+
+			$map_deets = $this->getMapDetails( $ride->id );
+			
+			echo $before_widget;
+
+			$max = 50;
+			$count = count( $map_deets->latlng );
+			$mod = (int) ( $count / $max );
+			$points = array();
+			for ( $i = 0; $i < $count; $i += $mod ) {
+				$point = $map_deets->latlng[$i];
+				$points[] = number_format( $point[0],4 ) . ',' . number_format( $point[1],4 );
+			}
+
+			$url_points = join( '|', $points );
+			echo "<img src='http://maps.google.com/maps/api/staticmap?maptype=terrain&size=390x260&sensor=false&path=color:0xFF0000BF|weight:2|{$url_points}' />";
+			/*
+			echo '<pre>';
+			print_r($map_deets);
+			echo '</pre>';
+			*/
+			echo $after_widget;
+		endif;
+	}
+	
+	private function getRides( $params ) {
+		$data = WPStrava::get_instance()->api->get( 'rides?' . implode( '&', explode( "\n", $params ) ), 1 ); //version 1
+		
+		if ( isset( $data->rides ) )
+			return $data->rides;
+		return array();
+	}
+
+	private function getRideInfo( $ride_id ) {
+		return WPStrava::get_instance()->api->get( "rides/{$ride_id}" );
+	}
+
+	private function getRidesLongerThan( $rides, $dist ) {
+		$meters = $this->som->distance_inverse( $dist );
+		
+		$long_rides = array();
+		foreach ( $rides as $ride ) {
+			$ride_info = $this->getRideInfo( $ride->id );
+			if ( $ride_info->ride->distance > $meters ) {
+				$long_rides[] = $ride_info;
+			}
+		}
+		return $long_rides;
+	}
+		
+	private function getMapDetails( $ride_id ) {
+		$token = WPStrava::get_instance()->settings->token;
+		return WPStrava::get_instance()->api->get( "rides/{$ride_id}/map_details?token={$token}" );
+	}
+}
\ No newline at end of file
diff --git a/lib/Rides.class.php b/lib/Rides.class.php
index 019e3ac..a31aea0 100755
--- a/lib/Rides.class.php
+++ b/lib/Rides.class.php
@@ -3,23 +3,9 @@
  * Rides is a class wrapper for the Strava REST API functions.
  */
 class WPStrava_Rides {
-	private $rideUrl = "http://www.strava.com/api/v1/rides/:id";
-	private $rideUrlV2 = "http://www.strava.com/api/v2/rides/:id";
-	private $ridesUrl = "http://www.strava.com/api/v1/rides";
-	private $authenticationUrl = "https://www.strava.com/api/v1/authentication/login";
-	private $authenticationUrlV2 = "https://www.strava.com/api/v2/authentication/login";
-	private $rideMapDetailsUrl = "http://www.strava.com/api/v1/rides/:id/map_details";
-	private $rideMapDetailsUrlV2 = "http://www.strava.com/api/v2/rides/:id/map_details";
-	
-	public $ridesLinkUrl = "http://app.strava.com/rides/";
-	public $athletesLinkUrl = "http://app.strava.com/athletes/";
 	public $stravaRides;
 	public $feedback;
-	
-	public function __construct() {
-		// Empty constructor
-	} // __construct
-	
+		
 	public function getRideDetails($rideId, $systemOfMeasurement) {
 		$url = preg_replace('/:id/', $rideId, $this->rideUrl);
 		$json = file_get_contents($url);
@@ -110,28 +96,7 @@ class WPStrava_Rides {
 			return false;
 		}
 	} // getLatestRides
-	
-	public function getAuthenticationToken($email, $password) {
-		require_once WPSTRAVA_PLUGIN_DIR . 'lib/Util.class.php';
-		$util = new WPStrava_Util();
-		$data = array('email' => $email, 'password' => $password);
-		$json = $util->makePostRequest($this->authenticationUrlV2, $data);
-
-		if($json) {
-			$strava_login = json_decode($json);
-			if(!isset($strava_login->error)) {
-				$this->feedback .= __('Successfully authenticated.', 'wp-strava');
-				return $strava_login->token;
-			} else {
-				$this->feedback .= __('Authentication failed, please check your credentials.', 'wp-strava');
-				return false;
-			}
-		} else {
-			$this->feedback .= __('There was an error pulling data of strava.com.', 'wp-strava');
-			return false;
-		}
-  } // getAuthenticationToken
-    
+	    
     public function getRideMap($rideId, $token, $efforts, $threshold) {
     	if($rideId != 0 AND $token != "") {
     		$url = preg_replace('/:id/', $rideId, $this->rideMapDetailsUrlV2);
diff --git a/lib/SOM.class.php b/lib/SOM.class.php
new file mode 100644
index 0000000..e114a33
--- /dev/null
+++ b/lib/SOM.class.php
@@ -0,0 +1,25 @@
+<?php
+
+abstract class WPStrava_SOM {
+
+	public static function get_som( $som = NULL ) {
+		$som = $som ? $som : WPStrava::get_instance()->settings->som;
+		if ( $som == 'metric' ) {
+			require_once WPSTRAVA_PLUGIN_DIR . 'lib/SOMMetric.class.php';
+			return new WPStrava_SOMMetric();
+		} else {
+			require_once WPSTRAVA_PLUGIN_DIR . 'lib/SOMEnglish.class.php';
+			return new WPStrava_SOMEnglish();
+		}
+			
+	}
+	
+	abstract public function distance( $m );
+	abstract public function distance_inverse( $dist );
+	abstract public function get_distance_label();
+	abstract public function speed( $mps );
+	abstract public function get_speed_label();
+	abstract public function elevation( $m );
+	abstract public function get_elevation_label();
+
+}
\ No newline at end of file
diff --git a/lib/SOMEnglish.class.php b/lib/SOMEnglish.class.php
new file mode 100644
index 0000000..ee9ee8f
--- /dev/null
+++ b/lib/SOMEnglish.class.php
@@ -0,0 +1,48 @@
+<?php
+
+class WPStrava_SOMEnglish extends WPStrava_SOM {
+
+	/**
+	 * @param string $m meters
+	 * @return string mi
+	 */
+	public function distance( $m ) {
+		return number_format( $m / 1609.344, 2 );
+	}
+
+	/**
+	 * @param string $dist miles
+	 * @return string meters
+	 */
+	public function distance_inverse( $dist ) {
+		return number_format( $dist * 1609.344, 2 );
+	}
+
+	public function get_distance_label() {
+		return __( 'mi.', 'wp-strava' );
+	}
+
+	/**
+	 * @param string $mps 
+	 * @return string mph
+	 */
+	public function speed( $mps ) {
+		return number_format( $mps * 2.2369, 2 );
+	}
+
+	public function get_speed_label() {
+		return __( 'mph', 'wp-strava' );
+	}
+
+	/**
+	 * @param string $m meters
+	 * @return string feet
+	 */
+	public function elevation( $m ) {
+		return number_format( $m / 0.3048, 2 );
+	}
+
+	public function get_elevation_label() {
+		return __( 'ft.', 'wp-strava' );
+	}
+}	
\ No newline at end of file
diff --git a/lib/SOMMetric.class.php b/lib/SOMMetric.class.php
new file mode 100644
index 0000000..087d13e
--- /dev/null
+++ b/lib/SOMMetric.class.php
@@ -0,0 +1,46 @@
+<?php
+
+class WPStrava_SOMMetric extends WPStrava_SOM
+	/**
+	 * @param $m meters
+	 * @return string km
+	 */
+	public function distance( $m ) {
+		return number_format( $m / 1000, 2 );
+	}
+
+	/**
+	 * @param string $dist km
+	 * @return string meters
+	 */
+	public function distance_inverse( $dist ) {
+		return number_format( $dist * 1000, 2 );
+	}
+
+	public function get_distance_label() {
+		return __( 'km', 'wp-strava' );
+	}
+
+	/**
+	 * @param $mps
+	 * @return string km/h
+	 */
+	public function speed( $mps ) {
+		return number_format( $mps * 3.6, 2 );
+	}
+
+	public function get_speed_label() {
+		return __( 'km/h', 'wp-strava' );
+	}
+
+	/**
+	 * @param $m meters
+	 * @return string meters
+	 */
+	public function elevation( $m ) {
+		return number_format( $m, 2 );
+	}
+
+	public function get_elevation_label() {
+		return __( 'meters', 'wp-strava' );
+	}
diff --git a/lib/Settings.class.php b/lib/Settings.class.php
index 5f6d368..9eee604 100644
--- a/lib/Settings.class.php
+++ b/lib/Settings.class.php
@@ -2,6 +2,8 @@
 
 class WPStrava_Settings {
 
+	private $feedback;
+	
 	//register admin menus
 	public function hook() {
 		add_action( 'admin_init', array( $this, 'register_strava_settings' ) );
@@ -80,15 +82,12 @@ class WPStrava_Settings {
 
 	public function sanitize_token( $token ) {
 		if ( ! empty( $_POST['strava_password'] ) ) {
-			require_once WPSTRAVA_PLUGIN_DIR . 'lib/Rides.class.php';
-			$ride = new WPStrava_Rides();
-			$email = get_option( 'strava_email' );
-			$token = $ride->getAuthenticationToken( $email, $_POST['strava_password'] );
+			$token = $this->get_authentication_token( $this->email, $_POST['strava_password'] );
 			if ( $token ) {
-				add_settings_error( 'strava_token', 'strava_token', sprintf( __( 'New Strava Token Retrieved: %s', 'wp-strava' ), $ride->feedback ) , 'updated' );
+				add_settings_error( 'strava_token', 'strava_token', sprintf( __( 'New Strava Token Retrieved: %s', 'wp-strava' ), $this->feedback ) , 'updated' );
 				return $token;
 			} else {
-				add_settings_error( 'strava_token', 'strava_token', $ride->feedback );
+				add_settings_error( 'strava_token', 'strava_token', $this->feedback );
 				return NULL;
 			}
 		}
@@ -96,6 +95,25 @@ class WPStrava_Settings {
 		return $token;
 	}
 
+	private function get_authentication_token( $email, $password ) {
+		$data = array( 'email' => $email, 'password' => $password );
+		$strava_login = WPStrava::get_instance()->api->post( 'authentication/login', $data );
+
+		if( $strava_login ) {
+			if( ! isset( $strava_login->error ) ) {
+				$this->feedback .= __( 'Successfully authenticated.', 'wp-strava' );
+				return $strava_login->token;
+			} else {
+				$this->feedback .= __( 'Authentication failed, please check your credentials.', 'wp-strava' );
+				return false;
+			}
+		} else {
+			$this->feedback .= __( 'There was an error pulling data of strava.com.', 'wp-strava' );
+			return false;
+		}
+	} // get_authentication_token
+
+	
 	public function print_options_label() {
 		?><p>Options</p><?php
 	}
diff --git a/lib/Strava.class.php b/lib/Strava.class.php
index 17d3d60..a24aa1a 100644
--- a/lib/Strava.class.php
+++ b/lib/Strava.class.php
@@ -1,12 +1,15 @@
 <?php
 
 require_once WPSTRAVA_PLUGIN_DIR . 'lib/Settings.class.php';
+require_once WPSTRAVA_PLUGIN_DIR . 'lib/SOM.class.php';
 require_once WPSTRAVA_PLUGIN_DIR . 'lib/LatestRidesWidget.class.php';
+require_once WPSTRAVA_PLUGIN_DIR . 'lib/LatestMapWidget.class.php';
 
 class WPStrava {
 
 	private static $instance = NULL;
 	private $settings = NULL;
+	private $api = NULL;
 	
 	private function __construct() {
 		$this->settings = new WPStrava_Settings();
@@ -17,6 +20,7 @@ class WPStrava {
 
 		// Register StravaLatestRidesWidget widget
 		add_action( 'widgets_init', function() {	return register_widget( 'WPStrava_LatestRidesWidget' ); } );
+		add_action( 'widgets_init', function() {	return register_widget( 'WPStrava_LatestMapWidget' ); } );
 		
 	}
 
@@ -30,6 +34,18 @@ class WPStrava {
 		if ( isset( $this->{$name} ) )
 			return $this->{$name};
 
+		if ( $name == 'api' )
+			return $this->get_api();
+
 		return NULL;
 	}
+
+	public function get_api() {
+		if ( ! $this->api ) {
+			require_once WPSTRAVA_PLUGIN_DIR . 'lib/API.class.php';
+			$this->api = new WPStrava_API();
+		}
+
+		return $this->api;
+	}
 }
\ No newline at end of file
diff --git a/lib/Util.class.php b/lib/Util.class.php
deleted file mode 100755
index 462ddf9..0000000
--- a/lib/Util.class.php
+++ /dev/null
@@ -1,57 +0,0 @@
-<?php
-/*
- * Util is a class with all the utility methods.
- */
-class WPStrava_Util {
-	public function makePostRequest ($url, $data) {
-    	$data = http_build_query($data);
-    	$url = parse_url($url);
-    	
-    	if ($url['scheme'] == "http") {
-    		$port = 80;
-    		$domain = "tcp://";
-    	} elseif ($url['scheme'] == "https") {
-    		$port = 443;
-    		$domain = "ssl://";
-    	} else {
-    		$this->feedback .= __('This function only support http and https', 'wp-strava');
-    		return false;
-    	}
-    	
-    	$host = $url['host'];
-    	$path = $url['path'];
-    	$protocol = $url['scheme'] . "://";
-    	
-    	// Open a socket connection to the specified port - timeout 30 seconds
-    	$fp = fsockopen($domain . $host, $port, $error_number, $error_string, 30);
-    	
-    	if ($fp) {
-    		// Build the request headers and data
-    		$request = "POST " . $protocol . $host . $path . " HTTP/1.0\r\n";
-    		$request .= "Host: " . $host . "\r\n"; 
-			$request .= "Content-Type: application/x-www-form-urlencoded\r\n";
-			$request .= "Content-Length: " . strlen($data) . "\r\n";
-			$request .= "Connection: close\r\n\r\n";
-			$request .= $data;
-
-			fputs($fp, $request);
-    		
-    		$result = "";
-    		while (!feof($fp)) {
-    			$result .= fgets($fp, 128);
-    		}
-    	} else {
-    		$this->feedback .= __('ERROR - ' . $error_string . '-' . $error_number, 'wp-strava');
-    		return false;
-    	}
-    	
-    	fclose($fp);
-    	
-    	// Split the result header from the content
-    	$result = explode("\r\n\r\n", $result, 2);
-	    $header = isset($result[0]) ? $result[0] : '';
-	    $content = isset($result[1]) ? $result[1] : '';
-	    
-	    return $content;
-    } // makePostRequest
-} // class Util
\ No newline at end of file
diff --git a/readme.txt b/readme.txt
index 75afa85..f9381f4 100755
--- a/readme.txt
+++ b/readme.txt
@@ -1,5 +1,5 @@
 === Plugin Name ===
-Contributors: cmanon (@cmanon), jrfoell
+Contributors: cmanon, jrfoell
 Donate link: http://cmanon.com/
 Tags: bicycle, cycling, strava
 Requires at least: 2.0

From f12989e5ff4b9dcfa43a7b590b8cf04f962bfa99 Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Sun, 31 Mar 2013 22:44:35 -0500
Subject: [PATCH 2/7] Added transient caching

---
 lib/LatestMapWidget.class.php | 96 ++++++++++++++++++++++++++---------
 lib/SOMEnglish.class.php      |  4 +-
 lib/SOMMetric.class.php       |  2 +-
 3 files changed, 75 insertions(+), 27 deletions(-)

diff --git a/lib/LatestMapWidget.class.php b/lib/LatestMapWidget.class.php
index d54aa64..06204a5 100644
--- a/lib/LatestMapWidget.class.php
+++ b/lib/LatestMapWidget.class.php
@@ -56,38 +56,85 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 		extract( $args );
 		$ride_index_params = $instance['ride_index_params'];
 		$distance_min = $instance['distance_min'];
-
-		$rides = $this->getRides( $ride_index_params );
-
-		if ( ! empty( $rides ) ):
-			
-			if ( ! empty( $distance_min ) )
-				$rides = $this->getRidesLongerThan( $rides, $distance_min );
+		$build_new = false;
 		
-			$ride = current( $rides );
+		//try our transient first
+		$ride_transient = get_transient( 'strava_latest_map_ride' );
+		$ride_option = get_option( 'strava_latest_map_ride' );
 
-			$map_deets = $this->getMapDetails( $ride->id );
+		if ( $ride_transient )
+			$ride = $ride_transient;
+		
+		if ( ! $ride ) {
+			$rides = $this->getRides( $ride_index_params );
+			if ( ! empty( $rides ) ) {
 			
-			echo $before_widget;
+				if ( ! empty( $distance_min ) )
+					$rides = $this->getRidesLongerThan( $rides, $distance_min );
 
-			$max = 50;
-			$count = count( $map_deets->latlng );
-			$mod = (int) ( $count / $max );
+				$ride = current( $rides );
+
+				//update transients & options
+				if ( $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 ( $ride ):			
+			echo $before_widget;
+			//die('<pre>' . print_r($ride, true));
+			?><h3 class="widget-title"><?php echo $ride->ride->name ?></h3>
+			<a href="http://app.strava.com/activities/<?php echo $ride->id ?>"><?php
+			echo $this->getStaticImage( $ride->id, $build_new );
+			?></a><?php
+			echo $after_widget;
+		endif;
+	}
+
+	private function buildImage( $map_details ) {
+		$url = 'http://maps.google.com/maps/api/staticmap?maptype=terrain&size=390x260&sensor=false&path=color:0xFF0000BF|weight:2|';
+		$url_len = strlen( $url );
+		$point_len = 0;
+		$num = 50;
+		$count = count( $map_details->latlng );
+		$full_url = '';
+		$max_chars = 1865;
+		
+		//get the longest usable URL
+		while ( $url_len + $point_len < $max_chars ) {
+			$mod = (int) ( $count / $num );
 			$points = array();
 			for ( $i = 0; $i < $count; $i += $mod ) {
-				$point = $map_deets->latlng[$i];
-				$points[] = number_format( $point[0],4 ) . ',' . number_format( $point[1],4 );
+				$point = $map_details->latlng[$i];
+				$points[] = number_format( $point[0], 4 ) . ',' . number_format( $point[1], 4 );
 			}
 
 			$url_points = join( '|', $points );
-			echo "<img src='http://maps.google.com/maps/api/staticmap?maptype=terrain&size=390x260&sensor=false&path=color:0xFF0000BF|weight:2|{$url_points}' />";
-			/*
-			echo '<pre>';
-			print_r($map_deets);
-			echo '</pre>';
-			*/
-			echo $after_widget;
-		endif;
+			$point_len = strlen( $url_points );
+			if ( $url_len + $point_len < $max_chars )
+				$full_url = $url . $url_points;
+			$num++;
+		}
+		
+		return "<img src='{$full_url}' />";
+	}
+
+	private function getStaticImage( $ride_id, $build_new ) {
+
+		$img = get_option( 'strava_latest_map' );
+		
+		if ( $build_new || ! $img ) {
+			$map_details = $this->getMapDetails( $ride_id );
+			$img = $this->buildImage( $map_details );
+			update_option( 'strava_latest_map', $img );
+		}
+
+		return $img;
 	}
 	
 	private function getRides( $params ) {
@@ -104,7 +151,7 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 
 	private function getRidesLongerThan( $rides, $dist ) {
 		$meters = $this->som->distance_inverse( $dist );
-		
+
 		$long_rides = array();
 		foreach ( $rides as $ride ) {
 			$ride_info = $this->getRideInfo( $ride->id );
@@ -112,6 +159,7 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 				$long_rides[] = $ride_info;
 			}
 		}
+		
 		return $long_rides;
 	}
 		
diff --git a/lib/SOMEnglish.class.php b/lib/SOMEnglish.class.php
index ee9ee8f..8cc6103 100644
--- a/lib/SOMEnglish.class.php
+++ b/lib/SOMEnglish.class.php
@@ -12,10 +12,10 @@ class WPStrava_SOMEnglish extends WPStrava_SOM {
 
 	/**
 	 * @param string $dist miles
-	 * @return string meters
+	 * @return float meters
 	 */
 	public function distance_inverse( $dist ) {
-		return number_format( $dist * 1609.344, 2 );
+		return $dist * 1609.344;
 	}
 
 	public function get_distance_label() {
diff --git a/lib/SOMMetric.class.php b/lib/SOMMetric.class.php
index 087d13e..08b92e6 100644
--- a/lib/SOMMetric.class.php
+++ b/lib/SOMMetric.class.php
@@ -14,7 +14,7 @@ class WPStrava_SOMMetric extends WPStrava_SOM
 	 * @return string meters
 	 */
 	public function distance_inverse( $dist ) {
-		return number_format( $dist * 1000, 2 );
+		return $dist * 1000;
 	}
 
 	public function get_distance_label() {

From 60ae33a0a08fec2e31001b078ca71b5afb85f04f Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Mon, 1 Apr 2013 07:54:07 -0500
Subject: [PATCH 3/7] Fixed syntax error in SOMMetric

---
 lib/SOMMetric.class.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/SOMMetric.class.php b/lib/SOMMetric.class.php
index 08b92e6..9d64678 100644
--- a/lib/SOMMetric.class.php
+++ b/lib/SOMMetric.class.php
@@ -1,6 +1,6 @@
 <?php
 
-class WPStrava_SOMMetric extends WPStrava_SOM
+class WPStrava_SOMMetric extends WPStrava_SOM {
 	/**
 	 * @param $m meters
 	 * @return string km

From 1e140cf700ef61851dade3d8d89c0711a4c8383a Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Mon, 1 Apr 2013 07:56:03 -0500
Subject: [PATCH 4/7] Fixed another syntax error in SOMMetric

---
 lib/SOMMetric.class.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/SOMMetric.class.php b/lib/SOMMetric.class.php
index 9d64678..0828c67 100644
--- a/lib/SOMMetric.class.php
+++ b/lib/SOMMetric.class.php
@@ -44,3 +44,4 @@ class WPStrava_SOMMetric extends WPStrava_SOM {
 	public function get_elevation_label() {
 		return __( 'meters', 'wp-strava' );
 	}
+}
\ No newline at end of file

From 380b73b7c7675ebacb4823fbcea850c5e03a1c52 Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Mon, 1 Apr 2013 15:18:11 -0500
Subject: [PATCH 5/7] Renamed widget title

---
 lib/LatestMapWidget.class.php | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/LatestMapWidget.class.php b/lib/LatestMapWidget.class.php
index 06204a5..a68cdea 100644
--- a/lib/LatestMapWidget.class.php
+++ b/lib/LatestMapWidget.class.php
@@ -87,9 +87,8 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 
 		if ( $ride ):			
 			echo $before_widget;
-			//die('<pre>' . print_r($ride, true));
-			?><h3 class="widget-title"><?php echo $ride->ride->name ?></h3>
-			<a href="http://app.strava.com/activities/<?php echo $ride->id ?>"><?php
+			?><h3 class="widget-title">Latest Ride</h3>
+			<a title="<?php echo $ride->ride->name ?>" href="http://app.strava.com/activities/<?php echo $ride->id ?>"><?php
 			echo $this->getStaticImage( $ride->id, $build_new );
 			?></a><?php
 			echo $after_widget;

From 52da31b09198fe1d3afd4000f60a4206771571bd Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Sun, 7 Apr 2013 17:56:07 -0500
Subject: [PATCH 6/7] Updated API & Rides to be more useful

---
 lib/API.class.php               |  47 +++++++++---
 lib/LatestMapWidget.class.php   |   6 +-
 lib/LatestRidesWidget.class.php |  84 +++++++++-----------
 lib/Rides.class.php             | 132 ++++++++++++--------------------
 lib/SOM.class.php               |   7 ++
 lib/Strava.class.php            |  18 ++++-
 readme.txt                      |   1 +
 wp-strava.php                   |   2 +-
 8 files changed, 149 insertions(+), 148 deletions(-)

diff --git a/lib/API.class.php b/lib/API.class.php
index d35f6c9..33631e7 100755
--- a/lib/API.class.php
+++ b/lib/API.class.php
@@ -17,10 +17,6 @@ class WPStrava_API {
 	private $rideMapDetailsUrlV2 = "http://www.strava.com/api/v2/rides/:id/map_details";
 	*/
 	
-	public $ridesLinkUrl = "http://app.strava.com/rides/";
-	public $athletesLinkUrl = "http://app.strava.com/athletes/";
-	public $feedback = '';
-	
 	public function post( $uri, $data = NULL, $version = 2 ) {
 		$url = ( $version == 2 ) ? self::STRAVA_V2_API : self::STRAVA_V1_API;
 
@@ -33,22 +29,49 @@ class WPStrava_API {
 
 		$response = wp_remote_post( $url . $uri, $args );
 
-		if ( empty( $response['response']['code'] ) || $response['response']['code'] != 200 ) {
-    		$this->feedback .= sprintf( __( 'ERROR - %s', 'wp-strava'), print_r( $response, true ) );
-			return false;
+		if ( $response['response']['code'] != 200 ) {
+			//see if there's useful info in the body
+			$body = json_decode( $response['body'] );
+			$error = '';
+			if ( ! empty( $body->error ) )
+				$error = $body->error;
+			else
+				$error = print_r( $response, true );
+
+			return new WP_Error( 'wp-strava_post',
+								 sprintf( __( 'ERROR %s %s - %s', 'wp-strava'), $response['response']['code'], $response['response']['message'], $error ),
+								 $response );
 		}
 
 		return json_decode( $response['body'] );
 	}
 
-	public function get( $uri, $version = 2 ) {
+	public function get( $uri, $args = NULL, $version = 2 ) {
 		$url = ( $version == 2 ) ? self::STRAVA_V2_API : self::STRAVA_V1_API;
 
-		$response = wp_remote_get( $url . $uri );
+		$url .= $uri;
 
-		if ( empty( $response['response']['code'] ) || $response['response']['code'] != 200 ) {
-    		$this->feedback .= sprintf( __( 'ERROR - %s', 'wp-strava'), print_r( $response, true ) );
-			return false;
+		if ( ! empty( $args ) )
+			$url = add_query_arg( $args, $url );
+
+		$response = wp_remote_get( $url );
+
+		if ( is_wp_error( $response ) )
+			return $response;
+		
+		if ( $response['response']['code'] != 200 ) {
+			die($url);
+			//see if there's useful info in the body
+			$body = json_decode( $response['body'] );
+			$error = '';
+			if ( ! empty( $body->error ) )
+				$error = $body->error;
+			else
+				$error = print_r( $response, true );
+
+			return new WP_Error( 'wp-strava_get',
+								 sprintf( __( 'ERROR %s %s - %s', 'wp-strava'), $response['response']['code'], $response['response']['message'], $error ),
+								 $response );
 		}
 
 		return json_decode( $response['body'] );
diff --git a/lib/LatestMapWidget.class.php b/lib/LatestMapWidget.class.php
index a68cdea..8cd6717 100644
--- a/lib/LatestMapWidget.class.php
+++ b/lib/LatestMapWidget.class.php
@@ -66,7 +66,9 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 			$ride = $ride_transient;
 		
 		if ( ! $ride ) {
-			$rides = $this->getRides( $ride_index_params );
+			die('<pre>'.print_r($ride_index_params, true));
+			$rides = WPStrava::get_instance()->rides->getRidesAdvanced( $ride_index_params );
+			//$rides = $this->getRides( $ride_index_params );
 			if ( ! empty( $rides ) ) {
 			
 				if ( ! empty( $distance_min ) )
@@ -164,6 +166,6 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 		
 	private function getMapDetails( $ride_id ) {
 		$token = WPStrava::get_instance()->settings->token;
-		return WPStrava::get_instance()->api->get( "rides/{$ride_id}/map_details?token={$token}" );
+		return WPStrava::get_instance()->api->get( "rides/{$ride_id}/map_details", array( 'token' => $token ) );
 	}
 }
\ No newline at end of file
diff --git a/lib/LatestRidesWidget.class.php b/lib/LatestRidesWidget.class.php
index 6af3bad..e84a4bb 100644
--- a/lib/LatestRidesWidget.class.php
+++ b/lib/LatestRidesWidget.class.php
@@ -4,15 +4,15 @@
  * WP Strava Latest Rides Widget Class
  */
 class WPStrava_LatestRidesWidget extends WP_Widget {
-
+	
 	public function __construct() {
 		$widget_ops = array('classname' => 'LatestRidesWidget', 'description' => __( 'Will publish your latest rides activity from strava.com.') );
 		parent::__construct('wp-strava', $name = 'Strava Latest Rides', $widget_ops);
-		wp_enqueue_style('wp-strava');
+		wp_enqueue_style('wp-strava'); //TODO only load this when wigit is loaded
 	}
 	
 	/** @see WP_Widget::widget */
-	public function widget($args, $instance) {
+	public function widget( $args, $instance ) {
 		extract($args);
 		
 		//$widget_id = $args['widget_id'];
@@ -21,9 +21,7 @@ class WPStrava_LatestRidesWidget extends WP_Widget {
 		$strava_search_id = empty($instance['strava_search_id']) ? '' : $instance['strava_search_id'];
 		$quantity = empty($instance['quantity']) ? '5' : $instance['quantity'];
 
-		$wpstrava = WPStrava::get_instance();
-		$strava_som_option = $wpstrava->settings->som;
-	   
+	   	$this->som = WPStrava_SOM::get_som();
 		?>
 		<?php echo $before_widget; ?>
 			<?php if ( $title ) echo $before_title . $title . $after_title; ?>
@@ -86,54 +84,44 @@ class WPStrava_LatestRidesWidget extends WP_Widget {
 	// The handler to the ajax call, we will avoid this if Strava support jsonp request and we can do it
 	// the parsing directly on the jQuery ajax call, the returned text will be enclosed in the $response variable.
 	private function strava_request_handler( $strava_search_option, $strava_search_id, $strava_som_option, $quantity ) {
-		$response = "";
 	
 		//Check if the username is empty.
-		if (empty($strava_search_id)) {
-			$response .= __("Please configure the Strava search id on the widget options.", "wp-strava");
-		} else {
-			require_once WPSTRAVA_PLUGIN_DIR . 'lib/Rides.class.php';
-			$strava_rides = new WPStrava_Rides(); 
-			$strava_rides->getLatestRides($strava_search_option, $strava_search_id, $quantity);
-			$rides_details = $strava_rides->getRidesDetails($strava_som_option);
+		if ( empty( $strava_search_id ) )
+			return __("Please configure the Strava search id on the widget options.", "wp-strava");		
+		//else
+		$strava_rides = WPStrava::get_instance()->rides;
 		
-			if ($strava_som_option == "metric") {
-				$units = array(
-					'elapsedTime' => __('hours','wp-strava'),
-					'movingTime' => __('hours','wp-strava'),
-					'distance' => __('km','wp-strava'),
-					'averageSpeed' => __('km/h','wp-strava'),
-					'maximumSpeed' => __('km/h','wp-strava'),
-					'elevationGain' => __('meters','wp-strava')
-							   );
-			} elseif ($strava_som_option == "english") {
-				$units = array(
-					'elapsedTime' => __('hours','wp-strava'),
-					'movingTime' => __('hours','wp-strava'),
-					'distance' => __('miles','wp-strava'),
-					'averageSpeed' => __('mph','wp-strava'),
-					'maximumSpeed' => __('mph','wp-strava'),
-					'elevationGain' => __('feet','wp-strava')
-							   );
-			}
+		$rides = $strava_rides->getRidesSimple( $strava_search_option, $strava_search_id );
+		if ( is_wp_error( $rides ) )
+			return $rides->get_error_message();
+
+		//adjust quantity
+		$rides = array_slice( $rides, 0, $quantity );
 		
-			$response .= "<ul id='rides'>";
-			foreach($rides_details as $ride) {
-				$response .= "<li class='ride'>";
-				$response .= "<a href='" . $strava_rides->ridesLinkUrl . $ride['id'] . "' >" . $ride['name'] . "</a>";
-				$response .= "<div class='ride-item'>";
-				$response .= __("On ", "wp-strava") . $ride['startDate'];
-				if ($strava_search_option == "club") {
-					$response .= " <a href='" . $strava_rides->athletesLinkUrl . $ride['athleteId'] . "'>" . $ride['athleteName'] . "</a>";
-				}
-				$response .= __(" rode ", "wp-strava") . $ride['distance'] . " " . $units['distance'];
-				$response .= __(" during ", "wp-strava") . $ride['elapsedTime'] . " " . $units['elapsedTime'];
-				$response .= __(" climbing ", "wp-strava") . $ride['elevationGain'] . " " . $units['elevationGain'] . ".";			
-				$response .= "</div>";
-				$response .= "</li>";
+		$rides_details = $strava_rides->getRidesDetails( $rides );
+		if ( is_wp_error( $rides_details ) )
+			return $rides_details->get_error_message();
+
+		$response = "<ul id='rides'>";
+		foreach($rides_details as $ride_obj) {
+			$ride = $ride_obj->ride;
+			$response .= "<li class='ride'>";
+			$response .= "<a href='" . WPStrava_Rides::RIDES_URL . $ride->id . "' >" . $ride->name . "</a>";
+			$response .= "<div class='ride-item'>";
+			$unixtime = strtotime( $ride->start_date_local );
+			$response .= sprintf( __("On %s %s", "wp-strava"), date_i18n( get_option( 'date_format' ), $unixtime ), date_i18n( get_option( 'time_format' ), $unixtime ) );
+			
+			if ($strava_search_option == "club") {
+				$response .= " <a href='" . WPStrava_Rides::ATHLETES_URL . $ride->athlete_id . "'>" . $ride->athlete_name . "</a>";
 			}
-			$response .= "</ul>";
+			
+			$response .= sprintf( __(" rode %s %s", "wp-strava"), $this->som->distance( $ride->distance ), $this->som->get_distance_label() );
+			$response .= sprintf( __( " during %s %s", "wp-strava" ), $this->som->time( $ride->elapsed_time ), $this->som->get_time_label() );
+			$response .= sprintf( __( " climbing %s %s", "wp-strava" ), $this->som->elevation( $ride->elevation_gain ), $this->som->get_elevation_label() );
+			$response .= "</div>";
+			$response .= "</li>";
 		}
+		$response .= "</ul>";
 		return $response;
 	} // Function strava_request_handler
 	
diff --git a/lib/Rides.class.php b/lib/Rides.class.php
index a31aea0..c8c7567 100755
--- a/lib/Rides.class.php
+++ b/lib/Rides.class.php
@@ -3,99 +3,65 @@
  * Rides is a class wrapper for the Strava REST API functions.
  */
 class WPStrava_Rides {
-	public $stravaRides;
-	public $feedback;
-		
-	public function getRideDetails($rideId, $systemOfMeasurement) {
-		$url = preg_replace('/:id/', $rideId, $this->rideUrl);
-		$json = file_get_contents($url);
 
-		if($json) {
-			$strava_ride = json_decode($json);
-			
-			//Transform data to a ready to be displayed format
-			$startDate = date("F j, Y - H:i a", strtotime($strava_ride->ride->startDateLocal));
-			$elapsedTime = date("H:i:s", mktime(0, 0, $strava_ride->ride->elapsedTime));
-			$movingTime = date("H:i:s", mktime(0, 0, $strava_ride->ride->movingTime));
-			
-			if ($systemOfMeasurement == "metric") {
-				//To km
-				$distance = number_format($strava_ride->ride->distance/1000, 2);
-				//To km/h
-				$averageSpeed = number_format($strava_ride->ride->averageSpeed * 3.6, 2);
-				//To km/h
-				// Removed on version 2 of the Strava API
-				//$maximumSpeed = number_format($strava_ride->ride->maximumSpeed/1000, 2);
-				//It is already in meters
-				$elevationGain = number_format($strava_ride->ride->elevationGain, 2);
-			} elseif ($systemOfMeasurement == "english") {
-				//To miles
-				$distance = number_format($strava_ride->ride->distance/1609.34, 2);
-				//To miles/h
-				$averageSpeed = number_format($strava_ride->ride->averageSpeed * 2.2369, 2);
-				//To miles/h
-				// Removed on version 2 of the Strava API
-				//$maximumSpeed = number_format($strava_ride->ride->maximumSpeed/1609.34, 2);
-				//To foot
-				$elevationGain = number_format($strava_ride->ride->elevationGain/0.3048, 2);
-			}
-			
-			$ride_details = array(
-				'id' => $rideId,
-				'name' => $strava_ride->ride->name,
-				'athleteId' => $strava_ride->ride->athlete->id,
-				'athleteName' => $strava_ride->ride->athlete->name,
-				'athleteUserName' => $strava_ride->ride->athlete->username,
-				'startDate' => $startDate,
-				'elapsedTime' => $elapsedTime,
-				'movingTime' => $movingTime,
-				'distance' => $distance,
-				'averageSpeed' => $averageSpeed,
-				//'maximumSpeed' => $maximumSpeed,
-				'elevationGain' => $elevationGain
-			);
-			return $ride_details;
-		} else {
-			$this->feedback .= _e("Could not get information from strava.com for the ride id: ") . $stravaRide->id;
-			return false;
-		}
-	} // getRideDetails
+	const RIDES_URL = "http://app.strava.com/rides/";
+	const ATHLETES_URL = "http://app.strava.com/athletes/";
 	
-	public function getRidesDetails($systemOfMeasurement) {
-		if($this->stravaRides) {
-			$rides_details = array();
-			foreach($this->stravaRides as $stravaRide) {
-				$rides_details[] = $this->getRideDetails($stravaRide->id, $systemOfMeasurement);
-			}
-			return $rides_details;
-		} else {
-			$this->feedback .= _e("Please provide the rides array to be processed.", "wp-strava");
-			return false;
+	public function getRideDetails( $rideId ) {
+		return WPStrava::get_instance()->api->get( "rides/{$rideId}" );
+	} // getRideDetails
+		
+	public function getRidesDetails( $rides ) {
+		$rides_details = array();
+		foreach ( $rides as $stravaRide ) {
+			$detail = $this->getRideDetails( $stravaRide->id );
+
+			if ( is_wp_error( $detail ) )
+				return $detail;
+				
+			$rides_details[] = $detail;
 		}
+		return $rides_details;
 	} // getRidesDetails
 	
-	public function getLatestRides($searchOption, $searchId, $quantity) {
-		$url = $this->ridesUrl;
+	public function getRidesSimple( $searchOption, $searchId ) {
+		$api = WPStrava::get_instance()->api;
+
+		$data = NULL;
 		//Get the json results using the constructor specified values.
-		if($searchOption == "athlete") {
-			if(is_numeric($searchId)) {
-				$json = file_get_contents($url . '?athleteId=' . urlencode($searchId));
+		if ( $searchOption == 'athlete' ) {
+			if ( is_numeric( $searchId ) ) {
+				$data = $api->get( 'rides', array( 'athleteId' => $searchId ), 1 );
 			} else {
-				$json = file_get_contents($url . '?athleteName=' . urlencode($searchId));
+				$data = $api->get( 'rides', array( 'athleteName' => $searchId ), 1 );
 			}
-		} elseif ($searchOption == "club" AND is_numeric($searchId)) {
-			$json = file_get_contents($url . '?clubId=' . urlencode($searchId));
+		} elseif ($searchOption == 'club' && is_numeric($searchId)) {
+			$data = $api->get( 'rides', array( 'clubId' => $searchId ), 1 );
 		} else {
-			$this->feedback .= _e("There's an error on the widget options combination.", "wp-strava");
+			return new WP_Error( 'wp-strava_options', __("There's an error in your simple options.", 'wp-strava') );
 		}
-		if($json) {
-			$strava_rides = json_decode($json);
-			$this->stravaRides = array_slice($strava_rides->rides, 0, $quantity);
-		} else {
-			$this->feedback .= _e("There was an error pulling data of strava.com.", "wp-strava");
-			return false;
-		}
-	} // getLatestRides
+
+		if ( is_wp_error( $data ) )
+			return $data;
+		
+		if ( isset( $data->rides ) )
+			return $data->rides;
+		
+		return array();	
+		
+	} // getRidesSimple
+
+	public function getRidesAdvanced( $params ) {	
+		$data = WPStrava::get_instance()->api->get( 'rides', explode( "\n", $params ), 1 ); //version 1
+
+		if ( is_wp_error( $data ) )
+			return $data;
+		
+		if ( isset( $data->rides ) )
+			return $data->rides;
+		
+		return array();	
+	}
 	    
     public function getRideMap($rideId, $token, $efforts, $threshold) {
     	if($rideId != 0 AND $token != "") {
diff --git a/lib/SOM.class.php b/lib/SOM.class.php
index e114a33..7340d76 100644
--- a/lib/SOM.class.php
+++ b/lib/SOM.class.php
@@ -22,4 +22,11 @@ abstract class WPStrava_SOM {
 	abstract public function elevation( $m );
 	abstract public function get_elevation_label();
 
+	public function time( $seconds ) {
+		return date( 'H:i:s', mktime( 0, 0, $seconds ) );
+	}
+
+	public function get_time_label() {
+		return __( 'hours', 'wp-strava' );
+	}
 }
\ No newline at end of file
diff --git a/lib/Strava.class.php b/lib/Strava.class.php
index a24aa1a..2f97d8d 100644
--- a/lib/Strava.class.php
+++ b/lib/Strava.class.php
@@ -10,6 +10,7 @@ class WPStrava {
 	private static $instance = NULL;
 	private $settings = NULL;
 	private $api = NULL;
+	private $rides = NULL;
 	
 	private function __construct() {
 		$this->settings = new WPStrava_Settings();
@@ -19,8 +20,8 @@ class WPStrava {
 		}
 
 		// Register StravaLatestRidesWidget widget
-		add_action( 'widgets_init', function() {	return register_widget( 'WPStrava_LatestRidesWidget' ); } );
-		add_action( 'widgets_init', function() {	return register_widget( 'WPStrava_LatestMapWidget' ); } );
+		add_action( 'widgets_init', function() { return register_widget( 'WPStrava_LatestRidesWidget' ); } );
+		add_action( 'widgets_init', function() { return register_widget( 'WPStrava_LatestMapWidget' ); } );
 		
 	}
 
@@ -34,9 +35,13 @@ class WPStrava {
 		if ( isset( $this->{$name} ) )
 			return $this->{$name};
 
+		//on-demand classes
 		if ( $name == 'api' )
 			return $this->get_api();
 
+		if ( $name == 'rides' )
+			return $this->get_rides();
+
 		return NULL;
 	}
 
@@ -48,4 +53,13 @@ class WPStrava {
 
 		return $this->api;
 	}
+	
+	public function get_rides() {
+		if ( ! $this->rides ) {
+			require_once WPSTRAVA_PLUGIN_DIR . 'lib/Rides.class.php';
+			$this->rides = new WPStrava_Rides();
+		}
+
+		return $this->rides;
+	}
 }
\ No newline at end of file
diff --git a/readme.txt b/readme.txt
index f9381f4..c2c68d8 100755
--- a/readme.txt
+++ b/readme.txt
@@ -16,6 +16,7 @@ This plugin uses the REST strava.com API to pull the data out and show the infor
 == Changelog ==
 
 = 0.70 =
+Use WordPress HTTP API for all remote calls
 Use WordPress Settings API for settings page
 
 = 0.62 =
diff --git a/wp-strava.php b/wp-strava.php
index e6e3eb4..045ab76 100755
--- a/wp-strava.php
+++ b/wp-strava.php
@@ -3,7 +3,7 @@
 Plugin Name: WP Strava
 Plugin URI: http://cmanon.com
 Description: Plugin to show your strava.com information in your wordpress blog. Some Icons are Copyright © Yusuke Kamiyamane. All rights reserved. Licensed under a Creative Commons Attribution 3.0 license.  
-Version: 0.63
+Version: 0.70
 Author: Carlos Santa Cruz (cmanon), Justin Foell <justin@foell.org>
 Author URI: http://cmanon.com
 License: GPL2

From a1a213f1b73265544aa893fd7982f4ebc8bdf7b6 Mon Sep 17 00:00:00 2001
From: Justin Foell <justin@9seeds.com>
Date: Sun, 7 Apr 2013 18:09:50 -0500
Subject: [PATCH 7/7] Moved appropriate methods from Map Widget to Rides

---
 lib/LatestMapWidget.class.php | 43 ++++++-----------------------------
 lib/Rides.class.php           | 24 ++++++++++++++++++-
 2 files changed, 30 insertions(+), 37 deletions(-)

diff --git a/lib/LatestMapWidget.class.php b/lib/LatestMapWidget.class.php
index 8cd6717..a77ab73 100644
--- a/lib/LatestMapWidget.class.php
+++ b/lib/LatestMapWidget.class.php
@@ -66,13 +66,15 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 			$ride = $ride_transient;
 		
 		if ( ! $ride ) {
-			die('<pre>'.print_r($ride_index_params, true));
-			$rides = WPStrava::get_instance()->rides->getRidesAdvanced( $ride_index_params );
-			//$rides = $this->getRides( $ride_index_params );
+			$strava_rides = WPStrava::get_instance()->rides;
+			$ride_index_params = implode( '&', explode( "\n", $ride_index_params ) );
+			parse_str( $ride_index_params, $params );
+			$rides = $strava_rides->getRidesAdvanced( $params );
+
 			if ( ! empty( $rides ) ) {
 			
 				if ( ! empty( $distance_min ) )
-					$rides = $this->getRidesLongerThan( $rides, $distance_min );
+					$rides = $strava_rides->getRidesLongerThan( $rides, $distance_min );
 
 				$ride = current( $rides );
 
@@ -126,11 +128,10 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 	}
 
 	private function getStaticImage( $ride_id, $build_new ) {
-
 		$img = get_option( 'strava_latest_map' );
 		
 		if ( $build_new || ! $img ) {
-			$map_details = $this->getMapDetails( $ride_id );
+			$map_details = WPStrava::get_instance()->rides->getMapDetails( $ride_id );
 			$img = $this->buildImage( $map_details );
 			update_option( 'strava_latest_map', $img );
 		}
@@ -138,34 +139,4 @@ class WPStrava_LatestMapWidget extends WP_Widget {
 		return $img;
 	}
 	
-	private function getRides( $params ) {
-		$data = WPStrava::get_instance()->api->get( 'rides?' . implode( '&', explode( "\n", $params ) ), 1 ); //version 1
-		
-		if ( isset( $data->rides ) )
-			return $data->rides;
-		return array();
-	}
-
-	private function getRideInfo( $ride_id ) {
-		return WPStrava::get_instance()->api->get( "rides/{$ride_id}" );
-	}
-
-	private function getRidesLongerThan( $rides, $dist ) {
-		$meters = $this->som->distance_inverse( $dist );
-
-		$long_rides = array();
-		foreach ( $rides as $ride ) {
-			$ride_info = $this->getRideInfo( $ride->id );
-			if ( $ride_info->ride->distance > $meters ) {
-				$long_rides[] = $ride_info;
-			}
-		}
-		
-		return $long_rides;
-	}
-		
-	private function getMapDetails( $ride_id ) {
-		$token = WPStrava::get_instance()->settings->token;
-		return WPStrava::get_instance()->api->get( "rides/{$ride_id}/map_details", array( 'token' => $token ) );
-	}
 }
\ No newline at end of file
diff --git a/lib/Rides.class.php b/lib/Rides.class.php
index c8c7567..d3e145b 100755
--- a/lib/Rides.class.php
+++ b/lib/Rides.class.php
@@ -52,7 +52,7 @@ class WPStrava_Rides {
 	} // getRidesSimple
 
 	public function getRidesAdvanced( $params ) {	
-		$data = WPStrava::get_instance()->api->get( 'rides', explode( "\n", $params ), 1 ); //version 1
+		$data = WPStrava::get_instance()->api->get( 'rides', $params, 1 ); //version 1
 
 		if ( is_wp_error( $data ) )
 			return $data;
@@ -81,5 +81,27 @@ class WPStrava_Rides {
 			return false;
     	}
     } // getRideDetails
+
+	public function getRidesLongerThan( $rides, $dist ) {
+		$som = WPStrava_SOM::get_som();		
+		$meters = $som->distance_inverse( $dist );
+
+		$long_rides = array();
+		foreach ( $rides as $ride ) {
+			$ride_info = $this->getRideDetails( $ride->id );
+			if ( $ride_info->ride->distance > $meters ) {
+				$long_rides[] = $ride_info;
+			}
+		}
+		
+		return $long_rides;
+	}
+		
+	public function getMapDetails( $ride_id ) {
+		$token = WPStrava::get_instance()->settings->token;
+		return WPStrava::get_instance()->api->get( "rides/{$ride_id}/map_details", array( 'token' => $token ) );
+	}
+	
+	
 } // class Rides
 ?>