Passed
Pull Request — master (#37)
by Raúl
02:27
created
controllers/front/notify.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
     private function getPmtOrderId()
197 197
     {
198 198
         try {
199
-            $this->pmtOrderId= Db::getInstance()->getValue(
199
+            $this->pmtOrderId = Db::getInstance()->getValue(
200 200
                 'select order_id from '._DB_PREFIX_.'pmt_order where id = '.$this->merchantOrderId
201 201
             );
202 202
 
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
                 Configuration::get('PS_OS_PAYMENT'),
283 283
                 $this->merchantOrder->getOrderTotal(true),
284 284
                 $this->module->displayName,
285
-                'pmtOrderId: ' . $this->pmtOrder->getId(),
285
+                'pmtOrderId: '.$this->pmtOrder->getId(),
286 286
                 array('transaction_id' => $this->pmtOrderId),
287 287
                 null,
288 288
                 false,
@@ -304,8 +304,8 @@  discard block
 block discarded – undo
304 304
             $this->orderClient->confirmOrder($this->pmtOrderId);
305 305
             try {
306 306
                 $this->saveLog(array(
307
-                    'message' => "Order CONFIRMED. " .
308
-                    'The order was confirmed by a ' .
307
+                    'message' => "Order CONFIRMED. ".
308
+                    'The order was confirmed by a '.
309 309
                     ($_SERVER['REQUEST_METHOD'] == 'POST') ? 'NOTIFICATION' : 'REDIRECTION'
310 310
                 ));
311 311
             } catch (\Exception $e) {
@@ -349,7 +349,7 @@  discard block
 block discarded – undo
349 349
     protected function unblockConcurrency()
350 350
     {
351 351
         try {
352
-            Db::getInstance()->delete('pmt_cart_process', 'timestamp < ' . (time() - 6));
352
+            Db::getInstance()->delete('pmt_cart_process', 'timestamp < '.(time() - 6));
353 353
         } catch (\Exception $exception) {
354 354
             throw new ConcurrencyException();
355 355
         }
@@ -367,15 +367,15 @@  discard block
 block discarded – undo
367 367
     {
368 368
         if ($this->merchantOrder && $this->processError === true) {
369 369
             sleep(5);
370
-            $id = (!is_null($this->pmtOrder))?$this->pmtOrder->getId():null;
371
-            $status = (!is_null($this->pmtOrder))?$this->pmtOrder->getStatus():null;
370
+            $id = (!is_null($this->pmtOrder)) ? $this->pmtOrder->getId() : null;
371
+            $status = (!is_null($this->pmtOrder)) ? $this->pmtOrder->getStatus() : null;
372 372
             $this->module->validateOrder(
373 373
                 $this->merchantOrderId,
374 374
                 Configuration::get('PS_OS_ERROR'),
375 375
                 $this->merchantOrder->getOrderTotal(true),
376 376
                 $this->module->displayName,
377
-                ' pmtOrderId: ' . $id .
378
-                ' pmtStatusId:' . $status,
377
+                ' pmtOrderId: '.$id.
378
+                ' pmtStatusId:'.$status,
379 379
                 null,
380 380
                 null,
381 381
                 false,
@@ -411,9 +411,9 @@  discard block
 block discarded – undo
411 411
             'id_cart' => $this->merchantOrderId,
412 412
             'key' => $this->config['secureKey'],
413 413
             'id_module' => $this->module->id,
414
-            'id_order' => ($this->pmtOrder)?$this->pmtOrder->getId(): null,
414
+            'id_order' => ($this->pmtOrder) ? $this->pmtOrder->getId() : null,
415 415
         );
416
-        $url = ($error)? $this->config['urlKO'] : $this->config['urlOK'];
416
+        $url = ($error) ? $this->config['urlKO'] : $this->config['urlOK'];
417 417
         return $this->redirect($url, $parameters);
418 418
     }
419 419
 }
Please login to merge, or discard this patch.