dshovchko / php_EasyPay

src/Provider31/Request/RAW.php

Indentation +71 added lines, -71 removed lines

@@ -16,42 +16,42 @@ 
 
 class RAW
 {
-    /**
-     *      @var string raw request
-     */
-    protected $raw_request;
+        /**
+         *      @var string raw request
+         */
+        protected $raw_request;
 
-    /**
-     *      RAW constructor
-     */
-    public function __construct()
-    {
+        /**
+         *      RAW constructor
+         */
+        public function __construct()
+        {
         $this->get_http_raw_post_data();
         $this->check_presence_request();
-    }
+        }
 
-    /**
-     *      Get request string content
-     *
-     *      @return string
-     */
-    public function str()
-    {
+        /**
+         *      Get request string content
+         *
+         *      @return string
+         */
+        public function str()
+        {
         return strval($this->raw_request);
-    }
+        }
 
-    /**
-     *      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
-     */
-    protected 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
+         */
+        protected function get_http_raw_post_data()
+        {
         Log::instance()->add('request from ' . $_SERVER['REMOTE_ADDR']);
 
         $this->raw_request = file_get_contents('php://input');
@@ -59,68 +59,68 @@ 
         Log::instance()->debug('request received: ');
         Log::instance()->debug($this->raw_request);
         Log::instance()->debug(' ');
-    }
+        }
 
-    /**
-     *      Check if presence request
-     *
-     *      @throws Exception\Structure
-     */
-    protected function check_presence_request()
-    {
+        /**
+         *      Check if presence request
+         *
+         *      @throws Exception\Structure
+         */
+        protected function check_presence_request()
+        {
         if (empty($this->raw_request))
         {
-            throw new Exception\Structure('An empty xml request', -50);
+                throw new Exception\Structure('An empty xml request', -50);
+        }
         }
-    }
 
-    /**
-     *      Get group of nodes from XML-request
-     *
-     *      @param string $name
-     *      @return array
-     */
-    public function get_nodes_from_request($name)
-    {
+        /**
+         *      Get group of nodes from XML-request
+         *
+         *      @param string $name
+         *      @return array
+         */
+        public function get_nodes_from_request($name)
+        {
         libxml_use_internal_errors(true);
         $doc = new \DOMDocument();
         if ( ! $doc->loadXML($this->raw_request))
         {
-            foreach(libxml_get_errors() as $e){
+                foreach(libxml_get_errors() as $e){
                 Log::instance()->error($e->message);
-            }
-            throw new Exception\Structure('The wrong XML is received', -51);
+                }
+                throw new Exception\Structure('The wrong XML is received', -51);
         }
 
         return $this->getNodes($doc, $name);
-    }
+        }
 
