dshovchko / php_EasyPay

src/Provider31/Response/Payment.php

Indentation +32 added lines, -32 removed lines

@@ -15,36 +15,36 @@
 
 final class Payment extends Response
 {
-    /**
-     *      @var string
-     */
-    protected $PaymentId;
-
-	/**
-     *      Payment constructor
-     *
-     *      @param string $paymentid
-     */
-    public function __construct($paymentid)
-    {
-		parent::__construct();
-
-		$this->setElementValue('StatusCode', 0);
-		$this->setElementValue('StatusDetail', 'checked');
-
-		$this->create_PaymentId($paymentid);
-	}
-
-	/**
-     *      Create PaymentId node
-     *
-     *      @param string $paymentid
-     */
-	public function create_PaymentId($paymentid)
-	{
-		if (isset($this->PaymentId)) return;
-
-		$this->PaymentId = self::createElement('PaymentId', $paymentid);
-		$this->Response->appendChild($this->PaymentId);
-	}
+        /**
+         *      @var string
+         */
+        protected $PaymentId;
+
+        /**
+         *      Payment constructor
+         *
+         *      @param string $paymentid
+         */
+        public function __construct($paymentid)
+        {
+                parent::__construct();
+
+                $this->setElementValue('StatusCode', 0);
+                $this->setElementValue('StatusDetail', 'checked');
+
+                $this->create_PaymentId($paymentid);
+        }
+
+        /**
+         *      Create PaymentId node
+         *
+         *      @param string $paymentid
+         */
+        public function create_PaymentId($paymentid)
+        {
+                if (isset($this->PaymentId)) return;
+
+                $this->PaymentId = self::createElement('PaymentId', $paymentid);
+                $this->Response->appendChild($this->PaymentId);
+        }
 }

Braces +3 added lines, -1 removed lines

