dshovchko / php_EasyPay

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

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/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));
+                }
+        }
 }

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

src/Provider31/Response/Confirm.php

Indentation +32 added lines, -32 removed lines

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

src/Provider31/Response/Payment.php

Indentation +32 added lines, -32 removed lines

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

src/OpenSSL.php

Indentation +53 added lines, -53 removed lines

@@ -4,76 +4,76 @@
 
 class OpenSSL
 {
-    public function __construct() {}
+        public function __construct() {}
 
-    /**
-     *      verify signature of xml
-     *
-     *      @param string $xml
-     *      @param string $bin_sign
-     *      @param resource $pub_key
-     *
-     *      @return integer result of checking
-     */
-    public function verify($xml, $bin_sign, $pub_key)
-    {
+        /**
+         *      verify signature of xml
+         *
+         *      @param string $xml
+         *      @param string $bin_sign
+         *      @param resource $pub_key
+         *
+         *      @return integer result of checking
+         */
+        public function verify($xml, $bin_sign, $pub_key)
+        {
         return @openssl_verify($xml, $bin_sign, $pub_key);
-    }
+        }
 
-    /**
-     *      generate signature of xml
-     *
-     *      @param string $xml
-     *      @param string $sign
-     *      @param resource $priv_key
-     *
-     *      @return bool result of signing
-     */
-    public function sign($xml, &$sign, $priv_key)
-    {
+        /**
+         *      generate signature of xml
+         *
+         *      @param string $xml
+         *      @param string $sign
+         *      @param resource $priv_key
+         *
+         *      @return bool result of signing
+         */
+        public function sign($xml, &$sign, $priv_key)
+        {
         return @openssl_sign($xml, $sign, $priv_key);
-    }
+        }
 
-    /**
-     *      get public key
-     *
-     *      @param mixed $certificate
-     *      @return resource
-     */
-    public function get_pub_key($certificate)
-    {
+        /**
+         *      get public key
+         *
+         *      @param mixed $certificate
+         *      @return resource
+         */
+        public function get_pub_key($certificate)
+        {
         $pub_key = @openssl_pkey_get_public($certificate);
         $this->is_key($pub_key);
 
         return $pub_key;
-    }
+        }
 
-    /**
-     *      get private key
-     *
-     *      @param mixed $certificate
-     *      @return resource
-     */
-    public function get_priv_key($certificate)
-    {
+        /**
+         *      get private key
+         *
+         *      @param mixed $certificate
+         *      @return resource
+         */
+        public function get_priv_key($certificate)
+        {
         $priv_key = @openssl_pkey_get_private($certificate);
         $this->is_key($priv_key);
 
         return $priv_key;
-    }
+        }
 
-    /**
-     *      check key
-     *
-     *      @param resource $key
-     *      @throws Exception\Runtime
-     */
-    public function is_key($key)
-    {
+        /**
+         *      check key
+         *
+         *      @param resource $key
+         *      @throws Exception\Runtime
+         */
+        public function is_key($key)
+        {
         if ($key === FALSE)
         {
-            throw new Exception\Runtime('Can not extract key from certificate!', -97);
+                throw new Exception\Runtime('Can not extract key from certificate!', -97);
+        }
         }
-    }
 
 }

src/Provider31.php

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