Completed
Push — master ( e84ecd...b63790 )
by Hiraku
04:17
created
src/Aspects/HttpGetRequest.php 2 patches
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -60,6 +60,9 @@  discard block
 block discarded – undo
60 60
         }
61 61
     }
62 62
 
63
+    /**
64
+     * @param string $url
65
+     */
63 66
     public function importURL($url)
64 67
     {
65 68
         $struct = parse_url($url);
@@ -78,6 +81,11 @@  discard block
 block discarded – undo
78 81
     }
79 82
 
80 83
     // utility for __construct
84
+
85
+    /**
86
+     * @param string $key
87
+     * @param string $default
88
+     */
81 89
     private static function setOr(array $struct, $key, $default=null)
82 90
     {
83 91
         if (!empty($struct[$key])) {
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
     public function importURL($url)
64 64
     {
65 65
         $struct = parse_url($url);
66
-        if (! $struct) {
66
+        if (!$struct) {
67 67
             throw new \InvalidArgumentException("$url is not valid URL");
68 68
         }
69 69
 
@@ -74,13 +74,13 @@  discard block
 block discarded – undo
74 74
         $this->username = self::setOr($struct, 'user', null);
75 75
         $this->password = self::setOr($struct, 'pass', null);
76 76
 
77
-        if (! empty($struct['query'])) {
77
+        if (!empty($struct['query'])) {
78 78
             parse_str($struct['query'], $this->query);
79 79
         }
80 80
     }
81 81
 
82 82
     // utility for __construct
83
-    private static function setOr(array $struct, $key, $default=null)
83
+    private static function setOr(array $struct, $key, $default = null)
84 84
     {
85 85
         if (!empty($struct[$key])) {
86 86
             return $struct[$key];
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
             CURLOPT_USERAGENT => $this->genUA(),
101 101
         );
102 102
 
103
-        $curlOpts[CURLOPT_VERBOSE] = (bool) $this->verbose;
103
+        $curlOpts[CURLOPT_VERBOSE] = (bool)$this->verbose;
104 104
 
105 105
         if ($this->username && $this->password) {
106 106
             $curlOpts[CURLOPT_USERPWD] = "$this->username:$this->password";
Please login to merge, or discard this patch.
src/CurlRemoteFilesystem.php 2 patches
Doc Comments   +9 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
     /**
80 80
      * Get the content.
81 81
      *
82
-     * @param string $originUrl The origin URL
82
+     * @param string $origin The origin URL
83 83
      * @param string $fileUrl   The file URL
84 84
      * @param bool   $progress  Display the progression
85 85
      * @param array  $options   Additional context options
@@ -97,6 +97,11 @@  discard block
 block discarded – undo
97 97
         });
98 98
     }
99 99
 
100
+    /**
101
+     * @param string $fileUrl
102
+     * @param boolean $progress
103
+     * @param \Closure $exec
104
+     */
100 105
     protected function fetch($origin, $fileUrl, $progress, $options, $exec)
101 106
     {
102 107
         do {
@@ -216,6 +221,9 @@  discard block
 block discarded – undo
216 221
         return 0;
217 222
     }
218 223
 
224
+    /**
225
+     * @param string $fileName
226
+     */
219 227
     public static function createFile($fileName)
220 228
     {
221 229
         if (is_dir($fileName)) {
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -56,9 +56,9 @@  discard block
 block discarded – undo
56 56
      *
57 57
      * @return bool true
58 58
      */
59
-    public function copy($origin, $fileUrl, $fileName, $progress=true, $options=array())
59
+    public function copy($origin, $fileUrl, $fileName, $progress = true, $options = array())
60 60
     {
61
-        return $this->fetch($origin, $fileUrl, $progress, $options, function ($ch, $request) use ($fileName) {
61
+        return $this->fetch($origin, $fileUrl, $progress, $options, function($ch, $request) use ($fileName) {
62 62
             $fp = $this->createFile($fileName);
63 63
             curl_setopt($ch, CURLOPT_RETURNTRANSFER, false);
64 64
             curl_setopt($ch, CURLOPT_FILE, $fp);
@@ -86,9 +86,9 @@  discard block
 block discarded – undo
86 86
      *
87 87
      * @return bool|string The content
88 88
      */
89
-    public function getContents($origin, $fileUrl, $progress=true, $options=array())
89
+    public function getContents($origin, $fileUrl, $progress = true, $options = array())
90 90
     {
91
-        return $this->fetch($origin, $fileUrl, $progress, $options, function ($ch, $request) {
91
+        return $this->fetch($origin, $fileUrl, $progress, $options, function($ch, $request) {
92 92
             // This order is important.
93 93
             curl_setopt($ch, CURLOPT_FILE, STDOUT);
94 94
             curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
Please login to merge, or discard this patch.
src/Factory.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
      * @param bool $auth
47 47
      * @return resource<curl>
48 48
      */
49
-    public static function getConnection($origin, $auth=false)
49
+    public static function getConnection($origin, $auth = false)
50 50
     {
51 51
         $instance = self::getInstance();
52 52
         if ($auth) {
Please login to merge, or discard this patch.
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,9 +1,9 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
- * hirak/prestissimo
4
- * @author Hiraku NAKANO
5
- * @license MIT https://github.com/hirak/prestissimo
6
- */
3
+     * hirak/prestissimo
4
+     * @author Hiraku NAKANO
5
+     * @license MIT https://github.com/hirak/prestissimo
6
+     */
7 7
 namespace Hirak\Prestissimo;
8 8
 
9 9
 /**
Please login to merge, or discard this patch.
src/Plugin.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
         }
108 108
 
109 109
         $config = $this->config->get('prestissimo');
110
-        if (! is_array($config)) {
110
+        if (!is_array($config)) {
111 111
             $config = array();
112 112
         }
113 113
         $config += array(
@@ -118,16 +118,16 @@  discard block
 block discarded – undo
118 118
             'privatePackages' => array(),
119 119
         );
120 120
 
121
-        if (! is_int($config['maxConnections']) || $config['maxConnections'] < 1) {
121
+        if (!is_int($config['maxConnections']) || $config['maxConnections'] < 1) {
122 122
             $config['maxConnections'] = 6;
123 123
         }
124
-        if (! is_int($config['minConnections']) || $config['minConnections'] > $config['maxConnections']) {
124
+        if (!is_int($config['minConnections']) || $config['minConnections'] > $config['maxConnections']) {
125 125
             $config['minConnections'] = 3;
126 126
         }
127
-        if (! is_bool($config['pipeline'])) {
127
+        if (!is_bool($config['pipeline'])) {
128 128
             $config['pipeline'] = (bool)$config['pipeline'];
129 129
         }
130
-        if (! is_array($config['privatePackages'])) {
130
+        if (!is_array($config['privatePackages'])) {
131 131
             $config['privatePackages'] = (array)$config['privatePackages'];
132 132
         }
133 133
 
Please login to merge, or discard this patch.
src/ParallelDownloader.php 1 patch
Switch Indentation   +38 added lines, -38 removed lines patch added patch discarded remove patch
@@ -116,46 +116,46 @@
 block discarded – undo
116 116
             // wait for any event
117 117
             do {
118 118
                 switch (curl_multi_select($mh, 5)) {
119
-                case -1:
120
-                    usleep(10);
121
-                    do {
122
-                        $stat = curl_multi_exec($mh, $running);
123
-                    } while ($stat === CURLM_CALL_MULTI_PERFORM);
124
-                    continue 2;
125
-                case 0:
126
-                    continue 2;
127
-                default:
128
-                    do {
129
-                        $stat = curl_multi_exec($mh, $running);
130
-                    } while ($stat === CURLM_CALL_MULTI_PERFORM);
131
-
132
-                    do {
133
-                        if ($raised = curl_multi_info_read($mh, $remains)) {
134
-                            $ch = $raised['handle'];
135
-                            $errno = curl_errno($ch);
136
-                            $info = curl_getinfo($ch);
137
-                            curl_setopt($ch, CURLOPT_FILE, STDOUT);
138
-                            $index = (int)$ch;
139
-                            $fileinfo = $chFpMap[$index];
140
-                            unset($chFpMap[$index]);
141
-                            $fp = $fileinfo['fp'];
142
-                            $filepath = $fileinfo['filepath'];
143
-                            fclose($fp);
144
-                            if (CURLE_OK === $errno && 200 === $info['http_code']) {
145
-                                ++$this->successCnt;
146
-                            } else {
147
-                                ++$this->failureCnt;
148
-                                unlink($filepath);
119
+                    case -1:
120
+                        usleep(10);
121
+                        do {
122
+                            $stat = curl_multi_exec($mh, $running);
123
+                        } while ($stat === CURLM_CALL_MULTI_PERFORM);
124
+                        continue 2;
125
+                    case 0:
126
+                        continue 2;
127
+                    default:
128
+                        do {
129
+                            $stat = curl_multi_exec($mh, $running);
130
+                        } while ($stat === CURLM_CALL_MULTI_PERFORM);
131
+
132
+                        do {
133
+                            if ($raised = curl_multi_info_read($mh, $remains)) {
134
+                                $ch = $raised['handle'];
135
+                                $errno = curl_errno($ch);
136
+                                $info = curl_getinfo($ch);
137
+                                curl_setopt($ch, CURLOPT_FILE, STDOUT);
138
+                                $index = (int)$ch;
139
+                                $fileinfo = $chFpMap[$index];
140
+                                unset($chFpMap[$index]);
141
+                                $fp = $fileinfo['fp'];
142
+                                $filepath = $fileinfo['filepath'];
143
+                                fclose($fp);
144
+                                if (CURLE_OK === $errno && 200 === $info['http_code']) {
145
+                                    ++$this->successCnt;
146
+                                } else {
147
+                                    ++$this->failureCnt;
148
+                                    unlink($filepath);
149
+                                }
150
+                                $this->io->write($this->makeDownloadingText($info['url']));
151
+                                curl_multi_remove_handle($mh, $ch);
152
+                                $unused[] = $ch;
149 153
                             }
150
-                            $this->io->write($this->makeDownloadingText($info['url']));
151
-                            curl_multi_remove_handle($mh, $ch);
152
-                            $unused[] = $ch;
153
-                        }
154
-                    } while ($remains);
154
+                        } while ($remains);
155 155
 
156
-                    if ($packages) {
157
-                        break 2;
158
-                    }
156
+                        if ($packages) {
157
+                            break 2;
158
+                        }
159 159
             }
160 160
             } while ($running);
161 161
         } while ($packages);
Please login to merge, or discard this patch.