dshovchko / php_EasyPay

src/Exception/Runtime.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
          *      @param string $message
          *      @param \Throwable $previous
          */
-        public function __construct($message, $code=0, \Exception $previous=null)
+        public function __construct($message, $code = 0, \Exception $previous = null)
         {
                 parent::__construct($message, $code, $previous, true);
         }

src/Exception/General.php

Spacing +2 added lines, -2 removed lines

@@ -21,9 +21,9 @@
          *      @param string $message
          *      @param \Throwable $previous
          */
-        public function __construct($message, $code=0, \Exception $previous=null, $tracelog=false)
+        public function __construct($message, $code = 0, \Exception $previous = null, $tracelog = false)
         {
-                Log::instance()->error($message.(($tracelog)?PHP_EOL.$this->getTraceAsString():null));
+                Log::instance()->error($message.(($tracelog) ?PHP_EOL.$this->getTraceAsString() : null));
                 parent::__construct($message, $code, $previous);
         }
 }

src/Key.php

Indentation +38 added lines, -38 removed lines

@@ -4,60 +4,60 @@
 
 class Key
 {
-    public function __construct() {}
-
-    /**
-     *      Get content of key file
-     *      @param string $fname
-     *      @param string $type
-     *      @return string
-     *      @throws Exception\Runtime
-     */
-    public function get($fname, $type)
-    {
+        public function __construct() {}
+
+        /**
+         *      Get content of key file
+         *      @param string $fname
+         *      @param string $type
+         *      @return string
+         *      @throws Exception\Runtime
+         */
+        public function get($fname, $type)
+        {
         try
         {
-            $this->check_exists($fname);
+                $this->check_exists($fname);
 
-            $key = $this->load($fname);
+                $key = $this->load($fname);
         }
         catch (\Exception $e)
         {
-            throw new Exception\Runtime('The file with the '.$type.' key was '.$e->getMessage().'!', -98);
+                throw new Exception\Runtime('The file with the '.$type.' key was '.$e->getMessage().'!', -98);
         }
 
         return $key;
-    }
-
-    /**
-     *      Check if exist key file
-     *
-     *      @param string $fname
-     *      @throws Exception\Runtime
-     */
-    protected function check_exists($fname)
-    {
+        }
+
+        /**
+         *      Check if exist key file
+         *
+         *      @param string $fname
+         *      @throws Exception\Runtime
+         */
+        protected function check_exists($fname)
+        {
         if ( ! file_exists($fname))
         {
-            throw new Exception\Runtime('not exists');
+                throw new Exception\Runtime('not exists');
         }
-    }
-
-    /**
-     *      Load key file
-     *
-     *      @param string $fname
-     *      @return string
-     *      @throws Exception\Runtime
-     */
-    protected function load($fname)
-    {
+        }
+
+        /**
+         *      Load key file
+         *
+         *      @param string $fname
+         *      @return string
+         *      @throws Exception\Runtime
+         */
+        protected function load($fname)
+        {
         $key = @file_get_contents($fname);
         if ($key === FALSE)
         {
-            throw new Exception\Runtime('not read');
+                throw new Exception\Runtime('not read');
         }
 
         return $key;
-    }
+        }
 }

Braces +1 added lines, -2 removed lines

@@ -20,8 +20,7 @@
             $this->check_exists($fname);
 
             $key = $this->load($fname);
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             throw new Exception\Runtime('The file with the '.$type.' key was '.$e->getMessage().'!', -98);
         }

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

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

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 \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();
 
         $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 \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']);
 
         $raw_request = file_get_contents('php://input');
@@ -70,5 +70,5 @@ 
         Log::instance()->debug(' ');
 
         return $raw_request;
-    }
+        }
 }

src/Provider31/Response.php

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

@@ -122,7 +122,9 @@ 
      */
     protected function sign($options)
     {
-        if (isset($this->Sign)) return;
+        if (isset($this->Sign)) {
+                return;
+        }
 
         if (isset($options['UseSign']) && ($options['UseSign'] === true))
         {
@@ -155,8 +157,7 @@ 
             );
 
             return strtoupper(bin2hex($sign));
-        }
-        catch (\Exception $e)
+        } catch (\Exception $e)
         {
             return null;
         }

Indentation +136 added lines, -136 removed lines

