reginaldojunior / winners

app/Vendor/PagSeguro/source/PagSeguroLibrary/service/PagSeguroConnectionData.class.php

Doc Comments +8 added lines

@@ -80,6 +80,10 @@ 
      * @param PagSeguroCredentials $credentials
      * @param $serviceName
      */
+
+    /**
+     * @param string $serviceName
+     */
     public function __construct(PagSeguroCredentials $credentials, $serviceName)
     {
 
@@ -298,6 +302,10 @@ 
      * @param $resource
      * @return mixed
      */
+
+    /**
+     * @param string $resource
+     */
     public function getResource($resource)
     {
         return $this->resources[$resource];

Indentation +206 added lines, -206 removed lines

@@ -1,25 +1,25 @@ 
 <?php
 /**
- * 2007-2014 [PagSeguro Internet Ltda.]
- *
- * NOTICE OF LICENSE
- *
- *Licensed under the Apache License, Version 2.0 (the "License");
- *you may not use this file except in compliance with the License.
- *You may obtain a copy of the License at
- *
- *http://www.apache.org/licenses/LICENSE-2.0
- *
- *Unless required by applicable law or agreed to in writing, software
- *distributed under the License is distributed on an "AS IS" BASIS,
- *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- *See the License for the specific language governing permissions and
- *limitations under the License.
- *
- *  @author    PagSeguro Internet Ltda.
- *  @copyright 2007-2014 PagSeguro Internet Ltda.
- *  @license   http://www.apache.org/licenses/LICENSE-2.0
- */
+	 * 2007-2014 [PagSeguro Internet Ltda.]
+	 *
+	 * NOTICE OF LICENSE
+	 *
+	 *Licensed under the Apache License, Version 2.0 (the "License");
+	 *you may not use this file except in compliance with the License.
+	 *You may obtain a copy of the License at
+	 *
+	 *http://www.apache.org/licenses/LICENSE-2.0
+	 *
+	 *Unless required by applicable law or agreed to in writing, software
+	 *distributed under the License is distributed on an "AS IS" BASIS,
+	 *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+	 *See the License for the specific language governing permissions and
+	 *limitations under the License.
+	 *
+	 *  @author    PagSeguro Internet Ltda.
+	 *  @copyright 2007-2014 PagSeguro Internet Ltda.
+	 *  @license   http://www.apache.org/licenses/LICENSE-2.0
+	 */
 
 /***
  * Class PagSeguroConnectionData
@@ -27,297 +27,297 @@ 
 class PagSeguroConnectionData
 {
 
-    /***
+	/***
      * @var
      */
-    private $serviceName;
-    /***
+	private $serviceName;
+	/***
      * @var PagSeguroCredentials
      */
-    private $credentials;
-    /***
+	private $credentials;
+	/***
      * @var
      */
-    private $resources;
-    /***
+	private $resources;
+	/***
      * @var
      */
-    private $environment;
-    /***
+	private $environment;
+	/***
      * @var
      */
-    private $webserviceUrl;
-    /***
+	private $webserviceUrl;
+	/***
      * @var
      */
-    private $paymentUrl;
-    /***
+	private $paymentUrl;
+	/***
      * @var
      */
-    private $baseUrl;
-    /***
+	private $baseUrl;
+	/***
      * @var
      */
-    private $installmentUrl;
-    /***
+	private $installmentUrl;
+	/***
      * @var
      */
-    private $sessionUrl;
-    /***
+	private $sessionUrl;
+	/***
      * @var
      */
-    private $servicePath;
-    /***
+	private $servicePath;
+	/***
      * @var
      */
-    private $serviceTimeout;
-    /***
+	private $serviceTimeout;
+	/***
      * @var
      */
-    private $charset;
+	private $charset;
 
-    /***
+	/***
      * @param PagSeguroCredentials $credentials
      * @param $serviceName
      */
-    public function __construct(PagSeguroCredentials $credentials, $serviceName)
-    {
-
-        $this->credentials = $credentials;
-        $this->serviceName = $serviceName;
-
-        try {
-            $this->setEnvironment(PagSeguroConfig::getEnvironment());
-            $this->setWebserviceUrl(PagSeguroResources::getWebserviceUrl($this->getEnvironment()));
-            $this->setPaymentUrl(PagSeguroResources::getPaymentUrl($this->getEnvironment()));
-            $this->setBaseUrl(PagSeguroResources::getBaseUrl($this->getEnvironment()));
-            $this->setInstallmentUrl(PagSeguroResources::getInstallmentUrl());
-            $this->setAuthorizationUrl(PagSeguroResources::getAuthorizationUrl());
-            $this->setSessionUrl(PagSeguroResources::getSessionUrl());
-            $this->setCharset(PagSeguroConfig::getApplicationCharset());
-
-            $this->resources = PagSeguroResources::getData($this->serviceName);
-            if (isset($this->resources['servicePath'])) {
-                $this->setServicePath($this->resources['servicePath']);
-            }
-            if (isset($this->resources['serviceTimeout'])) {
-                $this->setServiceTimeout($this->resources['serviceTimeout']);
-            }
-        } catch (Exception $e) {
-            throw $e;
-        }
-    }
-
-    /***
+	public function __construct(PagSeguroCredentials $credentials, $serviceName)
+	{
+
+		$this->credentials = $credentials;
+		$this->serviceName = $serviceName;
+
+		try {
+			$this->setEnvironment(PagSeguroConfig::getEnvironment());
+			$this->setWebserviceUrl(PagSeguroResources::getWebserviceUrl($this->getEnvironment()));
+			$this->setPaymentUrl(PagSeguroResources::getPaymentUrl($this->getEnvironment()));
+			$this->setBaseUrl(PagSeguroResources::getBaseUrl($this->getEnvironment()));
+			$this->setInstallmentUrl(PagSeguroResources::getInstallmentUrl());
+			$this->setAuthorizationUrl(PagSeguroResources::getAuthorizationUrl());
+			$this->setSessionUrl(PagSeguroResources::getSessionUrl());
+			$this->setCharset(PagSeguroConfig::getApplicationCharset());
+
+			$this->resources = PagSeguroResources::getData($this->serviceName);
+			if (isset($this->resources['servicePath'])) {
+				$this->setServicePath($this->resources['servicePath']);
+			}
+			if (isset($this->resources['serviceTimeout'])) {
+				$this->setServiceTimeout($this->resources['serviceTimeout']);
+			}
+		} catch (Exception $e) {
+			throw $e;
+		}
+	}
+
+	/***
      * @return PagSeguroCredentials
      */
-    public function getCredentials()
-    {
-        return $this->credentials;
-    }
+	public function getCredentials()
+	{
+		return $this->credentials;
+	}
 
-    /***
+	/***
      * @param PagSeguroCredentials $credentials
      */
-    public function setCredentials(PagSeguroCredentials $credentials)
-    {
-        $this->credentials = $credentials;
-    }
+	public function setCredentials(PagSeguroCredentials $credentials)
+	{
+		$this->credentials = $credentials;
+	}
 
-    /***
+	/***
      * @return string
      */
-    public function getCredentialsUrlQuery()
-    {
-        return http_build_query($this->credentials->getAttributesMap(), '', '&');
-    }
+	public function getCredentialsUrlQuery()
+	{
+		return http_build_query($this->credentials->getAttributesMap(), '', '&');
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getEnvironment()
-    {
-        return $this->environment;
-    }
+	public function getEnvironment()
+	{
+		return $this->environment;
+	}
 
-    /***
+	/***
      * @param $environment
      */
-    public function setEnvironment($environment)
-    {
-        $this->environment = $environment;
-    }
+	public function setEnvironment($environment)
+	{
+		$this->environment = $environment;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getWebserviceUrl()
-    {
-        return $this->webserviceUrl;
-    }
+	public function getWebserviceUrl()
+	{
+		return $this->webserviceUrl;
+	}
 
-    /***
+	/***
      * @param $webserviceUrl
      */
-    public function setWebserviceUrl($webserviceUrl)
-    {
-        $this->webserviceUrl = $webserviceUrl;
-    }
+	public function setWebserviceUrl($webserviceUrl)
+	{
+		$this->webserviceUrl = $webserviceUrl;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getPaymentUrl()
-    {
-        return $this->paymentUrl;
-    }
+	public function getPaymentUrl()
+	{
+		return $this->paymentUrl;
+	}
 
-    /***
+	/***
      * @param $paymentUrl
      */
-    public function setPaymentUrl($paymentUrl)
-    {
-        $this->paymentUrl = $paymentUrl;
-    }
+	public function setPaymentUrl($paymentUrl)
+	{
+		$this->paymentUrl = $paymentUrl;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getBaseUrl()
-    {
-        return $this->baseUrl;
-    }
+	public function getBaseUrl()
+	{
+		return $this->baseUrl;
+	}
 
-    /***
+	/***
      * @param $baseUrl
      */
-    public function setBaseUrl($baseUrl)
-    {
-        $this->baseUrl = $baseUrl;
-    }
+	public function setBaseUrl($baseUrl)
+	{
+		$this->baseUrl = $baseUrl;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getInstallmentUrl()
-    {
-        return $this->installmentUrl;
-    }
+	public function getInstallmentUrl()
+	{
+		return $this->installmentUrl;
+	}
 
-    /***
+	/***
      * @param $installmentUrl
      */
-    public function setInstallmentUrl($installmentUrl)
-    {
-        $this->installmentUrl = $installmentUrl;
-    }
+	public function setInstallmentUrl($installmentUrl)
+	{
+		$this->installmentUrl = $installmentUrl;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getAuthorizationUrl()
-    {
-        return $this->authorizationUrl;
-    }
+	public function getAuthorizationUrl()
+	{
+		return $this->authorizationUrl;
+	}
 
-    /***
+	/***
      * @param $installmentUrl
      */
-    public function setAuthorizationUrl($authorizationUrl)
-    {
-        $this->authorizationUrl = $authorizationUrl;
-    }
+	public function setAuthorizationUrl($authorizationUrl)
+	{
+		$this->authorizationUrl = $authorizationUrl;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getSessionUrl()
-    {
-        return $this->sessionUrl;
-    }
+	public function getSessionUrl()
+	{
+		return $this->sessionUrl;
+	}
 
-    /***
+	/***
      * @param $installmentUrl
      */
-    public function setSessionUrl($sessionUrl)
-    {
-        $this->sessionUrl = $sessionUrl;
-    }
+	public function setSessionUrl($sessionUrl)
+	{
+		$this->sessionUrl = $sessionUrl;
+	}
 
-    /***
+	/***
      * @param mixed $version
      * @return mixed
      */
-    public function getServicePath($version = null)
-    {
-        if ($version) {
-            return $this->servicePath[$version];
-        } else {
-            return $this->servicePath;
-        }
-    }
+	public function getServicePath($version = null)
+	{
+		if ($version) {
+			return $this->servicePath[$version];
+		} else {
+			return $this->servicePath;
+		}
+	}
 
-    /***
+	/***
      * @param $servicePath
      */
-    public function setServicePath($servicePath)
-    {
-        $this->servicePath = $servicePath;
-    }
+	public function setServicePath($servicePath)
+	{
+		$this->servicePath = $servicePath;
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getServiceTimeout()
-    {
-        return $this->serviceTimeout;
-    }
+	public function getServiceTimeout()
+	{
+		return $this->serviceTimeout;
+	}
 
-    /***
+	/***
      * @param $serviceTimeout
      */
-    public function setServiceTimeout($serviceTimeout)
-    {
-        $this->serviceTimeout = $serviceTimeout;
-    }
+	public function setServiceTimeout($serviceTimeout)
+	{
+		$this->serviceTimeout = $serviceTimeout;
+	}
 
-    /***
+	/***
      * @param mixed $version
      * @return string
      */
-    public function getServiceUrl($version = null)
-    {
-        if ($version) {
-            return $this->getWebserviceUrl() . $this->getServicePath($version);
-        } else {
-            return $this->getWebserviceUrl() . $this->getServicePath();
-        }
-    }
+	public function getServiceUrl($version = null)
+	{
+		if ($version) {
+			return $this->getWebserviceUrl() . $this->getServicePath($version);
+		} else {
+			return $this->getWebserviceUrl() . $this->getServicePath();
+		}
+	}
 
-    /***
+	/***
      * @param $resource
      * @return mixed
      */
-    public function getResource($resource)
-    {
-        return $this->resources[$resource];
-    }
+	public function getResource($resource)
+	{
+		return $this->resources[$resource];
+	}
 
-    /***
+	/***
      * @return mixed
      */
-    public function getCharset()
-    {
-        return $this->charset;
-    }
+	public function getCharset()
+	{
+		return $this->charset;
+	}
 
-    /***
+	/***
      * @param $charset
      */
-    public function setCharset($charset)
-    {
-        $this->charset = $charset;
-    }
+	public function setCharset($charset)
+	{
+		$this->charset = $charset;
+	}
 
 
 }

Spacing +2 added lines, -2 removed lines

@@ -288,9 +288,9 @@
     public function getServiceUrl($version = null)
     {
         if ($version) {
-            return $this->getWebserviceUrl() . $this->getServicePath($version);
+            return $this->getWebserviceUrl().$this->getServicePath($version);
         } else {
-            return $this->getWebserviceUrl() . $this->getServicePath();
+            return $this->getWebserviceUrl().$this->getServicePath();
         }
     }
 

app/Vendor/PagSeguro/source/PagSeguroLibrary/service/PagSeguroNotificationService.class.php

Doc Comments +8 added lines

@@ -39,6 +39,10 @@ 
      * @param $notificationCode
      * @return string
      */
+
+    /**
+     * @param string $notificationCode
+     */
     private static function buildTransactionNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
     {
         $url = $connectionData->getServiceUrl();
@@ -50,6 +54,10 @@ 
      * @param $notificationCode
      * @return string
      */
+
+    /**
+     * @param string $notificationCode
+     */
     private static function buildAuthorizationNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
     {
         $url = $connectionData->getWebserviceUrl() . '/' . $connectionData->getResource('applicationPath');

Indentation +132 added lines, -132 removed lines

@@ -1,25 +1,25 @@ 
 <?php
 /**
- * 2007-2014 [PagSeguro Internet Ltda.]
- *
- * NOTICE OF LICENSE
- *
- *Licensed under the Apache License, Version 2.0 (the "License");
- *you may not use this file except in compliance with the License.
- *You may obtain a copy of the License at
- *
- *http://www.apache.org/licenses/LICENSE-2.0
- *
- *Unless required by applicable law or agreed to in writing, software
- *distributed under the License is distributed on an "AS IS" BASIS,
- *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- *See the License for the specific language governing permissions and
- *limitations under the License.
- *
- *  @author    PagSeguro Internet Ltda.
- *  @copyright 2007-2014 PagSeguro Internet Ltda.
- *  @license   http://www.apache.org/licenses/LICENSE-2.0
- */
+	 * 2007-2014 [PagSeguro Internet Ltda.]
+	 *
+	 * NOTICE OF LICENSE
+	 *
+	 *Licensed under the Apache License, Version 2.0 (the "License");
+	 *you may not use this file except in compliance with the License.
+	 *You may obtain a copy of the License at
+	 *
+	 *http://www.apache.org/licenses/LICENSE-2.0
+	 *
+	 *Unless required by applicable law or agreed to in writing, software
+	 *distributed under the License is distributed on an "AS IS" BASIS,
+	 *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+	 *See the License for the specific language governing permissions and
+	 *limitations under the License.
+	 *
+	 *  @author    PagSeguro Internet Ltda.
+	 *  @copyright 2007-2014 PagSeguro Internet Ltda.
+	 *  @license   http://www.apache.org/licenses/LICENSE-2.0
+	 */
 
 /***
  * Encapsulates web service calls regarding PagSeguro notifications
@@ -27,36 +27,36 @@ 
 class PagSeguroNotificationService
 {
 
-    /***
+	/***
      *
      */
-    const SERVICE_NAME = 'notificationService';
+	const SERVICE_NAME = 'notificationService';
 
-    private static $logService;
+	private static $logService;
 
-    /***
+	/***
      * @param PagSeguroConnectionData $connectionData
      * @param $notificationCode
      * @return string
      */
-    private static function buildTransactionNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
-    {
-        $url = $connectionData->getServiceUrl();
-        return "{$url}/{$notificationCode}/?" . $connectionData->getCredentialsUrlQuery();
-    }
+	private static function buildTransactionNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
+	{
+		$url = $connectionData->getServiceUrl();
+		return "{$url}/{$notificationCode}/?" . $connectionData->getCredentialsUrlQuery();
+	}
 
-    /***
+	/***
      * @param PagSeguroConnectionData $connectionData
      * @param $notificationCode
      * @return string
      */
-    private static function buildAuthorizationNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
-    {
-        $url = $connectionData->getWebserviceUrl() . '/' . $connectionData->getResource('applicationPath');
-        return "{$url}/{$notificationCode}/?" . $connectionData->getCredentialsUrlQuery();
-    }
+	private static function buildAuthorizationNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
+	{
+		$url = $connectionData->getWebserviceUrl() . '/' . $connectionData->getResource('applicationPath');
+		return "{$url}/{$notificationCode}/?" . $connectionData->getCredentialsUrlQuery();
+	}
 
-    /***
+	/***
      * Returns a transaction from a notification code
      *
      * @param PagSeguroCredentials $credentials
@@ -66,35 +66,35 @@ 
      * @return PagSeguroTransaction
      * @see PagSeguroTransaction
      */
-    public static function checkTransaction(PagSeguroCredentials $credentials, $notificationCode)
-    {
+	public static function checkTransaction(PagSeguroCredentials $credentials, $notificationCode)
+	{
 
-        LogPagSeguro::info("PagSeguroNotificationService.CheckTransaction(notificationCode=$notificationCode) - begin");
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+		LogPagSeguro::info("PagSeguroNotificationService.CheckTransaction(notificationCode=$notificationCode) - begin");
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
 
-        try {
+		try {
 
-            $connection = new PagSeguroHttpConnection();
-            $connection->get(
-                self::buildTransactionNotificationUrl($connectionData, $notificationCode),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
+			$connection = new PagSeguroHttpConnection();
+			$connection->get(
+				self::buildTransactionNotificationUrl($connectionData, $notificationCode),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
 
-            $transaction = PagSeguroTransactionParser::readTransaction($connection->getResponse());
+			$transaction = PagSeguroTransactionParser::readTransaction($connection->getResponse());
 
-            self::$logService = "CheckTransaction";
-            return self::searchReturn($connection, $transaction, $notificationCode);
+			self::$logService = "CheckTransaction";
+			return self::searchReturn($connection, $transaction, $notificationCode);
 
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
-    }
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		} catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
+	}
 
-    /***
+	/***
      * Returns a authorization from a notification code
      *
      * @param PagSeguroCredentials $credentials
@@ -104,79 +104,79 @@ 
      * @return PagSeguroAuthorization
      * @see PagSeguroAuthorization
      */
-    public static function checkAuthorization(PagSeguroCredentials $credentials, $notificationCode)
-    {
-
-        LogPagSeguro::info(
-            "PagSeguroNotificationService.CheckAuthorization(notificationCode=$notificationCode) - begin"
-        );
-
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
-
-        try {
-
-            $connection = new PagSeguroHttpConnection();
-            $connection->get(
-                self::buildAuthorizationNotificationUrl($connectionData, $notificationCode),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
-
-            $authorization = PagSeguroAuthorizationParser::readAuthorization($connection->getResponse());
-            self::$logService = "CheckAuthorization";
-            return self::searchReturn($connection, $authorization, $notificationCode);
-
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        }
-        catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
-    }
-
-    /**
-     * @param PagSeguroHttpConnection $connection
-     * @param string $code
-     * @return bool|mixed|string
-     * @throws PagSeguroServiceException
-     */
-    private function searchReturn($connection, $parsers, $code)
-    {
-        $httpStatus = new PagSeguroHttpStatus($connection->getStatus());
-
-        switch ($httpStatus->getType()) {
-
-            case 'OK':
-
-                LogPagSeguro::info(
-                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - end ", self::$logService) .
-                    $parsers->toString() . ")"
-                );
-                break;
-
-            case 'BAD_REQUEST':
-
-                $errors = PagSeguroServiceParser::readErrors($connection->getResponse());
-
-                $err = new PagSeguroServiceException($httpStatus, $errors);
-                LogPagSeguro::info(
-                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService) .
-                    $err->getOneLineMessage()
-                );
-                throw $err;
-                break;
-
-            default:
-                $err = new PagSeguroServiceException($httpStatus);
-                LogPagSeguro::info(
-                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService) .
-                    $err->getOneLineMessage()
-                );
-                throw $err;
-                break;
-        }
-        return isset($parsers) ? $parsers : null;
-    }
+	public static function checkAuthorization(PagSeguroCredentials $credentials, $notificationCode)
+	{
+
+		LogPagSeguro::info(
+			"PagSeguroNotificationService.CheckAuthorization(notificationCode=$notificationCode) - begin"
+		);
+
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+
+		try {
+
+			$connection = new PagSeguroHttpConnection();
+			$connection->get(
+				self::buildAuthorizationNotificationUrl($connectionData, $notificationCode),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
+
+			$authorization = PagSeguroAuthorizationParser::readAuthorization($connection->getResponse());
+			self::$logService = "CheckAuthorization";
+			return self::searchReturn($connection, $authorization, $notificationCode);
+
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		}
+		catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
+	}
+
+	/**
+	 * @param PagSeguroHttpConnection $connection
+	 * @param string $code
+	 * @return bool|mixed|string
+	 * @throws PagSeguroServiceException
+	 */
+	private function searchReturn($connection, $parsers, $code)
+	{
+		$httpStatus = new PagSeguroHttpStatus($connection->getStatus());
+
+		switch ($httpStatus->getType()) {
+
+			case 'OK':
+
+				LogPagSeguro::info(
+					sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - end ", self::$logService) .
+					$parsers->toString() . ")"
+				);
+				break;
+
+			case 'BAD_REQUEST':
+
+				$errors = PagSeguroServiceParser::readErrors($connection->getResponse());
+
+				$err = new PagSeguroServiceException($httpStatus, $errors);
+				LogPagSeguro::info(
+					sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService) .
+					$err->getOneLineMessage()
+				);
+				throw $err;
+				break;
+
+			default:
+				$err = new PagSeguroServiceException($httpStatus);
+				LogPagSeguro::info(
+					sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService) .
+					$err->getOneLineMessage()
+				);
+				throw $err;
+				break;
+		}
+		return isset($parsers) ? $parsers : null;
+	}
 
 }

Spacing +9 added lines, -9 removed lines

@@ -42,7 +42,7 @@ 
     private static function buildTransactionNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
     {
         $url = $connectionData->getServiceUrl();
-        return "{$url}/{$notificationCode}/?" . $connectionData->getCredentialsUrlQuery();
+        return "{$url}/{$notificationCode}/?".$connectionData->getCredentialsUrlQuery();
     }
 
     /***
@@ -52,8 +52,8 @@ 
      */
     private static function buildAuthorizationNotificationUrl(PagSeguroConnectionData $connectionData, $notificationCode)
     {
-        $url = $connectionData->getWebserviceUrl() . '/' . $connectionData->getResource('applicationPath');
-        return "{$url}/{$notificationCode}/?" . $connectionData->getCredentialsUrlQuery();
+        $url = $connectionData->getWebserviceUrl().'/'.$connectionData->getResource('applicationPath');
+        return "{$url}/{$notificationCode}/?".$connectionData->getCredentialsUrlQuery();
     }
 
     /***
@@ -89,7 +89,7 @@ 
         } catch (PagSeguroServiceException $err) {
             throw $err;
         } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
     }
@@ -130,7 +130,7 @@ 
             throw $err;
         }
         catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
     }
@@ -150,8 +150,8 @@ 
             case 'OK':
 
                 LogPagSeguro::info(
-                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - end ", self::$logService) .
-                    $parsers->toString() . ")"
+                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - end ", self::$logService).
+                    $parsers->toString().")"
                 );
                 break;
 
@@ -161,7 +161,7 @@ 
 
                 $err = new PagSeguroServiceException($httpStatus, $errors);
                 LogPagSeguro::info(
-                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService) .
+                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService).
                     $err->getOneLineMessage()
                 );
                 throw $err;
@@ -170,7 +170,7 @@ 
             default:
                 $err = new PagSeguroServiceException($httpStatus);
                 LogPagSeguro::info(
-                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService) .
+                    sprintf("PagSeguroNotificationService.%s(notificationCode=$code) - error ", self::$logService).
                     $err->getOneLineMessage()
                 );
                 throw $err;

Braces +1 added lines, -2 removed lines

@@ -128,8 +128,7 @@
 
         } catch (PagSeguroServiceException $err) {
             throw $err;
-        }
-        catch (Exception $err) {
+        } catch (Exception $err) {
             LogPagSeguro::error("Exception: " . $err->getMessage());
             throw $err;
         }

app/Vendor/PagSeguro/source/PagSeguroLibrary/service/PagSeguroPaymentService.class.php

Doc Comments +4 added lines

@@ -61,6 +61,10 @@
      * @throws Exception|PagSeguroServiceException
      * @throws Exception
      */
+
+    /**
+     * @param boolean $onlyCheckoutCode
+     */
     public static function createCheckoutRequest(
         PagSeguroCredentials $credentials,
         PagSeguroPaymentRequest $paymentRequest,

Indentation +86 added lines, -86 removed lines

@@ -27,107 +27,107 @@
 class PagSeguroPaymentService
 {
 
-    /***
+	/***
      *
      */
-    const SERVICE_NAME = 'paymentService';
+	const SERVICE_NAME = 'paymentService';
 
-    /***
+	/***
      * @param PagSeguroConnectionData $connectionData
      * @return string
      */
-    private static function buildCheckoutRequestUrl(PagSeguroConnectionData $connectionData)
-    {
-        return $connectionData->getServiceUrl() . '/?' . $connectionData->getCredentialsUrlQuery();
-    }
+	private static function buildCheckoutRequestUrl(PagSeguroConnectionData $connectionData)
+	{
+		return $connectionData->getServiceUrl() . '/?' . $connectionData->getCredentialsUrlQuery();
+	}
 
-    /***
+	/***
      * @param PagSeguroConnectionData $connectionData
      * @param $code
      * @return string
      */
-    private static function buildCheckoutUrl(PagSeguroConnectionData $connectionData, $code)
-    {
-        return $connectionData->getPaymentUrl() . $connectionData->getResource('checkoutUrl') . "?code=$code";
-    }
-
-    // createCheckoutRequest is the actual implementation of the Register method
-    // This separation serves as test hook to validate the Uri
-    // against the code returned by the service
-    /***
+	private static function buildCheckoutUrl(PagSeguroConnectionData $connectionData, $code)
+	{
+		return $connectionData->getPaymentUrl() . $connectionData->getResource('checkoutUrl') . "?code=$code";
+	}
+
+	// createCheckoutRequest is the actual implementation of the Register method
+	// This separation serves as test hook to validate the Uri
+	// against the code returned by the service
+	/***
      * @param PagSeguroCredentials $credentials
      * @param PagSeguroPaymentRequest $paymentRequest
      * @return bool|string
      * @throws Exception|PagSeguroServiceException
      * @throws Exception
      */
-    public static function createCheckoutRequest(
-        PagSeguroCredentials $credentials,
-        PagSeguroPaymentRequest $paymentRequest,
-        $onlyCheckoutCode
-    ) {
-
-        LogPagSeguro::info("PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - begin");
-
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
-
-        try {
-
-            $connection = new PagSeguroHttpConnection();
-            $connection->post(
-                self::buildCheckoutRequestUrl($connectionData),
-                PagSeguroPaymentParser::getData($paymentRequest),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
-
-            $httpStatus = new PagSeguroHttpStatus($connection->getStatus());
-
-            switch ($httpStatus->getType()) {
-
-                case 'OK':
-                    $PaymentParserData = PagSeguroPaymentParser::readSuccessXml($connection->getResponse());
-
-                    if ($onlyCheckoutCode) {
-                        $paymentReturn = $PaymentParserData->getCode();
-                    } else {
-                        $paymentReturn = self::buildCheckoutUrl($connectionData, $PaymentParserData->getCode());
-                    }
-                    LogPagSeguro::info(
-                        "PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - end {1}" .
-                        $PaymentParserData->getCode()
-                    );
-                    break;
-
-                case 'BAD_REQUEST':
-                    $errors = PagSeguroPaymentParser::readErrors($connection->getResponse());
-                    $e = new PagSeguroServiceException($httpStatus, $errors);
-                    LogPagSeguro::error(
-                        "PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - error " .
-                        $e->getOneLineMessage()
-                    );
-                    throw $e;
-                    break;
-
-                default:
-                    $e = new PagSeguroServiceException($httpStatus);
-                    LogPagSeguro::error(
-                        "PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - error " .
-                        $e->getOneLineMessage()
-                    );
-                    throw $e;
-                    break;
-
-            }
-            return (isset($paymentReturn) ? $paymentReturn : false);
-
-        } catch (PagSeguroServiceException $e) {
-            throw $e;
-        }
-        catch (Exception $e) {
-            LogPagSeguro::error("Exception: " . $e->getMessage());
-            throw $e;
-        }
-
-    }
+	public static function createCheckoutRequest(
+		PagSeguroCredentials $credentials,
+		PagSeguroPaymentRequest $paymentRequest,
+		$onlyCheckoutCode
+	) {
+
+		LogPagSeguro::info("PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - begin");
+
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+
+		try {
+
+			$connection = new PagSeguroHttpConnection();
+			$connection->post(
+				self::buildCheckoutRequestUrl($connectionData),
+				PagSeguroPaymentParser::getData($paymentRequest),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
+
+			$httpStatus = new PagSeguroHttpStatus($connection->getStatus());
+
+			switch ($httpStatus->getType()) {
+
+				case 'OK':
+					$PaymentParserData = PagSeguroPaymentParser::readSuccessXml($connection->getResponse());
+
+					if ($onlyCheckoutCode) {
+						$paymentReturn = $PaymentParserData->getCode();
+					} else {
+						$paymentReturn = self::buildCheckoutUrl($connectionData, $PaymentParserData->getCode());
+					}
+					LogPagSeguro::info(
+						"PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - end {1}" .
+						$PaymentParserData->getCode()
+					);
+					break;
+
+				case 'BAD_REQUEST':
+					$errors = PagSeguroPaymentParser::readErrors($connection->getResponse());
+					$e = new PagSeguroServiceException($httpStatus, $errors);
+					LogPagSeguro::error(
+						"PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - error " .
+						$e->getOneLineMessage()
+					);
+					throw $e;
+					break;
+
+				default:
+					$e = new PagSeguroServiceException($httpStatus);
+					LogPagSeguro::error(
+						"PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - error " .
+						$e->getOneLineMessage()
+					);
+					throw $e;
+					break;
+
+			}
+			return (isset($paymentReturn) ? $paymentReturn : false);
+
+		} catch (PagSeguroServiceException $e) {
+			throw $e;
+		}
+		catch (Exception $e) {
+			LogPagSeguro::error("Exception: " . $e->getMessage());
+			throw $e;
+		}
+
+	}
 }

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
      */
     private static function buildCheckoutRequestUrl(PagSeguroConnectionData $connectionData)
     {
-        return $connectionData->getServiceUrl() . '/?' . $connectionData->getCredentialsUrlQuery();
+        return $connectionData->getServiceUrl().'/?'.$connectionData->getCredentialsUrlQuery();
     }
 
     /***
@@ -48,7 +48,7 @@ 
      */
     private static function buildCheckoutUrl(PagSeguroConnectionData $connectionData, $code)
     {
-        return $connectionData->getPaymentUrl() . $connectionData->getResource('checkoutUrl') . "?code=$code";
+        return $connectionData->getPaymentUrl().$connectionData->getResource('checkoutUrl')."?code=$code";
     }
 
     // createCheckoutRequest is the actual implementation of the Register method
@@ -67,7 +67,7 @@ 
         $onlyCheckoutCode
     ) {
 
-        LogPagSeguro::info("PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - begin");
+        LogPagSeguro::info("PagSeguroPaymentService.Register(".$paymentRequest->toString().") - begin");
 
         $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
 
@@ -94,7 +94,7 @@ 
                         $paymentReturn = self::buildCheckoutUrl($connectionData, $PaymentParserData->getCode());
                     }
                     LogPagSeguro::info(
-                        "PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - end {1}" .
+                        "PagSeguroPaymentService.Register(".$paymentRequest->toString().") - end {1}".
                         $PaymentParserData->getCode()
                     );
                     break;
@@ -103,7 +103,7 @@ 
                     $errors = PagSeguroPaymentParser::readErrors($connection->getResponse());
                     $e = new PagSeguroServiceException($httpStatus, $errors);
                     LogPagSeguro::error(
-                        "PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - error " .
+                        "PagSeguroPaymentService.Register(".$paymentRequest->toString().") - error ".
                         $e->getOneLineMessage()
                     );
                     throw $e;
@@ -112,7 +112,7 @@ 
                 default:
                     $e = new PagSeguroServiceException($httpStatus);
                     LogPagSeguro::error(
-                        "PagSeguroPaymentService.Register(" . $paymentRequest->toString() . ") - error " .
+                        "PagSeguroPaymentService.Register(".$paymentRequest->toString().") - error ".
                         $e->getOneLineMessage()
                     );
                     throw $e;
@@ -125,7 +125,7 @@ 
             throw $e;
         }
         catch (Exception $e) {
-            LogPagSeguro::error("Exception: " . $e->getMessage());
+            LogPagSeguro::error("Exception: ".$e->getMessage());
             throw $e;
         }
 

Braces +1 added lines, -2 removed lines

@@ -123,8 +123,7 @@
 
         } catch (PagSeguroServiceException $e) {
             throw $e;
-        }
-        catch (Exception $e) {
+        } catch (Exception $e) {
             LogPagSeguro::error("Exception: " . $e->getMessage());
             throw $e;
         }

app/Vendor/PagSeguro/source/PagSeguroLibrary/service/PagSeguroRefundService.class.php

Doc Comments +7 added lines

@@ -29,6 +29,9 @@ 
 
     const SERVICE_NAME = 'refundService';
 
+    /**
+     * @param PagSeguroConnectionData $connectionData
+     */
     private static function buildRefundURL($connectionData, $transactionCode, $refundValue = null)
     {
         if (is_null($refundValue)) {
@@ -40,6 +43,10 @@ 
         }
     }
 
+    /**
+     * @param string $transactionCode
+     * @param string $refundValue
+     */
     public static function createRefundRequest(
         PagSeguroCredentials $credentials,
         $transactionCode,

Indentation +85 added lines, -85 removed lines

@@ -1,25 +1,25 @@ 
 <?php
 /**
- * 2007-2014 [PagSeguro Internet Ltda.]
- *
- * NOTICE OF LICENSE
- *
- *Licensed under the Apache License, Version 2.0 (the "License");
- *you may not use this file except in compliance with the License.
- *You may obtain a copy of the License at
- *
- *http://www.apache.org/licenses/LICENSE-2.0
- *
- *Unless required by applicable law or agreed to in writing, software
- *distributed under the License is distributed on an "AS IS" BASIS,
- *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- *See the License for the specific language governing permissions and
- *limitations under the License.
- *
- *  @author    PagSeguro Internet Ltda.
- *  @copyright 2007-2014 PagSeguro Internet Ltda.
- *  @license   http://www.apache.org/licenses/LICENSE-2.0
- */
+	 * 2007-2014 [PagSeguro Internet Ltda.]
+	 *
+	 * NOTICE OF LICENSE
+	 *
+	 *Licensed under the Apache License, Version 2.0 (the "License");
+	 *you may not use this file except in compliance with the License.
+	 *You may obtain a copy of the License at
+	 *
+	 *http://www.apache.org/licenses/LICENSE-2.0
+	 *
+	 *Unless required by applicable law or agreed to in writing, software
+	 *distributed under the License is distributed on an "AS IS" BASIS,
+	 *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+	 *See the License for the specific language governing permissions and
+	 *limitations under the License.
+	 *
+	 *  @author    PagSeguro Internet Ltda.
+	 *  @copyright 2007-2014 PagSeguro Internet Ltda.
+	 *  @license   http://www.apache.org/licenses/LICENSE-2.0
+	 */
 
 /***
  * Encapsulates web service calls regarding PagSeguro refund requests
@@ -27,79 +27,79 @@ 
 class PagSeguroRefundService
 {
 
-    const SERVICE_NAME = 'refundService';
+	const SERVICE_NAME = 'refundService';
 
-    private static function buildRefundURL($connectionData, $transactionCode, $refundValue = null)
-    {
-        if (is_null($refundValue)) {
-            return  $connectionData->getServiceUrl() . '?' . $connectionData->getCredentialsUrlQuery()
-                    . "&transactionCode=" . $transactionCode;
-        } else {
-            return  $connectionData->getServiceUrl() . '?' . $connectionData->getCredentialsUrlQuery()
-                    . "&transactionCode=" . $transactionCode . "&refundValue=" . $refundValue;
-        }
-    }
+	private static function buildRefundURL($connectionData, $transactionCode, $refundValue = null)
+	{
+		if (is_null($refundValue)) {
+			return  $connectionData->getServiceUrl() . '?' . $connectionData->getCredentialsUrlQuery()
+					. "&transactionCode=" . $transactionCode;
+		} else {
+			return  $connectionData->getServiceUrl() . '?' . $connectionData->getCredentialsUrlQuery()
+					. "&transactionCode=" . $transactionCode . "&refundValue=" . $refundValue;
+		}
+	}
 
-    public static function createRefundRequest(
-        PagSeguroCredentials $credentials,
-        $transactionCode,
-        $refundValue = null
-    ){
+	public static function createRefundRequest(
+		PagSeguroCredentials $credentials,
+		$transactionCode,
+		$refundValue = null
+	){
 
-        LogPagSeguro::info("PagSeguroRefundService.Register(".$transactionCode.") - begin");
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+		LogPagSeguro::info("PagSeguroRefundService.Register(".$transactionCode.") - begin");
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
 
-        if (is_null($refundValue)) {
-            $url = self::buildRefundURL($connectionData, $transactionCode);
-        } else {
-            $url = self::buildRefundURL($connectionData, $transactionCode, $refundValue);
-        }
+		if (is_null($refundValue)) {
+			$url = self::buildRefundURL($connectionData, $transactionCode);
+		} else {
+			$url = self::buildRefundURL($connectionData, $transactionCode, $refundValue);
+		}
 
-        try {
+		try {
 
-           $connection = new PagSeguroHttpConnection();
-           $connection->post(
-                $url,
-                array(),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-           );
+		   $connection = new PagSeguroHttpConnection();
+		   $connection->post(
+				$url,
+				array(),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+		   );
 
-            $httpStatus = new PagSeguroHttpStatus($connection->getStatus());
+			$httpStatus = new PagSeguroHttpStatus($connection->getStatus());
 
-            switch ($httpStatus->getType()) {
-                case 'OK':
+			switch ($httpStatus->getType()) {
+				case 'OK':
 
-                    $result = PagSeguroRefundParser::readSuccessXml($connection->getResponse());
-                    LogPagSeguro::info(
-                        "PagSeguroRefundService.createRefundRequest(".$result.") - end "
-                    );
-                    break;
-                case 'BAD_REQUEST':
-                    $errors = PagSeguroRefundParser::readErrors($connection->getResponse());
-                    $err = new PagSeguroServiceException($httpStatus, $errors);
-                    LogPagSeguro::error(
-                        "PagSeguroRefundService.createRefundRequest() - error " .
-                        $err->getOneLineMessage()
-                    );
-                    throw $err;
-                    break;
-                default:
-                    $err = new PagSeguroServiceException($httpStatus);
-                    LogPagSeguro::error(
-                        "PagSeguroRefundService.createRefundRequest() - error " .
-                        $err->getOneLineMessage()
-                    );
-                    throw $err;
-                    break;
-            }
-            return isset($result) ? $result : false;
+					$result = PagSeguroRefundParser::readSuccessXml($connection->getResponse());
+					LogPagSeguro::info(
+						"PagSeguroRefundService.createRefundRequest(".$result.") - end "
+					);
+					break;
+				case 'BAD_REQUEST':
+					$errors = PagSeguroRefundParser::readErrors($connection->getResponse());
+					$err = new PagSeguroServiceException($httpStatus, $errors);
+					LogPagSeguro::error(
+						"PagSeguroRefundService.createRefundRequest() - error " .
+						$err->getOneLineMessage()
+					);
+					throw $err;
+					break;
+				default:
+					$err = new PagSeguroServiceException($httpStatus);
+					LogPagSeguro::error(
+						"PagSeguroRefundService.createRefundRequest() - error " .
+						$err->getOneLineMessage()
+					);
+					throw $err;
+					break;
+			}
+			return isset($result) ? $result : false;
 
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
-    }
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		} catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
+	}
 }
\ No newline at end of file

Spacing +8 added lines, -8 removed lines

@@ -32,11 +32,11 @@ 
     private static function buildRefundURL($connectionData, $transactionCode, $refundValue = null)
     {
         if (is_null($refundValue)) {
-            return  $connectionData->getServiceUrl() . '?' . $connectionData->getCredentialsUrlQuery()
-                    . "&transactionCode=" . $transactionCode;
+            return  $connectionData->getServiceUrl().'?'.$connectionData->getCredentialsUrlQuery()
+                    . "&transactionCode=".$transactionCode;
         } else {
-            return  $connectionData->getServiceUrl() . '?' . $connectionData->getCredentialsUrlQuery()
-                    . "&transactionCode=" . $transactionCode . "&refundValue=" . $refundValue;
+            return  $connectionData->getServiceUrl().'?'.$connectionData->getCredentialsUrlQuery()
+                    . "&transactionCode=".$transactionCode."&refundValue=".$refundValue;
         }
     }
 
@@ -44,7 +44,7 @@ 
         PagSeguroCredentials $credentials,
         $transactionCode,
         $refundValue = null
-    ){
+    ) {
 
         LogPagSeguro::info("PagSeguroRefundService.Register(".$transactionCode.") - begin");
         $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
@@ -79,7 +79,7 @@ 
                     $errors = PagSeguroRefundParser::readErrors($connection->getResponse());
                     $err = new PagSeguroServiceException($httpStatus, $errors);
                     LogPagSeguro::error(
-                        "PagSeguroRefundService.createRefundRequest() - error " .
+                        "PagSeguroRefundService.createRefundRequest() - error ".
                         $err->getOneLineMessage()
                     );
                     throw $err;
@@ -87,7 +87,7 @@ 
                 default:
                     $err = new PagSeguroServiceException($httpStatus);
                     LogPagSeguro::error(
-                        "PagSeguroRefundService.createRefundRequest() - error " .
+                        "PagSeguroRefundService.createRefundRequest() - error ".
                         $err->getOneLineMessage()
                     );
                     throw $err;
@@ -98,7 +98,7 @@ 
         } catch (PagSeguroServiceException $err) {
             throw $err;
         } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
     }

app/Vendor/PagSeguro/source/PagSeguroLibrary/service/PagSeguroSessionService.class.php

Doc Comments +4 added lines

@@ -44,6 +44,10 @@
      * @throws Exception|PagSeguroServiceException
      * @throws Exception
      */
+
+    /**
+     * @param PagSeguroAccountCredentials $credentials
+     */
     public static function getSession($credentials)
     {
         $connectionData = new PagSeguroConnectionData($credentials, 'sessionService');

Indentation +72 added lines, -72 removed lines

@@ -1,25 +1,25 @@ 
 <?php
 /**
- * 2007-2014 [PagSeguro Internet Ltda.]
- *
- * NOTICE OF LICENSE
- *
- *Licensed under the Apache License, Version 2.0 (the "License");
- *you may not use this file except in compliance with the License.
- *You may obtain a copy of the License at
- *
- *http://www.apache.org/licenses/LICENSE-2.0
- *
- *Unless required by applicable law or agreed to in writing, software
- *distributed under the License is distributed on an "AS IS" BASIS,
- *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- *See the License for the specific language governing permissions and
- *limitations under the License.
- *
- *  @author    PagSeguro Internet Ltda.
- *  @copyright 2007-2014 PagSeguro Internet Ltda.
- *  @license   http://www.apache.org/licenses/LICENSE-2.0
- */
+	 * 2007-2014 [PagSeguro Internet Ltda.]
+	 *
+	 * NOTICE OF LICENSE
+	 *
+	 *Licensed under the Apache License, Version 2.0 (the "License");
+	 *you may not use this file except in compliance with the License.
+	 *You may obtain a copy of the License at
+	 *
+	 *http://www.apache.org/licenses/LICENSE-2.0
+	 *
+	 *Unless required by applicable law or agreed to in writing, software
+	 *distributed under the License is distributed on an "AS IS" BASIS,
+	 *WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+	 *See the License for the specific language governing permissions and
+	 *limitations under the License.
+	 *
+	 *  @author    PagSeguro Internet Ltda.
+	 *  @copyright 2007-2014 PagSeguro Internet Ltda.
+	 *  @license   http://www.apache.org/licenses/LICENSE-2.0
+	 */
 
 /***
  * Encapsulates web service calls regarding PagSeguro session requests
@@ -27,80 +27,80 @@ 
 class PagSeguroSessionService
 {
 
-    /***
+	/***
      * Build URL for get session.
      * @param PagSeguroConnectionData $connectionData 
      * @return string session URL
      */
-    private static function buildSessionURL($connectionData)
-    {
-        return  $connectionData->getWebserviceUrl() . $connectionData->getSessionUrl();  
-    }
+	private static function buildSessionURL($connectionData)
+	{
+		return  $connectionData->getWebserviceUrl() . $connectionData->getSessionUrl();  
+	}
 
-    /***
+	/***
      * Get session for direct payment from webservice 
      * @param $credentials PagSeguroAccountCredentials 
      * @return bool|string
      * @throws Exception|PagSeguroServiceException
      * @throws Exception
      */
-    public static function getSession($credentials)
-    {
-        $connectionData = new PagSeguroConnectionData($credentials, 'sessionService');
+	public static function getSession($credentials)
+	{
+		$connectionData = new PagSeguroConnectionData($credentials, 'sessionService');
 
-        $url = self::buildSessionURL($connectionData)  . "?" . $connectionData->getCredentialsUrlQuery();
+		$url = self::buildSessionURL($connectionData)  . "?" . $connectionData->getCredentialsUrlQuery();
 
-        try {
+		try {
             
-            $connection = new PagSeguroHttpConnection();
-            $connection->post(
-                $url,
-                array(),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-             );
+			$connection = new PagSeguroHttpConnection();
+			$connection->post(
+				$url,
+				array(),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			 );
 
-            $httpStatus = new PagSeguroHttpStatus($connection->getStatus());
+			$httpStatus = new PagSeguroHttpStatus($connection->getStatus());
 
-            switch ($httpStatus->getType()) {
+			switch ($httpStatus->getType()) {
 
-                case 'OK':
+				case 'OK':
 
-                    $session = PagSeguroSessionParser::readResult($connection->getResponse());
+					$session = PagSeguroSessionParser::readResult($connection->getResponse());
 
-                    return $session->getId();
+					return $session->getId();
 
-                    LogPagSeguro::info(
-                        "PagSeguroSessionService.getSession()(" . $session->toString() . ") - end {1}"
-                    );
-                    break;
+					LogPagSeguro::info(
+						"PagSeguroSessionService.getSession()(" . $session->toString() . ") - end {1}"
+					);
+					break;
 
-                case 'BAD_REQUEST':
-                    $errors = PagSeguroSessionParser::readErrors($connection->getStatus());
-                    $e = new PagSeguroServiceException($httpStatus, $errors);
-                    LogPagSeguro::error(
-                        "PagSeguroSessionService.getSession() - error " . 
-                        $e->getOneLineMessage()
-                    );
-                    throw $e;
-                    break;
+				case 'BAD_REQUEST':
+					$errors = PagSeguroSessionParser::readErrors($connection->getStatus());
+					$e = new PagSeguroServiceException($httpStatus, $errors);
+					LogPagSeguro::error(
+						"PagSeguroSessionService.getSession() - error " . 
+						$e->getOneLineMessage()
+					);
+					throw $e;
+					break;
 
-                default:
+				default:
 
-                    $e = new PagSeguroServiceException($httpStatus);
-                    LogPagSeguro::error(
-                        "PagSeguroSessionService.getSession() - error " . 
-                        $e->getOneLineMessage() 
-                    );
-                    throw $e;
-                    break;
-            }
-        } catch (PagSeguroServiceException $e) {
-            throw $e;
-        } catch (Exception $e) {
-            LogPagSeguro::error("Exception: " . $e->getMessage());
-            throw $e;
-        }
-    }
+					$e = new PagSeguroServiceException($httpStatus);
+					LogPagSeguro::error(
+						"PagSeguroSessionService.getSession() - error " . 
+						$e->getOneLineMessage() 
+					);
+					throw $e;
+					break;
+			}
+		} catch (PagSeguroServiceException $e) {
+			throw $e;
+		} catch (Exception $e) {
+			LogPagSeguro::error("Exception: " . $e->getMessage());
+			throw $e;
+		}
+	}
 
 }

Spacing +6 added lines, -6 removed lines

@@ -34,7 +34,7 @@ 
      */
     private static function buildSessionURL($connectionData)
     {
-        return  $connectionData->getWebserviceUrl() . $connectionData->getSessionUrl();  
+        return  $connectionData->getWebserviceUrl().$connectionData->getSessionUrl();  
     }
 
     /***
@@ -48,7 +48,7 @@ 
     {
         $connectionData = new PagSeguroConnectionData($credentials, 'sessionService');
 
-        $url = self::buildSessionURL($connectionData)  . "?" . $connectionData->getCredentialsUrlQuery();
+        $url = self::buildSessionURL($connectionData)."?".$connectionData->getCredentialsUrlQuery();
 
         try {
             
@@ -71,7 +71,7 @@ 
                     return $session->getId();
 
                     LogPagSeguro::info(
-                        "PagSeguroSessionService.getSession()(" . $session->toString() . ") - end {1}"
+                        "PagSeguroSessionService.getSession()(".$session->toString().") - end {1}"
                     );
                     break;
 
@@ -79,7 +79,7 @@ 
                     $errors = PagSeguroSessionParser::readErrors($connection->getStatus());
                     $e = new PagSeguroServiceException($httpStatus, $errors);
                     LogPagSeguro::error(
-                        "PagSeguroSessionService.getSession() - error " . 
+                        "PagSeguroSessionService.getSession() - error ". 
                         $e->getOneLineMessage()
                     );
                     throw $e;
@@ -89,7 +89,7 @@ 
 
                     $e = new PagSeguroServiceException($httpStatus);
                     LogPagSeguro::error(
-                        "PagSeguroSessionService.getSession() - error " . 
+                        "PagSeguroSessionService.getSession() - error ". 
                         $e->getOneLineMessage() 
                     );
                     throw $e;
@@ -98,7 +98,7 @@ 
         } catch (PagSeguroServiceException $e) {
             throw $e;
         } catch (Exception $e) {
-            LogPagSeguro::error("Exception: " . $e->getMessage());
+            LogPagSeguro::error("Exception: ".$e->getMessage());
             throw $e;
         }
     }

app/Vendor/PagSeguro/source/PagSeguroLibrary/service/PagSeguroTransactionSearchService.class.php

Doc Comments +15 added lines, -15 removed lines

@@ -36,7 +36,7 @@ 
 
     /**
      * @param PagSeguroConnectionData $connectionData
-     * @param $transactionCode
+     * @param string $transactionCode
      * @return string
      */
     private static function buildSearchUrlByCode(PagSeguroConnectionData $connectionData, $transactionCode)
@@ -260,11 +260,11 @@ 
 
     /**
      * @param PagSeguroCredentials $credentials
-     * @param $reference
-     * @param null $initialDate
-     * @param null $finalDate
-     * @param null $pageNumber
-     * @param null $maxPageResults
+     * @param string $reference
+     * @param string $initialDate
+     * @param string $finalDate
+     * @param integer $pageNumber
+     * @param integer $maxPageResults
      * @throws Exception
      * @throws PagSeguroServiceException
      */
@@ -314,10 +314,10 @@ 
     }
 
     /**
-     * @param $pageNumber
-     * @param $maxPageResults
-     * @param $initialDate
-     * @param null $finalDate
+     * @param integer|null $pageNumber
+     * @param integer|null $maxPageResults
+     * @param string $initialDate
+     * @param string|null $finalDate
      * @return array
      */
     private function buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate = null)
@@ -334,8 +334,8 @@ 
     }
 
     /**
-     * @param $connection
-     * @param $code
+     * @param PagSeguroHttpConnection $connection
+     * @param string $code
      * @return bool|PagSeguroTransaction
      * @throws PagSeguroServiceException
      */
@@ -376,9 +376,9 @@ 
     }
 
     /**
-     * @param $connection
-     * @param null $initialDate
-     * @param null $finalDate
+     * @param PagSeguroHttpConnection $connection
+     * @param string $initialDate
+     * @param string|null $finalDate
      * @return bool|PagSeguroTransactionSearchResult
      * @throws PagSeguroServiceException
      */

Indentation +348 added lines, -348 removed lines

@@ -27,96 +27,96 @@ 
 class PagSeguroTransactionSearchService
 {
 
-    /**
-     *
-     */
-    const SERVICE_NAME = 'transactionSearchService';
-
-    private static $logService;
-
-    /**
-     * @param PagSeguroConnectionData $connectionData
-     * @param $transactionCode
-     * @return string
-     */
-    private static function buildSearchUrlByCode(PagSeguroConnectionData $connectionData, $transactionCode)
-    {
-        $url = $connectionData->getServiceUrl('v3');
-        return "{$url}/{$transactionCode}/?" . $connectionData->getCredentialsUrlQuery();
-    }
-
-    /**
-     * @param PagSeguroConnectionData $connectionData
-     * @param array $searchParams
-     * @return string
-     */
-    private static function buildSearchUrlByDate(PagSeguroConnectionData $connectionData, array $searchParams)
-    {
-        $url = $connectionData->getServiceUrl('v2');
-        $initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
-        $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
-        if ($searchParams['pageNumber'] != null) {
-            $page = "&page=" . $searchParams['pageNumber'];
-        }
-        if ($searchParams['maxPageResults'] != null) {
-            $maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
-        }
-        return "{$url}/?" . $connectionData->getCredentialsUrlQuery() .
-            "&initialDate={$initialDate}{$finalDate}{$page}{$maxPageResults}";
-    }
-
-    /**
-     * @param PagSeguroConnectionData $connectionData
-     * @param array $searchParams
-     * @return string
-     */
-    private static function buildSearchUrlAbandoned(PagSeguroConnectionData $connectionData, array $searchParams)
-    {
-        $url = $connectionData->getServiceUrl('v2');
-
-        $initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
-        $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
-        if ($searchParams['pageNumber'] != null) {
-            $page = "&page=" . $searchParams['pageNumber'];
-        }
-        if ($searchParams['maxPageResults'] != null) {
-            $maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
-        }
-        return "{$url}/abandoned/?" . $connectionData->getCredentialsUrlQuery() .
-            "&initialDate={$initialDate}&finalDate={$finalDate}{$page}{$maxPageResults}";
-    }
-
-    /**
-     * @param PagSeguroConnectionData $connectionData
-     * @param $reference
-     * @param null $searchParams
-     * @return string
-     */
-    private static function buildSearchUrlByReference(
-        PagSeguroConnectionData $connectionData,
-        $reference,
-        $searchParams = null
-    ){
-        $url = $connectionData->getServiceUrl('v2');
-        if ($searchParams == null) {
-            return "{$url}?" . $connectionData->getCredentialsUrlQuery() . "&reference=" . $reference;
-        } else {
-
-            $initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
-            $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
-            if ($searchParams['pageNumber'] != null) {
-                $page = "&page=" . $searchParams['pageNumber'];
-            }
-            if ($searchParams['maxPageResults'] != null) {
-                $maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
-            }
-
-            return "{$url}?" . $connectionData->getCredentialsUrlQuery() . "&reference=" . $reference
-                   . "&initialDate={$initialDate}&finalDate={$finalDate}{$page}{$maxPageResults}";
-        }
-    }
-
-    /***
+	/**
+	 *
+	 */
+	const SERVICE_NAME = 'transactionSearchService';
+
+	private static $logService;
+
+	/**
+	 * @param PagSeguroConnectionData $connectionData
+	 * @param $transactionCode
+	 * @return string
+	 */
+	private static function buildSearchUrlByCode(PagSeguroConnectionData $connectionData, $transactionCode)
+	{
+		$url = $connectionData->getServiceUrl('v3');
+		return "{$url}/{$transactionCode}/?" . $connectionData->getCredentialsUrlQuery();
+	}
+
+	/**
+	 * @param PagSeguroConnectionData $connectionData
+	 * @param array $searchParams
+	 * @return string
+	 */
+	private static function buildSearchUrlByDate(PagSeguroConnectionData $connectionData, array $searchParams)
+	{
+		$url = $connectionData->getServiceUrl('v2');
+		$initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
+		$finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
+		if ($searchParams['pageNumber'] != null) {
+			$page = "&page=" . $searchParams['pageNumber'];
+		}
+		if ($searchParams['maxPageResults'] != null) {
+			$maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
+		}
+		return "{$url}/?" . $connectionData->getCredentialsUrlQuery() .
+			"&initialDate={$initialDate}{$finalDate}{$page}{$maxPageResults}";
+	}
+
+	/**
+	 * @param PagSeguroConnectionData $connectionData
+	 * @param array $searchParams
+	 * @return string
+	 */
+	private static function buildSearchUrlAbandoned(PagSeguroConnectionData $connectionData, array $searchParams)
+	{
+		$url = $connectionData->getServiceUrl('v2');
+
+		$initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
+		$finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
+		if ($searchParams['pageNumber'] != null) {
+			$page = "&page=" . $searchParams['pageNumber'];
+		}
+		if ($searchParams['maxPageResults'] != null) {
+			$maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
+		}
+		return "{$url}/abandoned/?" . $connectionData->getCredentialsUrlQuery() .
+			"&initialDate={$initialDate}&finalDate={$finalDate}{$page}{$maxPageResults}";
+	}
+
+	/**
+	 * @param PagSeguroConnectionData $connectionData
+	 * @param $reference
+	 * @param null $searchParams
+	 * @return string
+	 */
+	private static function buildSearchUrlByReference(
+		PagSeguroConnectionData $connectionData,
+		$reference,
+		$searchParams = null
+	){
+		$url = $connectionData->getServiceUrl('v2');
+		if ($searchParams == null) {
+			return "{$url}?" . $connectionData->getCredentialsUrlQuery() . "&reference=" . $reference;
+		} else {
+
+			$initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
+			$finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
+			if ($searchParams['pageNumber'] != null) {
+				$page = "&page=" . $searchParams['pageNumber'];
+			}
+			if ($searchParams['maxPageResults'] != null) {
+				$maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
+			}
+
+			return "{$url}?" . $connectionData->getCredentialsUrlQuery() . "&reference=" . $reference
+				   . "&initialDate={$initialDate}&finalDate={$finalDate}{$page}{$maxPageResults}";
+		}
+	}
+
+	/***
      * Finds a transaction with a matching transaction code
      *
      * @param PagSeguroCredentials $credentials
@@ -126,35 +126,35 @@ 
      * @throws PagSeguroServiceException
      * @throws Exception
      */
-    public static function searchByCode(PagSeguroCredentials $credentials, $transactionCode)
-    {
+	public static function searchByCode(PagSeguroCredentials $credentials, $transactionCode)
+	{
 
-        LogPagSeguro::info("PagSeguroTransactionSearchService.SearchByCode($transactionCode) - begin");
+		LogPagSeguro::info("PagSeguroTransactionSearchService.SearchByCode($transactionCode) - begin");
 
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
 
-        try {
+		try {
 
-            $connection = new PagSeguroHttpConnection();
-            $connection->get(
-                self::buildSearchUrlByCode($connectionData, $transactionCode),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
+			$connection = new PagSeguroHttpConnection();
+			$connection->get(
+				self::buildSearchUrlByCode($connectionData, $transactionCode),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
 
-            return self::searchByCodeResult($connection, $transactionCode);
+			return self::searchByCodeResult($connection, $transactionCode);
 
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        }
-        catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		}
+		catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
 
-    }
+	}
 
-    /***
+	/***
  * Search transactions associated with this set of credentials within a date range
  *
  * @param PagSeguroCredentials $credentials
@@ -167,45 +167,45 @@ 
  * @throws PagSeguroServiceException
  * @throws Exception
  */
-    public static function searchByDate(
-        PagSeguroCredentials $credentials,
-        $pageNumber,
-        $maxPageResults,
-        $initialDate,
-        $finalDate = null
-    ) {
+	public static function searchByDate(
+		PagSeguroCredentials $credentials,
+		$pageNumber,
+		$maxPageResults,
+		$initialDate,
+		$finalDate = null
+	) {
 
-        LogPagSeguro::info(
-            "PagSeguroTransactionSearchService.SearchByDate(initialDate=" . PagSeguroHelper::formatDate($initialDate) .
-            ", finalDate=" . PagSeguroHelper::formatDate($finalDate) . ") - begin"
-        );
+		LogPagSeguro::info(
+			"PagSeguroTransactionSearchService.SearchByDate(initialDate=" . PagSeguroHelper::formatDate($initialDate) .
+			", finalDate=" . PagSeguroHelper::formatDate($finalDate) . ") - begin"
+		);
 
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
 
-        $searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
+		$searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
 
-        try {
+		try {
 
-            $connection = new PagSeguroHttpConnection();
-            $connection->get(
-                self::buildSearchUrlByDate($connectionData, $searchParams),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
+			$connection = new PagSeguroHttpConnection();
+			$connection->get(
+				self::buildSearchUrlByDate($connectionData, $searchParams),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
 
-            self::$logService = "SearchByDate";
-            return self::searchResult($connection, $initialDate, $finalDate);
+			self::$logService = "SearchByDate";
+			return self::searchResult($connection, $initialDate, $finalDate);
 
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		} catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
 
-    }
+	}
 
-    /***
+	/***
      * Search transactions abandoned associated with this set of credentials within a date range
      *
      * @param PagSeguroCredentials $credentials
@@ -218,211 +218,211 @@ 
      * @throws PagSeguroServiceException
      * @throws Exception
      */
-    public static function searchAbandoned(
-        PagSeguroCredentials $credentials,
-        $pageNumber,
-        $maxPageResults,
-        $initialDate,
-        $finalDate = null
-    ) {
-
-        LogPagSeguro::info(
-            "PagSeguroTransactionSearchService.searchAbandoned(initialDate=" .
-            PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-            PagSeguroHelper::formatDate($finalDate) . ") - begin"
-        );
-
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
-
-        $searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
-
-        try {
-
-            $connection = new PagSeguroHttpConnection();
-            $connection->get(
-                self::buildSearchUrlAbandoned($connectionData, $searchParams),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
-
-            self::$logService = "searchAbandoned";
-            return self::searchResult($connection, $initialDate, $finalDate);
-
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        }
-        catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
-
-    }
-
-    /**
-     * @param PagSeguroCredentials $credentials
-     * @param $reference
-     * @param null $initialDate
-     * @param null $finalDate
-     * @param null $pageNumber
-     * @param null $maxPageResults
-     * @throws Exception
-     * @throws PagSeguroServiceException
-     */
-    public static function searchByReference(
-        PagSeguroCredentials $credentials,
-        $reference,
-        $initialDate = null,
-        $finalDate = null,
-        $pageNumber = null,
-        $maxPageResults = null
-    ) {
-
-        LogPagSeguro::info(
-            "PagSeguroTransactionSearchService.SearchByReference(reference=".$reference.") - begin"
-        );
-
-        $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
-
-        if ($initialDate)
-            $searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
-        else
-            $searchParams = null;
-
-        try {
-
-            $connection = new PagSeguroHttpConnection();
-            $connection->get(
-                self::buildSearchUrlByReference(
-                    $connectionData,
-                    $reference,
-                    $searchParams
-                ),
-                $connectionData->getServiceTimeout(),
-                $connectionData->getCharset()
-            );
-
-            self::$logService = "SearchByReference";
-            return self::searchResult($connection);
-
-        } catch (PagSeguroServiceException $err) {
-            throw $err;
-        } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
-            throw $err;
-        }
-
-    }
-
-    /**
-     * @param $pageNumber
-     * @param $maxPageResults
-     * @param $initialDate
-     * @param null $finalDate
-     * @return array
-     */
-    private function buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate = null)
-    {
-        $searchParams = array(
-            'initialDate' => PagSeguroHelper::formatDate($initialDate),
-            'pageNumber' => $pageNumber,
-            'maxPageResults' => $maxPageResults
-        );
-
-        $searchParams['finalDate'] = $finalDate ? PagSeguroHelper::formatDate($finalDate) : null;
-
-        return $searchParams;
-    }
-
-    /**
-     * @param $connection
-     * @param $code
-     * @return bool|PagSeguroTransaction
-     * @throws PagSeguroServiceException
-     */
-    private function searchByCodeResult($connection, $code)
-    {
-        $httpStatus = new PagSeguroHttpStatus($connection->getStatus());
-
-        switch ($httpStatus->getType()) {
-
-            case 'OK':
-                $transaction = PagSeguroTransactionParser::readTransaction($connection->getResponse());
-                LogPagSeguro::info(
-                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - end " .
-                    $transaction->toString()
-                );
-                break;
-
-            case 'BAD_REQUEST':
-                $errors = PagSeguroTransactionParser::readErrors($connection->getResponse());
-                $err = new PagSeguroServiceException($httpStatus, $errors);
-                LogPagSeguro::error(
-                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error " .
-                    $err->getOneLineMessage()
-                );
-                throw $err;
-                break;
-
-            default:
-                $err = new PagSeguroServiceException($httpStatus);
-                LogPagSeguro::error(
-                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error " .
-                    $err->getOneLineMessage()
-                );
-                throw $err;
-                break;
-        }
-        return isset($transaction) ? $transaction : false;
-    }
-
-    /**
-     * @param $connection
-     * @param null $initialDate
-     * @param null $finalDate
-     * @return bool|PagSeguroTransactionSearchResult
-     * @throws PagSeguroServiceException
-     */
-    private function searchResult($connection, $initialDate = null, $finalDate = null)
-    {
-
-        $httpStatus = new PagSeguroHttpStatus($connection->getStatus());
-
-        switch ($httpStatus->getType()) {
-
-            case 'OK':
-
-                $searchResult = PagSeguroTransactionParser::readSearchResult($connection->getResponse());
-
-                LogPagSeguro::info(
-                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
-                    PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-                    PagSeguroHelper::formatDate($finalDate) . ") - end ", self::$logService) . $searchResult->toString()
-                );
-                break;
-
-            case 'BAD_REQUEST':
-                $errors = PagSeguroTransactionParser::readErrors($connection->getResponse());
-                $err = new PagSeguroServiceException($httpStatus, $errors);
-                LogPagSeguro::error(
-                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
-                    PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-                    PagSeguroHelper::formatDate($finalDate) . ") - end ", self::$logService) . $err->getOneLineMessage()
-                );
-                throw $err;
-                break;
-
-            default:
-                $err = new PagSeguroServiceException($httpStatus);
-                LogPagSeguro::error(
-                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
-                    PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-                    PagSeguroHelper::formatDate($finalDate) . ") - end ",  self::$logService) . $err->getOneLineMessage()
-                );
-                throw $err;
-                break;
-
-        }
-
-        return isset($searchResult) ? $searchResult : false;
-    }
+	public static function searchAbandoned(
+		PagSeguroCredentials $credentials,
+		$pageNumber,
+		$maxPageResults,
+		$initialDate,
+		$finalDate = null
+	) {
+
+		LogPagSeguro::info(
+			"PagSeguroTransactionSearchService.searchAbandoned(initialDate=" .
+			PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
+			PagSeguroHelper::formatDate($finalDate) . ") - begin"
+		);
+
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+
+		$searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
+
+		try {
+
+			$connection = new PagSeguroHttpConnection();
+			$connection->get(
+				self::buildSearchUrlAbandoned($connectionData, $searchParams),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
+
+			self::$logService = "searchAbandoned";
+			return self::searchResult($connection, $initialDate, $finalDate);
+
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		}
+		catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
+
+	}
+
+	/**
+	 * @param PagSeguroCredentials $credentials
+	 * @param $reference
+	 * @param null $initialDate
+	 * @param null $finalDate
+	 * @param null $pageNumber
+	 * @param null $maxPageResults
+	 * @throws Exception
+	 * @throws PagSeguroServiceException
+	 */
+	public static function searchByReference(
+		PagSeguroCredentials $credentials,
+		$reference,
+		$initialDate = null,
+		$finalDate = null,
+		$pageNumber = null,
+		$maxPageResults = null
+	) {
+
+		LogPagSeguro::info(
+			"PagSeguroTransactionSearchService.SearchByReference(reference=".$reference.") - begin"
+		);
+
+		$connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
+
+		if ($initialDate)
+			$searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
+		else
+			$searchParams = null;
+
+		try {
+
+			$connection = new PagSeguroHttpConnection();
+			$connection->get(
+				self::buildSearchUrlByReference(
+					$connectionData,
+					$reference,
+					$searchParams
+				),
+				$connectionData->getServiceTimeout(),
+				$connectionData->getCharset()
+			);
+
+			self::$logService = "SearchByReference";
+			return self::searchResult($connection);
+
+		} catch (PagSeguroServiceException $err) {
+			throw $err;
+		} catch (Exception $err) {
+			LogPagSeguro::error("Exception: " . $err->getMessage());
+			throw $err;
+		}
+
+	}
+
+	/**
+	 * @param $pageNumber
+	 * @param $maxPageResults
+	 * @param $initialDate
+	 * @param null $finalDate
+	 * @return array
+	 */
+	private function buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate = null)
+	{
+		$searchParams = array(
+			'initialDate' => PagSeguroHelper::formatDate($initialDate),
+			'pageNumber' => $pageNumber,
+			'maxPageResults' => $maxPageResults
+		);
+
+		$searchParams['finalDate'] = $finalDate ? PagSeguroHelper::formatDate($finalDate) : null;
+
+		return $searchParams;
+	}
+
+	/**
+	 * @param $connection
+	 * @param $code
+	 * @return bool|PagSeguroTransaction
+	 * @throws PagSeguroServiceException
+	 */
+	private function searchByCodeResult($connection, $code)
+	{
+		$httpStatus = new PagSeguroHttpStatus($connection->getStatus());
+
+		switch ($httpStatus->getType()) {
+
+			case 'OK':
+				$transaction = PagSeguroTransactionParser::readTransaction($connection->getResponse());
+				LogPagSeguro::info(
+					"PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - end " .
+					$transaction->toString()
+				);
+				break;
+
+			case 'BAD_REQUEST':
+				$errors = PagSeguroTransactionParser::readErrors($connection->getResponse());
+				$err = new PagSeguroServiceException($httpStatus, $errors);
+				LogPagSeguro::error(
+					"PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error " .
+					$err->getOneLineMessage()
+				);
+				throw $err;
+				break;
+
+			default:
+				$err = new PagSeguroServiceException($httpStatus);
+				LogPagSeguro::error(
+					"PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error " .
+					$err->getOneLineMessage()
+				);
+				throw $err;
+				break;
+		}
+		return isset($transaction) ? $transaction : false;
+	}
+
+	/**
+	 * @param $connection
+	 * @param null $initialDate
+	 * @param null $finalDate
+	 * @return bool|PagSeguroTransactionSearchResult
+	 * @throws PagSeguroServiceException
+	 */
+	private function searchResult($connection, $initialDate = null, $finalDate = null)
+	{
+
+		$httpStatus = new PagSeguroHttpStatus($connection->getStatus());
+
+		switch ($httpStatus->getType()) {
+
+			case 'OK':
+
+				$searchResult = PagSeguroTransactionParser::readSearchResult($connection->getResponse());
+
+				LogPagSeguro::info(
+					sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
+					PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
+					PagSeguroHelper::formatDate($finalDate) . ") - end ", self::$logService) . $searchResult->toString()
+				);
+				break;
+
+			case 'BAD_REQUEST':
+				$errors = PagSeguroTransactionParser::readErrors($connection->getResponse());
+				$err = new PagSeguroServiceException($httpStatus, $errors);
+				LogPagSeguro::error(
+					sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
+					PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
+					PagSeguroHelper::formatDate($finalDate) . ") - end ", self::$logService) . $err->getOneLineMessage()
+				);
+				throw $err;
+				break;
+
+			default:
+				$err = new PagSeguroServiceException($httpStatus);
+				LogPagSeguro::error(
+					sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
+					PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
+					PagSeguroHelper::formatDate($finalDate) . ") - end ",  self::$logService) . $err->getOneLineMessage()
+				);
+				throw $err;
+				break;
+
+		}
+
+		return isset($searchResult) ? $searchResult : false;
+	}
 }

Spacing +36 added lines, -36 removed lines

@@ -42,7 +42,7 @@ 
     private static function buildSearchUrlByCode(PagSeguroConnectionData $connectionData, $transactionCode)
     {
         $url = $connectionData->getServiceUrl('v3');
-        return "{$url}/{$transactionCode}/?" . $connectionData->getCredentialsUrlQuery();
+        return "{$url}/{$transactionCode}/?".$connectionData->getCredentialsUrlQuery();
     }
 
     /**
@@ -54,14 +54,14 @@ 
     {
         $url = $connectionData->getServiceUrl('v2');
         $initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
-        $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
+        $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=".$searchParams['finalDate']) : "";
         if ($searchParams['pageNumber'] != null) {
-            $page = "&page=" . $searchParams['pageNumber'];
+            $page = "&page=".$searchParams['pageNumber'];
         }
         if ($searchParams['maxPageResults'] != null) {
-            $maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
+            $maxPageResults = "&maxPageResults=".$searchParams['maxPageResults'];
         }
-        return "{$url}/?" . $connectionData->getCredentialsUrlQuery() .
+        return "{$url}/?".$connectionData->getCredentialsUrlQuery().
             "&initialDate={$initialDate}{$finalDate}{$page}{$maxPageResults}";
     }
 
@@ -75,14 +75,14 @@ 
         $url = $connectionData->getServiceUrl('v2');
 
         $initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
-        $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
+        $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=".$searchParams['finalDate']) : "";
         if ($searchParams['pageNumber'] != null) {
-            $page = "&page=" . $searchParams['pageNumber'];
+            $page = "&page=".$searchParams['pageNumber'];
         }
         if ($searchParams['maxPageResults'] != null) {
-            $maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
+            $maxPageResults = "&maxPageResults=".$searchParams['maxPageResults'];
         }
-        return "{$url}/abandoned/?" . $connectionData->getCredentialsUrlQuery() .
+        return "{$url}/abandoned/?".$connectionData->getCredentialsUrlQuery().
             "&initialDate={$initialDate}&finalDate={$finalDate}{$page}{$maxPageResults}";
     }
 
@@ -96,22 +96,22 @@ 
         PagSeguroConnectionData $connectionData,
         $reference,
         $searchParams = null
-    ){
+    ) {
         $url = $connectionData->getServiceUrl('v2');
         if ($searchParams == null) {
-            return "{$url}?" . $connectionData->getCredentialsUrlQuery() . "&reference=" . $reference;
+            return "{$url}?".$connectionData->getCredentialsUrlQuery()."&reference=".$reference;
         } else {
 
             $initialDate = $searchParams['initialDate'] != null ? $searchParams['initialDate'] : "";
-            $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=" . $searchParams['finalDate']) : "";
+            $finalDate = $searchParams['finalDate'] != null ? ("&finalDate=".$searchParams['finalDate']) : "";
             if ($searchParams['pageNumber'] != null) {
-                $page = "&page=" . $searchParams['pageNumber'];
+                $page = "&page=".$searchParams['pageNumber'];
             }
             if ($searchParams['maxPageResults'] != null) {
-                $maxPageResults = "&maxPageResults=" . $searchParams['maxPageResults'];
+                $maxPageResults = "&maxPageResults=".$searchParams['maxPageResults'];
             }
 
-            return "{$url}?" . $connectionData->getCredentialsUrlQuery() . "&reference=" . $reference
+            return "{$url}?".$connectionData->getCredentialsUrlQuery()."&reference=".$reference
                    . "&initialDate={$initialDate}&finalDate={$finalDate}{$page}{$maxPageResults}";
         }
     }
@@ -148,7 +148,7 @@ 
             throw $err;
         }
         catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
 
@@ -176,8 +176,8 @@ 
     ) {
 
         LogPagSeguro::info(
-            "PagSeguroTransactionSearchService.SearchByDate(initialDate=" . PagSeguroHelper::formatDate($initialDate) .
-            ", finalDate=" . PagSeguroHelper::formatDate($finalDate) . ") - begin"
+            "PagSeguroTransactionSearchService.SearchByDate(initialDate=".PagSeguroHelper::formatDate($initialDate).
+            ", finalDate=".PagSeguroHelper::formatDate($finalDate).") - begin"
         );
 
         $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
@@ -199,7 +199,7 @@ 
         } catch (PagSeguroServiceException $err) {
             throw $err;
         } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
 
@@ -227,9 +227,9 @@ 
     ) {
 
         LogPagSeguro::info(
-            "PagSeguroTransactionSearchService.searchAbandoned(initialDate=" .
-            PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-            PagSeguroHelper::formatDate($finalDate) . ") - begin"
+            "PagSeguroTransactionSearchService.searchAbandoned(initialDate=".
+            PagSeguroHelper::formatDate($initialDate).", finalDate=".
+            PagSeguroHelper::formatDate($finalDate).") - begin"
         );
 
         $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
@@ -252,7 +252,7 @@ 
             throw $err;
         }
         catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
 
@@ -307,7 +307,7 @@ 
         } catch (PagSeguroServiceException $err) {
             throw $err;
         } catch (Exception $err) {
-            LogPagSeguro::error("Exception: " . $err->getMessage());
+            LogPagSeguro::error("Exception: ".$err->getMessage());
             throw $err;
         }
 
@@ -348,7 +348,7 @@ 
             case 'OK':
                 $transaction = PagSeguroTransactionParser::readTransaction($connection->getResponse());
                 LogPagSeguro::info(
-                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - end " .
+                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - end ".
                     $transaction->toString()
                 );
                 break;
@@ -357,7 +357,7 @@ 
                 $errors = PagSeguroTransactionParser::readErrors($connection->getResponse());
                 $err = new PagSeguroServiceException($httpStatus, $errors);
                 LogPagSeguro::error(
-                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error " .
+                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error ".
                     $err->getOneLineMessage()
                 );
                 throw $err;
@@ -366,7 +366,7 @@ 
             default:
                 $err = new PagSeguroServiceException($httpStatus);
                 LogPagSeguro::error(
-                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error " .
+                    "PagSeguroTransactionSearchService.SearchByCode(transactionCode=$code) - error ".
                     $err->getOneLineMessage()
                 );
                 throw $err;
@@ -394,9 +394,9 @@ 
                 $searchResult = PagSeguroTransactionParser::readSearchResult($connection->getResponse());
 
                 LogPagSeguro::info(
-                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
-                    PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-                    PagSeguroHelper::formatDate($finalDate) . ") - end ", self::$logService) . $searchResult->toString()
+                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=".
+                    PagSeguroHelper::formatDate($initialDate).", finalDate=".
+                    PagSeguroHelper::formatDate($finalDate).") - end ", self::$logService).$searchResult->toString()
                 );
                 break;
 
@@ -404,9 +404,9 @@ 
                 $errors = PagSeguroTransactionParser::readErrors($connection->getResponse());
                 $err = new PagSeguroServiceException($httpStatus, $errors);
                 LogPagSeguro::error(
-                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
-                    PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-                    PagSeguroHelper::formatDate($finalDate) . ") - end ", self::$logService) . $err->getOneLineMessage()
+                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=".
+                    PagSeguroHelper::formatDate($initialDate).", finalDate=".
+                    PagSeguroHelper::formatDate($finalDate).") - end ", self::$logService).$err->getOneLineMessage()
                 );
                 throw $err;
                 break;
@@ -414,9 +414,9 @@ 
             default:
                 $err = new PagSeguroServiceException($httpStatus);
                 LogPagSeguro::error(
-                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=" .
-                    PagSeguroHelper::formatDate($initialDate) . ", finalDate=" .
-                    PagSeguroHelper::formatDate($finalDate) . ") - end ",  self::$logService) . $err->getOneLineMessage()
+                    sprintf("PagSeguroTransactionSearchService.%s(initialDate=".
+                    PagSeguroHelper::formatDate($initialDate).", finalDate=".
+                    PagSeguroHelper::formatDate($finalDate).") - end ", self::$logService).$err->getOneLineMessage()
                 );
                 throw $err;
                 break;

Braces +7 added lines, -8 removed lines

@@ -146,8 +146,7 @@ 
 
         } catch (PagSeguroServiceException $err) {
             throw $err;
-        }
-        catch (Exception $err) {
+        } catch (Exception $err) {
             LogPagSeguro::error("Exception: " . $err->getMessage());
             throw $err;
         }
@@ -250,8 +249,7 @@ 
 
         } catch (PagSeguroServiceException $err) {
             throw $err;
-        }
-        catch (Exception $err) {
+        } catch (Exception $err) {
             LogPagSeguro::error("Exception: " . $err->getMessage());
             throw $err;
         }
@@ -283,10 +281,11 @@ 
 
         $connectionData = new PagSeguroConnectionData($credentials, self::SERVICE_NAME);
 
-        if ($initialDate)
-            $searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
-        else
-            $searchParams = null;
+        if ($initialDate) {
+                    $searchParams = self::buildParams($pageNumber, $maxPageResults, $initialDate, $finalDate);
+        } else {
+                    $searchParams = null;
+        }
 
         try {
 

app/Vendor/PHPExcel/PHPExcel.php

Doc Comments +3 added lines, -3 removed lines

@@ -304,7 +304,7 @@ 
 	 * Get index for sheet
 	 *
 	 * @param PHPExcel_Worksheet $pSheet
-	 * @return Sheet index
+	 * @return integer|null index
 	 * @throws Exception
 	 */
 	public function getIndex(PHPExcel_Worksheet $pSheet)
@@ -321,7 +321,7 @@ 
 	 *
 	 * @param string $sheetName Sheet name to modify index for
 	 * @param int $newIndex New index for the sheet
-	 * @return New sheet index
+	 * @return integer sheet index
 	 * @throws Exception
 	 */
     public function setIndexByName($sheetName, $newIndex)
@@ -457,7 +457,7 @@ 
 	 * Add named range
 	 *
 	 * @param PHPExcel_NamedRange $namedRange
-	 * @return PHPExcel
+	 * @return boolean
 	 */
 	public function addNamedRange(PHPExcel_NamedRange $namedRange) {
 		if ($namedRange->getScope() == null) {

Indentation +76 added lines, -76 removed lines

@@ -1,29 +1,29 @@ 
 <?php
 /**
- * PHPExcel
- *
- * Copyright (c) 2006 - 2012 PHPExcel
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2.1 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
- *
- * @category   PHPExcel
- * @package    PHPExcel
- * @copyright  Copyright (c) 2006 - 2012 PHPExcel (http://www.codeplex.com/PHPExcel)
- * @license    http://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt	LGPL
- * @version    1.7.7, 2012-05-19
- */
+	 * PHPExcel
+	 *
+	 * Copyright (c) 2006 - 2012 PHPExcel
+	 *
+	 * This library is free software; you can redistribute it and/or
+	 * modify it under the terms of the GNU Lesser General Public
+	 * License as published by the Free Software Foundation; either
+	 * version 2.1 of the License, or (at your option) any later version.
+	 *
+	 * This library is distributed in the hope that it will be useful,
+	 * but WITHOUT ANY WARRANTY; without even the implied warranty of
+	 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+	 * Lesser General Public License for more details.
+	 *
+	 * You should have received a copy of the GNU Lesser General Public
+	 * License along with this library; if not, write to the Free Software
+	 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+	 *
+	 * @category   PHPExcel
+	 * @package    PHPExcel
+	 * @copyright  Copyright (c) 2006 - 2012 PHPExcel (http://www.codeplex.com/PHPExcel)
+	 * @license    http://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt	LGPL
+	 * @version    1.7.7, 2012-05-19
+	 */
 
 
 /** PHPExcel root directory */
@@ -191,48 +191,48 @@ 
 		return $this->_workSheetCollection[$this->_activeSheetIndex];
 	}
 
-    /**
-     * Create sheet and add it to this workbook
-     *
+	/**
+	 * Create sheet and add it to this workbook
+	 *
 	 * @param int|null $iSheetIndex Index where sheet should go (0,1,..., or null for last)
-     * @return PHPExcel_Worksheet
-     * @throws Exception
-     */
-    public function createSheet($iSheetIndex = null)
-    {
-        $newSheet = new PHPExcel_Worksheet($this);
-        $this->addSheet($newSheet, $iSheetIndex);
-        return $newSheet;
-    }
-
-    /**
-     * Add sheet
-     *
-     * @param PHPExcel_Worksheet $pSheet
+	 * @return PHPExcel_Worksheet
+	 * @throws Exception
+	 */
+	public function createSheet($iSheetIndex = null)
+	{
+		$newSheet = new PHPExcel_Worksheet($this);
+		$this->addSheet($newSheet, $iSheetIndex);
+		return $newSheet;
+	}
+
+	/**
+	 * Add sheet
+	 *
+	 * @param PHPExcel_Worksheet $pSheet
 	 * @param int|null $iSheetIndex Index where sheet should go (0,1,..., or null for last)
-     * @return PHPExcel_Worksheet
-     * @throws Exception
-     */
-    public function addSheet(PHPExcel_Worksheet $pSheet = null, $iSheetIndex = null)
-    {
-        if($iSheetIndex === NULL) {
-            $this->_workSheetCollection[] = $pSheet;
-        } else {
-            // Insert the sheet at the requested index
-            array_splice(
-                $this->_workSheetCollection,
-                $iSheetIndex,
-                0,
-                array($pSheet)
-                );
+	 * @return PHPExcel_Worksheet
+	 * @throws Exception
+	 */
+	public function addSheet(PHPExcel_Worksheet $pSheet = null, $iSheetIndex = null)
+	{
+		if($iSheetIndex === NULL) {
+			$this->_workSheetCollection[] = $pSheet;
+		} else {
+			// Insert the sheet at the requested index
+			array_splice(
+				$this->_workSheetCollection,
+				$iSheetIndex,
+				0,
+				array($pSheet)
+				);
 
 			// Adjust active sheet index if necessary
 			if ($this->_activeSheetIndex >= $iSheetIndex) {
 				++$this->_activeSheetIndex;
 			}
-        }
+		}
 		return $pSheet;
-    }
+	}
 
 	/**
 	 * Remove sheet by index
@@ -316,7 +316,7 @@ 
 		}
 	}
 
-    /**
+	/**
 	 * Set index for sheet by sheet name.
 	 *
 	 * @param string $sheetName Sheet name to modify index for
@@ -324,22 +324,22 @@ 
 	 * @return New sheet index
 	 * @throws Exception
 	 */
-    public function setIndexByName($sheetName, $newIndex)
-    {
-        $oldIndex = $this->getIndex($this->getSheetByName($sheetName));
-        $pSheet = array_splice(
-            $this->_workSheetCollection,
-            $oldIndex,
-            1
-            );
-        array_splice(
-            $this->_workSheetCollection,
-            $newIndex,
-            0,
-            $pSheet
-            );
-        return $newIndex;
-    }
+	public function setIndexByName($sheetName, $newIndex)
+	{
+		$oldIndex = $this->getIndex($this->getSheetByName($sheetName));
+		$pSheet = array_splice(
+			$this->_workSheetCollection,
+			$oldIndex,
+			1
+			);
+		array_splice(
+			$this->_workSheetCollection,
+			$newIndex,
+			0,
+			$pSheet
+			);
+		return $newIndex;
+	}
 
 	/**
 	 * Get sheet count
@@ -724,7 +724,7 @@ 
 	 */
 	public function garbageCollect()
 	{
-    	// how many references are there to each cellXf ?
+		// how many references are there to each cellXf ?
 		$countReferencesCellXf = array();
 		foreach ($this->_cellXfCollection as $index => $cellXf) {
 			$countReferencesCellXf[$index] = 0;

Spacing +16 added lines, -16 removed lines

@@ -27,9 +27,9 @@ 
 
 
 /** PHPExcel root directory */
-if (!defined('PHPEXCEL_ROOT')) {
-	define('PHPEXCEL_ROOT', dirname(__FILE__) . '/');
-	require(PHPEXCEL_ROOT . 'PHPExcel/Autoloader.php');
+if ( ! defined('PHPEXCEL_ROOT')) {
+	define('PHPEXCEL_ROOT', dirname(__FILE__).'/');
+	require(PHPEXCEL_ROOT.'PHPExcel/Autoloader.php');
 }
 
 
@@ -133,7 +133,7 @@ 
 	 *
 	 */
 	public function disconnectWorksheets() {
-		foreach($this->_workSheetCollection as $k => &$worksheet) {
+		foreach ($this->_workSheetCollection as $k => &$worksheet) {
 			$worksheet->disconnectCells();
 			$this->_workSheetCollection[$k] = null;
 		}
@@ -215,7 +215,7 @@ 
      */
     public function addSheet(PHPExcel_Worksheet $pSheet = null, $iSheetIndex = null)
     {
-        if($iSheetIndex === NULL) {
+        if ($iSheetIndex === NULL) {
             $this->_workSheetCollection[] = $pSheet;
         } else {
             // Insert the sheet at the requested index
@@ -392,7 +392,7 @@ 
 			return $worksheet;
 		}
 
-		throw new Exception('Workbook does not contain sheet:' . $pValue);
+		throw new Exception('Workbook does not contain sheet:'.$pValue);
 	}
 
 	/**
@@ -438,7 +438,7 @@ 
 		// update the cellXfs
 		foreach ($pSheet->getCellCollection(false) as $cellID) {
 			$cell = $pSheet->getCell($cellID);
-			$cell->setXfIndex( $cell->getXfIndex() + $countCellXfs );
+			$cell->setXfIndex($cell->getXfIndex() + $countCellXfs);
 		}
 
 		return $this->addSheet($pSheet, $iSheetIndex);
@@ -487,8 +487,8 @@ 
 			}
 
 			// then look for local defined name (has priority over global defined name if both names exist)
-			if (($pSheet !== NULL) && isset($this->_namedRanges[$pSheet->getTitle() . '!' . $namedRange])) {
-				$returnValue = $this->_namedRanges[$pSheet->getTitle() . '!' . $namedRange];
+			if (($pSheet !== NULL) && isset($this->_namedRanges[$pSheet->getTitle().'!'.$namedRange])) {
+				$returnValue = $this->_namedRanges[$pSheet->getTitle().'!'.$namedRange];
 			}
 		}
 
@@ -508,8 +508,8 @@ 
 				unset($this->_namedRanges[$namedRange]);
 			}
 		} else {
-			if (isset($this->_namedRanges[$pSheet->getTitle() . '!' . $namedRange])) {
-				unset($this->_namedRanges[$pSheet->getTitle() . '!' . $namedRange]);
+			if (isset($this->_namedRanges[$pSheet->getTitle().'!'.$namedRange])) {
+				unset($this->_namedRanges[$pSheet->getTitle().'!'.$namedRange]);
 			}
 		}
 		return $this;
@@ -545,7 +545,7 @@ 
 	 * Implement PHP __clone to create a deep clone, not just a shallow copy.
 	 */
 	public function __clone() {
-		foreach($this as $key => $val) {
+		foreach ($this as $key => $val) {
 			if (is_object($val) || (is_array($val))) {
 				$this->{$key} = unserialize(serialize($val));
 			}
@@ -633,7 +633,7 @@ 
 				foreach ($worksheet->getCellCollection(false) as $cellID) {
 					$cell = $worksheet->getCell($cellID);
 					$xfIndex = $cell->getXfIndex();
-					if ($xfIndex > $pIndex ) {
+					if ($xfIndex > $pIndex) {
 						// decrease xf index by 1
 						$cell->setXfIndex($xfIndex - 1);
 					} else if ($xfIndex == $pIndex) {
@@ -780,19 +780,19 @@ 
 			// for all cells
 			foreach ($sheet->getCellCollection(false) as $cellID) {
 				$cell = $sheet->getCell($cellID);
-				$cell->setXfIndex( $map[$cell->getXfIndex()] );
+				$cell->setXfIndex($map[$cell->getXfIndex()]);
 			}
 
 			// for all row dimensions
 			foreach ($sheet->getRowDimensions() as $rowDimension) {
 				if ($rowDimension->getXfIndex() !== null) {
-					$rowDimension->setXfIndex( $map[$rowDimension->getXfIndex()] );
+					$rowDimension->setXfIndex($map[$rowDimension->getXfIndex()]);
 				}
 			}
 
 			// for all column dimensions
 			foreach ($sheet->getColumnDimensions() as $columnDimension) {
-				$columnDimension->setXfIndex( $map[$columnDimension->getXfIndex()] );
+				$columnDimension->setXfIndex($map[$columnDimension->getXfIndex()]);
 			}
 		}
 

app/Vendor/PHPExcel/PHPExcel/CachedObjectStorage/APC.php

Doc Comments +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
 	 * @access	public
      * @param	string			$pCoord		Coordinate address of the cell to update
      * @param	PHPExcel_Cell	$cell		Cell to update
-	 * @return	void
+	 * @return	PHPExcel_Cell
      * @throws	Exception
      */
 	public function addCacheData($pCoord, PHPExcel_Cell $cell) {
@@ -102,7 +102,7 @@ 
 	 *
 	 * @access	public
 	 * @param	string		$pCoord		Coordinate address of the cell to check
-	 * @return	void
+	 * @return	boolean
 	 * @return	boolean
 	 */
 	public function isDataSet($pCoord) {

Indentation +26 added lines, -26 removed lines

@@ -52,14 +52,14 @@ 
 	private $_cacheTime = 600;
 
 
-    /**
-     * Store cell data in cache for the current cell object if it's "dirty",
-     *     and the 'nullify' the current cell object
-     *
+	/**
+	 * Store cell data in cache for the current cell object if it's "dirty",
+	 *     and the 'nullify' the current cell object
+	 *
 	 * @access	private
 	 * @return	void
-     * @throws	Exception
-     */
+	 * @throws	Exception
+	 */
 	private function _storeData() {
 		if ($this->_currentCellIsDirty) {
 			$this->_currentObject->detach();
@@ -74,15 +74,15 @@ 
 	}	//	function _storeData()
 
 
-    /**
-     * Add or Update a cell in cache identified by coordinate address
-     *
+	/**
+	 * Add or Update a cell in cache identified by coordinate address
+	 *
 	 * @access	public
-     * @param	string			$pCoord		Coordinate address of the cell to update
-     * @param	PHPExcel_Cell	$cell		Cell to update
+	 * @param	string			$pCoord		Coordinate address of the cell to update
+	 * @param	PHPExcel_Cell	$cell		Cell to update
 	 * @return	void
-     * @throws	Exception
-     */
+	 * @throws	Exception
+	 */
 	public function addCacheData($pCoord, PHPExcel_Cell $cell) {
 		if (($pCoord !== $this->_currentObjectID) && ($this->_currentObjectID !== null)) {
 			$this->_storeData();
@@ -124,14 +124,14 @@ 
 	}	//	function isDataSet()
 
 
-    /**
-     * Get cell at a specific coordinate
-     *
+	/**
+	 * Get cell at a specific coordinate
+	 *
 	 * @access	public
-     * @param 	string 			$pCoord		Coordinate of the cell
-     * @throws 	Exception
-     * @return 	PHPExcel_Cell 	Cell that was found, or null if not found
-     */
+	 * @param 	string 			$pCoord		Coordinate of the cell
+	 * @throws 	Exception
+	 * @return 	PHPExcel_Cell 	Cell that was found, or null if not found
+	 */
 	public function getCacheData($pCoord) {
 		if ($pCoord === $this->_currentObjectID) {
 			return $this->_currentObject;
@@ -162,13 +162,13 @@ 
 	}	//	function getCacheData()
 
 
-    /**
-     * Delete a cell in cache identified by coordinate address
-     *
+	/**
+	 * Delete a cell in cache identified by coordinate address
+	 *
 	 * @access	public
-     * @param	string			$pCoord		Coordinate address of the cell to delete
-     * @throws	Exception
-     */
+	 * @param	string			$pCoord		Coordinate address of the cell to delete
+	 * @throws	Exception
+	 */
 	public function deleteCacheData($pCoord) {
 		//	Delete the entry from APC
 		apc_delete($this->_cachePrefix.$pCoord.'.cache');

Spacing +8 added lines, -8 removed lines

@@ -64,7 +64,7 @@ 
 		if ($this->_currentCellIsDirty) {
 			$this->_currentObject->detach();
 
-			if (!apc_store($this->_cachePrefix.$this->_currentObjectID.'.cache',serialize($this->_currentObject),$this->_cacheTime)) {
+			if ( ! apc_store($this->_cachePrefix.$this->_currentObjectID.'.cache', serialize($this->_currentObject), $this->_cacheTime)) {
 				$this->__destruct();
 				throw new Exception('Failed to store cell '.$this->_currentObjectID.' in APC');
 			}
@@ -189,9 +189,9 @@ 
 		parent::copyCellCollection($parent);
 		//	Get a new id for the new file name
 		$baseUnique = $this->_getUniqueID();
-		$newCachePrefix = substr(md5($baseUnique),0,8).'.';
+		$newCachePrefix = substr(md5($baseUnique), 0, 8).'.';
 		$cacheList = $this->getCellList();
-		foreach($cacheList as $cellID) {
+		foreach ($cacheList as $cellID) {
 			if ($cellID != $this->_currentObjectID) {
 				$obj = apc_fetch($this->_cachePrefix.$cellID.'.cache');
 				if ($obj === false) {
@@ -199,7 +199,7 @@ 
 					parent::deleteCacheData($cellID);
 					throw new Exception('Cell entry '.$cellID.' no longer exists in APC');
 				}
-				if (!apc_store($newCachePrefix.$cellID.'.cache',$obj,$this->_cacheTime)) {
+				if ( ! apc_store($newCachePrefix.$cellID.'.cache', $obj, $this->_cacheTime)) {
 					$this->__destruct();
 					throw new Exception('Failed to store cell '.$cellID.' in APC');
 				}
@@ -239,11 +239,11 @@ 
 	 * @param	array of mixed		$arguments	Additional initialisation arguments
 	 */
 	public function __construct(PHPExcel_Worksheet $parent, $arguments) {
-		$cacheTime	= (isset($arguments['cacheTime']))	? $arguments['cacheTime']	: 600;
+		$cacheTime = (isset($arguments['cacheTime'])) ? $arguments['cacheTime'] : 600;
 
 		if ($this->_cachePrefix === NULL) {
 			$baseUnique = $this->_getUniqueID();
-			$this->_cachePrefix = substr(md5($baseUnique),0,8).'.';
+			$this->_cachePrefix = substr(md5($baseUnique), 0, 8).'.';
 			$this->_cacheTime = $cacheTime;
 
 			parent::__construct($parent);
@@ -258,7 +258,7 @@ 
 	 */
 	public function __destruct() {
 		$cacheList = $this->getCellList();
-		foreach($cacheList as $cellID) {
+		foreach ($cacheList as $cellID) {
 			apc_delete($this->_cachePrefix.$cellID.'.cache');
 		}
 	}	//	function __destruct()
@@ -272,7 +272,7 @@ 
 	 * @return	boolean
 	 */
 	public static function cacheMethodIsAvailable() {
-		if (!function_exists('apc_store')) {
+		if ( ! function_exists('apc_store')) {
 			return false;
 		}
 		if (apc_sma_info() === false) {

app/Vendor/PHPExcel/PHPExcel/CachedObjectStorage/Memcache.php

Doc Comments +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
      *
      * @param	string			$pCoord		Coordinate address of the cell to update
      * @param	PHPExcel_Cell	$cell		Cell to update
-	 * @return	void
+	 * @return	PHPExcel_Cell
      * @throws	Exception
      */
 	public function addCacheData($pCoord, PHPExcel_Cell $cell) {
@@ -107,7 +107,7 @@ 
 	 * Is a value set in the current PHPExcel_CachedObjectStorage_ICache for an indexed cell?
 	 *
 	 * @param	string		$pCoord		Coordinate address of the cell to check
-	 * @return	void
+	 * @return	boolean
 	 * @return	boolean
 	 */
 	public function isDataSet($pCoord) {

Indentation +26 added lines, -26 removed lines

@@ -57,13 +57,13 @@ 
 	private $_memcache = null;
 
 
-    /**
-     * Store cell data in cache for the current cell object if it's "dirty",
-     *     and the 'nullify' the current cell object
-     *
+	/**
+	 * Store cell data in cache for the current cell object if it's "dirty",
+	 *     and the 'nullify' the current cell object
+	 *
 	 * @return	void
-     * @throws	Exception
-     */
+	 * @throws	Exception
+	 */
 	private function _storeData() {
 		if ($this->_currentCellIsDirty) {
 			$this->_currentObject->detach();
@@ -81,14 +81,14 @@ 
 	}	//	function _storeData()
 
 
-    /**
-     * Add or Update a cell in cache identified by coordinate address
-     *
-     * @param	string			$pCoord		Coordinate address of the cell to update
-     * @param	PHPExcel_Cell	$cell		Cell to update
+	/**
+	 * Add or Update a cell in cache identified by coordinate address
+	 *
+	 * @param	string			$pCoord		Coordinate address of the cell to update
+	 * @param	PHPExcel_Cell	$cell		Cell to update
 	 * @return	void
-     * @throws	Exception
-     */
+	 * @throws	Exception
+	 */
 	public function addCacheData($pCoord, PHPExcel_Cell $cell) {
 		if (($pCoord !== $this->_currentObjectID) && ($this->_currentObjectID !== null)) {
 			$this->_storeData();
@@ -130,12 +130,12 @@ 
 
 
 	/**
-     * Get cell at a specific coordinate
-     *
-     * @param 	string 			$pCoord		Coordinate of the cell
-     * @throws 	Exception
-     * @return 	PHPExcel_Cell 	Cell that was found, or null if not found
-     */
+	 * Get cell at a specific coordinate
+	 *
+	 * @param 	string 			$pCoord		Coordinate of the cell
+	 * @throws 	Exception
+	 * @return 	PHPExcel_Cell 	Cell that was found, or null if not found
+	 */
 	public function getCacheData($pCoord) {
 		if ($pCoord === $this->_currentObjectID) {
 			return $this->_currentObject;
@@ -166,12 +166,12 @@ 
 	}	//	function getCacheData()
 
 
-    /**
-     * Delete a cell in cache identified by coordinate address
-     *
-     * @param	string			$pCoord		Coordinate address of the cell to delete
-     * @throws	Exception
-     */
+	/**
+	 * Delete a cell in cache identified by coordinate address
+	 *
+	 * @param	string			$pCoord		Coordinate address of the cell to delete
+	 * @throws	Exception
+	 */
 	public function deleteCacheData($pCoord) {
 		//	Delete the entry from Memcache
 		$this->_memcache->delete($this->_cachePrefix.$pCoord.'.cache');
@@ -264,7 +264,7 @@ 
 	 *
 	 * @param	string	$host		Memcache server
 	 * @param	integer	$port		Memcache port
-     * @throws	Exception
+	 * @throws	Exception
 	 */
 	public function failureCallback($host, $port) {
 		throw new Exception('memcache '.$host.':'.$port.' failed');

Spacing +13 added lines, -13 removed lines

@@ -69,8 +69,8 @@ 
 			$this->_currentObject->detach();
 
 			$obj = serialize($this->_currentObject);
-			if (!$this->_memcache->replace($this->_cachePrefix.$this->_currentObjectID.'.cache',$obj,NULL,$this->_cacheTime)) {
-				if (!$this->_memcache->add($this->_cachePrefix.$this->_currentObjectID.'.cache',$obj,NULL,$this->_cacheTime)) {
+			if ( ! $this->_memcache->replace($this->_cachePrefix.$this->_currentObjectID.'.cache', $obj, NULL, $this->_cacheTime)) {
+				if ( ! $this->_memcache->add($this->_cachePrefix.$this->_currentObjectID.'.cache', $obj, NULL, $this->_cacheTime)) {
 					$this->__destruct();
 					throw new Exception('Failed to store cell '.$this->_currentObjectID.' in MemCache');
 				}
@@ -191,9 +191,9 @@ 
 		parent::copyCellCollection($parent);
 		//	Get a new id for the new file name
 		$baseUnique = $this->_getUniqueID();
-		$newCachePrefix = substr(md5($baseUnique),0,8).'.';
+		$newCachePrefix = substr(md5($baseUnique), 0, 8).'.';
 		$cacheList = $this->getCellList();
-		foreach($cacheList as $cellID) {
+		foreach ($cacheList as $cellID) {
 			if ($cellID != $this->_currentObjectID) {
 				$obj = $this->_memcache->get($this->_cachePrefix.$cellID.'.cache');
 				if ($obj === false) {
@@ -201,7 +201,7 @@ 
 					parent::deleteCacheData($cellID);
 					throw new Exception('Cell entry '.$cellID.' no longer exists in MemCache');
 				}
-				if (!$this->_memcache->add($newCachePrefix.$cellID.'.cache',$obj,NULL,$this->_cacheTime)) {
+				if ( ! $this->_memcache->add($newCachePrefix.$cellID.'.cache', $obj, NULL, $this->_cacheTime)) {
 					$this->__destruct();
 					throw new Exception('Failed to store cell '.$cellID.' in MemCache');
 				}
@@ -217,7 +217,7 @@ 
 	 * @return	void
 	 */
 	public function unsetWorksheetCells() {
-		if(!is_null($this->_currentObject)) {
+		if ( ! is_null($this->_currentObject)) {
 			$this->_currentObject->detach();
 			$this->_currentObject = $this->_currentObjectID = null;
 		}
@@ -239,17 +239,17 @@ 
 	 * @param	array of mixed		$arguments	Additional initialisation arguments
 	 */
 	public function __construct(PHPExcel_Worksheet $parent, $arguments) {
-		$memcacheServer	= (isset($arguments['memcacheServer']))	? $arguments['memcacheServer']	: 'localhost';
-		$memcachePort	= (isset($arguments['memcachePort']))	? $arguments['memcachePort']	: 11211;
-		$cacheTime		= (isset($arguments['cacheTime']))		? $arguments['cacheTime']		: 600;
+		$memcacheServer = (isset($arguments['memcacheServer'])) ? $arguments['memcacheServer'] : 'localhost';
+		$memcachePort = (isset($arguments['memcachePort'])) ? $arguments['memcachePort'] : 11211;
+		$cacheTime = (isset($arguments['cacheTime'])) ? $arguments['cacheTime'] : 600;
 
 		if (is_null($this->_cachePrefix)) {
 			$baseUnique = $this->_getUniqueID();
-			$this->_cachePrefix = substr(md5($baseUnique),0,8).'.';
+			$this->_cachePrefix = substr(md5($baseUnique), 0, 8).'.';
 
 			//	Set a new Memcache object and connect to the Memcache server
 			$this->_memcache = new Memcache();
-			if (!$this->_memcache->addServer($memcacheServer, $memcachePort, false, 50, 5, 5, true, array($this, 'failureCallback'))) {
+			if ( ! $this->_memcache->addServer($memcacheServer, $memcachePort, false, 50, 5, 5, true, array($this, 'failureCallback'))) {
 				throw new Exception('Could not connect to MemCache server at '.$memcacheServer.':'.$memcachePort);
 			}
 			$this->_cacheTime = $cacheTime;
@@ -276,7 +276,7 @@ 
 	 */
 	public function __destruct() {
 		$cacheList = $this->getCellList();
-		foreach($cacheList as $cellID) {
+		foreach ($cacheList as $cellID) {
 			$this->_memcache->delete($this->_cachePrefix.$cellID.'.cache');
 		}
 	}	//	function __destruct()
@@ -288,7 +288,7 @@ 
 	 * @return	boolean
 	 */
 	public static function cacheMethodIsAvailable() {
-		if (!function_exists('memcache_add')) {
+		if ( ! function_exists('memcache_add')) {
 			return false;
 		}