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