rudiedirkx / fuelly

lib/fuelly/src/Client.php

Doc Comments +1 added lines

@@ -328,6 +328,7 @@
 
 	/**
 	 * HTTP POST
+	 * @param string $uri
 	 */
 	public function _post( $uri, $options = array() ) {
 		return $this->_http($uri, $options + array('method' => 'POST'));

Spacing +40 added lines, -40 removed lines

@@ -25,23 +25,23 @@ 
 	/**
 	 * Dependency constructor
 	 */
-	public function __construct( WebAuth $auth ) {
+	public function __construct(WebAuth $auth) {
 		$this->auth = $auth;
 	}
 
 	/**
 	 *
 	 */
-	public function getFuelUp( $id ) {
+	public function getFuelUp($id) {
 		$fuelup = compact('id');
 
 		$response = $this->_get('fuelups/' . $id . '/edit');
 
 		preg_match_all('#<input[\s\S]+?name="([^"]+)"[\s\S]+?>#', $response->body, $matches, PREG_SET_ORDER);
 		foreach ($matches as $match) {
-			if ( in_array($match[1], array('_token', 'miles_last_fuelup', 'price_per_unit', 'amount', 'fuelup_date')) ) {
+			if (in_array($match[1], array('_token', 'miles_last_fuelup', 'price_per_unit', 'amount', 'fuelup_date'))) {
 				preg_match('#value="([^"]+)"#', $match[0], $match2);
-				$fuelup[ $match[1] ] = $match2[1];
+				$fuelup[$match[1]] = $match2[1];
 			}
 		}
 
@@ -54,11 +54,11 @@ 
 	/**
 	 *
 	 */
-	public function updateFuelUp( $id, $data ) {
+	public function updateFuelUp($id, $data) {
 		$data = $this->_validateFuelUpData($data);
 		unset($data['id']);
 
-		if ( !isset($data['_token']) ) {
+		if (!isset($data['_token'])) {
 			$response = $this->_get('fuelups/' . $id . '/edit');
 			$token = $this->extractFormToken($response->body);
 		}
@@ -73,7 +73,7 @@ 
 	/**
 	 *
 	 */
-	public function _validateFuelUpData( $data ) {
+	public function _validateFuelUpData($data) {
 		$data += array(
 			'errorlevel' => 2,
 			'price_per_unit' => '',
@@ -90,7 +90,7 @@ 
 
 		$required = array('usercar_id', 'miles_last_fuelup', 'amount');
 		$missing = array_diff($required, array_keys(array_filter($data)));
-		if ( $missing ) {
+		if ($missing) {
 			throw new InvalidArgumentException('Missing params: ' . implode(', ', $missing));
 		}
 
@@ -100,7 +100,7 @@ 
 	/**
 	 *
 	 */
-	public function getFuelUpsWithIds( Vehicle $vehicle, $limit = 15 ) {
+	public function getFuelUpsWithIds(Vehicle $vehicle, $limit = 15) {
 		$query = http_build_query(array(
 			'iDisplayStart' => 0,
 			'iDisplayLength' => $limit,
@@ -108,11 +108,11 @@ 
 			'usercar_id' => $vehicle->id,
 		));
 		$response = $this->_get('ajax/fuelup-log?' . $query);
-		if ( $response->code == 200 ) {
-			if ( $response->response ) {
+		if ($response->code == 200) {
+			if ($response->response) {
 				$fuelups = array();
-				foreach ( $response->response['aaData'] as $fuelup ) {
-					if ( preg_match('#fuelups/(\d+)/edit#', $fuelup[0], $match) ) {
+				foreach ($response->response['aaData'] as $fuelup) {
+					if (preg_match('#fuelups/(\d+)/edit#', $fuelup[0], $match)) {
 						$fuelup = array(
 							'id' => $match[1],
 							'usercar_id' => $vehicle->id,
@@ -121,7 +121,7 @@ 
 							'amount' => $fuelup[4][0],
 						);
 
-						$fuelups[ $fuelup['id'] ] = new FuelUp($vehicle, $fuelup);
+						$fuelups[$fuelup['id']] = new FuelUp($vehicle, $fuelup);
 					}
 				}
 
@@ -138,21 +138,21 @@ 
 	/**
 	 *
 	 */
-	public function getAllFuelups( Vehicle $vehicle ) {
+	public function getAllFuelups(Vehicle $vehicle) {
 		$response = $this->_get('car/make/model/2001/username/' . $vehicle->id . '/export');
-		if ( $token = $this->extractFormToken($response->body) ) {
+		if ($token = $this->extractFormToken($response->body)) {
 			$response = $this->_post('exportfuelups', array(
 				'data' => array(
 					'_token' => $token,
 					'usercar_id' => $vehicle->id,
 				),
 			));
-			if ( $response->code == 200 ) {
+			if ($response->code == 200) {
 				$lines = array_filter(preg_split('#[\r\n]+#', $response->body));
 				$header = $rows = array();
 				foreach ($lines as $line) {
 					$row = array_map('trim', str_getcsv($line));
-					if ( !$header ) {
+					if (!$header) {
 						$header = $row;
 					}
 					else {
@@ -168,26 +168,26 @@ 
 	/**
 	 *
 	 */
-	public function addFuelUp( $data ) {
+	public function addFuelUp($data) {
 		$data = $this->_validateFuelUpData($data);
 
 		// GET /fuelups/create
 		$response = $this->_get('fuelups/create');
 
-		if ( $token = $this->extractFormToken($response->body) ) {
+		if ($token = $this->extractFormToken($response->body)) {
 			$data['_token'] = $token;
 
 			// POST /fuelups/create
 			$response = $this->_post('fuelups', array(
 				'data' => $data,
 			));
-			if ( $response->code == 302 ) {
+			if ($response->code == 302) {
 				$response = $this->_get($response->headers['location'][0]);
 
 				// Take new fuelup ID from response and add it
-				if ( $response->code == 200 ) {
+				if ($response->code == 200) {
 					$regex = '#' . preg_quote($this->base, '#') . 'fuelups/(\d+)/edit#';
-					if ( preg_match($regex, $response->body, $match) ) {
+					if (preg_match($regex, $response->body, $match)) {
 						$response->fuelup_id = $match[1];
 					}
 				}
@@ -200,7 +200,7 @@ 
 	/**
 	 *
 	 */
-	public function getVehicle( $id ) {
+	public function getVehicle($id) {
 		$vehicles = $this->getVehicles();
 		return @$vehicles[$id];
 	}
@@ -216,11 +216,11 @@ 
 	/**
 	 *
 	 */
-	protected function extractVehicles( $html ) {
+	protected function extractVehicles($html) {
 		$regex = '#<ul class="dashboard-vehicle" data-clickable="([^"]+)">[\w\W]+?</ul>#';
 		$vehicles = array();
-		if ( preg_match_all($regex, $html, $matches) ) {
-			foreach ( $matches[0] as $i => $html ) {
+		if (preg_match_all($regex, $html, $matches)) {
+			foreach ($matches[0] as $i => $html) {
 				$url = $matches[1][$i];
 
 				preg_match('#/(\d+)$#', $url, $match);
@@ -246,14 +246,14 @@ 
 	 *
 	 */
 	public function logIn() {
-		if ( !$this->auth->mail || !$this->auth->pass ) {
+		if (!$this->auth->mail || !$this->auth->pass) {
 			return false;
 		}
 
 		// GET /login
 		$response = $this->_get('login', array('login' => true));
 
-		if ( $token = $this->extractFormToken($response->body) ) {
+		if ($token = $this->extractFormToken($response->body)) {
 			// POST /login
 			$response = $this->_post('login', array(
 				'login' => true,
@@ -275,14 +275,14 @@ 
 	 *
 	 */
 	public function checkSession() {
-		if ( !$this->auth->session ) {
+		if (!$this->auth->session) {
 			return false;
 		}
 
 		$response = $this->_get('dashboard');
-		if ( $response->code == 200 ) {
+		if ($response->code == 200) {
 			$regex = '#<a href="' . preg_quote($this->base, '#') . 'driver/([\w\d]+)/edit">Settings</a>#';
-			if ( preg_match($regex, $response->body, $match) ) {
+			if (preg_match($regex, $response->body, $match)) {
 				$this->username = $match[1];
 
 				// Since we're downloading /dashboard anyway, let's extract our vehicles from it
@@ -299,7 +299,7 @@ 
 	 *
 	 */
 	public function refreshSession() {
-		if ( !$this->checkSession() ) {
+		if (!$this->checkSession()) {
 			return $this->logIn();
 		}
 
@@ -309,9 +309,9 @@ 
 	/**
 	 *
 	 */
-	protected function extractFormToken( $html ) {
-		if ( preg_match('#<input.+?name="_token".+?>#i', $html, $match) ) {
-			if ( preg_match('#value="([^"]+)"#', $match[0], $match) ) {
+	protected function extractFormToken($html) {
+		if (preg_match('#<input.+?name="_token".+?>#i', $html, $match)) {
+			if (preg_match('#value="([^"]+)"#', $match[0], $match)) {
 				return $match[1];
 			}
 		}
@@ -322,21 +322,21 @@ 
 	/**
 	 * HTTP GET
 	 */
-	public function _get( $uri, $options = array() ) {
+	public function _get($uri, $options = array()) {
 		return $this->_http($uri, $options + array('method' => 'GET'));
 	}
 
 	/**
 	 * HTTP POST
 	 */
-	public function _post( $uri, $options = array() ) {
+	public function _post($uri, $options = array()) {
 		return $this->_http($uri, $options + array('method' => 'POST'));
 	}
 
 	/**
 	 * HTTP URL
 	 */
-	public function _url( $uri, $options = array() ) {
+	public function _url($uri, $options = array()) {
 		$base = !empty($options['login']) ? $this->loginBase : $this->base;
 		$url = strpos($uri, '://') ? $uri : $base . $uri;
 		return $url;
@@ -345,7 +345,7 @@ 
 	/**
 	 * HTTP REQUEST
 	 */
-	public function _http( $uri, $options = array() ) {
+	public function _http($uri, $options = array()) {
 		if ($this->auth->session) {
 			$options['cookies'][] = array('fuelly_session', $this->auth->session);
 		}

Braces +1 added lines, -2 removed lines

@@ -154,8 +154,7 @@
 					$row = array_map('trim', str_getcsv($line));
 					if ( !$header ) {
 						$header = $row;
-					}
-					else {
+					} else {
 						$rows[] = array_combine($header, $row);
 					}
 				}

test.edit.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 echo "Vehicle:\n";
 $vehicles = $client->vehicles;
-$vehicle = $vehicles[ array_rand($vehicles) ];
+$vehicle = $vehicles[array_rand($vehicles)];
 print_r($vehicle);
 echo "\n\n";
 

test.fuelup.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 
 echo "Vehicle:\n";
 $vehicles = $client->vehicles;
-$vehicle = $vehicles[ array_rand($vehicles) ];
+$vehicle = $vehicles[array_rand($vehicles)];
 print_r($vehicle);
 echo "\n\n";
 
@@ -20,7 +20,7 @@ 
 	'amount' => 10,
 	'note' => 'TEST TEST TEST',
 ));
-if ( !empty($response->fuelup_id) ) {
+if (!empty($response->fuelup_id)) {
 	var_dump($response->fuelup_id);
 }
 else {

Braces +1 added lines, -2 removed lines

@@ -22,8 +22,7 @@
 ));
 if ( !empty($response->fuelup_id) ) {
 	var_dump($response->fuelup_id);
-}
-else {
+} else {
 	print_r($response);
 }
 echo "\n\n";

test.fuelups.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 echo "Vehicle:\n";
 $vehicles = $client->vehicles;
-$vehicle = $vehicles[ array_rand($vehicles) ];
+$vehicle = $vehicles[array_rand($vehicles)];
 print_r($vehicle);
 echo "\n\n";
 

lib/fuelly/autoload.php

Spacing +3 added lines, -3 removed lines

@@ -1,10 +1,10 @@
 <?php
 
-spl_autoload_register(function( $class ) {
+spl_autoload_register(function($class) {
 	$ns = explode('\\', $class);
 	$lib = array_splice($ns, 0, 2);
-	if ( $lib == array('rdx', 'fuelly') ) {
-		if ( file_exists($file = __DIR__ . '/src/' . implode('/', $ns) . '.php') ) {
+	if ($lib == array('rdx', 'fuelly')) {
+		if (file_exists($file = __DIR__ . '/src/' . implode('/', $ns) . '.php')) {
 			include $file;
 		}
 	}

lib/fuelly/src/FuelUp.php

Spacing +4 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 	/**
 	 *
 	 */
-	public function __construct( Vehicle $vehicle, array $fuelup ) {
+	public function __construct(Vehicle $vehicle, array $fuelup) {
 		// $this->vehicle = $vehicle;
 
 		// @todo Parse date correctly
@@ -23,8 +23,7 @@ 
 		// @todo Parse distance correctly
 
 		$this->date =
-			DateTime::createFromFormat('Y-m-d H:i:s', $fuelup['fuelup_date']) ?:
-			DateTime::createFromFormat('d-m-y', $fuelup['fuelup_date']);
+			DateTime::createFromFormat('Y-m-d H:i:s', $fuelup['fuelup_date']) ?: DateTime::createFromFormat('d-m-y', $fuelup['fuelup_date']);
 
 		$this->amount = self::number(@$fuelup['raw_amount'] ?: $fuelup['amount']);
 		$this->distance = self::number($fuelup['miles_last_fuelup']);
@@ -37,14 +36,14 @@ 
 	/**
 	 *
 	 */
-	public static function number( $str ) {
+	public static function number($str) {
 		return (float) str_replace(',', '.', $str);
 	}
 
 	/**
 	 *
 	 */
-	public static function dateCmp( FuelUp $a, FuelUp $b ) {
+	public static function dateCmp(FuelUp $a, FuelUp $b) {
 		return $b->date->getTimestamp() - $a->date->getTimestamp();
 	}
 

lib/fuelly/src/Vehicle.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 	/**
 	 *
 	 */
-	public function __construct( Client $client, array $vehicle ) {
+	public function __construct(Client $client, array $vehicle) {
 		// $this->client = $client;
 
 		$this->id = $vehicle['id'];
@@ -26,8 +26,8 @@ 
 		$this->name = $vehicle['name'];
 		$this->image = $vehicle['image'];
 
-		if ( isset($vehicle['trend']) ) {
-			foreach ( $vehicle['trend'] as $fuelup) {
+		if (isset($vehicle['trend'])) {
+			foreach ($vehicle['trend'] as $fuelup) {
 				$this->trend[] = new FuelUp($this, $fuelup);
 			}
 

lib/fuelly/src/WebAuth.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 	/**
 	 * Dependency constructor
 	 */
-	public function __construct( $mail, $pass, $session = '' ) {
+	public function __construct($mail, $pass, $session = '') {
 		$this->mail = $mail;
 		$this->pass = $pass;
 		$this->session = $session;