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/Provider31/Request/General.php

Spacing +5 added lines, -5 removed lines

@@ -44,7 +44,7 @@ 
         /**
          *      @var array list of possible operations
          */
-        protected $operations = array('Check','Payment','Confirm','Cancel');
+        protected $operations = array('Check', 'Payment', 'Confirm', 'Cancel');
 
         /**
          *      General constructor
@@ -114,7 +114,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);
@@ -226,7 +226,7 @@ 
          */
         public function verify_sign($options)
         {
-                if (!isset($options['UseSign']) || ($options['UseSign'] === false))
+                if ( ! isset($options['UseSign']) || ($options['UseSign'] === false))
                 {
                         return null;
                 }
@@ -280,9 +280,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 +6 added lines, -12 removed lines

@@ -133,18 +133,15 @@ 
                         if ($child->nodeName == 'DateTime')
                         {
                                 $this->parse_request_node($child, 'DateTime');
-                        }
-                        elseif ($child->nodeName == 'Sign')
+                        } elseif ($child->nodeName == 'Sign')
                         {
                                 $this->parse_request_node($child, 'Sign');
-                        }
-                        elseif (in_array($child->nodeName, $this->operations))
+                        } elseif (in_array($child->nodeName, $this->operations))
                         {
                                 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);
                                 }
@@ -181,8 +178,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);
                 }
@@ -264,8 +260,7 @@ 
                 if ($check == -1)
                 {
                         throw new Exception\Sign('Error verify signature of request!', -96);
-                }
-                elseif ($check == 0)
+                } elseif ($check == 0)
                 {
                         throw new Exception\Sign('Signature of request is incorrect!', -95);
                 }
@@ -287,8 +282,7 @@ 
                         if ($child->nodeName == $name)
                         {
                                 array_push($ret, $child);
-                        }
-                        else
+                        } else
                         {
                                 if (count($child->childNodes) > 0)
                                 {

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

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

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

Indentation +101 added lines, -101 removed lines

@@ -16,93 +16,93 @@ 
 
 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;
         //$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: ');
@@ -112,62 +112,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 = $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)
+        {
         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));
-    }
+        }
 }

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 +117 added lines, -117 removed lines

@@ -13,201 +13,201 @@
 
 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\General
-     */
-    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\General
+         */
+        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);
-    }
-
-    /**
-     *      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);
-    }
+        }
 }

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