@@ -42,7 +42,9 @@
      */
 	public function create_PaymentId($paymentid)
 	{
-		if (isset($this->PaymentId)) return;
+		if (isset($this->PaymentId)) {
+		        return;
+		}
 
 		$this->PaymentId = self::createElement('PaymentId', $paymentid);
 		$this->Response->appendChild($this->PaymentId);

src/Provider31/Response/Cancel.php

Indentation +31 added lines, -31 removed lines

@@ -15,35 +15,35 @@
 
 final class Cancel extends Response
 {
-    /**
-     *      @var string
-     */
-    protected $CancelDate;
-
-	/**
-     *      Cancel constructor
-     *
-     *      @param string $orderdate
-     */
-	public function __construct($canceldate) {
-		parent::__construct();
-
-		$this->setElementValue('StatusCode', 0);
-		$this->setElementValue('StatusDetail', 'checked');
-
-		$this->create_CancelDate($canceldate);
-	}
-
-	/**
-     *      Create CancelDate node
-     *
-     *      @param string $canceldate
-     */
-	public function create_CancelDate($canceldate)
-	{
-		if (isset($this->CancelDate)) return;
-
-		$this->CancelDate = self::createElement('CancelDate', $canceldate);
-		$this->Response->appendChild($this->CancelDate);
-	}
+        /**
+         *      @var string
+         */
+        protected $CancelDate;
+
+        /**
+         *      Cancel constructor
+         *
+         *      @param string $orderdate
+         */
+        public function __construct($canceldate) {
+                parent::__construct();
+
+                $this->setElementValue('StatusCode', 0);
+                $this->setElementValue('StatusDetail', 'checked');
+
+                $this->create_CancelDate($canceldate);
+        }
+
+        /**
+         *      Create CancelDate node
+         *
+         *      @param string $canceldate
+         */
+        public function create_CancelDate($canceldate)
+        {
+                if (isset($this->CancelDate)) return;
+
+                $this->CancelDate = self::createElement('CancelDate', $canceldate);
+                $this->Response->appendChild($this->CancelDate);
+        }
 }

Braces +3 added lines, -1 removed lines

@@ -41,7 +41,9 @@
      */
 	public function create_CancelDate($canceldate)
 	{
-		if (isset($this->CancelDate)) return;
+		if (isset($this->CancelDate)) {
+		        return;
+		}
 
 		$this->CancelDate = self::createElement('CancelDate', $canceldate);
 		$this->Response->appendChild($this->CancelDate);

src/Provider31/Response/Confirm.php

Indentation +32 added lines, -32 removed lines

@@ -15,36 +15,36 @@
 
 final class Confirm extends Response
 {
-    /**
-     *      @var string
-     */
-    protected $OrderDate;
-
-	/**
-     *      Confirm constructor
-     *
-     *      @param string $orderdate
-     */
-    public function __construct($orderdate)
-    {
-		parent::__construct();
-
-		$this->setElementValue('StatusCode', 0);
-		$this->setElementValue('StatusDetail', 'checked');
-
-		$this->create_OrderDate($orderdate);
-	}
-
-	/**
-     *      Create OrderDate node
-     *
-     *      @param string $orderdate
-     */
-	public function create_OrderDate($orderdate)
-	{
-		if (isset($this->OrderDate)) return;
-
-		$this->OrderDate = self::createElement('OrderDate', $orderdate);
-		$this->Response->appendChild($this->OrderDate);
-	}
+        /**
+         *      @var string
+         */
+        protected $OrderDate;
+
+        /**
+         *      Confirm constructor
+         *
+         *      @param string $orderdate
+         */
+        public function __construct($orderdate)
+        {
+                parent::__construct();
+
+                $this->setElementValue('StatusCode', 0);
+                $this->setElementValue('StatusDetail', 'checked');
+
+                $this->create_OrderDate($orderdate);
+        }
+
+        /**
+         *      Create OrderDate node
+         *
+         *      @param string $orderdate
+         */
+        public function create_OrderDate($orderdate)
+        {
+                if (isset($this->OrderDate)) return;
+
+                $this->OrderDate = self::createElement('OrderDate', $orderdate);
+                $this->Response->appendChild($this->OrderDate);
+        }
 }

Braces +3 added lines, -1 removed lines

@@ -42,7 +42,9 @@
      */
 	public function create_OrderDate($orderdate)
 	{
-		if (isset($this->OrderDate)) return;
+		if (isset($this->OrderDate)) {
+		        return;
+		}
 
 		$this->OrderDate = self::createElement('OrderDate', $orderdate);
 		$this->Response->appendChild($this->OrderDate);

src/Provider31/Response/Check.php

Indentation +37 added lines, -37 removed lines

@@ -16,41 +16,41 @@
 
 final class Check extends Response
 {
-    /**
-     *      @var string
-     */
-    protected $AccountInfo;
-
-	/**
-     *      Check constructor
-     *
-     *      @param AccountInfo $accountinfo account information set
-     */
-    public function __construct(AccountInfo $accountinfo)
-    {
-		parent::__construct();
-
-		$this->setElementValue('StatusCode', 0);
-		$this->setElementValue('StatusDetail', 'checked');
-
-		$this->create_AccountInfo($accountinfo);
-	}
-
-	/**
-     *      Create AccountInfo node with child nodes
-     *
-     *      @param AccountInfo $accountinfo account information set
-     */
-	public function create_AccountInfo($accountinfo)
-	{
-		if (isset($this->AccountInfo)) return;
-
-		$this->AccountInfo = self::createElement('AccountInfo');
-		$this->Response->appendChild($this->AccountInfo);
-
-		foreach($accountinfo as $parameter=>$value)
-		{
-			$this->AccountInfo->appendChild(self::createElement($parameter, $value));
-		}
-	}
+        /**
+         *      @var string
+         */
+        protected $AccountInfo;
+
+        /**
+         *      Check constructor
+         *
+         *      @param AccountInfo $accountinfo account information set
+         */
+        public function __construct(AccountInfo $accountinfo)
+        {
+                parent::__construct();
+
+                $this->setElementValue('StatusCode', 0);
+                $this->setElementValue('StatusDetail', 'checked');
+
+                $this->create_AccountInfo($accountinfo);
+        }
+
+        /**
+         *      Create AccountInfo node with child nodes
+         *
+         *      @param AccountInfo $accountinfo account information set
+         */
+        public function create_AccountInfo($accountinfo)
+        {
+                if (isset($this->AccountInfo)) return;
+
+                $this->AccountInfo = self::createElement('AccountInfo');
+                $this->Response->appendChild($this->AccountInfo);
+
+                foreach($accountinfo as $parameter=>$value)
+                {
+                        $this->AccountInfo->appendChild(self::createElement($parameter, $value));
+                }
+        }
 }

Braces +3 added lines, -1 removed lines

@@ -43,7 +43,9 @@
      */
 	public function create_AccountInfo($accountinfo)
 	{
-		if (isset($this->AccountInfo)) return;
+		if (isset($this->AccountInfo)) {
+		        return;
+		}
 
 		$this->AccountInfo = self::createElement('AccountInfo');
 		$this->Response->appendChild($this->AccountInfo);

src/Provider31/Response/ErrorInfo.php

Indentation +12 added lines, -12 removed lines

@@ -15,17 +15,17 @@
 
 final class ErrorInfo extends Response
 {
-	/**
-     *      ErrorInfo constructor
-     *
-     *      @param integer $code Error code
-     *      @param string $message Error message text
-     */
-    public function __construct($code, $message)
-    {
-		parent::__construct();
+        /**
+         *      ErrorInfo constructor
+         *
+         *      @param integer $code Error code
+         *      @param string $message Error message text
+         */
+        public function __construct($code, $message)
+        {
+                parent::__construct();
 
-		$this->setElementValue('StatusCode', $code);
-		$this->setElementValue('StatusDetail', $message);
-	}
+                $this->setElementValue('StatusCode', $code);
+                $this->setElementValue('StatusDetail', $message);
+        }
 }

src/Provider31/Request.php

Indentation +30 added lines, -30 removed lines

@@ -15,52 +15,52 @@ 
 
 final class Request
 {
-    /**
-     *      static method to create a specific class of request
-     *
-     *      @return Request\General Request class of the appropriate type
-     *      @throws Exception\Structure
-     */
-    public static function get()
-    {
+        /**
+         *      static method to create a specific class of request
+         *
+         *      @return Request\General Request class of the appropriate type
+         *      @throws Exception\Structure
+         */
+        public static function get()
+        {
         $raw = self::get_http_raw_post_data();
 
         $r = new Request\General($raw);
 
         switch ($r->Operation())
         {
-            case 'Check':
+                case 'Check':
                 return new Request\Check($raw);
 
-            case 'Payment':
+                case 'Payment':
                 return new Request\Payment($raw);
 
-            case 'Confirm':
+                case 'Confirm':
                 return new Request\Confirm($raw);
 
-            case 'Cancel';
+                case 'Cancel';
                 return new Request\Cancel($raw);
 
-            default:
+                default:
                 throw new Exception\Structure('There is not supported value of Operation in xml-request!', -99);
         }
-    }
+        }
 
-    /**
-     *      Get data from the body of the http request
-     *
-     *      - with the appropriate configuration of php.ini they can be found
-     *        in the global variable $HTTP_RAW_POST_DATA
-     *
-     *      - but it's easier just to read the data from the php://input stream,
-     *        which does not depend on the php.ini directives and allows you to read
-     *        raw data from the request body
-     *
-     *      @return string Http raw post data
-     *
-     */
-    private static function get_http_raw_post_data()
-    {
+        /**
+         *      Get data from the body of the http request
+         *
+         *      - with the appropriate configuration of php.ini they can be found
+         *        in the global variable $HTTP_RAW_POST_DATA
+         *
+         *      - but it's easier just to read the data from the php://input stream,
+         *        which does not depend on the php.ini directives and allows you to read
+         *        raw data from the request body
+         *
+         *      @return string Http raw post data
+         *
+         */
+        private static function get_http_raw_post_data()
+        {
         Log::instance()->add('request from ' . $_SERVER['REMOTE_ADDR']);
 
         $raw_request = file_get_contents('php://input');
@@ -70,5 +70,5 @@ 
         Log::instance()->debug(' ');
 
         return $raw_request;
-    }
+        }
 }

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
      */
     private static function get_http_raw_post_data()
     {
-        Log::instance()->add('request from ' . $_SERVER['REMOTE_ADDR']);
+        Log::instance()->add('request from '.$_SERVER['REMOTE_ADDR']);
 
         $raw_request = file_get_contents('php://input');
 

src/Provider31/Response.php

Indentation +105 added lines, -105 removed lines

@@ -16,96 +16,96 @@ 
 
 abstract class Response extends \DomDocument
 {
-    /**
-     *      @var string
-     *
-     */
-    const TEMPLATE = '<Response><StatusCode></StatusCode><StatusDetail></StatusDetail><DateTime></DateTime></Response>';
-
-    /**
-     *      @var string
-     */
-    protected $Response;
-
-    /**
-     *      @var string
-     */
-    protected $Sign;
-
-    /**
-     *      Response constructor
-     *
-     */
-    public function __construct()
-    {
+        /**
+         *      @var string
+         *
+         */
+        const TEMPLATE = '<Response><StatusCode></StatusCode><StatusDetail></StatusDetail><DateTime></DateTime></Response>';
+
+        /**
+         *      @var string
+         */
+        protected $Response;
+
+        /**
+         *      @var string
+         */
+        protected $Sign;
+
+        /**
+         *      Response constructor
+         *
+         */
+        public function __construct()
+        {
         parent::__construct('1.0', 'UTF-8');
 
         self::loadXML(self::TEMPLATE);
 
         $this->Response = $this->firstChild;
         $this->setElementValue('DateTime', date('Y-m-d\TH:i:s', time()));
-    }
-
-    /**
-     *      Create new element node
-     *
-     *      @param string $name
-     *      @param string $value (optional)
-     */
-    public function createElement($name, $value=NULL)
-    {
+        }
+
+        /**
+         *      Create new element node
+         *
+         *      @param string $name
+         *      @param string $value (optional)
+         */
+        public function createElement($name, $value=NULL)
+        {
         return parent::createElement($name, $value);
-    }
-
-    /**
-     *      Create new node attribute
-     *
-     *      @param string $name
-     *      @param string $value
-     */
-    public function create_attr($name, $value)
-    {
+        }
+
+        /**
+         *      Create new node attribute
+         *
+         *      @param string $name
+         *      @param string $value
+         */
+        public function create_attr($name, $value)
+        {
         return new DOMAttr($name, $value);
-    }
-
-    /**
-     *      Set node value
-     *
-     *      @param string $name
-     *      @param string $value
-     */
-    public function setElementValue($name, $value)
-    {
+        }
+
+        /**
+         *      Set node value
+         *
+         *      @param string $name
+         *      @param string $value
+         */
+        public function setElementValue($name, $value)
+        {
         foreach ($this->Response->childNodes as $child)
         {
-            if ($child->nodeName == $name)
-            {
+                if ($child->nodeName == $name)
+                {
                 $child->nodeValue = $value;
-            }
+                }
+        }
         }
-    }
-
-    /**
-     *      Dumps response into a string
-     *
-     *      @return string XML
-     */
-    public function friendly()
-    {
+
+        /**
+         *      Dumps response into a string
+         *
+         *      @return string XML
+         */
+        public function friendly()
+        {
         $this->encoding = 'UTF-8';
         $this->formatOutput = true;
         //$this->save('/tmp/test1.xml');
 
         return $this->saveXML(NULL, LIBXML_NOEMPTYTAG);
-    }
-
-    /**
-     *      Send response
-     *
-     *      @param array $options
-     */
-    public function out($options)
-    {
+        }
+
+        /**
+         *      Send response
+         *
+         *      @param array $options
+         */
+        public function out($options)
+        {
         $this->sign($options);
 
         Log::instance()->debug('response sends: ');
@@ -115,62 +115,62 @@ 
         header("Content-Type: text/xml; charset=utf-8");
         echo $this->friendly();
         exit;
-    }
-
-    /**
-     *      Add Sign (if hasn't yet done)
-     *
-     *      @param array $options
-     */
-    protected function sign($options)
-    {
+        }
+
+        /**
+         *      Add Sign (if hasn't yet done)
+         *
+         *      @param array $options
+         */
+        protected function sign($options)
+        {
         if (isset($this->Sign)) return;
 
         if (isset($options['UseSign']) && ($options['UseSign'] === true))
         {
-            $this->Sign = self::createElement('Sign');
-            $this->Response->appendChild($this->Sign);
+                $this->Sign = self::createElement('Sign');
+                $this->Response->appendChild($this->Sign);
 
-            $sign = $this->generate_sign($options);
+                $sign = $this->generate_sign($options);
 
-            $this->Sign->nodeValue = $sign;
+                $this->Sign->nodeValue = $sign;
+        }
         }
-    }
-
-    /**
-     *      Generate signature of response
-     *
-     *      @param array $options
-     *      @return string
-     */
-    public function generate_sign($options)
-    {
+
+        /**
+         *      Generate signature of response
+         *
+         *      @param array $options
+         *      @return string
+         */
+        public function generate_sign($options)
+        {
         if ( ! isset($options['ProviderPKey']))
         {
-            Log::instance()->error('The parameter ProviderPKey is not set!');
-            return null;
+                Log::instance()->error('The parameter ProviderPKey is not set!');
+                return null;
         }
         try
         {
-            $pkeyid = (new Key())->get($options['ProviderPKey'], 'private');
+                $pkeyid = (new Key())->get($options['ProviderPKey'], 'private');
         }
         catch (\Exception $e)
         {
-            return null;
+                return null;
         }
 
         $pr_key = openssl_pkey_get_private($pkeyid);
         if ($pr_key === FALSE)
         {
-            Log::instance()->error('Can not extract the private key from certificate!');
-            return null;
+                Log::instance()->error('Can not extract the private key from certificate!');
+                return null;
         }
         if (openssl_sign($this->friendly(), $sign, $pr_key) === FALSE)
         {
-            Log::instance()->error('Can not generate signature!');
-            return null;
+                Log::instance()->error('Can not generate signature!');
+                return null;
         }
 
         return strtoupper(bin2hex($sign));
-    }
+        }
 }

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      *      @param string $name
      *      @param string $value (optional)
      */
-    public function createElement($name, $value=NULL)
+    public function createElement($name, $value = NULL)
     {
         return parent::createElement($name, $value);
     }

Braces +4 added lines, -3 removed lines

@@ -124,7 +124,9 @@ 
      */
     protected function sign($options)
     {
-        if (isset($this->Sign)) return;
+        if (isset($this->Sign)) {
+                return;
+        }
 
         if (isset($options['UseSign']) && ($options['UseSign'] === true))
         {
@@ -153,8 +155,7 @@ 
         try
         {
             $pkeyid = (new Key())->get($options['ProviderPKey'], 'private');
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             return null;
         }

src/Provider31.php

Indentation +117 added lines, -117 removed lines

@@ -13,202 +13,202 @@
 
 class Provider31
 {
-    /**
-     *      @var array
-     */
-    protected static $options = array(
+        /**
+         *      @var array
+         */
+        protected static $options = array(
         'ServiceId' => 0,
         'UseSign' => false,
         'EasySoftPKey' => '',
         'ProviderPKey' => '',
-    );
-
-    /**
-     *      @var Callback
-     */
-    protected static $cb;
-
-    /**
-     *      @var Request
-     */
-    private $request;
-
-    /**
-     *      Provider31 constructor
-     *
-     *      @param array $options
-     *      @param Callback $cb
-     *      @param \Debulog\LoggerInterface $log
-     */
-    public function __construct(array $options, Callback $cb, \Debulog\LoggerInterface $log)
-    {
+        );
+
+        /**
+         *      @var Callback
+         */
+        protected static $cb;
+
+        /**
+         *      @var Request
+         */
+        private $request;
+
+        /**
+         *      Provider31 constructor
+         *
+         *      @param array $options
+         *      @param Callback $cb
+         *      @param \Debulog\LoggerInterface $log
+         */
+        public function __construct(array $options, Callback $cb, \Debulog\LoggerInterface $log)
+        {
         self::$options = array_merge(self::$options, $options);
         self::$cb = $cb;
 
         Log::set($log);
-    }
-
-    /**
-     *      Get and process request, echo response
-     *
-     */
-    public function process()
-    {
+        }
+
+        /**
+         *      Get and process request, echo response
+         *
+         */
+        public function process()
+        {
         try
         {
-            //      get request
-            $this->request = Provider31\Request::get();
+                //      get request
+                $this->request = Provider31\Request::get();
 
-            //      validate request
-            $this->request->validate_request(self::$options);
-            Log::instance()->debug('request is valid');
+                //      validate request
+                $this->request->validate_request(self::$options);
+                Log::instance()->debug('request is valid');
 
-            //      verify sign
-            $this->request->verify_sign(self::$options);
-            Log::instance()->debug('signature of request is correct');
+                //      verify sign
+                $this->request->verify_sign(self::$options);
+                Log::instance()->debug('signature of request is correct');
 
-            //      get response
-            $this->response = $this->get_response();
+                //      get response
+                $this->response = $this->get_response();
 
-            Log::instance()->add('the request was processed successfully');
+                Log::instance()->add('the request was processed successfully');
         }
         catch (Exception\Structure $e)
         {
-            $this->response = $this->get_error_response($e->getCode(), 'Error in request');
+                $this->response = $this->get_error_response($e->getCode(), 'Error in request');
         }
         catch (Exception\Sign $e)
         {
-            $this->response = $this->get_error_response($e->getCode(), 'Signature error!');
+                $this->response = $this->get_error_response($e->getCode(), 'Signature error!');
         }
         catch (Exception\Runtime $e)
         {
-            $this->response = $this->get_error_response($e->getCode(), 'Error while processing request');
+                $this->response = $this->get_error_response($e->getCode(), 'Error while processing request');
         }
         catch (\Exception $e)
         {
-            $this->response = $this->get_error_response($e->getCode(), $e->getMessage());
+                $this->response = $this->get_error_response($e->getCode(), $e->getMessage());
         }
 
         //      output response
         $this->response->out(self::$options);
         exit;
-    }
-
-    /**
-     *      Process request and generate response
-     *
-     *      @throws Exception\Structure
-     */
-    private function get_response()
-    {
+        }
+
+        /**
+         *      Process request and generate response
+         *
+         *      @throws Exception\Structure
+         */
+        private function get_response()
+        {
         switch ($this->request->Operation())
         {
-            case 'Check':
+                case 'Check':
 
                 return $this->response_check();
 
-            case 'Payment':
+                case 'Payment':
 
                 return $this->response_payment();
 
-            case 'Confirm':
+                case 'Confirm':
 
                 return $this->response_confirm();
 
-            case 'Cancel';
+                case 'Cancel';
 
                 return $this->response_cancel();
 
-            default:
+                default:
                 break;
         }
 
         throw new Exception\Structure('There is not supported value of Operation in xml-request!', -99);
-    }
-
-    /**
-     *      run check callback and generate a response
-     *
-     *      @return Provider31\Response\Check
-     */
-    private function response_check()
-    {
+        }
+
+        /**
+         *      run check callback and generate a response
+         *
+         *      @return Provider31\Response\Check
+         */
+        private function response_check()
+        {
         Log::instance()->add(sprintf('Check("%s")', $this->request->Account()));
 
         $accountinfo = self::$cb->check(
-            $this->request->Account()
+                $this->request->Account()
         );
 
         // Sending a response
         return new Provider31\Response\Check($accountinfo);
-    }
-
-    /**
-     *      run payment callback and generate a response
-     *
-     *      @return Provider31\Response\Payment
-     */
-    private function response_payment()
-    {
+        }
+
+        /**
+         *      run payment callback and generate a response
+         *
+         *      @return Provider31\Response\Payment
+         */
+        private function response_payment()
+        {
         Log::instance()->add(sprintf('Payment("%s", "%s", "%s")', $this->request->Account(), $this->request->OrderId(), $this->request->Amount()));
 
         $paymentid = self::$cb->payment(
-            $this->request->Account(),
-            $this->request->OrderId(),
-            $this->request->Amount()
+                $this->request->Account(),
+                $this->request->OrderId(),
+                $this->request->Amount()
         );
 
         // Sending a response
         return new Provider31\Response\Payment($paymentid);
-    }
-
-    /**
-     *      run confirm callback and generate a response
-     *
-     *      @return Provider31\Response\Confirm
-     */
-    private function response_confirm()
-    {
+        }
+
+        /**
+         *      run confirm callback and generate a response
+         *
+         *      @return Provider31\Response\Confirm
+         */
+        private function response_confirm()
+        {
         Log::instance()->add(sprintf('Confirm("%s")', $this->request->PaymentId()));
 
         $orderdate = self::$cb->confirm(
-            $this->request->PaymentId()
+                $this->request->PaymentId()
         );
 
         // Sending a response
         return new Provider31\Response\Confirm($orderdate);
-    }
-
-    /**
-     *      run cancel callback and generate a response
-     *
-     *      @return Provider31\Response\Cancel
-     */
-    private function response_cancel()
-    {
+        }
+
+        /**
+         *      run cancel callback and generate a response
+         *
+         *      @return Provider31\Response\Cancel
+         */
+        private function response_cancel()
+        {
         Log::instance()->add(sprintf('Cancel("%s")', $this->request->PaymentId()));
 
         $canceldate = self::$cb->cancel(
-            $this->request->PaymentId()
+                $this->request->PaymentId()
         );
 
         // Sending a response
         return new Provider31\Response\Cancel($canceldate);
-    }
-
-    /**
-     *      Generates an xml with an error message
-     *
-     *      @param integer $code
-     *      @param string $message
-     *
-     *      @return Provider31\Response\ErrorInfo
-     */
-    private function get_error_response($code, $message)
-    {
+        }
+
+        /**
+         *      Generates an xml with an error message
+         *
+         *      @param integer $code
+         *      @param string $message
+         *
+         *      @return Provider31\Response\ErrorInfo
+         */
+        private function get_error_response($code, $message)
+        {
         Log::instance()->add('the request was processed with an error');
 
         // Sending a response
         return new Provider31\Response\ErrorInfo($code, $message);
-    }
+        }
 }

Braces +4 added lines, -8 removed lines

@@ -71,20 +71,16 @@
             $this->response = $this->get_response();
 
             Log::instance()->add('the request was processed successfully');
-        }
-        catch (Exception\Structure $e)
+        } catch (Exception\Structure $e)
         {
             $this->response = $this->get_error_response($e->getCode(), 'Error in request');
-        }
-        catch (Exception\Sign $e)
+        } catch (Exception\Sign $e)
         {
             $this->response = $this->get_error_response($e->getCode(), 'Signature error!');
-        }
-        catch (Exception\Runtime $e)
+        } catch (Exception\Runtime $e)
         {
             $this->response = $this->get_error_response($e->getCode(), 'Error while processing request');
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             $this->response = $this->get_error_response($e->getCode(), $e->getMessage());
         }