@@ -17,194 +17,194 @@
 
 abstract class Response extends \DomDocument
 {
-    /**
-     *      @var string
-     *
-     */
-    const TEMPLATE = '<Response><StatusCode></StatusCode><StatusDetail></StatusDetail><DateTime></DateTime></Response>';
-
-    /**
-     *      @var \DOMNode
-     */
-    protected $Response;
-
-    /**
-     *      @var \DOMElement
-     */
-    protected $Sign;
-
-    /**
-     *      Response constructor
-     *
-     */
-    public function __construct()
-    {
+        /**
+         *      @var string
+         *
+         */
+        const TEMPLATE = '<Response><StatusCode></StatusCode><StatusDetail></StatusDetail><DateTime></DateTime></Response>';
+
+        /**
+         *      @var \DOMNode
+         */
+        protected $Response;
+
+        /**
+         *      @var \DOMElement
+         */
+        protected $Sign;
+
+        /**
+         *      Response constructor
+         *
+         */
+        public function __construct()
+        {
         parent::__construct('1.0', 'UTF-8');
 
         $this->loadXML(self::TEMPLATE);
 
         $this->Response = $this->firstChild;
         $this->set_DateTime();
-    }
+        }
 
-    /**
-     *      Set DateTime element value by current time
-     */
-    public function set_DateTime()
-    {
+        /**
+         *      Set DateTime element value by current time
+         */
+        public function set_DateTime()
+        {
         $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);
-    }
-
-    /**
-     *      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;
 
         return $this->saveXML(NULL, LIBXML_NOEMPTYTAG);
-    }
-
-    /**
-     *      Sign and send response
-     *
-     *      @param array $options
-     */
-    public function sign_and_out($options)
-    {
+        }
+
+        /**
+         *      Sign and send response
+         *
+         *      @param array $options
+         */
+        public function sign_and_out($options)
+        {
         $this->sign($options);
         $this->out_header();
         $this->out_body($this->friendly());
-    }
+        }
 
-    /**
-     *      Send header
-     */
-    protected function out_header()
-    {
+        /**
+         *      Send header
+         */
+        protected function out_header()
+        {
         ob_clean();
         @header("Content-Type: text/xml; charset=utf-8");
-    }
-
-    /**
-     *      Send body
-     *
-     *      @param string $body
-     */
-    protected function out_body($body)
-    {
+        }
+
+        /**
+         *      Send body
+         *
+         *      @param string $body
+         */
+        protected function out_body($body)
+        {
         Log::instance()->debug('response sends: ');
         Log::instance()->debug($body);
 
         echo $body;
-    }
-
-    /**
-     *      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 = $this->createElement('Sign');
-            $this->Response->appendChild($this->Sign);
+                $this->Sign = $this->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)
+        {
         try
         {
-            $sign = '';
-            $this->check_sign_result(
+                $sign = '';
+                $this->check_sign_result(
                 $result = (new OpenSSL())->sign(
-                    $this->friendly(),
-                    $sign,
-                    (new OpenSSL())->get_priv_key($this->get_priv_key($options))
+                        $this->friendly(),
+                        $sign,
+                        (new OpenSSL())->get_priv_key($this->get_priv_key($options))
                 )
-            );
+                );
 
-            return strtoupper(bin2hex($sign));
+                return strtoupper(bin2hex($sign));
         }
         catch (\Exception $e)
         {
-            return null;
+                return null;
+        }
         }
-    }
 
-    /**
-     *      load file with provider private key
-     *
-     *      @param array $options
-     *      @throws Exception\Runtime
-     *      @return string
-     */
-    protected function get_priv_key($options)
-    {
+        /**
+         *      load file with provider private key
+         *
+         *      @param array $options
+         *      @throws Exception\Runtime
+         *      @return string
+         */
+        protected function get_priv_key($options)
+        {
         if ( ! isset($options['ProviderPKey']))
         {
-            throw new \EasyPay\Exception\Runtime('The parameter ProviderPKey is not set!', -94);
+                throw new \EasyPay\Exception\Runtime('The parameter ProviderPKey is not set!', -94);
         }
 
         return (new Key())->get($options['ProviderPKey'], 'private');
-    }
-
-    /**
-     *      check result of openssl sign
-     *
-     *      @param bool $result
-     *      @throws Exception\Sign
-     */
-    protected function check_sign_result($result)
-    {
+        }
+
+        /**
+         *      check result of openssl sign
+         *
+         *      @param bool $result
+         *      @throws Exception\Sign
+         */
+        protected function check_sign_result($result)
+        {
         if ($result === FALSE)
         {
-            throw new \EasyPay\Exception\Sign('Can not generate signature!', -96);
+                throw new \EasyPay\Exception\Sign('Can not generate signature!', -96);
+        }
         }
-    }
 }

