las93 / venus3

bundles/core/UrlManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	 * @access public
 	 * @param  string $sCode code of the url between "routes" and "route" in Route.conf
 	 * @param  array $aParams parameters to create the url
-	 * @return void
+	 * @return string
 	 *
 	 * @tutorial	If I have this route I could make my URL:
 	 *

Spacing +3 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
 
 						$aRoutes = array();
 
-						foreach($oHost->routes as $sKey => $oRoute) {
+						foreach ($oHost->routes as $sKey => $oRoute) {
 
 							if ($sKey === $sCode) {
 
@@ -135,8 +135,8 @@ 
 			throw new \Exception();
 		}
 		
-		$sStringToEncode = str_replace(['à','á','â','ã','ä','ç','è','é','ê','ë','ì','í','î','ï','ñ','ò','ó','ô','õ','ö','ù','ú','û','ü','ý','ÿ','À','Á','Â','Ã','Ä','Ç','È','É','Ê','Ë','Ì','Í','Î','Ï','Ñ','Ò','Ó','Ô','Õ','Ö','Ù','Ú','Û','Ü','Ý'],
-			['a','a','a','a','a','c','e','e','e','e','i','i','i','i','n','o','o','o','o','o','u','u','u','u','y','y','A','A','A','A','A','C','E','E','E','E','I','I','I','I','N','O','O','O','O','O','U','U','U','U','Y'],
+		$sStringToEncode = str_replace(['à', 'á', 'â', 'ã', 'ä', 'ç', 'è', 'é', 'ê', 'ë', 'ì', 'í', 'î', 'ï', 'ñ', 'ò', 'ó', 'ô', 'õ', 'ö', 'ù', 'ú', 'û', 'ü', 'ý', 'ÿ', 'À', 'Á', 'Â', 'Ã', 'Ä', 'Ç', 'È', 'É', 'Ê', 'Ë', 'Ì', 'Í', 'Î', 'Ï', 'Ñ', 'Ò', 'Ó', 'Ô', 'Õ', 'Ö', 'Ù', 'Ú', 'Û', 'Ü', 'Ý'],
+			['a', 'a', 'a', 'a', 'a', 'c', 'e', 'e', 'e', 'e', 'i', 'i', 'i', 'i', 'n', 'o', 'o', 'o', 'o', 'o', 'u', 'u', 'u', 'u', 'y', 'y', 'A', 'A', 'A', 'A', 'A', 'C', 'E', 'E', 'E', 'E', 'I', 'I', 'I', 'I', 'N', 'O', 'O', 'O', 'O', 'O', 'U', 'U', 'U', 'U', 'Y'],
 				$sStringToEncode);
 
 		$sStringToEncode = preg_replace('/[^a-zA-Z0-9_]+/', '_', preg_quote($sStringToEncode));

Braces +2 added lines, -4 removed lines

@@ -96,12 +96,10 @@
 
 										if (preg_match('#'.$sType.'#', $aParams[$sName])) {
 
-											if ($aParams[$sName]) { $sRoute = str_replace('[/:'.$sName.']', '/'.$aParams[$sName], $sRoute); }
-											else { $sRoute = str_replace('[/:'.$sName.']', '', $sRoute); }
+											if ($aParams[$sName]) { $sRoute = str_replace('[/:'.$sName.']', '/'.$aParams[$sName], $sRoute); } else { $sRoute = str_replace('[/:'.$sName.']', '', $sRoute); }
 											$sRoute = str_replace('[:'.$sName.']', $aParams[$sName], $sRoute);
 											continue;
-										}
-										else if (isset($oRoute->defaults_constraints) 
+										} else if (isset($oRoute->defaults_constraints) 
 											&& isset($oRoute->defaults_constraints->{$sName}) 
 											&& preg_match('#'.$sType.'#', $oRoute->defaults_constraints->{$sName})) {
 

bundles/ext/facebook/base_facebook.php

Doc Comments +6 added lines, -2 removed lines

@@ -503,7 +503,7 @@ 
    * Get the UID of the connected user, or 0
    * if the Facebook user is not connected.
    *
-   * @return string the UID if available.
+   * @return integer the UID if available.
    */
   public function getUser() {
     if ($this->user !== null) {
@@ -757,6 +757,7 @@ 
    * either logged in to Facebook or has granted an offline access permission.
    *
    * @param string $code An authorization code.
+   * @param string $redirect_uri
    * @return mixed An access token exchanged for the authorization code, or
    *               false if an access token could not be generated.
    */
@@ -1135,7 +1136,7 @@ 
   /**
    * Build the URL for given domain alias, path and parameters.
    *
-   * @param $name string The name of the domain
+   * @param string $name string The name of the domain
    * @param $path string Optional path (without a leading slash)
    * @param $params array Optional query parameters
    *
@@ -1402,6 +1403,9 @@ 
     return self::endsWith($big, '.'.$small);
   }
 
+  /**
+   * @param string $small
+   */
   protected static function endsWith($big, $small) {
     $len = strlen($small);
     if ($len === 0) {

Indentation +739 added lines, -739 removed lines

@@ -40,24 +40,24 @@ 
    * @param array $result The result from the API server
    */
   public function __construct($result) {
-    $this->result = $result;
-
-    $code = isset($result['error_code']) ? $result['error_code'] : 0;
-
-    if (isset($result['error_description'])) {
-      // OAuth 2.0 Draft 10 style
-      $msg = $result['error_description'];
-    } else if (isset($result['error']) && is_array($result['error'])) {
-      // OAuth 2.0 Draft 00 style
-      $msg = $result['error']['message'];
-    } else if (isset($result['error_msg'])) {
-      // Rest server style
-      $msg = $result['error_msg'];
-    } else {
-      $msg = 'Unknown Error. Check getResult()';
-    }
-
-    parent::__construct($msg, $code);
+	$this->result = $result;
+
+	$code = isset($result['error_code']) ? $result['error_code'] : 0;
+
+	if (isset($result['error_description'])) {
+	  // OAuth 2.0 Draft 10 style
+	  $msg = $result['error_description'];
+	} else if (isset($result['error']) && is_array($result['error'])) {
+	  // OAuth 2.0 Draft 00 style
+	  $msg = $result['error']['message'];
+	} else if (isset($result['error_msg'])) {
+	  // Rest server style
+	  $msg = $result['error_msg'];
+	} else {
+	  $msg = 'Unknown Error. Check getResult()';
+	}
+
+	parent::__construct($msg, $code);
   }
 
   /**
@@ -66,7 +66,7 @@ 
    * @return array The result from the API server
    */
   public function getResult() {
-    return $this->result;
+	return $this->result;
   }
 
   /**
@@ -76,20 +76,20 @@ 
    * @return string
    */
   public function getType() {
-    if (isset($this->result['error'])) {
-      $error = $this->result['error'];
-      if (is_string($error)) {
-        // OAuth 2.0 Draft 10 style
-        return $error;
-      } else if (is_array($error)) {
-        // OAuth 2.0 Draft 00 style
-        if (isset($error['type'])) {
-          return $error['type'];
-        }
-      }
-    }
-
-    return 'Exception';
+	if (isset($this->result['error'])) {
+	  $error = $this->result['error'];
+	  if (is_string($error)) {
+		// OAuth 2.0 Draft 10 style
+		return $error;
+	  } else if (is_array($error)) {
+		// OAuth 2.0 Draft 00 style
+		if (isset($error['type'])) {
+		  return $error['type'];
+		}
+	  }
+	}
+
+	return 'Exception';
   }
 
   /**
@@ -98,11 +98,11 @@ 
    * @return string The string representation of the error
    */
   public function __toString() {
-    $str = $this->getType() . ': ';
-    if ($this->code != 0) {
-      $str .= $this->code . ': ';
-    }
-    return $str . $this->message;
+	$str = $this->getType() . ': ';
+	if ($this->code != 0) {
+	  $str .= $this->code . ': ';
+	}
+	return $str . $this->message;
   }
 }
 
@@ -131,10 +131,10 @@ 
    * Default options for curl.
    */
   public static $CURL_OPTS = array(
-    CURLOPT_CONNECTTIMEOUT => 10,
-    CURLOPT_RETURNTRANSFER => true,
-    CURLOPT_TIMEOUT        => 60,
-    CURLOPT_USERAGENT      => 'facebook-php-3.2',
+	CURLOPT_CONNECTTIMEOUT => 10,
+	CURLOPT_RETURNTRANSFER => true,
+	CURLOPT_TIMEOUT        => 60,
+	CURLOPT_USERAGENT      => 'facebook-php-3.2',
   );
 
   /**
@@ -142,21 +142,21 @@ 
    * the current URL.
    */
   protected static $DROP_QUERY_PARAMS = array(
-    'code',
-    'state',
-    'signed_request',
+	'code',
+	'state',
+	'signed_request',
   );
 
   /**
    * Maps aliases to Facebook domains.
    */
   public static $DOMAIN_MAP = array(
-    'api'         => 'https://api.facebook.com/',
-    'api_video'   => 'https://api-video.facebook.com/',
-    'api_read'    => 'https://api-read.facebook.com/',
-    'graph'       => 'https://graph.facebook.com/',
-    'graph_video' => 'https://graph-video.facebook.com/',
-    'www'         => 'https://www.facebook.com/',
+	'api'         => 'https://api.facebook.com/',
+	'api_video'   => 'https://api-video.facebook.com/',
+	'api_read'    => 'https://api-read.facebook.com/',
+	'graph'       => 'https://graph.facebook.com/',
+	'graph_video' => 'https://graph-video.facebook.com/',
+	'www'         => 'https://www.facebook.com/',
   );
 
   /**
@@ -223,18 +223,18 @@ 
    * @param array $config The application configuration
    */
   public function __construct($config) {
-    $this->setAppId($config['appId']);
-    $this->setAppSecret($config['secret']);
-    if (isset($config['fileUpload'])) {
-      $this->setFileUploadSupport($config['fileUpload']);
-    }
-    if (isset($config['trustForwarded']) && $config['trustForwarded']) {
-      $this->trustForwarded = true;
-    }
-    $state = $this->getPersistentData('state');
-    if (!empty($state)) {
-      $this->state = $state;
-    }
+	$this->setAppId($config['appId']);
+	$this->setAppSecret($config['secret']);
+	if (isset($config['fileUpload'])) {
+	  $this->setFileUploadSupport($config['fileUpload']);
+	}
+	if (isset($config['trustForwarded']) && $config['trustForwarded']) {
+	  $this->trustForwarded = true;
+	}
+	$state = $this->getPersistentData('state');
+	if (!empty($state)) {
+	  $this->state = $state;
+	}
   }
 
   /**
@@ -244,8 +244,8 @@ 
    * @return BaseFacebook
    */
   public function setAppId($appId) {
-    $this->appId = $appId;
-    return $this;
+	$this->appId = $appId;
+	return $this;
   }
 
   /**
@@ -254,7 +254,7 @@ 
    * @return string the Application ID
    */
   public function getAppId() {
-    return $this->appId;
+	return $this->appId;
   }
 
   /**
@@ -265,8 +265,8 @@ 
    * @deprecated
    */
   public function setApiSecret($apiSecret) {
-    $this->setAppSecret($apiSecret);
-    return $this;
+	$this->setAppSecret($apiSecret);
+	return $this;
   }
 
   /**
@@ -276,8 +276,8 @@ 
    * @return BaseFacebook
    */
   public function setAppSecret($appSecret) {
-    $this->appSecret = $appSecret;
-    return $this;
+	$this->appSecret = $appSecret;
+	return $this;
   }
 
   /**
@@ -287,7 +287,7 @@ 
    * @deprecated
    */
   public function getApiSecret() {
-    return $this->getAppSecret();
+	return $this->getAppSecret();
   }
 
   /**
@@ -296,7 +296,7 @@ 
    * @return string the App Secret
    */
   public function getAppSecret() {
-    return $this->appSecret;
+	return $this->appSecret;
   }
 
   /**
@@ -306,8 +306,8 @@ 
    * @return BaseFacebook
    */
   public function setFileUploadSupport($fileUploadSupport) {
-    $this->fileUploadSupport = $fileUploadSupport;
-    return $this;
+	$this->fileUploadSupport = $fileUploadSupport;
+	return $this;
   }
 
   /**
@@ -316,7 +316,7 @@ 
    * @return boolean true if and only if the server supports file upload.
    */
   public function getFileUploadSupport() {
-    return $this->fileUploadSupport;
+	return $this->fileUploadSupport;
   }
 
   /**
@@ -327,7 +327,7 @@ 
    * @return boolean true if and only if the server supports file upload.
    */
   public function useFileUploadSupport() {
-    return $this->getFileUploadSupport();
+	return $this->getFileUploadSupport();
   }
 
   /**
@@ -339,8 +339,8 @@ 
    * @return BaseFacebook
    */
   public function setAccessToken($access_token) {
-    $this->accessToken = $access_token;
-    return $this;
+	$this->accessToken = $access_token;
+	return $this;
   }
 
   /**
@@ -349,41 +349,41 @@ 
    * for the workaround.
    */
   public function setExtendedAccessToken() {
-    try {
-      // need to circumvent json_decode by calling _oauthRequest
-      // directly, since response isn't JSON format.
-      $access_token_response = $this->_oauthRequest(
-        $this->getUrl('graph', '/oauth/access_token'),
-        $params = array(
-          'client_id' => $this->getAppId(),
-          'client_secret' => $this->getAppSecret(),
-          'grant_type' => 'fb_exchange_token',
-          'fb_exchange_token' => $this->getAccessToken(),
-        )
-      );
-    }
-    catch (FacebookApiException $e) {
-      // most likely that user very recently revoked authorization.
-      // In any event, we don't have an access token, so say so.
-      return false;
-    }
-
-    if (empty($access_token_response)) {
-      return false;
-    }
-
-    $response_params = array();
-    parse_str($access_token_response, $response_params);
-
-    if (!isset($response_params['access_token'])) {
-      return false;
-    }
-
-    $this->destroySession();
-
-    $this->setPersistentData(
-      'access_token', $response_params['access_token']
-    );
+	try {
+	  // need to circumvent json_decode by calling _oauthRequest
+	  // directly, since response isn't JSON format.
+	  $access_token_response = $this->_oauthRequest(
+		$this->getUrl('graph', '/oauth/access_token'),
+		$params = array(
+		  'client_id' => $this->getAppId(),
+		  'client_secret' => $this->getAppSecret(),
+		  'grant_type' => 'fb_exchange_token',
+		  'fb_exchange_token' => $this->getAccessToken(),
+		)
+	  );
+	}
+	catch (FacebookApiException $e) {
+	  // most likely that user very recently revoked authorization.
+	  // In any event, we don't have an access token, so say so.
+	  return false;
+	}
+
+	if (empty($access_token_response)) {
+	  return false;
+	}
+
+	$response_params = array();
+	parse_str($access_token_response, $response_params);
+
+	if (!isset($response_params['access_token'])) {
+	  return false;
+	}
+
+	$this->destroySession();
+
+	$this->setPersistentData(
+	  'access_token', $response_params['access_token']
+	);
   }
 
   /**
@@ -396,21 +396,21 @@ 
    * @return string The access token
    */
   public function getAccessToken() {
-    if ($this->accessToken !== null) {
-      // we've done this already and cached it.  Just return.
-      return $this->accessToken;
-    }
-
-    // first establish access token to be the application
-    // access token, in case we navigate to the /oauth/access_token
-    // endpoint, where SOME access token is required.
-    $this->setAccessToken($this->getApplicationAccessToken());
-    $user_access_token = $this->getUserAccessToken();
-    if ($user_access_token) {
-      $this->setAccessToken($user_access_token);
-    }
-
-    return $this->accessToken;
+	if ($this->accessToken !== null) {
+	  // we've done this already and cached it.  Just return.
+	  return $this->accessToken;
+	}
+
+	// first establish access token to be the application
+	// access token, in case we navigate to the /oauth/access_token
+	// endpoint, where SOME access token is required.
+	$this->setAccessToken($this->getApplicationAccessToken());
+	$user_access_token = $this->getUserAccessToken();
+	if ($user_access_token) {
+	  $this->setAccessToken($user_access_token);
+	}
+
+	return $this->accessToken;
   }
 
   /**
@@ -424,60 +424,60 @@ 
    *                could not be determined.
    */
   protected function getUserAccessToken() {
-    // first, consider a signed request if it's supplied.
-    // if there is a signed request, then it alone determines
-    // the access token.
-    $signed_request = $this->getSignedRequest();
-    if ($signed_request) {
-      // apps.facebook.com hands the access_token in the signed_request
-      if (array_key_exists('oauth_token', $signed_request)) {
-        $access_token = $signed_request['oauth_token'];
-        $this->setPersistentData('access_token', $access_token);
-        return $access_token;
-      }
-
-      // the JS SDK puts a code in with the redirect_uri of ''
-      if (array_key_exists('code', $signed_request)) {
-        $code = $signed_request['code'];
-        if ($code && $code == $this->getPersistentData('code')) {
-          // short-circuit if the code we have is the same as the one presented
-          return $this->getPersistentData('access_token');
-        }
-
-        $access_token = $this->getAccessTokenFromCode($code, '');
-        if ($access_token) {
-          $this->setPersistentData('code', $code);
-          $this->setPersistentData('access_token', $access_token);
-          return $access_token;
-        }
-      }
-
-      // signed request states there's no access token, so anything
-      // stored should be cleared.
-      $this->clearAllPersistentData();
-      return false; // respect the signed request's data, even
-                    // if there's an authorization code or something else
-    }
-
-    $code = $this->getCode();
-    if ($code && $code != $this->getPersistentData('code')) {
-      $access_token = $this->getAccessTokenFromCode($code);
-      if ($access_token) {
-        $this->setPersistentData('code', $code);
-        $this->setPersistentData('access_token', $access_token);
-        return $access_token;
-      }
-
-      // code was bogus, so everything based on it should be invalidated.
-      $this->clearAllPersistentData();
-      return false;
-    }
-
-    // as a fallback, just return whatever is in the persistent
-    // store, knowing nothing explicit (signed request, authorization
-    // code, etc.) was present to shadow it (or we saw a code in $_REQUEST,
-    // but it's the same as what's in the persistent store)
-    return $this->getPersistentData('access_token');
+	// first, consider a signed request if it's supplied.
+	// if there is a signed request, then it alone determines
+	// the access token.
+	$signed_request = $this->getSignedRequest();
+	if ($signed_request) {
+	  // apps.facebook.com hands the access_token in the signed_request
+	  if (array_key_exists('oauth_token', $signed_request)) {
+		$access_token = $signed_request['oauth_token'];
+		$this->setPersistentData('access_token', $access_token);
+		return $access_token;
+	  }
+
+	  // the JS SDK puts a code in with the redirect_uri of ''
+	  if (array_key_exists('code', $signed_request)) {
+		$code = $signed_request['code'];
+		if ($code && $code == $this->getPersistentData('code')) {
+		  // short-circuit if the code we have is the same as the one presented
+		  return $this->getPersistentData('access_token');
+		}
+
+		$access_token = $this->getAccessTokenFromCode($code, '');
+		if ($access_token) {
+		  $this->setPersistentData('code', $code);
+		  $this->setPersistentData('access_token', $access_token);
+		  return $access_token;
+		}
+	  }
+
+	  // signed request states there's no access token, so anything
+	  // stored should be cleared.
+	  $this->clearAllPersistentData();
+	  return false; // respect the signed request's data, even
+					// if there's an authorization code or something else
+	}
+
+	$code = $this->getCode();
+	if ($code && $code != $this->getPersistentData('code')) {
+	  $access_token = $this->getAccessTokenFromCode($code);
+	  if ($access_token) {
+		$this->setPersistentData('code', $code);
+		$this->setPersistentData('access_token', $access_token);
+		return $access_token;
+	  }
+
+	  // code was bogus, so everything based on it should be invalidated.
+	  $this->clearAllPersistentData();
+	  return false;
+	}
+
+	// as a fallback, just return whatever is in the persistent
+	// store, knowing nothing explicit (signed request, authorization
+	// code, etc.) was present to shadow it (or we saw a code in $_REQUEST,
+	// but it's the same as what's in the persistent store)
+	return $this->getPersistentData('access_token');
   }
 
   /**
@@ -487,16 +487,16 @@ 
    * @return string the signed request, if available, or null otherwise.
    */
   public function getSignedRequest() {
-    if (!$this->signedRequest) {
-      if (!empty($_REQUEST['signed_request'])) {
-        $this->signedRequest = $this->parseSignedRequest(
-          $_REQUEST['signed_request']);
-      } else if (!empty($_COOKIE[$this->getSignedRequestCookieName()])) {
-        $this->signedRequest = $this->parseSignedRequest(
-          $_COOKIE[$this->getSignedRequestCookieName()]);
-      }
-    }
-    return $this->signedRequest;
+	if (!$this->signedRequest) {
+	  if (!empty($_REQUEST['signed_request'])) {
+		$this->signedRequest = $this->parseSignedRequest(
+		  $_REQUEST['signed_request']);
+	  } else if (!empty($_COOKIE[$this->getSignedRequestCookieName()])) {
+		$this->signedRequest = $this->parseSignedRequest(
+		  $_COOKIE[$this->getSignedRequestCookieName()]);
+	  }
+	}
+	return $this->signedRequest;
   }
 
   /**
@@ -506,12 +506,12 @@ 
    * @return string the UID if available.
    */
   public function getUser() {
-    if ($this->user !== null) {
-      // we've already determined this and cached the value.
-      return $this->user;
-    }
+	if ($this->user !== null) {
+	  // we've already determined this and cached the value.
+	  return $this->user;
+	}
 
-    return $this->user = $this->getUserFromAvailableData();
+	return $this->user = $this->getUserFromAvailableData();
   }
 
   /**
@@ -523,45 +523,45 @@ 
    *                 or 0 if no such user exists.
    */
   protected function getUserFromAvailableData() {
-    // if a signed request is supplied, then it solely determines
-    // who the user is.
-    $signed_request = $this->getSignedRequest();
-    if ($signed_request) {
-      if (array_key_exists('user_id', $signed_request)) {
-        $user = $signed_request['user_id'];
-
-        if($user != $this->getPersistentData('user_id')){
-          $this->clearAllPersistentData();
-        }
-
-        $this->setPersistentData('user_id', $signed_request['user_id']);
-        return $user;
-      }
-
-      // if the signed request didn't present a user id, then invalidate
-      // all entries in any persistent store.
-      $this->clearAllPersistentData();
-      return 0;
-    }
-
-    $user = $this->getPersistentData('user_id', $default = 0);
-    $persisted_access_token = $this->getPersistentData('access_token');
-
-    // use access_token to fetch user id if we have a user access_token, or if
-    // the cached access token has changed.
-    $access_token = $this->getAccessToken();
-    if ($access_token &&
-        $access_token != $this->getApplicationAccessToken() &&
-        !($user && $persisted_access_token == $access_token)) {
-      $user = $this->getUserFromAccessToken();
-      if ($user) {
-        $this->setPersistentData('user_id', $user);
-      } else {
-        $this->clearAllPersistentData();
-      }
-    }
-
-    return $user;
+	// if a signed request is supplied, then it solely determines
+	// who the user is.
+	$signed_request = $this->getSignedRequest();
+	if ($signed_request) {
+	  if (array_key_exists('user_id', $signed_request)) {
+		$user = $signed_request['user_id'];
+
+		if($user != $this->getPersistentData('user_id')){
+		  $this->clearAllPersistentData();
+		}
+
+		$this->setPersistentData('user_id', $signed_request['user_id']);
+		return $user;
+	  }
+
+	  // if the signed request didn't present a user id, then invalidate
+	  // all entries in any persistent store.
+	  $this->clearAllPersistentData();
+	  return 0;
+	}
+
+	$user = $this->getPersistentData('user_id', $default = 0);
+	$persisted_access_token = $this->getPersistentData('access_token');
+
+	// use access_token to fetch user id if we have a user access_token, or if
+	// the cached access token has changed.
+	$access_token = $this->getAccessToken();
+	if ($access_token &&
+		$access_token != $this->getApplicationAccessToken() &&
+		!($user && $persisted_access_token == $access_token)) {
+	  $user = $this->getUserFromAccessToken();
+	  if ($user) {
+		$this->setPersistentData('user_id', $user);
+	  } else {
+		$this->clearAllPersistentData();
+	  }
+	}
+
+	return $user;
   }
 
   /**
@@ -577,23 +577,23 @@ 
    * @return string The URL for the login flow
    */
   public function getLoginUrl($params=array()) {
-    $this->establishCSRFTokenState();
-    $currentUrl = $this->getCurrentUrl();
-
-    // if 'scope' is passed as an array, convert to comma separated list
-    $scopeParams = isset($params['scope']) ? $params['scope'] : null;
-    if ($scopeParams && is_array($scopeParams)) {
-      $params['scope'] = implode(',', $scopeParams);
-    }
-
-    return $this->getUrl(
-      'www',
-      'dialog/oauth',
-      array_merge(array(
-                    'client_id' => $this->getAppId(),
-                    'redirect_uri' => $currentUrl, // possibly overwritten
-                    'state' => $this->state),
-                  $params));
+	$this->establishCSRFTokenState();
+	$currentUrl = $this->getCurrentUrl();
+
+	// if 'scope' is passed as an array, convert to comma separated list
+	$scopeParams = isset($params['scope']) ? $params['scope'] : null;
+	if ($scopeParams && is_array($scopeParams)) {
+	  $params['scope'] = implode(',', $scopeParams);
+	}
+
+	return $this->getUrl(
+	  'www',
+	  'dialog/oauth',
+	  array_merge(array(
+					'client_id' => $this->getAppId(),
+					'redirect_uri' => $currentUrl, // possibly overwritten
+					'state' => $this->state),
+				  $params));
   }
 
   /**
@@ -606,14 +606,14 @@ 
    * @return string The URL for the logout flow
    */
   public function getLogoutUrl($params=array()) {
-    return $this->getUrl(
-      'www',
-      'logout.php',
-      array_merge(array(
-        'next' => $this->getCurrentUrl(),
-        'access_token' => $this->getUserAccessToken(),
-      ), $params)
-    );
+	return $this->getUrl(
+	  'www',
+	  'logout.php',
+	  array_merge(array(
+		'next' => $this->getCurrentUrl(),
+		'access_token' => $this->getUserAccessToken(),
+	  ), $params)
+	);
   }
 
   /**
@@ -628,17 +628,17 @@ 
    * @return string The URL for the logout flow
    */
   public function getLoginStatusUrl($params=array()) {
-    return $this->getUrl(
-      'www',
-      'extern/login_status.php',
-      array_merge(array(
-        'api_key' => $this->getAppId(),
-        'no_session' => $this->getCurrentUrl(),
-        'no_user' => $this->getCurrentUrl(),
-        'ok_session' => $this->getCurrentUrl(),
-        'session_version' => 3,
-      ), $params)
-    );
+	return $this->getUrl(
+	  'www',
+	  'extern/login_status.php',
+	  array_merge(array(
+		'api_key' => $this->getAppId(),
+		'no_session' => $this->getCurrentUrl(),
+		'no_user' => $this->getCurrentUrl(),
+		'ok_session' => $this->getCurrentUrl(),
+		'session_version' => 3,
+	  ), $params)
+	);
   }
 
   /**
@@ -647,12 +647,12 @@ 
    * @return mixed The decoded response
    */
   public function api(/* polymorphic */) {
-    $args = func_get_args();
-    if (is_array($args[0])) {
-      return $this->_restserver($args[0]);
-    } else {
-      return call_user_func_array(array($this, '_graph'), $args);
-    }
+	$args = func_get_args();
+	if (is_array($args[0])) {
+	  return $this->_restserver($args[0]);
+	} else {
+	  return call_user_func_array(array($this, '_graph'), $args);
+	}
   }
 
   /**
@@ -665,7 +665,7 @@ 
    *         the signed request value.
    */
   protected function getSignedRequestCookieName() {
-    return 'fbsr_'.$this->getAppId();
+	return 'fbsr_'.$this->getAppId();
   }
 
   /**
@@ -676,7 +676,7 @@ 
    * @return string the name of the cookie that would house metadata.
    */
   protected function getMetadataCookieName() {
-    return 'fbm_'.$this->getAppId();
+	return 'fbm_'.$this->getAppId();
   }
 
   /**
@@ -688,22 +688,22 @@ 
    *               code could not be determined.
    */
   protected function getCode() {
-    if (isset($_REQUEST['code'])) {
-      if ($this->state !== null &&
-          isset($_REQUEST['state']) &&
-          $this->state === $_REQUEST['state']) {
-
-        // CSRF state has done its job, so clear it
-        $this->state = null;
-        $this->clearPersistentData('state');
-        return $_REQUEST['code'];
-      } else {
-        self::errorLog('CSRF state token does not match one provided.');
-        return false;
-      }
-    }
-
-    return false;
+	if (isset($_REQUEST['code'])) {
+	  if ($this->state !== null &&
+		  isset($_REQUEST['state']) &&
+		  $this->state === $_REQUEST['state']) {
+
+		// CSRF state has done its job, so clear it
+		$this->state = null;
+		$this->clearPersistentData('state');
+		return $_REQUEST['code'];
+	  } else {
+		self::errorLog('CSRF state token does not match one provided.');
+		return false;
+	  }
+	}
+
+	return false;
   }
 
   /**
@@ -717,12 +717,12 @@ 
    *                 if the Facebook user could not be determined.
    */
   protected function getUserFromAccessToken() {
-    try {
-      $user_info = $this->api('/me');
-      return $user_info['id'];
-    } catch (FacebookApiException $e) {
-      return 0;
-    }
+	try {
+	  $user_info = $this->api('/me');
+	  return $user_info['id'];
+	} catch (FacebookApiException $e) {
+	  return 0;
+	}
   }
 
   /**
@@ -733,7 +733,7 @@ 
    *                public information about users and applications.
    */
   protected function getApplicationAccessToken() {
-    return $this->appId.'|'.$this->appSecret;
+	return $this->appId.'|'.$this->appSecret;
   }
 
   /**
@@ -742,10 +742,10 @@ 
    * @return void
    */
   protected function establishCSRFTokenState() {
-    if ($this->state === null) {
-      $this->state = md5(uniqid(mt_rand(), true));
-      $this->setPersistentData('state', $this->state);
-    }
+	if ($this->state === null) {
+	  $this->state = md5(uniqid(mt_rand(), true));
+	  $this->setPersistentData('state', $this->state);
+	}
   }
 
   /**
@@ -761,41 +761,41 @@ 
    *               false if an access token could not be generated.
    */
   protected function getAccessTokenFromCode($code, $redirect_uri = null) {
-    if (empty($code)) {
-      return false;
-    }
-
-    if ($redirect_uri === null) {
-      $redirect_uri = $this->getCurrentUrl();
-    }
-
-    try {
-      // need to circumvent json_decode by calling _oauthRequest
-      // directly, since response isn't JSON format.
-      $access_token_response =
-        $this->_oauthRequest(
-          $this->getUrl('graph', '/oauth/access_token'),
-          $params = array('client_id' => $this->getAppId(),
-                          'client_secret' => $this->getAppSecret(),
-                          'redirect_uri' => $redirect_uri,
-                          'code' => $code));
-    } catch (FacebookApiException $e) {
-      // most likely that user very recently revoked authorization.
-      // In any event, we don't have an access token, so say so.
-      return false;
-    }
-
-    if (empty($access_token_response)) {
-      return false;
-    }
-
-    $response_params = array();
-    parse_str($access_token_response, $response_params);
-    if (!isset($response_params['access_token'])) {
-      return false;
-    }
-
-    return $response_params['access_token'];
+	if (empty($code)) {
+	  return false;
+	}
+
+	if ($redirect_uri === null) {
+	  $redirect_uri = $this->getCurrentUrl();
+	}
+
+	try {
+	  // need to circumvent json_decode by calling _oauthRequest
+	  // directly, since response isn't JSON format.
+	  $access_token_response =
+		$this->_oauthRequest(
+		  $this->getUrl('graph', '/oauth/access_token'),
+		  $params = array('client_id' => $this->getAppId(),
+						  'client_secret' => $this->getAppSecret(),
+						  'redirect_uri' => $redirect_uri,
+						  'code' => $code));
+	} catch (FacebookApiException $e) {
+	  // most likely that user very recently revoked authorization.
+	  // In any event, we don't have an access token, so say so.
+	  return false;
+	}
+
+	if (empty($access_token_response)) {
+	  return false;
+	}
+
+	$response_params = array();
+	parse_str($access_token_response, $response_params);
+	if (!isset($response_params['access_token'])) {
+	  return false;
+	}
+
+	return $response_params['access_token'];
   }
 
   /**
@@ -807,29 +807,29 @@ 
    * @throws FacebookApiException
    */
   protected function _restserver($params) {
-    // generic application level parameters
-    $params['api_key'] = $this->getAppId();
-    $params['format'] = 'json-strings';
-
-    $result = json_decode($this->_oauthRequest(
-      $this->getApiUrl($params['method']),
-      $params
-    ), true);
-
-    // results are returned, errors are thrown
-    if (is_array($result) && isset($result['error_code'])) {
-      $this->throwAPIException($result);
-      // @codeCoverageIgnoreStart
-    }
-    // @codeCoverageIgnoreEnd
-
-    $method = strtolower($params['method']);
-    if ($method === 'auth.expiresession' ||
-        $method === 'auth.revokeauthorization') {
-      $this->destroySession();
-    }
-
-    return $result;
+	// generic application level parameters
+	$params['api_key'] = $this->getAppId();
+	$params['format'] = 'json-strings';
+
+	$result = json_decode($this->_oauthRequest(
+	  $this->getApiUrl($params['method']),
+	  $params
+	), true);
+
+	// results are returned, errors are thrown
+	if (is_array($result) && isset($result['error_code'])) {
+	  $this->throwAPIException($result);
+	  // @codeCoverageIgnoreStart
+	}
+	// @codeCoverageIgnoreEnd
+
+	$method = strtolower($params['method']);
+	if ($method === 'auth.expiresession' ||
+		$method === 'auth.revokeauthorization') {
+	  $this->destroySession();
+	}
+
+	return $result;
   }
 
   /**
@@ -841,10 +841,10 @@ 
    * @return boolean true if this is video post
    */
   protected function isVideoPost($path, $method = 'GET') {
-    if ($method == 'POST' && preg_match("/^(\/)(.+)(\/)(videos)$/", $path)) {
-      return true;
-    }
-    return false;
+	if ($method == 'POST' && preg_match("/^(\/)(.+)(\/)(videos)$/", $path)) {
+	  return true;
+	}
+	return false;
   }
 
   /**
@@ -858,31 +858,31 @@ 
    * @throws FacebookApiException
    */
   protected function _graph($path, $method = 'GET', $params = array()) {
-    if (is_array($method) && empty($params)) {
-      $params = $method;
-      $method = 'GET';
-    }
-    $params['method'] = $method; // method override as we always do a POST
-
-    if ($this->isVideoPost($path, $method)) {
-      $domainKey = 'graph_video';
-    } else {
-      $domainKey = 'graph';
-    }
-
-    $result = json_decode($this->_oauthRequest(
-      $this->getUrl($domainKey, $path),
-      $params
-    ), true);
-
-    // results are returned, errors are thrown
-    if (is_array($result) && isset($result['error'])) {
-      $this->throwAPIException($result);
-      // @codeCoverageIgnoreStart
-    }
-    // @codeCoverageIgnoreEnd
-
-    return $result;
+	if (is_array($method) && empty($params)) {
+	  $params = $method;
+	  $method = 'GET';
+	}
+	$params['method'] = $method; // method override as we always do a POST
+
+	if ($this->isVideoPost($path, $method)) {
+	  $domainKey = 'graph_video';
+	} else {
+	  $domainKey = 'graph';
+	}
+
+	$result = json_decode($this->_oauthRequest(
+	  $this->getUrl($domainKey, $path),
+	  $params
+	), true);
+
+	// results are returned, errors are thrown
+	if (is_array($result) && isset($result['error'])) {
+	  $this->throwAPIException($result);
+	  // @codeCoverageIgnoreStart
+	}
+	// @codeCoverageIgnoreEnd
+
+	return $result;
   }
 
   /**
@@ -895,22 +895,22 @@ 
    * @throws FacebookApiException
    */
   protected function _oauthRequest($url, $params) {
-    if (!isset($params['access_token'])) {
-      $params['access_token'] = $this->getAccessToken();
-    }
-
-    if (isset($params['access_token'])) {
-      $params['appsecret_proof'] = $this->getAppSecretProof($params['access_token']);
-    }
-
-    // json_encode all params values that are not strings
-    foreach ($params as $key => $value) {
-      if (!is_string($value)) {
-        $params[$key] = json_encode($value);
-      }
-    }
-
-    return $this->makeRequest($url, $params);
+	if (!isset($params['access_token'])) {
+	  $params['access_token'] = $this->getAccessToken();
+	}
+
+	if (isset($params['access_token'])) {
+	  $params['appsecret_proof'] = $this->getAppSecretProof($params['access_token']);
+	}
+
+	// json_encode all params values that are not strings
+	foreach ($params as $key => $value) {
+	  if (!is_string($value)) {
+		$params[$key] = json_encode($value);
+	  }
+	}
+
+	return $this->makeRequest($url, $params);
   }
 
   /**
@@ -923,7 +923,7 @@ 
    * @return string The sha256 hash of the access_token
    */
   protected function getAppSecretProof($access_token) {
-    return hash_hmac('sha256', $access_token, $this->getAppSecret());
+	return hash_hmac('sha256', $access_token, $this->getAppSecret());
   }
 
   /**
@@ -938,71 +938,71 @@ 
    * @return string The response text
    */
   protected function makeRequest($url, $params, $ch=null) {
-    if (!$ch) {
-      $ch = curl_init();
-    }
-
-    $opts = self::$CURL_OPTS;
-    if ($this->getFileUploadSupport()) {
-      $opts[CURLOPT_POSTFIELDS] = $params;
-    } else {
-      $opts[CURLOPT_POSTFIELDS] = http_build_query($params, null, '&');
-    }
-    $opts[CURLOPT_URL] = $url;
-
-    // disable the 'Expect: 100-continue' behaviour. This causes CURL to wait
-    // for 2 seconds if the server does not support this header.
-    if (isset($opts[CURLOPT_HTTPHEADER])) {
-      $existing_headers = $opts[CURLOPT_HTTPHEADER];
-      $existing_headers[] = 'Expect:';
-      $opts[CURLOPT_HTTPHEADER] = $existing_headers;
-    } else {
-      $opts[CURLOPT_HTTPHEADER] = array('Expect:');
-    }
-
-    curl_setopt_array($ch, $opts);
-    $result = curl_exec($ch);
-
-    if (curl_errno($ch) == 60) { // CURLE_SSL_CACERT
-      self::errorLog('Invalid or no certificate authority found, '.
-                     'using bundled information');
-      curl_setopt($ch, CURLOPT_CAINFO,
-                  dirname(__FILE__) . '/fb_ca_chain_bundle.crt');
-      $result = curl_exec($ch);
-    }
-
-    // With dual stacked DNS responses, it's possible for a server to
-    // have IPv6 enabled but not have IPv6 connectivity.  If this is
-    // the case, curl will try IPv4 first and if that fails, then it will
-    // fall back to IPv6 and the error EHOSTUNREACH is returned by the
-    // operating system.
-    if ($result === false && empty($opts[CURLOPT_IPRESOLVE])) {
-        $matches = array();
-        $regex = '/Failed to connect to ([^:].*): Network is unreachable/';
-        if (preg_match($regex, curl_error($ch), $matches)) {
-          if (strlen(@inet_pton($matches[1])) === 16) {
-            self::errorLog('Invalid IPv6 configuration on server, '.
-                           'Please disable or get native IPv6 on your server.');
-            self::$CURL_OPTS[CURLOPT_IPRESOLVE] = CURL_IPRESOLVE_V4;
-            curl_setopt($ch, CURLOPT_IPRESOLVE, CURL_IPRESOLVE_V4);
-            $result = curl_exec($ch);
-          }
-        }
-    }
-
-    if ($result === false) {
-      $e = new FacebookApiException(array(
-        'error_code' => curl_errno($ch),
-        'error' => array(
-        'message' => curl_error($ch),
-        'type' => 'CurlException',
-        ),
-      ));
-      curl_close($ch);
-      throw $e;
-    }
-    curl_close($ch);
-    return $result;
+	if (!$ch) {
+	  $ch = curl_init();
+	}
+
+	$opts = self::$CURL_OPTS;
+	if ($this->getFileUploadSupport()) {
+	  $opts[CURLOPT_POSTFIELDS] = $params;
+	} else {
+	  $opts[CURLOPT_POSTFIELDS] = http_build_query($params, null, '&');
+	}
+	$opts[CURLOPT_URL] = $url;
+
+	// disable the 'Expect: 100-continue' behaviour. This causes CURL to wait
+	// for 2 seconds if the server does not support this header.
+	if (isset($opts[CURLOPT_HTTPHEADER])) {
+	  $existing_headers = $opts[CURLOPT_HTTPHEADER];
+	  $existing_headers[] = 'Expect:';
+	  $opts[CURLOPT_HTTPHEADER] = $existing_headers;
+	} else {
+	  $opts[CURLOPT_HTTPHEADER] = array('Expect:');
+	}
+
+	curl_setopt_array($ch, $opts);
+	$result = curl_exec($ch);
+
+	if (curl_errno($ch) == 60) { // CURLE_SSL_CACERT
+	  self::errorLog('Invalid or no certificate authority found, '.
+					 'using bundled information');
+	  curl_setopt($ch, CURLOPT_CAINFO,
+				  dirname(__FILE__) . '/fb_ca_chain_bundle.crt');
+	  $result = curl_exec($ch);
+	}
+
+	// With dual stacked DNS responses, it's possible for a server to
+	// have IPv6 enabled but not have IPv6 connectivity.  If this is
+	// the case, curl will try IPv4 first and if that fails, then it will
+	// fall back to IPv6 and the error EHOSTUNREACH is returned by the
+	// operating system.
+	if ($result === false && empty($opts[CURLOPT_IPRESOLVE])) {
+		$matches = array();
+		$regex = '/Failed to connect to ([^:].*): Network is unreachable/';
+		if (preg_match($regex, curl_error($ch), $matches)) {
+		  if (strlen(@inet_pton($matches[1])) === 16) {
+			self::errorLog('Invalid IPv6 configuration on server, '.
+						   'Please disable or get native IPv6 on your server.');
+			self::$CURL_OPTS[CURLOPT_IPRESOLVE] = CURL_IPRESOLVE_V4;
+			curl_setopt($ch, CURLOPT_IPRESOLVE, CURL_IPRESOLVE_V4);
+			$result = curl_exec($ch);
+		  }
+		}
+	}
+
+	if ($result === false) {
+	  $e = new FacebookApiException(array(
+		'error_code' => curl_errno($ch),
+		'error' => array(
+		'message' => curl_error($ch),
+		'type' => 'CurlException',
+		),
+	  ));
+	  curl_close($ch);
+	  throw $e;
+	}
+	curl_close($ch);
+	return $result;
   }
 
   /**
@@ -1012,27 +1012,27 @@ 
    * @return array The payload inside it or null if the sig is wrong
    */
   protected function parseSignedRequest($signed_request) {
-    list($encoded_sig, $payload) = explode('.', $signed_request, 2);
-
-    // decode the data
-    $sig = self::base64UrlDecode($encoded_sig);
-    $data = json_decode(self::base64UrlDecode($payload), true);
-
-    if (strtoupper($data['algorithm']) !== self::SIGNED_REQUEST_ALGORITHM) {
-      self::errorLog(
-        'Unknown algorithm. Expected ' . self::SIGNED_REQUEST_ALGORITHM);
-      return null;
-    }
-
-    // check sig
-    $expected_sig = hash_hmac('sha256', $payload,
-                              $this->getAppSecret(), $raw = true);
-    if ($sig !== $expected_sig) {
-      self::errorLog('Bad Signed JSON signature!');
-      return null;
-    }
-
-    return $data;
+	list($encoded_sig, $payload) = explode('.', $signed_request, 2);
+
+	// decode the data
+	$sig = self::base64UrlDecode($encoded_sig);
+	$data = json_decode(self::base64UrlDecode($payload), true);
+
+	if (strtoupper($data['algorithm']) !== self::SIGNED_REQUEST_ALGORITHM) {
+	  self::errorLog(
+		'Unknown algorithm. Expected ' . self::SIGNED_REQUEST_ALGORITHM);
+	  return null;
+	}
+
+	// check sig
+	$expected_sig = hash_hmac('sha256', $payload,
+							  $this->getAppSecret(), $raw = true);
+	if ($sig !== $expected_sig) {
+	  self::errorLog('Bad Signed JSON signature!');
+	  return null;
+	}
+
+	return $data;
   }
 
   /**
@@ -1042,17 +1042,17 @@ 
    * @return string The signed request.
    */
   protected function makeSignedRequest($data) {
-    if (!is_array($data)) {
-      throw new InvalidArgumentException(
-        'makeSignedRequest expects an array. Got: ' . print_r($data, true));
-    }
-    $data['algorithm'] = self::SIGNED_REQUEST_ALGORITHM;
-    $data['issued_at'] = time();
-    $json = json_encode($data);
-    $b64 = self::base64UrlEncode($json);
-    $raw_sig = hash_hmac('sha256', $b64, $this->getAppSecret(), $raw = true);
-    $sig = self::base64UrlEncode($raw_sig);
-    return $sig.'.'.$b64;
+	if (!is_array($data)) {
+	  throw new InvalidArgumentException(
+		'makeSignedRequest expects an array. Got: ' . print_r($data, true));
+	}
+	$data['algorithm'] = self::SIGNED_REQUEST_ALGORITHM;
+	$data['issued_at'] = time();
+	$json = json_encode($data);
+	$b64 = self::base64UrlEncode($json);
+	$raw_sig = hash_hmac('sha256', $b64, $this->getAppSecret(), $raw = true);
+	$sig = self::base64UrlEncode($raw_sig);
+	return $sig.'.'.$b64;
   }
 
   /**
@@ -1062,74 +1062,74 @@ 
    * @return string The URL for the given parameters
    */
   protected function getApiUrl($method) {
-    static $READ_ONLY_CALLS =
-      array('admin.getallocation' => 1,
-            'admin.getappproperties' => 1,
-            'admin.getbannedusers' => 1,
-            'admin.getlivestreamvialink' => 1,
-            'admin.getmetrics' => 1,
-            'admin.getrestrictioninfo' => 1,
-            'application.getpublicinfo' => 1,
-            'auth.getapppublickey' => 1,
-            'auth.getsession' => 1,
-            'auth.getsignedpublicsessiondata' => 1,
-            'comments.get' => 1,
-            'connect.getunconnectedfriendscount' => 1,
-            'dashboard.getactivity' => 1,
-            'dashboard.getcount' => 1,
-            'dashboard.getglobalnews' => 1,
-            'dashboard.getnews' => 1,
-            'dashboard.multigetcount' => 1,
-            'dashboard.multigetnews' => 1,
-            'data.getcookies' => 1,
-            'events.get' => 1,
-            'events.getmembers' => 1,
-            'fbml.getcustomtags' => 1,
-            'feed.getappfriendstories' => 1,
-            'feed.getregisteredtemplatebundlebyid' => 1,
-            'feed.getregisteredtemplatebundles' => 1,
-            'fql.multiquery' => 1,
-            'fql.query' => 1,
-            'friends.arefriends' => 1,
-            'friends.get' => 1,
-            'friends.getappusers' => 1,
-            'friends.getlists' => 1,
-            'friends.getmutualfriends' => 1,
-            'gifts.get' => 1,
-            'groups.get' => 1,
-            'groups.getmembers' => 1,
-            'intl.gettranslations' => 1,
-            'links.get' => 1,
-            'notes.get' => 1,
-            'notifications.get' => 1,
-            'pages.getinfo' => 1,
-            'pages.isadmin' => 1,
-            'pages.isappadded' => 1,
-            'pages.isfan' => 1,
-            'permissions.checkavailableapiaccess' => 1,
-            'permissions.checkgrantedapiaccess' => 1,
-            'photos.get' => 1,
-            'photos.getalbums' => 1,
-            'photos.gettags' => 1,
-            'profile.getinfo' => 1,
-            'profile.getinfooptions' => 1,
-            'stream.get' => 1,
-            'stream.getcomments' => 1,
-            'stream.getfilters' => 1,
-            'users.getinfo' => 1,
-            'users.getloggedinuser' => 1,
-            'users.getstandardinfo' => 1,
-            'users.hasapppermission' => 1,
-            'users.isappuser' => 1,
-            'users.isverified' => 1,
-            'video.getuploadlimits' => 1);
-    $name = 'api';
-    if (isset($READ_ONLY_CALLS[strtolower($method)])) {
-      $name = 'api_read';
-    } else if (strtolower($method) == 'video.upload') {
-      $name = 'api_video';
-    }
-    return self::getUrl($name, 'restserver.php');
+	static $READ_ONLY_CALLS =
+	  array('admin.getallocation' => 1,
+			'admin.getappproperties' => 1,
+			'admin.getbannedusers' => 1,
+			'admin.getlivestreamvialink' => 1,
+			'admin.getmetrics' => 1,
+			'admin.getrestrictioninfo' => 1,
+			'application.getpublicinfo' => 1,
+			'auth.getapppublickey' => 1,
+			'auth.getsession' => 1,
+			'auth.getsignedpublicsessiondata' => 1,
+			'comments.get' => 1,
+			'connect.getunconnectedfriendscount' => 1,
+			'dashboard.getactivity' => 1,
+			'dashboard.getcount' => 1,
+			'dashboard.getglobalnews' => 1,
+			'dashboard.getnews' => 1,
+			'dashboard.multigetcount' => 1,
+			'dashboard.multigetnews' => 1,
+			'data.getcookies' => 1,
+			'events.get' => 1,
+			'events.getmembers' => 1,
+			'fbml.getcustomtags' => 1,
+			'feed.getappfriendstories' => 1,
+			'feed.getregisteredtemplatebundlebyid' => 1,
+			'feed.getregisteredtemplatebundles' => 1,
+			'fql.multiquery' => 1,
+			'fql.query' => 1,
+			'friends.arefriends' => 1,
+			'friends.get' => 1,
+			'friends.getappusers' => 1,
+			'friends.getlists' => 1,
+			'friends.getmutualfriends' => 1,
+			'gifts.get' => 1,
+			'groups.get' => 1,
+			'groups.getmembers' => 1,
+			'intl.gettranslations' => 1,
+			'links.get' => 1,
+			'notes.get' => 1,
+			'notifications.get' => 1,
+			'pages.getinfo' => 1,
+			'pages.isadmin' => 1,
+			'pages.isappadded' => 1,
+			'pages.isfan' => 1,
+			'permissions.checkavailableapiaccess' => 1,
+			'permissions.checkgrantedapiaccess' => 1,
+			'photos.get' => 1,
+			'photos.getalbums' => 1,
+			'photos.gettags' => 1,
+			'profile.getinfo' => 1,
+			'profile.getinfooptions' => 1,
+			'stream.get' => 1,
+			'stream.getcomments' => 1,
+			'stream.getfilters' => 1,
+			'users.getinfo' => 1,
+			'users.getloggedinuser' => 1,
+			'users.getstandardinfo' => 1,
+			'users.hasapppermission' => 1,
+			'users.isappuser' => 1,
+			'users.isverified' => 1,
+			'video.getuploadlimits' => 1);
+	$name = 'api';
+	if (isset($READ_ONLY_CALLS[strtolower($method)])) {
+	  $name = 'api_read';
+	} else if (strtolower($method) == 'video.upload') {
+	  $name = 'api_video';
+	}
+	return self::getUrl($name, 'restserver.php');
   }
 
   /**
@@ -1142,59 +1142,59 @@ 
    * @return string The URL for the given parameters
    */
   protected function getUrl($name, $path='', $params=array()) {
-    $url = self::$DOMAIN_MAP[$name];
-    if ($path) {
-      if ($path[0] === '/') {
-        $path = substr($path, 1);
-      }
-      $url .= $path;
-    }
-    if ($params) {
-      $url .= '?' . http_build_query($params, null, '&');
-    }
-
-    return $url;
+	$url = self::$DOMAIN_MAP[$name];
+	if ($path) {
+	  if ($path[0] === '/') {
+		$path = substr($path, 1);
+	  }
+	  $url .= $path;
+	}
+	if ($params) {
+	  $url .= '?' . http_build_query($params, null, '&');
+	}
+
+	return $url;
   }
 
   protected function getHttpHost() {
-    if ($this->trustForwarded && isset($_SERVER['HTTP_X_FORWARDED_HOST'])) {
-      return $_SERVER['HTTP_X_FORWARDED_HOST'];
-    }
-    return $_SERVER['HTTP_HOST'];
+	if ($this->trustForwarded && isset($_SERVER['HTTP_X_FORWARDED_HOST'])) {
+	  return $_SERVER['HTTP_X_FORWARDED_HOST'];
+	}
+	return $_SERVER['HTTP_HOST'];
   }
 
   protected function getHttpProtocol() {
-    if ($this->trustForwarded && isset($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
-      if ($_SERVER['HTTP_X_FORWARDED_PROTO'] === 'https') {
-        return 'https';
-      }
-      return 'http';
-    }
-    /*apache + variants specific way of checking for https*/
-    if (isset($_SERVER['HTTPS']) &&
-        ($_SERVER['HTTPS'] === 'on' || $_SERVER['HTTPS'] == 1)) {
-      return 'https';
-    }
-    /*nginx way of checking for https*/
-    if (isset($_SERVER['SERVER_PORT']) &&
-        ($_SERVER['SERVER_PORT'] === '443')) {
-      return 'https';
-    }
-    return 'http';
+	if ($this->trustForwarded && isset($_SERVER['HTTP_X_FORWARDED_PROTO'])) {
+	  if ($_SERVER['HTTP_X_FORWARDED_PROTO'] === 'https') {
+		return 'https';
+	  }
+	  return 'http';
+	}
+	/*apache + variants specific way of checking for https*/
+	if (isset($_SERVER['HTTPS']) &&
+		($_SERVER['HTTPS'] === 'on' || $_SERVER['HTTPS'] == 1)) {
+	  return 'https';
+	}
+	/*nginx way of checking for https*/
+	if (isset($_SERVER['SERVER_PORT']) &&
+		($_SERVER['SERVER_PORT'] === '443')) {
+	  return 'https';
+	}
+	return 'http';
   }
 
   /**
    * Get the base domain used for the cookie.
    */
   protected function getBaseDomain() {
-    // The base domain is stored in the metadata cookie if not we fallback
-    // to the current hostname
-    $metadata = $this->getMetadataCookie();
-    if (array_key_exists('base_domain', $metadata) &&
-        !empty($metadata['base_domain'])) {
-      return trim($metadata['base_domain'], '.');
-    }
-    return $this->getHttpHost();
+	// The base domain is stored in the metadata cookie if not we fallback
+	// to the current hostname
+	$metadata = $this->getMetadataCookie();
+	if (array_key_exists('base_domain', $metadata) &&
+		!empty($metadata['base_domain'])) {
+	  return trim($metadata['base_domain'], '.');
+	}
+	return $this->getHttpHost();
   }
 
   /**
@@ -1206,36 +1206,36 @@ 
    * @return string The current URL
    */
   protected function getCurrentUrl() {
-    $protocol = $this->getHttpProtocol() . '://';
-    $host = $this->getHttpHost();
-    $currentUrl = $protocol.$host.$_SERVER['REQUEST_URI'];
-    $parts = parse_url($currentUrl);
-
-    $query = '';
-    if (!empty($parts['query'])) {
-      // drop known fb params
-      $params = explode('&', $parts['query']);
-      $retained_params = array();
-      foreach ($params as $param) {
-        if ($this->shouldRetainParam($param)) {
-          $retained_params[] = $param;
-        }
-      }
-
-      if (!empty($retained_params)) {
-        $query = '?'.implode($retained_params, '&');
-      }
-    }
-
-    // use port if non default
-    $port =
-      isset($parts['port']) &&
-      (($protocol === 'http://' && $parts['port'] !== 80) ||
-       ($protocol === 'https://' && $parts['port'] !== 443))
-      ? ':' . $parts['port'] : '';
-
-    // rebuild
-    return $protocol . $parts['host'] . $port . $parts['path'] . $query;
+	$protocol = $this->getHttpProtocol() . '://';
+	$host = $this->getHttpHost();
+	$currentUrl = $protocol.$host.$_SERVER['REQUEST_URI'];
+	$parts = parse_url($currentUrl);
+
+	$query = '';
+	if (!empty($parts['query'])) {
+	  // drop known fb params
+	  $params = explode('&', $parts['query']);
+	  $retained_params = array();
+	  foreach ($params as $param) {
+		if ($this->shouldRetainParam($param)) {
+		  $retained_params[] = $param;
+		}
+	  }
+
+	  if (!empty($retained_params)) {
+		$query = '?'.implode($retained_params, '&');
+	  }
+	}
+
+	// use port if non default
+	$port =
+	  isset($parts['port']) &&
+	  (($protocol === 'http://' && $parts['port'] !== 80) ||
+	   ($protocol === 'https://' && $parts['port'] !== 443))
+	  ? ':' . $parts['port'] : '';
+
+	// rebuild
+	return $protocol . $parts['host'] . $port . $parts['path'] . $query;
   }
 
   /**
@@ -1250,13 +1250,13 @@ 
    * @return boolean
    */
   protected function shouldRetainParam($param) {
-    foreach (self::$DROP_QUERY_PARAMS as $drop_query_param) {
-      if (strpos($param, $drop_query_param.'=') === 0) {
-        return false;
-      }
-    }
+	foreach (self::$DROP_QUERY_PARAMS as $drop_query_param) {
+	  if (strpos($param, $drop_query_param.'=') === 0) {
+		return false;
+	  }
+	}
 
-    return true;
+	return true;
   }
 
   /**
@@ -1268,25 +1268,25 @@ 
    *                      by a failed API call.
    */
   protected function throwAPIException($result) {
-    $e = new FacebookApiException($result);
-    switch ($e->getType()) {
-      // OAuth 2.0 Draft 00 style
-      case 'OAuthException':
-        // OAuth 2.0 Draft 10 style
-      case 'invalid_token':
-        // REST server errors are just Exceptions
-      case 'Exception':
-        $message = $e->getMessage();
-        if ((strpos($message, 'Error validating access token') !== false) ||
-            (strpos($message, 'Invalid OAuth access token') !== false) ||
-            (strpos($message, 'An active access token must be used') !== false)
-        ) {
-          $this->destroySession();
-        }
-        break;
-    }
-
-    throw $e;
+	$e = new FacebookApiException($result);
+	switch ($e->getType()) {
+	  // OAuth 2.0 Draft 00 style
+	  case 'OAuthException':
+		// OAuth 2.0 Draft 10 style
+	  case 'invalid_token':
+		// REST server errors are just Exceptions
+	  case 'Exception':
+		$message = $e->getMessage();
+		if ((strpos($message, 'Error validating access token') !== false) ||
+			(strpos($message, 'Invalid OAuth access token') !== false) ||
+			(strpos($message, 'An active access token must be used') !== false)
+		) {
+		  $this->destroySession();
+		}
+		break;
+	}
+
+	throw $e;
   }
 
 
@@ -1296,14 +1296,14 @@ 
    * @param string $msg Log message
    */
   protected static function errorLog($msg) {
-    // disable error log if we are running in a CLI environment
-    // @codeCoverageIgnoreStart
-    if (php_sapi_name() != 'cli') {
-      error_log($msg);
-    }
-    // uncomment this if you want to see the errors on the page
-    // print 'error_log: '.$msg."\n";
-    // @codeCoverageIgnoreEnd
+	// disable error log if we are running in a CLI environment
+	// @codeCoverageIgnoreStart
+	if (php_sapi_name() != 'cli') {
+	  error_log($msg);
+	}
+	// uncomment this if you want to see the errors on the page
+	// print 'error_log: '.$msg."\n";
+	// @codeCoverageIgnoreEnd
   }
 
   /**
@@ -1317,7 +1317,7 @@ 
    * @return string
    */
   protected static function base64UrlDecode($input) {
-    return base64_decode(strtr($input, '-_', '+/'));
+	return base64_decode(strtr($input, '-_', '+/'));
   }
 
   /**
@@ -1330,38 +1330,38 @@ 
    * @return string base64Url encoded string
    */
   protected static function base64UrlEncode($input) {
-    $str = strtr(base64_encode($input), '+/', '-_');
-    $str = str_replace('=', '', $str);
-    return $str;
+	$str = strtr(base64_encode($input), '+/', '-_');
+	$str = str_replace('=', '', $str);
+	return $str;
   }
 
   /**
    * Destroy the current session
    */
   public function destroySession() {
-    $this->accessToken = null;
-    $this->signedRequest = null;
-    $this->user = null;
-    $this->clearAllPersistentData();
-
-    // Javascript sets a cookie that will be used in getSignedRequest that we
-    // need to clear if we can
-    $cookie_name = $this->getSignedRequestCookieName();
-    if (array_key_exists($cookie_name, $_COOKIE)) {
-      unset($_COOKIE[$cookie_name]);
-      if (!headers_sent()) {
-        $base_domain = $this->getBaseDomain();
-        setcookie($cookie_name, '', 1, '/', '.'.$base_domain);
-      } else {
-        // @codeCoverageIgnoreStart
-        self::errorLog(
-          'There exists a cookie that we wanted to clear that we couldn\'t '.
-          'clear because headers was already sent. Make sure to do the first '.
-          'API call before outputing anything.'
-        );
-        // @codeCoverageIgnoreEnd
-      }
-    }
+	$this->accessToken = null;
+	$this->signedRequest = null;
+	$this->user = null;
+	$this->clearAllPersistentData();
+
+	// Javascript sets a cookie that will be used in getSignedRequest that we
+	// need to clear if we can
+	$cookie_name = $this->getSignedRequestCookieName();
+	if (array_key_exists($cookie_name, $_COOKIE)) {
+	  unset($_COOKIE[$cookie_name]);
+	  if (!headers_sent()) {
+		$base_domain = $this->getBaseDomain();
+		setcookie($cookie_name, '', 1, '/', '.'.$base_domain);
+	  } else {
+		// @codeCoverageIgnoreStart
+		self::errorLog(
+		  'There exists a cookie that we wanted to clear that we couldn\'t '.
+		  'clear because headers was already sent. Make sure to do the first '.
+		  'API call before outputing anything.'
+		);
+		// @codeCoverageIgnoreEnd
+	  }
+	}
   }
 
   /**
@@ -1370,44 +1370,44 @@ 
    * @return  an array mapping key to value
    */
   protected function getMetadataCookie() {
-    $cookie_name = $this->getMetadataCookieName();
-    if (!array_key_exists($cookie_name, $_COOKIE)) {
-      return array();
-    }
-
-    // The cookie value can be wrapped in "-characters so remove them
-    $cookie_value = trim($_COOKIE[$cookie_name], '"');
-
-    if (empty($cookie_value)) {
-      return array();
-    }
-
-    $parts = explode('&', $cookie_value);
-    $metadata = array();
-    foreach ($parts as $part) {
-      $pair = explode('=', $part, 2);
-      if (!empty($pair[0])) {
-        $metadata[urldecode($pair[0])] =
-          (count($pair) > 1) ? urldecode($pair[1]) : '';
-      }
-    }
-
-    return $metadata;
+	$cookie_name = $this->getMetadataCookieName();
+	if (!array_key_exists($cookie_name, $_COOKIE)) {
+	  return array();
+	}
+
+	// The cookie value can be wrapped in "-characters so remove them
+	$cookie_value = trim($_COOKIE[$cookie_name], '"');
+
+	if (empty($cookie_value)) {
+	  return array();
+	}
+
+	$parts = explode('&', $cookie_value);
+	$metadata = array();
+	foreach ($parts as $part) {
+	  $pair = explode('=', $part, 2);
+	  if (!empty($pair[0])) {
+		$metadata[urldecode($pair[0])] =
+		  (count($pair) > 1) ? urldecode($pair[1]) : '';
+	  }
+	}
+
+	return $metadata;
   }
 
   protected static function isAllowedDomain($big, $small) {
-    if ($big === $small) {
-      return true;
-    }
-    return self::endsWith($big, '.'.$small);
+	if ($big === $small) {
+	  return true;
+	}
+	return self::endsWith($big, '.'.$small);
   }
 
   protected static function endsWith($big, $small) {
-    $len = strlen($small);
-    if ($len === 0) {
-      return true;
-    }
-    return substr($big, -$len) === $small;
+	$len = strlen($small);
+	if ($len === 0) {
+	  return true;
+	}
+	return substr($big, -$len) === $small;
   }
 
   /**

Spacing +16 added lines, -16 removed lines

@@ -98,11 +98,11 @@ 
    * @return string The string representation of the error
    */
   public function __toString() {
-    $str = $this->getType() . ': ';
+    $str = $this->getType().': ';
     if ($this->code != 0) {
-      $str .= $this->code . ': ';
+      $str .= $this->code.': ';
     }
-    return $str . $this->message;
+    return $str.$this->message;
   }
 }
 
@@ -530,7 +530,7 @@ 
       if (array_key_exists('user_id', $signed_request)) {
         $user = $signed_request['user_id'];
 
-        if($user != $this->getPersistentData('user_id')){
+        if ($user != $this->getPersistentData('user_id')) {
           $this->clearAllPersistentData();
         }
 
@@ -576,7 +576,7 @@ 
    * @param array $params Provide custom parameters
    * @return string The URL for the login flow
    */
-  public function getLoginUrl($params=array()) {
+  public function getLoginUrl($params = array()) {
     $this->establishCSRFTokenState();
     $currentUrl = $this->getCurrentUrl();
 
@@ -605,7 +605,7 @@ 
    * @param array $params Provide custom parameters
    * @return string The URL for the logout flow
    */
-  public function getLogoutUrl($params=array()) {
+  public function getLogoutUrl($params = array()) {
     return $this->getUrl(
       'www',
       'logout.php',
@@ -627,7 +627,7 @@ 
    * @param array $params Provide custom parameters
    * @return string The URL for the logout flow
    */
-  public function getLoginStatusUrl($params=array()) {
+  public function getLoginStatusUrl($params = array()) {
     return $this->getUrl(
       'www',
       'extern/login_status.php',
@@ -937,7 +937,7 @@ 
    *
    * @return string The response text
    */
-  protected function makeRequest($url, $params, $ch=null) {
+  protected function makeRequest($url, $params, $ch = null) {
     if (!$ch) {
       $ch = curl_init();
     }
@@ -967,7 +967,7 @@ 
       self::errorLog('Invalid or no certificate authority found, '.
                      'using bundled information');
       curl_setopt($ch, CURLOPT_CAINFO,
-                  dirname(__FILE__) . '/fb_ca_chain_bundle.crt');
+                  dirname(__FILE__).'/fb_ca_chain_bundle.crt');
       $result = curl_exec($ch);
     }
 
@@ -1020,7 +1020,7 @@ 
 
     if (strtoupper($data['algorithm']) !== self::SIGNED_REQUEST_ALGORITHM) {
       self::errorLog(
-        'Unknown algorithm. Expected ' . self::SIGNED_REQUEST_ALGORITHM);
+        'Unknown algorithm. Expected '.self::SIGNED_REQUEST_ALGORITHM);
       return null;
     }
 
@@ -1044,7 +1044,7 @@ 
   protected function makeSignedRequest($data) {
     if (!is_array($data)) {
       throw new InvalidArgumentException(
-        'makeSignedRequest expects an array. Got: ' . print_r($data, true));
+        'makeSignedRequest expects an array. Got: '.print_r($data, true));
     }
     $data['algorithm'] = self::SIGNED_REQUEST_ALGORITHM;
     $data['issued_at'] = time();
@@ -1141,7 +1141,7 @@ 
    *
    * @return string The URL for the given parameters
    */
-  protected function getUrl($name, $path='', $params=array()) {
+  protected function getUrl($name, $path = '', $params = array()) {
     $url = self::$DOMAIN_MAP[$name];
     if ($path) {
       if ($path[0] === '/') {
@@ -1150,7 +1150,7 @@ 
       $url .= $path;
     }
     if ($params) {
-      $url .= '?' . http_build_query($params, null, '&');
+      $url .= '?'.http_build_query($params, null, '&');
     }
 
     return $url;
@@ -1206,7 +1206,7 @@ 
    * @return string The current URL
    */
   protected function getCurrentUrl() {
-    $protocol = $this->getHttpProtocol() . '://';
+    $protocol = $this->getHttpProtocol().'://';
     $host = $this->getHttpHost();
     $currentUrl = $protocol.$host.$_SERVER['REQUEST_URI'];
     $parts = parse_url($currentUrl);
@@ -1232,10 +1232,10 @@ 
       isset($parts['port']) &&
       (($protocol === 'http://' && $parts['port'] !== 80) ||
        ($protocol === 'https://' && $parts['port'] !== 443))
-      ? ':' . $parts['port'] : '';
+      ? ':'.$parts['port'] : '';
 
     // rebuild
-    return $protocol . $parts['host'] . $port . $parts['path'] . $query;
+    return $protocol.$parts['host'].$port.$parts['path'].$query;
   }
 
   /**

Braces +1 added lines, -2 removed lines

@@ -361,8 +361,7 @@
           'fb_exchange_token' => $this->getAccessToken(),
         )
       );
-    }
-    catch (FacebookApiException $e) {
+    } catch (FacebookApiException $e) {
       // most likely that user very recently revoked authorization.
       // In any event, we don't have an access token, so say so.
       return false;

bundles/ext/facebook/facebook.php

Doc Comments +3 added lines

@@ -150,6 +150,9 @@
     return self::FBSS_COOKIE_NAME . '_' . $this->getAppId();
   }
 
+  /**
+   * @param string $key
+   */
   protected function constructSessionVariableName($key) {
     $parts = array('fb', $this->getAppId(), $key);
     if ($this->sharedSessionID) {

Indentation +77 added lines, -77 removed lines

@@ -45,52 +45,52 @@ 
    * @see BaseFacebook::__construct in facebook.php
    */
   public function __construct($config) {
-    if (!session_id()) {
-      session_start();
-    }
-    parent::__construct($config);
-    if (!empty($config['sharedSession'])) {
-      $this->initSharedSession();
-    }
+	if (!session_id()) {
+	  session_start();
+	}
+	parent::__construct($config);
+	if (!empty($config['sharedSession'])) {
+	  $this->initSharedSession();
+	}
   }
 
   protected static $kSupportedKeys =
-    array('state', 'code', 'access_token', 'user_id');
+	array('state', 'code', 'access_token', 'user_id');
 
   protected function initSharedSession() {
-    $cookie_name = $this->getSharedSessionCookieName();
-    if (isset($_COOKIE[$cookie_name])) {
-      $data = $this->parseSignedRequest($_COOKIE[$cookie_name]);
-      if ($data && !empty($data['domain']) &&
-          self::isAllowedDomain($this->getHttpHost(), $data['domain'])) {
-        // good case
-        $this->sharedSessionID = $data['id'];
-        return;
-      }
-      // ignoring potentially unreachable data
-    }
-    // evil/corrupt/missing case
-    $base_domain = $this->getBaseDomain();
-    $this->sharedSessionID = md5(uniqid(mt_rand(), true));
-    $cookie_value = $this->makeSignedRequest(
-      array(
-        'domain' => $base_domain,
-        'id' => $this->sharedSessionID,
-      )
-    );
-    $_COOKIE[$cookie_name] = $cookie_value;
-    if (!headers_sent()) {
-      $expire = time() + self::FBSS_COOKIE_EXPIRE;
-      setcookie($cookie_name, $cookie_value, $expire, '/', '.'.$base_domain);
-    } else {
-      // @codeCoverageIgnoreStart
-      self::errorLog(
-        'Shared session ID cookie could not be set! You must ensure you '.
-        'create the Facebook instance before headers have been sent. This '.
-        'will cause authentication issues after the first request.'
-      );
-      // @codeCoverageIgnoreEnd
-    }
+	$cookie_name = $this->getSharedSessionCookieName();
+	if (isset($_COOKIE[$cookie_name])) {
+	  $data = $this->parseSignedRequest($_COOKIE[$cookie_name]);
+	  if ($data && !empty($data['domain']) &&
+		  self::isAllowedDomain($this->getHttpHost(), $data['domain'])) {
+		// good case
+		$this->sharedSessionID = $data['id'];
+		return;
+	  }
+	  // ignoring potentially unreachable data
+	}
+	// evil/corrupt/missing case
+	$base_domain = $this->getBaseDomain();
+	$this->sharedSessionID = md5(uniqid(mt_rand(), true));
+	$cookie_value = $this->makeSignedRequest(
+	  array(
+		'domain' => $base_domain,
+		'id' => $this->sharedSessionID,
+	  )
+	);
+	$_COOKIE[$cookie_name] = $cookie_value;
+	if (!headers_sent()) {
+	  $expire = time() + self::FBSS_COOKIE_EXPIRE;
+	  setcookie($cookie_name, $cookie_value, $expire, '/', '.'.$base_domain);
+	} else {
+	  // @codeCoverageIgnoreStart
+	  self::errorLog(
+		'Shared session ID cookie could not be set! You must ensure you '.
+		'create the Facebook instance before headers have been sent. This '.
+		'will cause authentication issues after the first request.'
+	  );
+	  // @codeCoverageIgnoreEnd
+	}
   }
 
   /**
@@ -100,61 +100,61 @@ 
    * access tokens.
    */
   protected function setPersistentData($key, $value) {
-    if (!in_array($key, self::$kSupportedKeys)) {
-      self::errorLog('Unsupported key passed to setPersistentData.');
-      return;
-    }
+	if (!in_array($key, self::$kSupportedKeys)) {
+	  self::errorLog('Unsupported key passed to setPersistentData.');
+	  return;
+	}
 
-    $session_var_name = $this->constructSessionVariableName($key);
-    $_SESSION[$session_var_name] = $value;
+	$session_var_name = $this->constructSessionVariableName($key);
+	$_SESSION[$session_var_name] = $value;
   }
 
   protected function getPersistentData($key, $default = false) {
-    if (!in_array($key, self::$kSupportedKeys)) {
-      self::errorLog('Unsupported key passed to getPersistentData.');
-      return $default;
-    }
-
-    $session_var_name = $this->constructSessionVariableName($key);
-    return isset($_SESSION[$session_var_name]) ?
-      $_SESSION[$session_var_name] : $default;
+	if (!in_array($key, self::$kSupportedKeys)) {
+	  self::errorLog('Unsupported key passed to getPersistentData.');
+	  return $default;
+	}
+
+	$session_var_name = $this->constructSessionVariableName($key);
+	return isset($_SESSION[$session_var_name]) ?
+	  $_SESSION[$session_var_name] : $default;
   }
 
   protected function clearPersistentData($key) {
-    if (!in_array($key, self::$kSupportedKeys)) {
-      self::errorLog('Unsupported key passed to clearPersistentData.');
-      return;
-    }
+	if (!in_array($key, self::$kSupportedKeys)) {
+	  self::errorLog('Unsupported key passed to clearPersistentData.');
+	  return;
+	}
 
-    $session_var_name = $this->constructSessionVariableName($key);
-    unset($_SESSION[$session_var_name]);
+	$session_var_name = $this->constructSessionVariableName($key);
+	unset($_SESSION[$session_var_name]);
   }
 
   protected function clearAllPersistentData() {
-    foreach (self::$kSupportedKeys as $key) {
-      $this->clearPersistentData($key);
-    }
-    if ($this->sharedSessionID) {
-      $this->deleteSharedSessionCookie();
-    }
+	foreach (self::$kSupportedKeys as $key) {
+	  $this->clearPersistentData($key);
+	}
+	if ($this->sharedSessionID) {
+	  $this->deleteSharedSessionCookie();
+	}
   }
 
   protected function deleteSharedSessionCookie() {
-    $cookie_name = $this->getSharedSessionCookieName();
-    unset($_COOKIE[$cookie_name]);
-    $base_domain = $this->getBaseDomain();
-    setcookie($cookie_name, '', 1, '/', '.'.$base_domain);
+	$cookie_name = $this->getSharedSessionCookieName();
+	unset($_COOKIE[$cookie_name]);
+	$base_domain = $this->getBaseDomain();
+	setcookie($cookie_name, '', 1, '/', '.'.$base_domain);
   }
 
   protected function getSharedSessionCookieName() {
-    return self::FBSS_COOKIE_NAME . '_' . $this->getAppId();
+	return self::FBSS_COOKIE_NAME . '_' . $this->getAppId();
   }
 
   protected function constructSessionVariableName($key) {
-    $parts = array('fb', $this->getAppId(), $key);
-    if ($this->sharedSessionID) {
-      array_unshift($parts, $this->sharedSessionID);
-    }
-    return implode('_', $parts);
+	$parts = array('fb', $this->getAppId(), $key);
+	if ($this->sharedSessionID) {
+	  array_unshift($parts, $this->sharedSessionID);
+	}
+	return implode('_', $parts);
   }
 }

Spacing +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
   }
 
   protected function getSharedSessionCookieName() {
-    return self::FBSS_COOKIE_NAME . '_' . $this->getAppId();
+    return self::FBSS_COOKIE_NAME.'_'.$this->getAppId();
   }
 
   protected function constructSessionVariableName($key) {

bundles/lib/Cache/Mock.php

Doc Comments +5 added lines, -6 removed lines

@@ -37,7 +37,7 @@ 
 	 * @param  string $sName name of the session
 	 * @param  int $iFlags flags
 	 * @param  int $iTimeout expiration of cache
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function get(string $sName, int &$iFlags = null, int $iTimeout = 0)
 	{ 
@@ -52,7 +52,7 @@ 
 	 * @param  mixed $mValue value of this sesion var
 	 * @param  int $iFlag unused
 	 * @param  int $iExpire expiration of cache
-	 * @return \Venus\lib\Cache\Apc
+	 * @return boolean
 	 */
 	public function set(string $sName, $mValue, int $iFlag = 0, int $iExpire = false)
 	{ 
@@ -63,7 +63,7 @@ 
 	 * flush the cache
 	 *
 	 * @access public
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function flush()
 	{
@@ -75,7 +75,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sName name of the session
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function delete(string $sName)
 	{
@@ -88,9 +88,8 @@ 
 	 * @access public
 	 * @param  string $sName name of the session
 	 * @param  mixed $mValue value of this sesion var
-	 * @param  int $iFlag unused
 	 * @param  int $iExpire expiration of cache
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function add($sName, $mValue, $iExpire = false)
 	{ 

Indentation +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
 	 */
 	public function get(string $sName, int &$iFlags = null, int $iTimeout = 0)
 	{ 
-	    return false;
+		return false;
 	}
 	
 	/**
@@ -79,7 +79,7 @@ 
 	 */
 	public function delete(string $sName)
 	{
-        return false;
+		return false;
 	}
 
 	/**

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
 	 * @param  int $iTimeout expiration of cache
 	 * @return mixed
 	 */
-	public function get(string $sName, int &$iFlags = null, int $iTimeout = 0)
+	public function get(string $sName, int&$iFlags = null, int $iTimeout = 0)
 	{ 
 	    return parent::get($sName);
 	}

bundles/lib/Date.php

Doc Comments -2 removed lines

@@ -192,8 +192,6 @@
 	 * set name of image
 	 *
 	 * @access public
-	 * @param  int $iWeek number of week
-	 * @param  int $iYear year
 	 * @return array
 	 */
 	public static function getActualMiddleWeek() : array

Spacing +10 added lines, -10 removed lines

@@ -41,12 +41,12 @@ 
 	 */
 	public static function getWeek(int $iWeek, int $iYear, string $sFormat = "Y-m-d") : Date
 	{
-		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
+		$iFirstDayInYear = date("N", mktime(0, 0, 0, 1, 1, $iYear));
 
 		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
 		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek - 1) * 604800; }
 		else { $iWeekInSeconds = 0; }
 
 		$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift;
@@ -158,12 +158,12 @@ 
 	 */
 	public static function getMiddleWeek(int $iWeek, int $iYear, string $sFormat = "Y-m-d") : array
 	{
-		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
+		$iFirstDayInYear = date("N", mktime(0, 0, 0, 1, 1, $iYear));
 
 		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
 		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek - 1) * 604800; }
 		else { $iWeekInSeconds = 0; }
 
 		if (date('N') > 2) {
@@ -182,7 +182,7 @@ 
 		if (preg_replace('/^([0-9]+)-[0-9]+-[0-9]+$/', '$1', $aDates[0]) != date('Y')) {
 
 			$aDates[0] = preg_replace('/^[0-9]+(-[0-9]+-[0-9]+)$/', date('Y').'$1', $aDates[0]);
-			$aDates[1] = preg_replace('/^[0-9]+(-[0-9]+-[0-9]+)$/', (date('Y')+1).'$1', $aDates[1]);
+			$aDates[1] = preg_replace('/^[0-9]+(-[0-9]+-[0-9]+)$/', (date('Y') + 1).'$1', $aDates[1]);
 		}
 
 		return $aDates;
@@ -225,10 +225,10 @@ 
 		$oDateTime = DateTime::createFromFormat('Y-m-d H:i:s', $sDateTime);
 		$iTimeStamp = time() - $oDateTime->getTimestamp();
 
-		if ($iTimeStamp < 3600) { return $sStartReturn.' '.(int)($iTimeStamp/60).' '.$sMinutes.$sEndReturn; }
-		if ($iTimeStamp < 86400) { return $sStartReturn.' '.(int)($iTimeStamp/3600).' '.$sHours.$sEndReturn; }
-		if ($iTimeStamp < 2592000) { return $sStartReturn.' '.(int)($iTimeStamp/86400).' '.$sDays.$sEndReturn; }
-		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp/2592000).' '.$sMonths.$sEndReturn; }
-		else { return $sStartReturn.' '.(int)($iTimeStamp/31536000).' '.$sYears.$sEndReturn; }
+		if ($iTimeStamp < 3600) { return $sStartReturn.' '.(int)($iTimeStamp / 60).' '.$sMinutes.$sEndReturn; }
+		if ($iTimeStamp < 86400) { return $sStartReturn.' '.(int)($iTimeStamp / 3600).' '.$sHours.$sEndReturn; }
+		if ($iTimeStamp < 2592000) { return $sStartReturn.' '.(int)($iTimeStamp / 86400).' '.$sDays.$sEndReturn; }
+		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp / 2592000).' '.$sMonths.$sEndReturn; }
+		else { return $sStartReturn.' '.(int)($iTimeStamp / 31536000).' '.$sYears.$sEndReturn; }
 	}
 }

Braces +9 added lines, -33 removed lines

@@ -43,11 +43,9 @@ 
 	{
 		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
 
-		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
-		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
+		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; } else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
-		else { $iWeekInSeconds = 0; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; } else { $iWeekInSeconds = 0; }
 
 		$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift;
 		$iTimestampLastDay = mktime(0, 0, 0, 1, 6, $iYear) + $iWeekInSeconds + $iShift + 604800;
@@ -78,18 +76,7 @@ 
 	{
 		if ($sLanguage == 'fr') {
 
-			if ($sMonth == '01' || $sMonth == 1) { return 'Janvier'; }
-			else if ($sMonth == '02' || $sMonth == 2) { return 'Février'; }
-			else if ($sMonth == '03' || $sMonth == 3) { return 'Mars'; }
-			else if ($sMonth == '04' || $sMonth == 4) { return 'Avril'; }
-			else if ($sMonth == '05' || $sMonth == 5) { return 'Mai'; }
-			else if ($sMonth == '06' || $sMonth == 6) { return 'Juin'; }
-			else if ($sMonth == '07' || $sMonth == 7) { return 'Juillet'; }
-			else if ($sMonth == '08' || $sMonth == 8) { return 'Août'; }
-			else if ($sMonth == '09' || $sMonth == 9) { return 'Septembre'; }
-			else if ($sMonth == 10) { return 'Octobre'; }
-			else if ($sMonth == 11) { return 'Novembre'; }
-			else if ($sMonth == 12) { return 'Décembre'; }
+			if ($sMonth == '01' || $sMonth == 1) { return 'Janvier'; } else if ($sMonth == '02' || $sMonth == 2) { return 'Février'; } else if ($sMonth == '03' || $sMonth == 3) { return 'Mars'; } else if ($sMonth == '04' || $sMonth == 4) { return 'Avril'; } else if ($sMonth == '05' || $sMonth == 5) { return 'Mai'; } else if ($sMonth == '06' || $sMonth == 6) { return 'Juin'; } else if ($sMonth == '07' || $sMonth == 7) { return 'Juillet'; } else if ($sMonth == '08' || $sMonth == 8) { return 'Août'; } else if ($sMonth == '09' || $sMonth == 9) { return 'Septembre'; } else if ($sMonth == 10) { return 'Octobre'; } else if ($sMonth == 11) { return 'Novembre'; } else if ($sMonth == 12) { return 'Décembre'; }
 		}
 	}
 
@@ -105,13 +92,7 @@ 
 	{
 		if ($sLanguage == 'fr') {
 
-			if ($sDay == 0) { return 'dimanche'; }
-			else if ($sDay == 1) { return 'lundi'; }
-			else if ($sDay == 2) { return 'mardi'; }
-			else if ($sDay == 3) { return 'mercredi'; }
-			else if ($sDay == 4) { return 'jeudi'; }
-			else if ($sDay == 5) { return 'vendredi'; }
-			else if ($sDay == 6) { return 'samedi'; }
+			if ($sDay == 0) { return 'dimanche'; } else if ($sDay == 1) { return 'lundi'; } else if ($sDay == 2) { return 'mardi'; } else if ($sDay == 3) { return 'mercredi'; } else if ($sDay == 4) { return 'jeudi'; } else if ($sDay == 5) { return 'vendredi'; } else if ($sDay == 6) { return 'samedi'; }
 		}
 	}
 
@@ -138,8 +119,7 @@ 
 			if ($iMonth == $aToday['mois']) {
 
 				if ($iDay > $aToday['jour']) { $iYears--; }
-			}
-			else {
+			} else {
 
 				$iYears--;
 			}
@@ -160,18 +140,15 @@ 
 	{
 		$iFirstDayInYear = date("N",mktime(0, 0, 0, 1, 1, $iYear));
 
-		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; }
-		else { $iShift = (8 - $iFirstDayInYear) * 86400; }
+		if ($iFirstDayInYear < 5) { $iShift = -($iFirstDayInYear - 1) * 86400; } else { $iShift = (8 - $iFirstDayInYear) * 86400; }
 
-		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; }
-		else { $iWeekInSeconds = 0; }
+		if ($iWeek > 1) { $iWeekInSeconds = ($iWeek-1) * 604800; } else { $iWeekInSeconds = 0; }
 
 		if (date('N') > 2) {
 
 			$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift + 172800;
 			$iTimestampLastDay = $iTimestamp + 604800;
-		}
-		else {
+		} else {
 
 			$iTimestamp = mktime(0, 0, 0, 1, 1, $iYear) + $iWeekInSeconds + $iShift - 432000;
 			$iTimestampLastDay = $iTimestamp + 604800;
@@ -228,7 +205,6 @@ 
 		if ($iTimeStamp < 3600) { return $sStartReturn.' '.(int)($iTimeStamp/60).' '.$sMinutes.$sEndReturn; }
 		if ($iTimeStamp < 86400) { return $sStartReturn.' '.(int)($iTimeStamp/3600).' '.$sHours.$sEndReturn; }
 		if ($iTimeStamp < 2592000) { return $sStartReturn.' '.(int)($iTimeStamp/86400).' '.$sDays.$sEndReturn; }
-		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp/2592000).' '.$sMonths.$sEndReturn; }
-		else { return $sStartReturn.' '.(int)($iTimeStamp/31536000).' '.$sYears.$sEndReturn; }
+		if ($iTimeStamp < 31536000) { return $sStartReturn.' '.(int)($iTimeStamp/2592000).' '.$sMonths.$sEndReturn; } else { return $sStartReturn.' '.(int)($iTimeStamp/31536000).' '.$sYears.$sEndReturn; }
 	}
 }

bundles/lib/Form.php

Doc Comments +1 added lines, -2 removed lines

@@ -304,7 +304,7 @@ 
 	 * get global object form
 	 *
 	 * @access public
-	 * @return object
+	 * @return \stdClass
 	 */
 	public function getFormInObject()
 	{
@@ -411,7 +411,6 @@ 
 	 * set the entity to synchronize with the formular
 	 *
 	 * @access public
-	 * @param  string $sSeparator separator between the fields
 	 * @param  int $iId id of the primary key
 	 * @return \Venus\lib\Form
 	 */

Indentation +52 added lines, -52 removed lines

@@ -141,14 +141,14 @@ 
 	 */
 	public function add($sName, $mType, $sLabel = null, $mValue = null, $mOptions = null)
 	{
-	    if ($mType instanceof Container) {
+		if ($mType instanceof Container) {
 	        
-	        $this->_aElement[$sName] = $mType;
-	    }
+			$this->_aElement[$sName] = $mType;
+		}
 		else if ($mType === 'text' || $mType === 'submit' || $mType === 'password' || $mType === 'file' || $mType === 'tel'
-	        || $mType === 'url' || $mType === 'email' || $mType === 'search' || $mType === 'date' || $mType === 'time'
-	        || $mType === 'datetime' || $mType === 'month' || $mType === 'week' || $mType === 'number' || $mType === 'range'
-	        || $mType === 'color') {
+			|| $mType === 'url' || $mType === 'email' || $mType === 'search' || $mType === 'date' || $mType === 'time'
+			|| $mType === 'datetime' || $mType === 'month' || $mType === 'week' || $mType === 'number' || $mType === 'range'
+			|| $mType === 'color') {
 
 			$this->_aElement[$sName] = new Input($sName, $mType, $sLabel, $mValue);
 		}
@@ -294,7 +294,7 @@ 
 
 		$oContainer = new Container;
 		$oContainer->setView($sFormContent)
-		           ->setForm($this);
+				   ->setForm($this);
 		
 		return $oContainer;
 	}
@@ -308,67 +308,67 @@ 
 	 */
 	public function getFormInObject()
 	{
-	    if ($this->_iIdEntity > 0 && $this->_sSynchronizeEntity !== null && count($_POST) < 1) {
+		if ($this->_iIdEntity > 0 && $this->_sSynchronizeEntity !== null && count($_POST) < 1) {
 	
-	        $sModelName = str_replace('Entity', 'Model', $this->_sSynchronizeEntity);
-	        $oModel = new $sModelName;
+			$sModelName = str_replace('Entity', 'Model', $this->_sSynchronizeEntity);
+			$oModel = new $sModelName;
 	        	
-	        $oEntity = new $this->_sSynchronizeEntity;
-	        $sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
-	        $sMethodName = 'findOneBy'.$sPrimaryKey;
-	        $oCompleteEntity = call_user_func_array(array(&$oModel, $sMethodName), array($this->_iIdEntity));
+			$oEntity = new $this->_sSynchronizeEntity;
+			$sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
+			$sMethodName = 'findOneBy'.$sPrimaryKey;
+			$oCompleteEntity = call_user_func_array(array(&$oModel, $sMethodName), array($this->_iIdEntity));
 	
-	        if (is_object($oCompleteEntity)) {
+			if (is_object($oCompleteEntity)) {
 	
-	            foreach ($this->_aElement as $sKey => $sValue) {
+				foreach ($this->_aElement as $sKey => $sValue) {
 	
-	                if ($sValue instanceof \Venus\lib\Form\Radio) {
+					if ($sValue instanceof \Venus\lib\Form\Radio) {
 	
-	                    $sExKey = $sKey;
-	                    $sKey = substr($sKey, 0, -6);
-	                }
+						$sExKey = $sKey;
+						$sKey = substr($sKey, 0, -6);
+					}
 	                	
-	                if ($sValue instanceof Form) {
+					if ($sValue instanceof Form) {
 	                    
-	                    ;
-	                }
-	                else {
+						;
+					}
+					else {
 	                 
-    	                $sMethodNameInEntity = 'get_'.$sKey;
-    	                $mValue = $oCompleteEntity->$sMethodNameInEntity();
+						$sMethodNameInEntity = 'get_'.$sKey;
+						$mValue = $oCompleteEntity->$sMethodNameInEntity();
     	
-    	                if ($sValue instanceof \Venus\lib\Form\Radio && method_exists($this->_aElement[$sExKey], 'setValueChecked')) {
+						if ($sValue instanceof \Venus\lib\Form\Radio && method_exists($this->_aElement[$sExKey], 'setValueChecked')) {
     	
-    	                    $this->_aElement[$sExKey]->setValueChecked($mValue);
-    	                }
-    	                else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
+							$this->_aElement[$sExKey]->setValueChecked($mValue);
+						}
+						else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
     	
-    	                    $this->_aElement[$sKey]->setValue($mValue);
-    	                }
-	                }
-	            }
-	        }
-	    }
+							$this->_aElement[$sKey]->setValue($mValue);
+						}
+					}
+				}
+			}
+		}
 	
-	    $oForm = new \StdClass();
-	    $oForm->start = '<form name="form'.$this->_iFormNumber.'" method="post"><input type="hidden" value="1" name="validform'.$this->_iFormNumber.'">';
-	    $oForm->form = array();
+		$oForm = new \StdClass();
+		$oForm->start = '<form name="form'.$this->_iFormNumber.'" method="post"><input type="hidden" value="1" name="validform'.$this->_iFormNumber.'">';
+		$oForm->form = array();
 	
-	    foreach ($this->_aElement as $sKey => $sValue) {
+		foreach ($this->_aElement as $sKey => $sValue) {
 
-	        if ($sValue instanceof Container) {
+			if ($sValue instanceof Container) {
 	        
-	            $oForm->form[$sKey] = $sValue;
-	        }
-	        else {
+				$oForm->form[$sKey] = $sValue;
+			}
+			else {
 	            
-	            $oForm->form[$sKey] = $sValue->fetch();
-	        }
-	    }
+				$oForm->form[$sKey] = $sValue->fetch();
+			}
+		}
 	
-	    $oForm->end = '</form>';
+		$oForm->end = '</form>';
 	
-	    return $oForm;
+		return $oForm;
 	}
 
 	/**
@@ -449,7 +449,7 @@ 
 	 */
 	public function getElement() {
 	    
-	    return $this->_aElement;
+		return $this->_aElement;
 	}
 	
 	/**
@@ -460,7 +460,7 @@ 
 	 */
 	public function getIdEntity() {
 	    
-	    return $this->_iIdEntity;
+		return $this->_iIdEntity;
 	}
 	
 	/**
@@ -471,6 +471,6 @@ 
 	 */
 	public function getSynchronizeEntity() {
 	    
-	    return $this->_sSynchronizeEntity;
+		return $this->_sSynchronizeEntity;
 	}
 }

Spacing +3 added lines, -3 removed lines

@@ -181,13 +181,13 @@ 
 		}
 		else  if ($mType === 'radio') {
 
-			$this->_aElement[$sName.rand(100000,999999)] = new Radio($sName, $sLabel, $mValue, $mOptions);
+			$this->_aElement[$sName.rand(100000, 999999)] = new Radio($sName, $sLabel, $mValue, $mOptions);
 		}
 		else  if ($mType === 'date') {
 
 			$aDay = array();
 
-			for ($i = 1 ; $i <= 31 ; $i++) {
+			for ($i = 1; $i <= 31; $i++) {
 
 				if ($i < 10) { $aDay['0'.$i] = '0'.$i; }
 				else { $aDay[$i] = $i; }
@@ -214,7 +214,7 @@ 
 
 			$aYear = array();
 
-			for ($i = 1900 ; $i <= 2013 ; $i++) {
+			for ($i = 1900; $i <= 2013; $i++) {
 
 				$aYear[$i] = $i;
 			}

Braces +12 added lines, -24 removed lines

@@ -144,27 +144,22 @@ 
 	    if ($mType instanceof Container) {
 	        
 	        $this->_aElement[$sName] = $mType;
-	    }
-		else if ($mType === 'text' || $mType === 'submit' || $mType === 'password' || $mType === 'file' || $mType === 'tel'
+	    } else if ($mType === 'text' || $mType === 'submit' || $mType === 'password' || $mType === 'file' || $mType === 'tel'
 	        || $mType === 'url' || $mType === 'email' || $mType === 'search' || $mType === 'date' || $mType === 'time'
 	        || $mType === 'datetime' || $mType === 'month' || $mType === 'week' || $mType === 'number' || $mType === 'range'
 	        || $mType === 'color') {
 
 			$this->_aElement[$sName] = new Input($sName, $mType, $sLabel, $mValue);
-		}
-		elseif ($mType === 'textarea') {
+		} elseif ($mType === 'textarea') {
 
 			$this->_aElement[$sName] = new Textarea($sName, $sLabel, $mValue);
-		}
-		else  if ($mType === 'select') {
+		} else  if ($mType === 'select') {
 
 			$this->_aElement[$sName] = new Select($sName, $mOptions, $sLabel, $mValue);
-		}
-		else  if ($mType === 'label') {
+		} else  if ($mType === 'label') {
 
 			$this->_aElement[$sName] = new Label($sName);
-		}
-		else  if ($mType === 'list_checkbox') {
+		} else  if ($mType === 'list_checkbox') {
 
 			$i = 0;
 			
@@ -174,23 +169,19 @@ 
 			
 				$this->_aElement[$sName.'_'.$i++] = new Checkbox($sName, $sValue, $mKey, $mOptions);
 			}
-		}
-		else  if ($mType === 'checkbox') {
+		} else  if ($mType === 'checkbox') {
 
 			$this->_aElement[$sName] = new Checkbox($sName, $sLabel, $mValue, $mOptions);
-		}
-		else  if ($mType === 'radio') {
+		} else  if ($mType === 'radio') {
 
 			$this->_aElement[$sName.rand(100000,999999)] = new Radio($sName, $sLabel, $mValue, $mOptions);
-		}
-		else  if ($mType === 'date') {
+		} else  if ($mType === 'date') {
 
 			$aDay = array();
 
 			for ($i = 1 ; $i <= 31 ; $i++) {
 
-				if ($i < 10) { $aDay['0'.$i] = '0'.$i; }
-				else { $aDay[$i] = $i; }
+				if ($i < 10) { $aDay['0'.$i] = '0'.$i; } else { $aDay[$i] = $i; }
 			}
 
 			$this->_aElement[$sName.'_day'] = new Select($sName, $aDay);
@@ -331,8 +322,7 @@ 
 	                if ($sValue instanceof Form) {
 	                    
 	                    ;
-	                }
-	                else {
+	                } else {
 	                 
     	                $sMethodNameInEntity = 'get_'.$sKey;
     	                $mValue = $oCompleteEntity->$sMethodNameInEntity();
@@ -340,8 +330,7 @@ 
     	                if ($sValue instanceof \Venus\lib\Form\Radio && method_exists($this->_aElement[$sExKey], 'setValueChecked')) {
     	
     	                    $this->_aElement[$sExKey]->setValueChecked($mValue);
-    	                }
-    	                else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
+    	                } else if (isset($mValue) && method_exists($this->_aElement[$sKey], 'setValue')) {
     	
     	                    $this->_aElement[$sKey]->setValue($mValue);
     	                }
@@ -359,8 +348,7 @@ 
 	        if ($sValue instanceof Container) {
 	        
 	            $oForm->form[$sKey] = $sValue;
-	        }
-	        else {
+	        } else {
 	            
 	            $oForm->form[$sKey] = $sValue->fetch();
 	        }

bundles/lib/Form/Container.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
 	 * get the Value
 	 *
 	 * @access public
-	 * @return object
+	 * @return \stdClass
 	 */
 	public function createView()
 	{

Indentation +50 added lines, -50 removed lines

@@ -73,24 +73,24 @@ 
 	 */
 	public function createView()
 	{
-	    $oView = new \stdClass;
-	    $oView->form = $this->_sView;
-	    $oView->form_start = $this->_oForm->getFormInObject()->start;
-	    $oView->form_end = $this->_oForm->getFormInObject()->end;
-	    $oView->form_row = array();
+		$oView = new \stdClass;
+		$oView->form = $this->_sView;
+		$oView->form_start = $this->_oForm->getFormInObject()->start;
+		$oView->form_end = $this->_oForm->getFormInObject()->end;
+		$oView->form_row = array();
 	    
-	    foreach ($this->_oForm->getFormInObject()->form as $sKey => $mValue) {
+		foreach ($this->_oForm->getFormInObject()->form as $sKey => $mValue) {
 	        
-	        if ($mValue instanceof Container) {
+			if ($mValue instanceof Container) {
 	            
-	            $oNewForm = $mValue->createView();
-	            $oView->form_row[$sKey] = $oNewForm->form_row;
-	        }
-	        else {
+				$oNewForm = $mValue->createView();
+				$oView->form_row[$sKey] = $oNewForm->form_row;
+			}
+			else {
 	            
-	           $oView->form_row[$sKey] = $mValue;
-	        }
-	    }
+			   $oView->form_row[$sKey] = $mValue;
+			}
+		}
 	       
 		return $oView;
 	}
@@ -117,48 +117,48 @@ 
 	 */
 	public function handleRequest(array $aRequest) : bool
 	{
-	    if (!count($_POST)) { return true; }
+		if (!count($_POST)) { return true; }
 	    
-	    // Validation
-	    foreach ($this->_oForm->getElement() as $sKey => $sValue) {
+		// Validation
+		foreach ($this->_oForm->getElement() as $sKey => $sValue) {
 	    
-	        if (!$sValue instanceof self && !$this->_validate($sValue)) {
+			if (!$sValue instanceof self && !$this->_validate($sValue)) {
 
-	            return false;
-	        }
-	    }
+				return false;
+			}
+		}
 	    
-	    // Save
+		// Save
 		if ($this->_oForm->getIdEntity() > 0 && $this->_oForm->getSynchronizeEntity() !== null && count($aRequest) > 0) {
 		
-		    $sModelName = str_replace('Entity', 'Model', $this->_oForm->getSynchronizeEntity());
-		    $oModel = new $sModelName;
+			$sModelName = str_replace('Entity', 'Model', $this->_oForm->getSynchronizeEntity());
+			$oModel = new $sModelName;
 		
-		    $oEntity = new $this->_oForm->getSynchronizeEntity();
-		    $sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
-		    $sMethodName = 'set_'.$sPrimaryKey;
+			$oEntity = new $this->_oForm->getSynchronizeEntity();
+			$sPrimaryKey = LibEntity::getPrimaryKeyNameWithoutMapping($oEntity);
+			$sMethodName = 'set_'.$sPrimaryKey;
 		
-		    call_user_func_array(array(&$oEntity, $sMethodName), array($this->_oForm->getIdEntity()));
+			call_user_func_array(array(&$oEntity, $sMethodName), array($this->_oForm->getIdEntity()));
 		
-		    foreach ($this->_oForm->getElement() as $sKey => $sValue) {
+			foreach ($this->_oForm->getElement() as $sKey => $sValue) {
 		
-		        $sMethodName = 'set_'.$sValue->getName().'';
-		        call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
-		    }
+				$sMethodName = 'set_'.$sValue->getName().'';
+				call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
+			}
 		
-		    $oEntity->save();
+			$oEntity->save();
 		}
 		else if ($this->_oForm->getSynchronizeEntity() !== null && isset($aRequest) && count($aRequest) > 0) {
 		
-		    $oEntity = new $this->_oForm->_sSynchronizeEntity;
+			$oEntity = new $this->_oForm->_sSynchronizeEntity;
 		
-		    foreach ($this->_oForm->getElement() as $sKey => $sValue) {
+			foreach ($this->_oForm->getElement() as $sKey => $sValue) {
 		
-		        $sMethodName = 'set_'.$sValue->getName().'';
-		        call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
-		    }
+				$sMethodName = 'set_'.$sValue->getName().'';
+				call_user_func_array(array(&$oEntity, $sMethodName), array($aRequest[$sValue->getName()]));
+			}
 		
-		    $this->_oForm->setIdEntityCreated($oEntity->save());
+			$this->_oForm->setIdEntityCreated($oEntity->save());
 		}
 		
 		$this->_bHandleRequestActivate = true;
@@ -199,13 +199,13 @@ 
 	 */
 	public function isSubmitted() : bool
 	{
-	    if (isset($_POST['validform'.$this->_oForm->getFormNumber()]) && $_POST['validform'.$this->_oForm->getFormNumber()] == 1) {
+		if (isset($_POST['validform'.$this->_oForm->getFormNumber()]) && $_POST['validform'.$this->_oForm->getFormNumber()] == 1) {
 	        
-	        return true;
-	    }
+			return true;
+		}
 		else { 
 		    
-		    return false;
+			return false;
 		}
 	}
 
@@ -218,7 +218,7 @@ 
 	 */
 	public function isClicked(string $sElementName) : bool
 	{
-	    if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; }
+		if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; }
 		else { return false; }
 	}
 
@@ -231,14 +231,14 @@ 
 	 */
 	private function _validate($oElement) : bool
 	{
-	    foreach ($oElement->getConstraint() as $oConstraint) {
+		foreach ($oElement->getConstraint() as $oConstraint) {
 	        
-	        if (!$oConstraint->validate($_POST[$oElement->getName()])) {
+			if (!$oConstraint->validate($_POST[$oElement->getName()])) {
 	            
-	            return false;
-	        }
-	    }
+				return false;
+			}
+		}
 	    
-	    return true;
+		return true;
 	}
 }

Braces +5 added lines, -10 removed lines

@@ -85,8 +85,7 @@ 
 	            
 	            $oNewForm = $mValue->createView();
 	            $oView->form_row[$sKey] = $oNewForm->form_row;
-	        }
-	        else {
+	        } else {
 	            
 	           $oView->form_row[$sKey] = $mValue;
 	        }
@@ -147,8 +146,7 @@ 
 		    }
 		
 		    $oEntity->save();
-		}
-		else if ($this->_oForm->getSynchronizeEntity() !== null && isset($aRequest) && count($aRequest) > 0) {
+		} else if ($this->_oForm->getSynchronizeEntity() !== null && isset($aRequest) && count($aRequest) > 0) {
 		
 		    $oEntity = new $this->_oForm->_sSynchronizeEntity;
 		
@@ -187,8 +185,7 @@ 
 	 */
 	public function isValid() : bool
 	{
-		if ($this->_bHandleRequestActivate === true) { return true; }
-		else { return false; }
+		if ($this->_bHandleRequestActivate === true) { return true; } else { return false; }
 	}
 
 	/**
@@ -202,8 +199,7 @@ 
 	    if (isset($_POST['validform'.$this->_oForm->getFormNumber()]) && $_POST['validform'.$this->_oForm->getFormNumber()] == 1) {
 	        
 	        return true;
-	    }
-		else { 
+	    } else { 
 		    
 		    return false;
 		}
@@ -218,8 +214,7 @@ 
 	 */
 	public function isClicked(string $sElementName) : bool
 	{
-	    if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; }
-		else { return false; }
+	    if (isset($_POST[$sElementName]) && $_POST[$sElementName]) { return true; } else { return false; }
 	}
 
 	/**

bundles/lib/Form/Input.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
 	 *
 	 * @access public
 	 * @param  string $sType type of input;
-	 * @return bool
+	 * @return string
 	 */
 	public function isClicked(string $sType) : string
 	{

Indentation +4 added lines, -4 removed lines

@@ -174,10 +174,10 @@
 		$sContent = '';
 
 		if ($this->getType() === 'text' || $this->getType() === 'password' || $this->getType() === 'file'
-            || $this->getType() === 'tel' || $this->getType() === 'url' || $this->getType() === 'email'
-            || $this->getType() === 'search' || $this->getType() === 'date' || $this->getType() === 'time'
-            || $this->getType() === 'datetime' || $this->getType() === 'month' || $this->getType() === 'week'
-            || $this->getType() === 'number' || $this->getType() === 'range' || $this->getType() === 'color') {
+			|| $this->getType() === 'tel' || $this->getType() === 'url' || $this->getType() === 'email'
+			|| $this->getType() === 'search' || $this->getType() === 'date' || $this->getType() === 'time'
+			|| $this->getType() === 'datetime' || $this->getType() === 'month' || $this->getType() === 'week'
+			|| $this->getType() === 'number' || $this->getType() === 'range' || $this->getType() === 'color') {
 
 			$sContent .= '<label>'.$this->getLabel().'</label> ';
 		}

Braces +1 added lines, -2 removed lines

@@ -60,8 +60,7 @@
 		$this->setName($sName);
 		$this->setValue($sValue);
 
-		if ($sLabel !== null) { $this->setLabel($sLabel); }
-		else { $this->setLabel($sName); }
+		if ($sLabel !== null) { $this->setLabel($sLabel); } else { $this->setLabel($sName); }
 	}
 
 	/**

bundles/lib/Form/Radio.php

Doc Comments +3 added lines, -3 removed lines

@@ -96,7 +96,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sValue Value of input;
-	 * @return \Venus\lib\Form\Input
+	 * @return Radio
 	 */
 	public function setValue(string $sValue) : Input
 	{
@@ -120,7 +120,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sValueChecked Value of input;
-	 * @return \Venus\lib\Form\Input
+	 * @return Radio
 	 */
 	public function setValueChecked(string $sValueChecked) : Input
 	{
@@ -144,7 +144,7 @@ 
 	 *
 	 * @access public
 	 * @param  string $sLabel Label of input;
-	 * @return \Venus\lib\Form\Input
+	 * @return Radio
 	 */
 	public function setLabel(string $sLabel) : Input
 	{