upyun / php-sdk

src/Upyun/Api/Pretreat.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Upyun\Api;
 
-use GuzzleHttp\Psr7;
 use GuzzleHttp\Client;
 use Upyun\Config;
 use Upyun\Signature;

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
             'accept' => 'json'
         );
 
-        $url = $this->url . '/pretreatment';
+        $url = $this->url.'/pretreatment';
         $signature = Signature::getSignature($this->config, $params, Signature::SIGN_VIDEO);
         $response = $client->request('POST', $url, [
             'headers' => array('Authorization' => "UPYUN {$this->config->operatorName}:$signature"),
@@ -53,7 +53,7 @@ 
             'task_ids' => implode(',', $taskIds)
         );
 
-        $url = $this->url . $path;
+        $url = $this->url.$path;
         $signature = Signature::getSignature($this->config, $params, Signature::SIGN_VIDEO);
         $response = $client->request('GET', $url, [
             'headers' => array('Authorization' => "UPYUN {$this->config->operatorName}:$signature"),

tests/UpyunTest.php

Spacing +11 added lines, -11 removed lines

@@ -3,7 +3,7 @@ 
 use Upyun\Config;
 use Upyun\Upyun;
 
-class UpyunTest extends \PHPUnit_Framework_TestCase{
+class UpyunTest extends \PHPUnit_Framework_TestCase {
 
     /**
      * @var Upyun
@@ -20,7 +20,7 @@ 
         $config->setFormApiKey('Mv83tlocuzkmfKKUFbz2s04FzTw=');
         $config->processNotifyUrl = 'http://localhost:9999';
         self::$upyun        = new Upyun($config);
-        self::$tempFilePath = __DIR__ . '/assets/test.txt';
+        self::$tempFilePath = __DIR__.'/assets/test.txt';
         touch(self::$tempFilePath);
     }
 
@@ -38,8 +38,8 @@ 
 
     public function testWriteStream() {
         $filename = 'test.jpeg';
-        $f = fopen(__DIR__ . '/assets/sample.jpeg', 'rb');
-        if(!$f) {
+        $f = fopen(__DIR__.'/assets/sample.jpeg', 'rb');
+        if (!$f) {
             throw new \Exception('open test file failed!');
         }
         self::$upyun->write($filename, $f);
@@ -50,8 +50,8 @@ 
     public function testWriteWithAsyncProcess() {
         $filename = 'test_async.jpeg';
         $newFilename = 'test_async.png';
-        $f = fopen(__DIR__ . '/assets/sample.jpeg', 'rb');
-        if(!$f) {
+        $f = fopen(__DIR__.'/assets/sample.jpeg', 'rb');
+        if (!$f) {
             throw new \Exception('open test file failed!');
         }
         $result = self::$upyun->write($filename, $f, array(
@@ -72,8 +72,8 @@ 
         $fs = new Upyun(new Config(BUCKET, USER_NAME, 'error-password'));
         try {
             $fs->write('test.txt', 'test file content');
-        } catch(\Exception $e) {
-            return ;
+        } catch (\Exception $e) {
+            return;
         }
         throw new \Exception('should get sign error.');
     }
@@ -104,8 +104,8 @@ 
         $r = self::$upyun->delete('test-delete.txt');
         try {
             self::$upyun->read('test-delete.txt');
-        } catch(\Exception $e) {
-            return ;
+        } catch (\Exception $e) {
+            return;
         }
         throw new \Exception('delete file failed');
     }
@@ -189,7 +189,7 @@ 
 
     public function testProcess() {
         $source = 'php-sdk-sample.mp4';
-        self::$upyun->write($source, fopen(__DIR__ . '/assets/SampleVideo_640x360_1mb.mp4', 'r'));
+        self::$upyun->write($source, fopen(__DIR__.'/assets/SampleVideo_640x360_1mb.mp4', 'r'));
         $result = self::$upyun->process($source, array(
             array('type' => 'video', 'avopts' => '/s/240p(4:3)/as/1/r/30', 'return_info' => true, 'save_as' => '/video/result.mp4')
         ));

tests/config.php

Spacing +1 added lines, -1 removed lines

@@ -3,5 +3,5 @@
     'user_name' => 'tester',
     'pwd' => 'grjxv2mxELR3',
     'bucket' => 'sdkimg',
-    'picture_path' => dirname(__FILE__) . '/assets/sample.jpeg'
+    'picture_path' => dirname(__FILE__).'/assets/sample.jpeg'
 );
\ No newline at end of file

tests/bootstrap.php

Spacing +6 added lines, -6 removed lines

@@ -1,15 +1,15 @@ 
 <?php
-require dirname(__DIR__) . '/vendor/autoload.php';
-$config = require_once __DIR__ . '/config.php';
+require dirname(__DIR__).'/vendor/autoload.php';
+$config = require_once __DIR__.'/config.php';
 
 define('USER_NAME', $config['user_name']);
 define('PWD', $config['pwd']);
 define('BUCKET', $config['bucket']);
-define('PIC_PATH' , $config['picture_path']);
-define('PIC_SIZE' , filesize(PIC_PATH));
+define('PIC_PATH', $config['picture_path']);
+define('PIC_SIZE', filesize(PIC_PATH));
 
 function getFileUrl($path) {
-    return "http://" . BUCKET . ".b0.upaiyun.com/" . ltrim($path, '/');
+    return "http://".BUCKET.".b0.upaiyun.com/".ltrim($path, '/');
 }
 
 function getUpyunFileSize($path) {
@@ -22,7 +22,7 @@ 
     preg_match('~^HTTP/1.1 (\d{3})~', $return, $match1);
     preg_match('~Content-Length: (\d+)~', $return, $match2);
 
-    if(isset($match1[1]) && $match1[1] == 200) {
+    if (isset($match1[1]) && $match1[1] == 200) {
         return isset($match2[1]) ? intval($match2[1]) : false;
     } else {
         return false;

tests/Api/MultiTest.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 use GuzzleHttp\Psr7;
 use Upyun\Upyun;
 
-class MultiTest extends \PHPUnit_Framework_TestCase{
+class MultiTest extends \PHPUnit_Framework_TestCase {
 
     /**
      * @var Multi;
@@ -20,7 +20,7 @@ 
     }
 
     public function testUpload() {
-        $filePath = __DIR__ . '/../assets/sample.jpeg';
+        $filePath = __DIR__.'/../assets/sample.jpeg';
         $stream = Psr7\stream_for(fopen($filePath, 'rb'));
         $r = $this->multiPart->upload('test-sample.jpeg', $stream, md5_file($filePath));
         $this->assertEquals($r->getStatusCode(), 200);

src/Upyun/Config.php

Spacing +4 added lines, -4 removed lines

@@ -82,17 +82,17 @@ 
     /**
      * 分块上传接口请求地址
      */
-    const ED_FORM            = 'm0.api.upyun.com';
+    const ED_FORM = 'm0.api.upyun.com';
 
     /**
      * 异步云处理接口地址
      */
-    const ED_VIDEO           = 'p0.api.upyun.com';
+    const ED_VIDEO = 'p0.api.upyun.com';
 
     /**
      * 刷新接口地址
      */
-    const ED_PURGE           = 'http://purge.upyun.com/purge/';
+    const ED_PURGE = 'http://purge.upyun.com/purge/';
 
     public function __construct($bucketName, $operatorName, $operatorPassword) {
         $this->bucketName = $bucketName;
@@ -107,7 +107,7 @@ 
     }
 
     public function getFormApiKey() {
-        if(! $this->formApiKey) {
+        if (!$this->formApiKey) {
             throw new \Exception('form api key is empty.');
         }
 

src/Upyun/Upyun.php

Spacing +10 added lines, -10 removed lines

@@ -86,7 +86,7 @@ 
      * @throws \Exception 上传失败时，抛出异常
      */
     public function write($path, $content, $params = array(), $withAsyncProcess = false) {
-        if(!$content) {
+        if (!$content) {
             throw new \Exception('write content can not be empty.');
         }
 
@@ -122,8 +122,8 @@ 
         $params = Util::getHeaderParams($response->getHeaders());
 
 
-        if(! isset($params['x-upyun-list-iter'])) {
-            if(is_resource($saveHandler)) {
+        if (!isset($params['x-upyun-list-iter'])) {
+            if (is_resource($saveHandler)) {
                 Psr7\copy_to_stream($response->getBody(), Psr7\stream_for($saveHandler));
                 return true;
             } else {
@@ -150,9 +150,9 @@ 
         try {
             $req->request('HEAD', $path)
                             ->send();
-        } catch(GuzzleHttp\Exception\BadResponseException $e) {
+        } catch (GuzzleHttp\Exception\BadResponseException $e) {
             $statusCode = $e->getResponse()->getStatusCode();
-            if($statusCode === 404) {
+            if ($statusCode === 404) {
                 return false;
             } else {
                 throw $e;
@@ -191,7 +191,7 @@ 
     public function delete($path, $async = false) {
         $req = new Rest($this->config);
         $req->request('DELETE', $path);
-        if($async) {
+        if ($async) {
             $req->withHeader('x-upyun-async', 'true');
         }
         $res = $req->send();
@@ -207,7 +207,7 @@ 
      * @throws \Exception
      */
     public function createDir($path) {
-        $path = rtrim($path, '/') . '/';
+        $path = rtrim($path, '/').'/';
         $req = new Rest($this->config);
         $res = $req->request('POST', $path)
             ->withHeader('folder', 'true')
@@ -236,9 +236,9 @@ 
      */
     public function usage($path = '/') {
 
-        $path = rtrim($path, '/') . '/';
+        $path = rtrim($path, '/').'/';
         $req = new Rest($this->config);
-        $response = $req->request('GET', $path . '?usage')
+        $response = $req->request('GET', $path.'?usage')
             ->withHeader('folder', 'true')
             ->send();
 
@@ -254,7 +254,7 @@ 
      */
     public function purge($urls) {
         $urlString = $urls;
-        if(is_array($urls)) {
+        if (is_array($urls)) {
             $urlString = implode("\n", $urls);
         }
 

src/Upyun/Api/Multi.php

Spacing +7 added lines, -7 removed lines

@@ -17,7 +17,7 @@ 
 
     public function __construct(Config $config) {
         $this->config = $config;
-        $this->url = ($this->config->useSsl ? 'https://' : 'http://') . Config::ED_FORM . '/'.
+        $this->url = ($this->config->useSsl ? 'https://' : 'http://').Config::ED_FORM.'/'.
                      $this->config->bucketName;
     }
 
@@ -31,20 +31,20 @@ 
      * @throws \Exception
      */
     public function upload($path, $stream, $fileHash, $params = []) {
-        $path = '/' . ltrim($path, '/');
+        $path = '/'.ltrim($path, '/');
         $initInfo = $this->initRequest($path, $stream, $fileHash, $params);
         $blockStatus = $initInfo->status;
 
         $newBlockStatus = $blockStatus;
 
-        for($blockId = 0; $blockId < $initInfo->blocks; $blockId++) {
-            if($blockStatus[$blockId] === 0) {
+        for ($blockId = 0; $blockId < $initInfo->blocks; $blockId++) {
+            if ($blockStatus[$blockId] === 0) {
                 $return = $this->blockUpload($initInfo, $blockId, $stream);
                 $newBlockStatus = $return->status;
             }
         }
 
-        if(array_sum($newBlockStatus) === $initInfo->blocks) {
+        if (array_sum($newBlockStatus) === $initInfo->blocks) {
             return $this->endRequest($initInfo, $params);
         } else {
             throw new \Exception(sprintf("chunk upload failed! current every block status is : [%s]", implode(',', $newBlockStatus)));
@@ -102,13 +102,13 @@ 
         );
 
         $multipart = [];
-        foreach($postData as $key => $value) {
+        foreach ($postData as $key => $value) {
            $multipart[] = ['name' => $key, 'contents' => $value];
         }
         $multipart[] = [
             'name' => 'file',
             'contents' => $fileBlock,
-            'filename' => 'file',  //this value must be file
+            'filename' => 'file', //this value must be file
             'headers' => ['Content-Type' => 'application/octet-stream']
         ];
         $postData['file'] = $fileBlock;

src/Upyun/Api/Rest.php

Spacing +5 added lines, -5 removed lines

@@ -26,12 +26,12 @@ 
 
     public function __construct(Config $config) {
         $this->config   = $config;
-        $this->endpoint = Config::$restApiEndPoint . '/' . $config->bucketName;
+        $this->endpoint = Config::$restApiEndPoint.'/'.$config->bucketName;
     }
     
     public function request($method, $storagePath) {
         $this->method = strtoupper($method);
-        $this->storagePath = '/' . ltrim($storagePath, '/');
+        $this->storagePath = '/'.ltrim($storagePath, '/');
         return $this;
     }
 
@@ -56,10 +56,10 @@ 
             'timeout' => $this->config->timeout,
         ]);
 
-        $url = ($this->config->useSsl ? 'https://' : 'http://') . $this->endpoint . $this->storagePath;
+        $url = ($this->config->useSsl ? 'https://' : 'http://').$this->endpoint.$this->storagePath;
         $bodySize = 0;
         $body = null;
-        if($this->file && $this->method === 'PUT') {
+        if ($this->file && $this->method === 'PUT') {
             $bodySize = $this->file->getSize();
             $body = $this->file;
         }
@@ -81,7 +81,7 @@ 
     }
 
     public function withHeaders($headers) {
-        if(is_array($headers)) {
+        if (is_array($headers)) {
             foreach ($headers as $header => $value) {
                 $this->withHeader($header, $value);
             }