Completed
Push — master ( d0121a...036b67 )
by Nicolas
02:10
created
src/utils/S3Utils.php 3 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@  discard block
 block discarded – undo
3 3
  * This class allows you to call the two S3 scripts
4 4
  * to download and upload files.
5 5
  * The scripts are executed using the CommandExecuter
6
-  */
6
+ */
7 7
 
8 8
 require_once __DIR__ . '/CommandExecuter.php';
9 9
 
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
             $cmd,
90 90
             1,
91 91
             array(1 => array("pipe", "w"),
92
-                  2 => array("pipe", "w")),
92
+                    2 => array("pipe", "w")),
93 93
             $callback,
94 94
             $callbackParams, 
95 95
             true,
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -5,7 +5,7 @@  discard block
 block discarded – undo
5 5
  * The scripts are executed using the CommandExecuter
6 6
   */
7 7
 
8
-require_once __DIR__ . '/CommandExecuter.php';
8
+require_once __DIR__.'/CommandExecuter.php';
9 9
 
10 10
 use SA\CpeSdk;
11 11
 
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
         $callbackParams = null,
37 37
         $logKey = null)
38 38
     {   
39
-        $cmd = "php " . __DIR__ . self::GET_FROM_S3;
39
+        $cmd = "php ".__DIR__.self::GET_FROM_S3;
40 40
         $cmd .= " --bucket $bucket";
41 41
         $cmd .= " --file $filename";
42 42
         $cmd .= " --to $saveFileTo";
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
         $callbackParams = null,
61 61
         $logKey = null)
62 62
     {
63
-        $cmd  = "php " . __DIR__ . self::PUT_IN_S3;
63
+        $cmd  = "php ".__DIR__.self::PUT_IN_S3;
64 64
         $cmd .= " --bucket $bucket";
65 65
         $cmd .= " --file $filename";
66 66
         $cmd .= " --from $pathToFileToSend";
Please login to merge, or discard this patch.
Braces   +21 added lines, -14 removed lines patch added patch discarded remove patch
@@ -22,8 +22,9 @@  discard block
 block discarded – undo
22 22
     
23 23
     public function __construct($cpeLogger = null)
24 24
     {
25
-        if (!$cpeLogger)
26
-            $this->cpeLogger = new CpeSdk\CpeLogger(null, 'S3Utils');
25
+        if (!$cpeLogger) {
26
+                    $this->cpeLogger = new CpeSdk\CpeLogger(null, 'S3Utils');
27
+        }
27 28
         $this->cpeLogger = $cpeLogger;
28 29
     }
29 30
 
@@ -64,10 +65,12 @@  discard block
 block discarded – undo
64 65
         $cmd .= " --bucket $bucket";
65 66
         $cmd .= " --file $filename";
66 67
         $cmd .= " --from $pathToFileToSend";
67
-        if ($options['rrs'])
68
-            $cmd .= " --rrs";
69
-        if ($options['encrypt'])
70
-            $cmd .= " --encrypt";
68
+        if ($options['rrs']) {
69
+                    $cmd .= " --rrs";
70
+        }
71
+        if ($options['encrypt']) {
72
+                    $cmd .= " --encrypt";
73
+        }
71 74
     
72 75
         // HAndle execution
73 76
         return ($this->handle_s3_ops(
@@ -96,21 +99,25 @@  discard block
 block discarded – undo
96 99
             2
97 100
         );
98 101
         
99
-        if ($out['outErr'])
100
-            throw new CpeSdk\CpeException($out['outErr'],
102
+        if ($out['outErr']) {
103
+                    throw new CpeSdk\CpeException($out['outErr'],
101 104
                 self::S3_OPS_FAILED);
105
+        }
102 106
 
103
-        if (!$out['out'])
104
-            throw new CpeSdk\CpeException("Script '$caller' didn't return any data !",
107
+        if (!$out['out']) {
108
+                    throw new CpeSdk\CpeException("Script '$caller' didn't return any data !",
105 109
                 self::NO_OUTPUT_DATA);
110
+        }
106 111
     
107
-        if (!($decoded = json_decode($out['out'], true)))
108
-            throw new CpeSdk\CpeException($out['out'],
112
+        if (!($decoded = json_decode($out['out'], true))) {
113
+                    throw new CpeSdk\CpeException($out['out'],
109 114
                 self::S3_OPS_FAILED);
115
+        }
110 116
     
111
-        if ($decoded["status"] == "ERROR")
112
-            throw new CpeSdk\CpeException($decoded["msg"],
117
+        if ($decoded["status"] == "ERROR") {
118
+                    throw new CpeSdk\CpeException($decoded["msg"],
113 119
                 self::S3_OPS_FAILED);
120
+        }
114 121
 
115 122
         return ($decoded);
116 123
     }
Please login to merge, or discard this patch.