reginaldojunior / winners

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,

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;
         }
     }

Indentation +65 added lines, -65 removed lines

@@ -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

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');

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;
         }
     }

Indentation +52 added lines, -52 removed lines

@@ -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;
+		}
+	}
 
 }

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) {

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()]);
 			}
 		}
 

Indentation +52 added lines, -52 removed lines

@@ -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;

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;
 		}
 

app/Vendor/PHPExcel/PHPExcel/CachedObjectStorage/PHPTemp.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,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) {

Indentation +20 added lines, -20 removed lines

@@ -49,13 +49,13 @@ 
 	 */
 	private $_memoryCacheSize = 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();
@@ -72,14 +72,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();
@@ -93,13 +93,13 @@ 
 	}	//	function addCacheData()
 
 
-    /**
-     * 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;

Spacing +13 added lines, -13 removed lines

@@ -60,10 +60,10 @@ 
 		if ($this->_currentCellIsDirty) {
 			$this->_currentObject->detach();
 
-			fseek($this->_fileHandle,0,SEEK_END);
+			fseek($this->_fileHandle, 0, SEEK_END);
 			$offset = ftell($this->_fileHandle);
 			fwrite($this->_fileHandle, serialize($this->_currentObject));
-			$this->_cellCache[$this->_currentObjectID]	= array('ptr' => $offset,
+			$this->_cellCache[$this->_currentObjectID] = array('ptr' => $offset,
 																'sz'  => ftell($this->_fileHandle) - $offset
 															   );
 			$this->_currentCellIsDirty = false;
@@ -107,15 +107,15 @@ 
 		$this->_storeData();
 
 		//	Check if the entry that has been requested actually exists
-		if (!isset($this->_cellCache[$pCoord])) {
+		if ( ! isset($this->_cellCache[$pCoord])) {
 			//	Return null if requested entry doesn't exist in cache
 			return null;
 		}
 
 		//	Set current entry to the requested entry
 		$this->_currentObjectID = $pCoord;
-		fseek($this->_fileHandle,$this->_cellCache[$pCoord]['ptr']);
-		$this->_currentObject = unserialize(fread($this->_fileHandle,$this->_cellCache[$pCoord]['sz']));
+		fseek($this->_fileHandle, $this->_cellCache[$pCoord]['ptr']);
+		$this->_currentObject = unserialize(fread($this->_fileHandle, $this->_cellCache[$pCoord]['sz']));
 		//	Re-attach the parent worksheet
 		$this->_currentObject->attach($this->_parent);
 
@@ -133,11 +133,11 @@ 
 	public function copyCellCollection(PHPExcel_Worksheet $parent) {
 		parent::copyCellCollection($parent);
 		//	Open a new stream for the cell cache data
-		$newFileHandle = fopen('php://temp/maxmemory:'.$this->_memoryCacheSize,'a+');
+		$newFileHandle = fopen('php://temp/maxmemory:'.$this->_memoryCacheSize, 'a+');
 		//	Copy the existing cell cache data to the new stream
-		fseek($this->_fileHandle,0);
-		while (!feof($this->_fileHandle)) {
-			fwrite($newFileHandle,fread($this->_fileHandle, 1024));
+		fseek($this->_fileHandle, 0);
+		while ( ! feof($this->_fileHandle)) {
+			fwrite($newFileHandle, fread($this->_fileHandle, 1024));
 		}
 		$this->_fileHandle = $newFileHandle;
 	}	//	function copyCellCollection()
@@ -149,7 +149,7 @@ 
 	 * @return	void
 	 */
 	public function unsetWorksheetCells() {
-		if(!is_null($this->_currentObject)) {
+		if ( ! is_null($this->_currentObject)) {
 			$this->_currentObject->detach();
 			$this->_currentObject = $this->_currentObjectID = null;
 		}
@@ -170,11 +170,11 @@ 
 	 * @param	array of mixed		$arguments	Additional initialisation arguments
 	 */
 	public function __construct(PHPExcel_Worksheet $parent, $arguments) {
-		$this->_memoryCacheSize	= (isset($arguments['memoryCacheSize']))	? $arguments['memoryCacheSize']	: '1MB';
+		$this->_memoryCacheSize = (isset($arguments['memoryCacheSize'])) ? $arguments['memoryCacheSize'] : '1MB';
 
 		parent::__construct($parent);
 		if (is_null($this->_fileHandle)) {
-			$this->_fileHandle = fopen('php://temp/maxmemory:'.$this->_memoryCacheSize,'a+');
+			$this->_fileHandle = fopen('php://temp/maxmemory:'.$this->_memoryCacheSize, 'a+');
 		}
 	}	//	function __construct()
 
@@ -183,7 +183,7 @@ 
 	 * Destroy this cell collection
 	 */
 	public function __destruct() {
-		if (!is_null($this->_fileHandle)) {
+		if ( ! is_null($this->_fileHandle)) {
 			fclose($this->_fileHandle);
 		}
 		$this->_fileHandle = null;

app/Vendor/PHPExcel/PHPExcel/CachedObjectStorage/Wincache.php

Doc Comments +1 added lines, -1 removed lines

@@ -85,7 +85,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) {

Indentation +6 added lines, -6 removed lines

@@ -50,13 +50,13 @@
 	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
+	 *
 	 * @return	void
-     * @throws	Exception
-     */
+	 * @throws	Exception
+	 */
 	private function _storeData() {
 		if ($this->_currentCellIsDirty) {
 			$this->_currentObject->detach();

Spacing +10 added lines, -10 removed lines

@@ -63,12 +63,12 @@ 
 
 			$obj = serialize($this->_currentObject);
 			if (wincache_ucache_exists($this->_cachePrefix.$this->_currentObjectID.'.cache')) {
-				if (!wincache_ucache_set($this->_cachePrefix.$this->_currentObjectID.'.cache', $obj, $this->_cacheTime)) {
+				if ( ! wincache_ucache_set($this->_cachePrefix.$this->_currentObjectID.'.cache', $obj, $this->_cacheTime)) {
 					$this->__destruct();
 					throw new Exception('Failed to store cell '.$this->_currentObjectID.' in WinCache');
 				}
 			} else {
-				if (!wincache_ucache_add($this->_cachePrefix.$this->_currentObjectID.'.cache', $obj, $this->_cacheTime)) {
+				if ( ! wincache_ucache_add($this->_cachePrefix.$this->_currentObjectID.'.cache', $obj, $this->_cacheTime)) {
 					$this->__destruct();
 					throw new Exception('Failed to store cell '.$this->_currentObjectID.' in WinCache');
 				}
@@ -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) {
 				$success = false;
 				$obj = wincache_ucache_get($this->_cachePrefix.$cellID.'.cache', $success);
@@ -202,7 +202,7 @@ 
 					parent::deleteCacheData($cellID);
 					throw new Exception('Cell entry '.$cellID.' no longer exists in Wincache');
 				}
-				if (!wincache_ucache_add($newCachePrefix.$cellID.'.cache', $obj, $this->_cacheTime)) {
+				if ( ! wincache_ucache_add($newCachePrefix.$cellID.'.cache', $obj, $this->_cacheTime)) {
 					$this->__destruct();
 					throw new Exception('Failed to store cell '.$cellID.' in Wincache');
 				}
@@ -218,7 +218,7 @@ 
 	 * @return	void
 	 */
 	public function unsetWorksheetCells() {
-		if(!is_null($this->_currentObject)) {
+		if ( ! is_null($this->_currentObject)) {
 			$this->_currentObject->detach();
 			$this->_currentObject = $this->_currentObjectID = null;
 		}
@@ -240,11 +240,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 (is_null($this->_cachePrefix)) {
 			$baseUnique = $this->_getUniqueID();
-			$this->_cachePrefix = substr(md5($baseUnique),0,8).'.';
+			$this->_cachePrefix = substr(md5($baseUnique), 0, 8).'.';
 			$this->_cacheTime = $cacheTime;
 
 			parent::__construct($parent);
@@ -257,7 +257,7 @@ 
 	 */
 	public function __destruct() {
 		$cacheList = $this->getCellList();
-		foreach($cacheList as $cellID) {
+		foreach ($cacheList as $cellID) {
 			wincache_ucache_delete($this->_cachePrefix.$cellID.'.cache');
 		}
 	}	//	function __destruct()
@@ -270,7 +270,7 @@ 
 	 * @return	boolean
 	 */
 	public static function cacheMethodIsAvailable() {
-		if (!function_exists('wincache_ucache_add')) {
+		if ( ! function_exists('wincache_ucache_add')) {
 			return false;
 		}