@@ -44,7 +44,7 @@ discard block |
||
| 44 | 44 | /** |
| 45 | 45 | * @var array list of possible operations |
| 46 | 46 | */ |
| 47 | - protected $operations = array('Check','Payment','Confirm','Cancel'); |
|
| 47 | + protected $operations = array('Check', 'Payment', 'Confirm', 'Cancel'); |
|
| 48 | 48 | |
| 49 | 49 | /** |
| 50 | 50 | * General constructor |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | $doc = new \DOMDocument(); |
| 115 | 115 | if ( ! $doc->loadXML($this->raw_request)) |
| 116 | 116 | { |
| 117 | - foreach(libxml_get_errors() as $e){ |
|
| 117 | + foreach (libxml_get_errors() as $e) { |
|
| 118 | 118 | Log::instance()->error($e->message); |
| 119 | 119 | } |
| 120 | 120 | throw new Exception\Structure('The wrong XML is received', -51); |
@@ -226,7 +226,7 @@ discard block |
||
| 226 | 226 | */ |
| 227 | 227 | public function verify_sign($options) |
| 228 | 228 | { |
| 229 | - if (!isset($options['UseSign']) || ($options['UseSign'] === false)) |
|
| 229 | + if ( ! isset($options['UseSign']) || ($options['UseSign'] === false)) |
|
| 230 | 230 | { |
| 231 | 231 | return null; |
| 232 | 232 | } |
@@ -280,9 +280,9 @@ discard block |
||
| 280 | 280 | * |
| 281 | 281 | * @return array nodes with the name |
| 282 | 282 | */ |
| 283 | - protected function getNodes($dom, $name, $ret=array()) |
|
| 283 | + protected function getNodes($dom, $name, $ret = array()) |
|
| 284 | 284 | { |
| 285 | - foreach($dom->childNodes as $child) |
|
| 285 | + foreach ($dom->childNodes as $child) |
|
| 286 | 286 | { |
| 287 | 287 | if ($child->nodeName == $name) |
| 288 | 288 | { |
@@ -133,18 +133,15 @@ discard block |
||
| 133 | 133 | if ($child->nodeName == 'DateTime') |
| 134 | 134 | { |
| 135 | 135 | $this->parse_request_node($child, 'DateTime'); |
| 136 | - } |
|
| 137 | - elseif ($child->nodeName == 'Sign') |
|
| 136 | + } elseif ($child->nodeName == 'Sign') |
|
| 138 | 137 | { |
| 139 | 138 | $this->parse_request_node($child, 'Sign'); |
| 140 | - } |
|
| 141 | - elseif (in_array($child->nodeName, $this->operations)) |
|
| 139 | + } elseif (in_array($child->nodeName, $this->operations)) |
|
| 142 | 140 | { |
| 143 | 141 | if ( ! isset($this->Operation)) |
| 144 | 142 | { |
| 145 | 143 | $this->Operation = $child->nodeName; |
| 146 | - } |
|
| 147 | - else |
|
| 144 | + } else |
|
| 148 | 145 | { |
| 149 | 146 | throw new Exception\Structure('There is more than one Operation type element in the xml-query!', -53); |
| 150 | 147 | } |
@@ -181,8 +178,7 @@ discard block |
||
| 181 | 178 | if ( ! isset($this->$name)) |
| 182 | 179 | { |
| 183 | 180 | $this->$name = $n->nodeValue; |
| 184 | - } |
|
| 185 | - else |
|
| 181 | + } else |
|
| 186 | 182 | { |
| 187 | 183 | throw new Exception\Structure('There is more than one '.$name.' element in the xml-query!', -56); |
| 188 | 184 | } |
@@ -264,8 +260,7 @@ discard block |
||
| 264 | 260 | if ($check == -1) |
| 265 | 261 | { |
| 266 | 262 | throw new Exception\Sign('Error verify signature of request!', -96); |
| 267 | - } |
|
| 268 | - elseif ($check == 0) |
|
| 263 | + } elseif ($check == 0) |
|
| 269 | 264 | { |
| 270 | 265 | throw new Exception\Sign('Signature of request is incorrect!', -95); |
| 271 | 266 | } |
@@ -287,8 +282,7 @@ discard block |
||
| 287 | 282 | if ($child->nodeName == $name) |
| 288 | 283 | { |
| 289 | 284 | array_push($ret, $child); |
| 290 | - } |
|
| 291 | - else |
|
| 285 | + } else |
|
| 292 | 286 | { |
| 293 | 287 | if (count($child->childNodes) > 0) |
| 294 | 288 | { |
@@ -4,60 +4,60 @@ |
||
| 4 | 4 | |
| 5 | 5 | class Key |
| 6 | 6 | { |
| 7 | - public function __construct() {} |
|
| 8 | - |
|
| 9 | - /** |
|
| 10 | - * Get content of key file |
|
| 11 | - * @param string $fname |
|
| 12 | - * @param string $type |
|
| 13 | - * @return string |
|
| 14 | - * @throws Exception\Runtime |
|
| 15 | - */ |
|
| 16 | - public function get($fname, $type) |
|
| 17 | - { |
|
| 7 | + public function __construct() {} |
|
| 8 | + |
|
| 9 | + /** |
|
| 10 | + * Get content of key file |
|
| 11 | + * @param string $fname |
|
| 12 | + * @param string $type |
|
| 13 | + * @return string |
|
| 14 | + * @throws Exception\Runtime |
|
| 15 | + */ |
|
| 16 | + public function get($fname, $type) |
|
| 17 | + { |
|
| 18 | 18 | try |
| 19 | 19 | { |
| 20 | - $this->check_exists($fname); |
|
| 20 | + $this->check_exists($fname); |
|
| 21 | 21 | |
| 22 | - $key = $this->load($fname); |
|
| 22 | + $key = $this->load($fname); |
|
| 23 | 23 | } |
| 24 | 24 | catch (\Exception $e) |
| 25 | 25 | { |
| 26 | - throw new Exception\Runtime('The file with the '.$type.' key was '.$e->getMessage().'!', -98); |
|
| 26 | + throw new Exception\Runtime('The file with the '.$type.' key was '.$e->getMessage().'!', -98); |
|
| 27 | 27 | } |
| 28 | 28 | |
| 29 | 29 | return $key; |
| 30 | - } |
|
| 31 | - |
|
| 32 | - /** |
|
| 33 | - * Check if exist key file |
|
| 34 | - * |
|
| 35 | - * @param string $fname |
|
| 36 | - * @throws Exception\Runtime |
|
| 37 | - */ |
|
| 38 | - protected function check_exists($fname) |
|
| 39 | - { |
|
| 30 | + } |
|
| 31 | + |
|
| 32 | + /** |
|
| 33 | + * Check if exist key file |
|
| 34 | + * |
|
| 35 | + * @param string $fname |
|
| 36 | + * @throws Exception\Runtime |
|
| 37 | + */ |
|
| 38 | + protected function check_exists($fname) |
|
| 39 | + { |
|
| 40 | 40 | if ( ! file_exists($fname)) |
| 41 | 41 | { |
| 42 | - throw new Exception\Runtime('not exists'); |
|
| 42 | + throw new Exception\Runtime('not exists'); |
|
| 43 | 43 | } |
| 44 | - } |
|
| 45 | - |
|
| 46 | - /** |
|
| 47 | - * Load key file |
|
| 48 | - * |
|
| 49 | - * @param string $fname |
|
| 50 | - * @return string |
|
| 51 | - * @throws Exception\Runtime |
|
| 52 | - */ |
|
| 53 | - protected function load($fname) |
|
| 54 | - { |
|
| 44 | + } |
|
| 45 | + |
|
| 46 | + /** |
|
| 47 | + * Load key file |
|
| 48 | + * |
|
| 49 | + * @param string $fname |
|
| 50 | + * @return string |
|
| 51 | + * @throws Exception\Runtime |
|
| 52 | + */ |
|
| 53 | + protected function load($fname) |
|
| 54 | + { |
|
| 55 | 55 | $key = @file_get_contents($fname); |
| 56 | 56 | if ($key === FALSE) |
| 57 | 57 | { |
| 58 | - throw new Exception\Runtime('not read'); |
|
| 58 | + throw new Exception\Runtime('not read'); |
|
| 59 | 59 | } |
| 60 | 60 | |
| 61 | 61 | return $key; |
| 62 | - } |
|
| 62 | + } |
|
| 63 | 63 | } |
@@ -20,8 +20,7 @@ |
||
| 20 | 20 | $this->check_exists($fname); |
| 21 | 21 | |
| 22 | 22 | $key = $this->load($fname); |
| 23 | - } |
|
| 24 | - catch (\Exception $e) |
|
| 23 | + } catch (\Exception $e) |
|
| 25 | 24 | { |
| 26 | 25 | throw new Exception\Runtime('The file with the '.$type.' key was '.$e->getMessage().'!', -98); |
| 27 | 26 | } |