-    /**
-     *      Selects nodes by name
-     *
-     *      @param \DOMDocument $dom
-     *      @param string $name
-     *      @param array $ret
-     *
-     *      @return array nodes with the name
-     */
-    protected function getNodes($dom, $name, $ret=array())
-    {
+        /**
+         *      Selects nodes by name
+         *
+         *      @param \DOMDocument $dom
+         *      @param string $name
+         *      @param array $ret
+         *
+         *      @return array nodes with the name
+         */
+        protected function getNodes($dom, $name, $ret=array())
+        {
         foreach($dom->childNodes as $child)
         {
-            if ($child->nodeName == $name)
-            {
+                if ($child->nodeName == $name)
+                {
                 array_push($ret, $child);
-            }
-            else
-            {
+                }
+                else
+                {
                 if (count($child->childNodes) > 0)
                 {
-                    $ret = $this->getNodes($child, $name, $ret);
+                        $ret = $this->getNodes($child, $name, $ret);
+                }
                 }
-            }
         }
 
         return $ret;
-    }
+        }
 }

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
      */
     protected function get_http_raw_post_data()
     {
-        Log::instance()->add('request from ' . $_SERVER['REMOTE_ADDR']);
+        Log::instance()->add('request from '.$_SERVER['REMOTE_ADDR']);
 
         $this->raw_request = file_get_contents('php://input');
 
@@ -86,7 +86,7 @@ 
         $doc = new \DOMDocument();
         if ( ! $doc->loadXML($this->raw_request))
         {
-            foreach(libxml_get_errors() as $e){
+            foreach (libxml_get_errors() as $e) {
                 Log::instance()->error($e->message);
             }
             throw new Exception\Structure('The wrong XML is received', -51);
@@ -104,9 +104,9 @@ 
      *
      *      @return array nodes with the name
      */
-    protected function getNodes($dom, $name, $ret=array())
+    protected function getNodes($dom, $name, $ret = array())
     {
-        foreach($dom->childNodes as $child)
+        foreach ($dom->childNodes as $child)
         {
             if ($child->nodeName == $name)
             {

Braces +1 added lines, -2 removed lines

@@ -111,8 +111,7 @@
             if ($child->nodeName == $name)
             {
                 array_push($ret, $child);
-            }
-            else
+            } else
             {
                 if (count($child->childNodes) > 0)
                 {

src/Provider31/Request/Check.php

Indentation +37 added lines, -37 removed lines

@@ -16,57 +16,57 @@
 
 class Check extends General
 {
-    /**
-     *      @var string 'Account' node
-     */
-    protected $Account;
+        /**
+         *      @var string 'Account' node
+         */
+        protected $Account;
 
-    /**
-     *      Check constructor
-     *
-     *      @param string $raw Raw request data
-     */
-    public function __construct($raw)
-    {
+        /**
+         *      Check constructor
+         *
+         *      @param string $raw Raw request data
+         */
+        public function __construct($raw)
+        {
         parent::__construct($raw);
-    }
+        }
 
-    /**
-     *      Get Account
-     *
-     *      @return string
-     */
-    public function Account()
-    {
+        /**
+         *      Get Account
+         *
+         *      @return string
+         */
+        public function Account()
+        {
         return $this->Account;
-    }
+        }
 
-    /**
-     *      Parse xml-request, which was previously "extracted" from the body of the http request
-     *
-     */
-    protected function parse_request_data()
-    {
+        /**
+         *      Parse xml-request, which was previously "extracted" from the body of the http request
+         *
+         */
+        protected function parse_request_data()
+        {
         parent::parse_request_data();
 
         $r = $this->raw_request->get_nodes_from_request('Check');
 
         foreach ($r[0]->childNodes as $child)
         {
-            $this->check_and_parse_request_node($child, 'Account');
+                $this->check_and_parse_request_node($child, 'Account');
+        }
         }
-    }
 
-    /**
-     *      validate Check request
-     *
-     *      @param array $options
-     *      @throws Exception\Structure
-     */
-    public function validate_request($options)
-    {
+        /**
+         *      validate Check request
+         *
+         *      @param array $options
+         *      @throws Exception\Structure
+         */
+        public function validate_request($options)
+        {
         parent::validate_request($options);
 
         $this->validate_element('Account');
-    }
+        }
 }

src/Provider31/Request/Payment.php

Indentation +63 added lines, -63 removed lines

@@ -16,91 +16,91 @@
 
 class Payment extends General
 {
-    /**
-     *      @var string 'Account' node
-     */
-    protected $Account;
+        /**
+         *      @var string 'Account' node
+         */
+        protected $Account;
 
-    /**
-     *      @var string 'OrderId' node
-     */
-    protected $OrderId;
+        /**
+         *      @var string 'OrderId' node
+         */
+        protected $OrderId;
 
-    /**
-     *      @var string 'Amount' node
-     */
-    protected $Amount;
+        /**
+         *      @var string 'Amount' node
+         */
+        protected $Amount;
 
-    /**
-     *      Payment constructor
-     *
-     *      @param string $raw Raw request data
-     */
-    public function __construct($raw)
-    {
+        /**
+         *      Payment constructor
+         *
+         *      @param string $raw Raw request data
+         */
+        public function __construct($raw)
+        {
         parent::__construct($raw);
-    }
+        }
 
-    /**
-     *      Get Account
-     *
-     *      @return string
-     */
-    public function Account()
-    {
+        /**
+         *      Get Account
+         *
+         *      @return string
+         */
+        public function Account()
+        {
         return $this->Account;
-    }
+        }
 
-    /**
-     *      Get OrderId
-     *
-     *      @return string
-     */
-    public function OrderId()
-    {
+        /**
+         *      Get OrderId
+         *
+         *      @return string
+         */
+        public function OrderId()
+        {
         return $this->OrderId;
-    }
+        }
 
-    /**
-     *      Get Amount
-     *
-     *      @return string
-     */
-    public function Amount()
-    {
+        /**
+         *      Get Amount
+         *
+         *      @return string
+         */
+        public function Amount()
+        {
         return $this->Amount;
-    }
+        }
 
-    /**
-     *      Parse xml-request, which was previously "extracted" from the body of the http request
-     *
-     */
-    protected function parse_request_data()
-    {
+        /**
+         *      Parse xml-request, which was previously "extracted" from the body of the http request
+         *
+         */
+        protected function parse_request_data()
+        {
         parent::parse_request_data();
 
         $r = $this->raw_request->get_nodes_from_request('Payment');
 
         foreach ($r[0]->childNodes as $child)
         {
-            $this->check_and_parse_request_node($child, 'Account');
-            $this->check_and_parse_request_node($child, 'OrderId');
-            $this->check_and_parse_request_node($child, 'Amount');
+                $this->check_and_parse_request_node($child, 'Account');
+                $this->check_and_parse_request_node($child, 'OrderId');
+                $this->check_and_parse_request_node($child, 'Amount');
+        }
         }
-    }
 
-    /**
-     *      validate Payment request
-     *
-     *      @param array $options
-     *      @throws Exception\Structure
-     */
-    public function validate_request($options)
-    {
+        /**
+         *      validate Payment request
+         *
+         *      @param array $options
+         *      @throws Exception\Structure
+         */
+        public function validate_request($options)
+        {
         parent::validate_request($options);
 
         $this->validate_element('Account');
         $this->validate_element('OrderId');
         $this->validate_element('Amount');
-    }
+        }
 }

src/Provider31/Request/General.php

Indentation +178 added lines, -178 removed lines

@@ -19,124 +19,124 @@ 
 
 class General
 {
-    /**
-     *      @var EasyPay\Provider31\Request\RAW raw request
-     */
-    protected $raw_request;
-
-    /**
-     *      @var string 'DateTime' node
-     */
-    protected $DateTime;
-
-    /**
-     *      @var string 'Sign' node
-     */
-    protected $Sign;
-
-    /**
-     *      @var string 'Operation' type
-     */
-    protected $Operation;
-
-    /**
-     *      @var string 'ServiceId' node
-     */
-    protected $ServiceId;
-
-    /**
-     *      @var array list of possible operations
-     */
-    protected $operations = array('Check','Payment','Confirm','Cancel');
-
-    /**
-     *      General constructor
-     *
-     *      @param EasyPay\Provider31\Request\RAW $raw Raw request data
-     */
-    public function __construct($raw)
-    {
+        /**
+         *      @var EasyPay\Provider31\Request\RAW raw request
+         */
+        protected $raw_request;
+
+        /**
+         *      @var string 'DateTime' node
+         */
+        protected $DateTime;
+
+        /**
+         *      @var string 'Sign' node
+         */
+        protected $Sign;
+
+        /**
+         *      @var string 'Operation' type
+         */
+        protected $Operation;
+
+        /**
+         *      @var string 'ServiceId' node
+         */
+        protected $ServiceId;
+
+        /**
+         *      @var array list of possible operations
+         */
+        protected $operations = array('Check','Payment','Confirm','Cancel');
+
+        /**
+         *      General constructor
+         *
+         *      @param EasyPay\Provider31\Request\RAW $raw Raw request data
+         */
+        public function __construct($raw)
+        {
         $this->raw_request = $raw;
 
         $this->parse_request_data();
-    }
-
-    /**
-     *      Get DateTime
-     *
-     *      @return string
-     */
-    public function DateTime()
-    {
+        }
+
+        /**
+         *      Get DateTime
+         *
+         *      @return string
+         */
+        public function DateTime()
+        {
         return $this->DateTime;
-    }
-
-    /**
-     *      Get Sign
-     *
-     *      @return string
-     */
-    public function Sign()
-    {
+        }
+
+        /**
+         *      Get Sign
+         *
+         *      @return string
+         */
+        public function Sign()
+        {
         return $this->Sign;
-    }
-
-    /**
-     *      Get Operation type
-     *
-     *      @return string
-     */
-    public function Operation()
-    {
+        }
+
+        /**
+         *      Get Operation type
+         *
+         *      @return string
+         */
+        public function Operation()
+        {
         return $this->Operation;
-    }
-
-    /**
-     *      Get ServiceId
-     *
-     *      @return string
-     */
-    public function ServiceId()
-    {
+        }
+
+        /**
+         *      Get ServiceId
+         *
+         *      @return string
+         */
+        public function ServiceId()
+        {
         return $this->ServiceId;
-    }
-
-    /**
-     *      Parse xml-request, which was previously "extracted" from the body of the http request
-     *
-     *      @throws Exception\Structure
-     */
-    protected function parse_request_data()
-    {
+        }
+
+        /**
+         *      Parse xml-request, which was previously "extracted" from the body of the http request
+         *
+         *      @throws Exception\Structure
+         */
+        protected function parse_request_data()
+        {
         // process <Request> group
         $r = $this->raw_request->get_nodes_from_request('Request');
 
         if (count($r) < 1)
         {
-            throw new Exception\Structure('The xml-query does not contain any element Request!', -52);
+                throw new Exception\Structure('The xml-query does not contain any element Request!', -52);
         }
 
         foreach ($r[0]->childNodes as $child)
         {
-            $this->check_and_parse_request_node($child, 'DateTime');
-            $this->check_and_parse_request_node($child, 'Sign');
+                $this->check_and_parse_request_node($child, 'DateTime');
+                $this->check_and_parse_request_node($child, 'Sign');
 
-            if (in_array($child->nodeName, $this->operations))
-            {
+                if (in_array($child->nodeName, $this->operations))
+                {
                 if ( ! isset($this->Operation))
                 {
-                    $this->Operation = $child->nodeName;
+                        $this->Operation = $child->nodeName;
                 }
                 else
                 {
-                    throw new Exception\Structure('There is more than one Operation type element in the xml-query!', -53);
+                        throw new Exception\Structure('There is more than one Operation type element in the xml-query!', -53);
+                }
                 }
-            }
         }
 
         if ( ! isset($this->Operation))
         {
-            throw new Exception\Structure('There is no Operation type element in the xml request!', -55);
+                throw new Exception\Structure('There is no Operation type element in the xml request!', -55);
         }
 
         // process <Operation> group
@@ -144,53 +144,53 @@ 
 
         foreach ($r[0]->childNodes as $child)
         {
-            $this->check_and_parse_request_node($child, 'ServiceId');
+                $this->check_and_parse_request_node($child, 'ServiceId');
         }
-    }
-
-    /**
-     *      Check if present node of request and then parse it
-     *
-     *      @param \DOMNode $n
-     *      @param string $name
-     */
-    protected function check_and_parse_request_node($n, $name)
-    {
+        }
+
+        /**
+         *      Check if present node of request and then parse it
+         *
+         *      @param \DOMNode $n
+         *      @param string $name
+         */
+        protected function check_and_parse_request_node($n, $name)
+        {
         if ($n->nodeName == $name)
         {
-            $this->parse_request_node($n, $name);
+                $this->parse_request_node($n, $name);
+        }
         }
-    }
-
-    /**
-     *      Parse node of request
-     *
-     *      @param \DOMNode $n
-     *      @param string $name
-     *
-     *      @throws Exception\Structure
-     */
-    protected function parse_request_node($n, $name)
-    {
+
+        /**
+         *      Parse node of request
+         *
+         *      @param \DOMNode $n
+         *      @param string $name
+         *
+         *      @throws Exception\Structure
+         */
+        protected function parse_request_node($n, $name)
+        {
         if ( ! isset($this->$name))
         {
-            $this->$name = $n->nodeValue;
+                $this->$name = $n->nodeValue;
         }
         else
         {
-            throw new Exception\Structure('There is more than one '.$name.' element in the xml-query!', -56);
+                throw new Exception\Structure('There is more than one '.$name.' element in the xml-query!', -56);
+        }
         }
-    }
-
-    /**
-     *      "Rough" validation of the received xml request
-     *
-     *      @param array $options
-     *      @throws Exception\Data
-     *      @throws Exception\Structure
-     */
-    public function validate_request($options)
-    {
+
+        /**
+         *      "Rough" validation of the received xml request
+         *
+         *      @param array $options
+         *      @throws Exception\Data
+         *      @throws Exception\Structure
+         */
+        public function validate_request($options)
+        {
         $this->validate_element('DateTime');
         $this->validate_element('Sign');
         $this->validate_element('ServiceId');
@@ -198,75 +198,75 @@ 
         // compare received value ServiceId with option ServiceId
         if (intval($options['ServiceId']) != intval($this->ServiceId))
         {
-            throw new Exception\Data('This request is not for our ServiceId!', -58);
+                throw new Exception\Data('This request is not for our ServiceId!', -58);
+        }
         }
-    }
-
-    /**
-     *      Validation of xml-element
-     *
-     *      @param string $name
-     */
-    public function validate_element($name)
-    {
+
+        /**
+         *      Validation of xml-element
+         *
+         *      @param string $name
+         */
+        public function validate_element($name)
+        {
         if ( ! isset($this->$name))
         {
-            throw new Exception\Structure('There is no '.$name.' element in the xml request!', -57);
+                throw new Exception\Structure('There is no '.$name.' element in the xml request!', -57);
         }
-    }
-
-    /**
-     *      Verify signature of request
-     *
-     *      @param array $options
-     */
-    public function verify_sign($options)
-    {
+        }
+
+        /**
+         *      Verify signature of request
+         *
+         *      @param array $options
+         */
+        public function verify_sign($options)
+        {
         if (isset($options['UseSign']) && ($options['UseSign'] === true))
         {
-            $this->check_verify_sign_result(
+                $this->check_verify_sign_result(
                 $result = (new OpenSSL())->verify(
-                    str_replace($this->Sign, '', $this->raw_request->str()),
-                    pack("H*", $this->Sign),
-                    (new OpenSSL())->get_pub_key($this->get_pub_key($options))
+                        str_replace($this->Sign, '', $this->raw_request->str()),
+                        pack("H*", $this->Sign),
+                        (new OpenSSL())->get_pub_key($this->get_pub_key($options))
                 )
-            );
+                );
         }
-    }
-
-    /**
-     *      load file with easysoft public key
-     *
-     *      @param array $options
-     *      @throws Exception\Runtime
-     *      @return string
-     */
-    protected function get_pub_key($options)
-    {
+        }
+
+        /**
+         *      load file with easysoft public key
+         *
+         *      @param array $options
+         *      @throws Exception\Runtime
+         *      @return string
+         */
+        protected function get_pub_key($options)
+        {
         if ( ! isset($options['EasySoftPKey']))
         {
-            throw new Exception\Runtime('The parameter EasySoftPKey is not set!', -94);
+                throw new Exception\Runtime('The parameter EasySoftPKey is not set!', -94);
         }
 
         return (new Key())->get($options['EasySoftPKey'], 'public');
-    }
-
-    /**
-     *      check result of openssl verify signature
-     *
-     *      @param integer $result
-     *      @throws Exception\Sign
-     */
-    protected function check_verify_sign_result($result)
-    {
+        }
+
+        /**
+         *      check result of openssl verify signature
+         *
+         *      @param integer $result
+         *      @throws Exception\Sign
+         */
+        protected function check_verify_sign_result($result)
+        {
         if ($result == -1)
         {
-            throw new Exception\Sign('Error verify signature of request!', -96);
+                throw new Exception\Sign('Error verify signature of request!', -96);
         }
         elseif ($result == 0)
         {
-            throw new Exception\Sign('Signature of request is incorrect!', -95);
+                throw new Exception\Sign('Signature of request is incorrect!', -95);
+        }
         }
-    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     /**
      *      @var array list of possible operations
      */
-    protected $operations = array('Check','Payment','Confirm','Cancel');
+    protected $operations = array('Check', 'Payment', 'Confirm', 'Cancel');
 
     /**
      *      General constructor

Braces +3 added lines, -6 removed lines

@@ -126,8 +126,7 @@ 
                 if ( ! isset($this->Operation))
                 {
                     $this->Operation = $child->nodeName;
-                }
-                else
+                } else
                 {
                     throw new Exception\Structure('There is more than one Operation type element in the xml-query!', -53);
                 }
@@ -175,8 +174,7 @@ 
         if ( ! isset($this->$name))
         {
             $this->$name = $n->nodeValue;
-        }
-        else
+        } else
         {
             throw new Exception\Structure('There is more than one '.$name.' element in the xml-query!', -56);
         }
@@ -262,8 +260,7 @@ 
         if ($result == -1)
         {
             throw new Exception\Sign('Error verify signature of request!', -96);
-        }
-        elseif ($result == 0)
+        } elseif ($result == 0)
         {
             throw new Exception\Sign('Signature of request is incorrect!', -95);
         }

src/Provider31/Request/Confirm.php

Indentation +37 added lines, -37 removed lines

@@ -16,57 +16,57 @@
 
 class Confirm extends General
 {
-    /**
-     *      @var string 'PaymentId' node
-     */
-    protected $PaymentId;
+        /**
+         *      @var string 'PaymentId' node
+         */
+        protected $PaymentId;
 
-    /**
-     *      Check constructor
-     *
-     *      @param string $raw Raw request data
-     */
-    public function __construct($raw)
-    {
+        /**
+         *      Check constructor
+         *
+         *      @param string $raw Raw request data
+         */
+        public function __construct($raw)
+        {
         parent::__construct($raw);
-    }
+        }
 
-    /**
-     *      Get PaymentId
-     *
-     *      @return string
-     */
-    public function PaymentId()
-    {
+        /**
+         *      Get PaymentId
+         *
+         *      @return string
+         */
+        public function PaymentId()
+        {
         return $this->PaymentId;
-    }
+        }
 
-    /**
-     *      Parse xml-request, which was previously "extracted" from the body of the http request
-     *
-     */
-    protected function parse_request_data()
-    {
+        /**
+         *      Parse xml-request, which was previously "extracted" from the body of the http request
+         *
+         */
+        protected function parse_request_data()
+        {
         parent::parse_request_data();
 
         $r = $this->raw_request->get_nodes_from_request('Confirm');
 
         foreach ($r[0]->childNodes as $child)
         {
-            $this->check_and_parse_request_node($child, 'PaymentId');
+                $this->check_and_parse_request_node($child, 'PaymentId');
+        }
         }
-    }
 
-    /**
-     *      validate Confirm request
-     *
-     *      @param array $options
-     *      @throws Exception\Structure
-     */
-    public function validate_request($options)
-    {
+        /**
+         *      validate Confirm request
+         *
+         *      @param array $options
+         *      @throws Exception\Structure
+         */
+        public function validate_request($options)
+        {
         parent::validate_request($options);
 
         $this->validate_element('PaymentId');
-    }
+        }
 }

src/Provider31/Request/Cancel.php

Indentation +37 added lines, -37 removed lines

@@ -16,57 +16,57 @@
 
 class Cancel extends General
 {
-    /**
-     *      @var string 'PaymentId' node
-     */
-    protected $PaymentId;
+        /**
+         *      @var string 'PaymentId' node
+         */
+        protected $PaymentId;
 
-    /**
-     *      Check constructor
-     *
-     *      @param string $raw Raw request data
-     */
-    public function __construct($raw)
-    {
+        /**
+         *      Check constructor
+         *
+         *      @param string $raw Raw request data
+         */
+        public function __construct($raw)
+        {
         parent::__construct($raw);
-    }
+        }
 
-    /**
-     *      Get PaymentId
-     *
-     *      @return string
-     */
-    public function PaymentId()
-    {
+        /**
+         *      Get PaymentId
+         *
+         *      @return string
+         */
+        public function PaymentId()
+        {
         return $this->PaymentId;
-    }
+        }
 
-    /**
-     *      Parse xml-request, which was previously "extracted" from the body of the http request
-     *
-     */
-    protected function parse_request_data()
-    {
+        /**
+         *      Parse xml-request, which was previously "extracted" from the body of the http request
+         *
+         */
+        protected function parse_request_data()
+        {
         parent::parse_request_data();
 
         $r = $this->raw_request->get_nodes_from_request('Cancel');
 
         foreach ($r[0]->childNodes as $child)
         {
-            $this->check_and_parse_request_node($child, 'PaymentId');
+                $this->check_and_parse_request_node($child, 'PaymentId');
+        }
         }
-    }
 
-    /**
-     *      validate Cancel request
-     *
-     *      @param array $options
-     *      @throws Exception\Structure
-     */
-    public function validate_request($options)
-    {
+        /**
+         *      validate Cancel request
+         *
+         *      @param array $options
+         *      @throws Exception\Structure
+         */
+        public function validate_request($options)
+        {
         parent::validate_request($options);
 
         $this->validate_element('PaymentId');
-    }
+        }
 }

src/Provider31/Request.php

Indentation +28 added lines, -28 removed lines

@@ -15,14 +15,14 @@ 
 
 final class Request
 {
-    /**
-     *      static method to create a specific class of request
-     *
-     *      @return Request\General Request class of the appropriate type
-     *      @throws \EasyPay\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 \EasyPay\Exception\Structure
+         */
+        public static function get()
+        {
         //$raw = self::get_http_raw_post_data();
         $raw = new Request\RAW();
 
@@ -30,37 +30,37 @@ 
 
         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 \EasyPay\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']);