src/Provider31.php

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

Indentation +122 added lines, -122 removed lines

@@ -13,207 +13,207 @@
 
 class Provider31
 {
-    /**
-     *      @var array
-     */
-    protected static $options = array(
+        /**
+         *      @var array
+         */
+        protected static $options = array(
         'ServiceId' => 0,
         'UseSign' => false,
         'EasySoftPKey' => '',
         'ProviderPKey' => '',
-    );
-
-    /**
-     *      @var \EasyPay\Callback
-     */
-    protected static $cb;
-
-    /**
-     *      @var mixed
-     */
-    private $request;
-
-    /**
-     *      @var Provider31\Response
-     */
-    private $response;
-
-    /**
-     *      Provider31 constructor
-     *
-     *      @param array $options
-     *      @param Callback $cb
-     *      @param \Debulog\LoggerInterface $log
-     */
-    public function __construct(array $options, \EasyPay\Callback $cb, \Debulog\LoggerInterface $log)
-    {
+        );
+
+        /**
+         *      @var \EasyPay\Callback
+         */
+        protected static $cb;
+
+        /**
+         *      @var mixed
+         */
+        private $request;
+
+        /**
+         *      @var Provider31\Response
+         */
+        private $response;
+
+        /**
+         *      Provider31 constructor
+         *
+         *      @param array $options
+         *      @param Callback $cb
+         *      @param \Debulog\LoggerInterface $log
+         */
+        public function __construct(array $options, \EasyPay\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->sign_and_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':
                 $response = $this->response_check();
                 break;
 
-            case 'Payment':
+                case 'Payment':
                 $response = $this->response_payment();
                 break;
 
-            case 'Confirm':
+                case 'Confirm':
                 $response = $this->response_confirm();
                 break;
 
-            case 'Cancel';
+                case 'Cancel';
                 $response = $this->response_cancel();
                 break;
 
-            default:
+                default:
                 throw new Exception\Structure('There is not supported value of Operation in xml-request!', -99);
         }
 
         return $response;
-    }
-
-    /**
-     *      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);
-    }
+        }
 }

src/Provider31/Response/Check.php

Indentation +37 added lines, -37 removed lines

@@ -16,41 +16,41 @@
 
 final class Check extends Response
 {
-    /**
-     *      @var \DOMElement
-     */
-    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 = $this->createElement('AccountInfo');
-		$this->Response->appendChild($this->AccountInfo);
-
-		foreach($accountinfo as $parameter=>$value)
-		{
-			$this->AccountInfo->appendChild($this->createElement($parameter, $value));
-		}
-	}
+        /**
+         *      @var \DOMElement
+         */
+        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 = $this->createElement('AccountInfo');
+                $this->Response->appendChild($this->AccountInfo);
+
+                foreach($accountinfo as $parameter=>$value)
+                {
+                        $this->AccountInfo->appendChild($this->createElement($parameter, $value));
+                }
+        }
 }

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 		$this->AccountInfo = $this->createElement('AccountInfo');
 		$this->Response->appendChild($this->AccountInfo);
 
-		foreach($accountinfo as $parameter=>$value)
+		foreach ($accountinfo as $parameter=>$value)
 		{
 			$this->AccountInfo->appendChild($this->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 = $this->createElement('AccountInfo');
 		$this->Response->appendChild($this->AccountInfo);

src/Provider31/Response/Cancel.php

Indentation +31 added lines, -31 removed lines

@@ -15,35 +15,35 @@
 
 final class Cancel extends Response
 {
-    /**
-     *      @var \DOMElement
-     */
-    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 = $this->createElement('CancelDate', $canceldate);
-		$this->Response->appendChild($this->CancelDate);
-	}
+        /**
+         *      @var \DOMElement
+         */
+        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 = $this->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 = $this->createElement('CancelDate', $canceldate);
 		$this->Response->appendChild($this->CancelDate);