fix compatibility with php 5.3

lihsai0 · lihsai0 · commit 6e7df9cb07ff · 2023-04-13T10:38:52.000+08:00
diff --git a/.github/workflows/test-ci.yml b/.github/workflows/test-ci.yml
@@ -10,12 +10,22 @@ jobs:
       fail-fast: false
       max-parallel: 1
       matrix:
-        php-versions: ['5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
+        php-versions: ['5.3', '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v2
 
+      - name: Setup php for mock server
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.2'
+
+      - name: Setup build-in server
+        run: |
+          nohup php -S localhost:9000 -t ./tests/mock-server/ > phpd.log 2>&1 &
+          echo $! > mock-server.pid
+
       - name: Setup php
         uses: shivammathur/setup-php@v2
         with:
@@ -28,19 +38,22 @@ jobs:
 
       - name: Run cases
         run: |
-          nohup php -S localhost:9000 -t ./tests/mock-server/ > phpd.log 2>&1 &
-          export PHP_SERVER_PID=$!
           ./vendor/bin/phpcs --standard=PSR2 src
           ./vendor/bin/phpcs --standard=PSR2 examples
           ./vendor/bin/phpcs --standard=PSR2 tests
           ./vendor/bin/phpunit --coverage-clover=coverage.xml
-          kill $PHP_SERVER_PID
+          cat mock-server.pid | xargs kill
 
         env:
           QINIU_ACCESS_KEY: ${{ secrets.QINIU_ACCESS_KEY }}
           QINIU_SECRET_KEY: ${{ secrets.QINIU_SECRET_KEY }}
           QINIU_TEST_BUCKET: ${{ secrets.QINIU_TEST_BUCKET }}
           QINIU_TEST_DOMAIN: ${{ secrets.QINIU_TEST_DOMAIN }}
 
+      - name: Print mock servion log
+        if: ${{ failure() }}
+        run: |
+          cat phpd.log
+
       - name: After_success
         run: bash <(curl -s https://codecov.io/bash)
diff --git a/composer.json b/composer.json
@@ -19,12 +19,12 @@
     ],
     "require": {
         "php": ">=5.3.3",
-        "myclabs/php-enum": "~1.6.6 || ~1.7.7 || ~1.8.4"
+        "myclabs/php-enum": "~1.5.2 || ~1.6.6 || ~1.7.7 || ~1.8.4"
     },
     "require-dev": {
         "paragonie/random_compat": ">=2",
         "phpunit/phpunit": "^4.8 || ^5.0 || ^6.0 || ^7.0 || ^8.0 || ^9.3.4",
-        "squizlabs/php_codesniffer": "~3.6"
+        "squizlabs/php_codesniffer": "^2.3 || ~3.6"
     },
     "autoload": {
         "psr-4": {
diff --git a/src/Qiniu/Enum/QiniuEnum.php b/src/Qiniu/Enum/QiniuEnum.php
@@ -1,6 +1,6 @@
 <?php
-// phpcs:disable
-// PSR1.Classes.ClassDeclaration.MultipleClasses
+// @codingStandardsIgnoreStart
+// phpcs:disable PSR1.Classes.ClassDeclaration.MultipleClasses
 
 namespace Qiniu\Enum;
 
@@ -9,6 +9,8 @@
 if (method_exists("MyCLabs\\Enum\\Enum", "from")) {
     abstract class QiniuEnum extends Enum
     {
+        // @codingStandardsIgnoreEnd
+        // @codingStandardsIgnoreStart
     }
 } else {
     /**
@@ -18,6 +20,7 @@ abstract class QiniuEnum extends Enum
      */
     abstract class QiniuEnum extends Enum
     {
+        // @codingStandardsIgnoreEnd
         /**
          * @param mixed $value
          * @return static
@@ -45,5 +48,6 @@ private static function assertValidValueReturningKey($value)
 
             return $key;
         }
+        // @codingStandardsIgnoreStart
     }
 }
diff --git a/src/Qiniu/Http/Header.php b/src/Qiniu/Http/Header.php
@@ -83,7 +83,7 @@ public static function normalizeKey($key)
             return $key;
         }
 
-        return ucwords(strtolower($key), '-');
+        return \Qiniu\ucwords(strtolower($key), '-');
     }
 
     /**
diff --git a/src/Qiniu/Storage/BucketManager.php b/src/Qiniu/Storage/BucketManager.php
@@ -705,7 +705,11 @@ public function changeMime($bucket, $key, $mime)
      *
      * @param string $bucket 待操作资源所在空间
      * @param string $key 待操作资源文件名
-     * @param int $fileType 0 表示标准存储；1 表示低频存储；2 表示归档存储；3 表示深度归档存储
+     * @param int $fileType 对象存储类型
+     *   0 表示标准存储；
+     *   1 表示低频存储；
+     *   2 表示归档存储；
+     *   3 表示深度归档存储；
      *
      * @return array
      * @link  https://developer.qiniu.com/kodo/api/3710/chtype
@@ -931,10 +935,18 @@ public function deleteAfterDays($bucket, $key, $days)
      *
      * @param string $bucket 空间名
      * @param string $key 目标资源
-     * @param int $to_line_after_days 多少天后将文件转为低频存储，设置为 -1 表示取消已设置的转低频存储的生命周期规则， 0 表示不修改转低频生命周期规则。
-     * @param int $to_archive_after_days 多少天后将文件转为归档存储，设置为 -1 表示取消已设置的转归档存储的生命周期规则， 0 表示不修改转归档生命周期规则。
-     * @param int $to_deep_archive_after_days 多少天后将文件转为深度归档存储，设置为 -1 表示取消已设置的转深度归档存储的生命周期规则， 0 表示不修改转深度归档生命周期规则。
-     * @param int $delete_after_days 多少天后将文件删除，设置为 -1 表示取消已设置的删除存储的生命周期规则， 0 表示不修改删除存储的生命周期规则。
+     * @param int $to_line_after_days 多少天后将文件转为低频存储。
+     *   -1 表示取消已设置的转低频存储的生命周期规则；
+     *   0 表示不修改转低频生命周期规则。
+     * @param int $to_archive_after_days 多少天后将文件转为归档存储。
+     *   -1 表示取消已设置的转归档存储的生命周期规则；
+     *   0 表示不修改转归档生命周期规则。
+     * @param int $to_deep_archive_after_days 多少天后将文件转为深度归档存储。
+     *   -1 表示取消已设置的转深度归档存储的生命周期规则；
+     *   0 表示不修改转深度归档生命周期规则。
+     * @param int $delete_after_days 多少天后将文件删除。
+     *   -1 表示取消已设置的删除存储的生命周期规则；
+     *   0 表示不修改删除存储的生命周期规则。
      * @return array
      */
     public function setObjectLifecycle(
@@ -961,11 +973,20 @@ public function setObjectLifecycle(
      *
      * @param string $bucket 空间名
      * @param string $key 目标资源
-     * @param int $to_line_after_days 多少天后将文件转为低频存储，设置为 -1 表示取消已设置的转低频存储的生命周期规则， 0 表示不修改转低频生命周期规则。
-     * @param int $to_archive_after_days 多少天后将文件转为归档存储，设置为 -1 表示取消已设置的转归档存储的生命周期规则， 0 表示不修改转归档生命周期规则。
-     * @param int $to_deep_archive_after_days 多少天后将文件转为深度归档存储，设置为 -1 表示取消已设置的转深度归档存储的生命周期规则， 0 表示不修改转深度归档生命周期规则。
-     * @param int $delete_after_days 多少天后将文件删除，设置为 -1 表示取消已设置的删除存储的生命周期规则， 0 表示不修改删除存储的生命周期规则。
-     * @param array<string, mixed> $cond 匹配条件，只有条件匹配才会设置成功，目前支持：hash、mime、fsize、putTime
+     * @param int $to_line_after_days 多少天后将文件转为低频存储。
+     *   设置为 -1 表示取消已设置的转低频存储的生命周期规则；
+     *   0 表示不修改转低频生命周期规则。
+     * @param int $to_archive_after_days 多少天后将文件转为归档存储。
+     *   -1 表示取消已设置的转归档存储的生命周期规则；
+     *   0 表示不修改转归档生命周期规则。
+     * @param int $to_deep_archive_after_days 多少天后将文件转为深度归档存储。
+     *   -1 表示取消已设置的转深度归档存储的生命周期规则；
+     *   0 表示不修改转深度归档生命周期规则。
+     * @param int $delete_after_days 多少天后将文件删除。
+     *   -1 表示取消已设置的删除存储的生命周期规则；
+     *   0 表示不修改删除存储的生命周期规则。
+     * @param array<string, mixed> $cond 匹配条件，只有条件匹配才会设置成功。
+     *   目前支持：hash、mime、fsize、putTime
      * @return array
      */
     public function setObjectLifecycleWithCond(
@@ -1124,10 +1145,18 @@ public static function buildBatchDeleteAfterDays($bucket, $key_day_pairs)
     /**
      * @param string $bucket 空间名
      * @param array<string> $keys 目标资源
-     * @param int $to_line_after_days 多少天后将文件转为低频存储，设置为 -1 表示取消已设置的转低频存储的生命周期规则， 0 表示不修改转低频生命周期规则。
-     * @param int $to_archive_after_days 多少天后将文件转为归档存储，设置为 -1 表示取消已设置的转归档存储的生命周期规则， 0 表示不修改转归档生命周期规则。
-     * @param int $to_deep_archive_after_days 多少天后将文件转为深度归档存储，设置为 -1 表示取消已设置的转深度归档存储的生命周期规则， 0 表示不修改转深度归档生命周期规则。
-     * @param int $delete_after_days 多少天后将文件删除，设置为 -1 表示取消已设置的删除存储的生命周期规则， 0 表示不修改删除存储的生命周期规则。
+     * @param int $to_line_after_days 多少天后将文件转为低频存储。
+     *   -1 表示取消已设置的转低频存储的生命周期规则；
+     *   0 表示不修改转低频生命周期规则。
+     * @param int $to_archive_after_days 多少天后将文件转为归档存储。
+     *   -1 表示取消已设置的转归档存储的生命周期规则；
+     *   0 表示不修改转归档生命周期规则。
+     * @param int $to_deep_archive_after_days 多少天后将文件转为深度归档存储。
+     *   -1 表示取消已设置的转深度归档存储的生命周期规则；
+     *   0 表示不修改转深度归档生命周期规则。
+     * @param int $delete_after_days 多少天后将文件删除。
+     *   -1 表示取消已设置的删除存储的生命周期规则；
+     *   0 表示不修改删除存储的生命周期规则。
      *
      * @retrun array<string>
      */
diff --git a/src/Qiniu/functions.php b/src/Qiniu/functions.php
@@ -278,4 +278,23 @@ function explodeUpToken($upToken)
         $bucket = $scopeItems[0];
         return array($accessKey, $bucket, null);
     }
+
+    // polyfill ucwords for php version < 5.4.32
+    if (phpversion() < "5.4.32") {
+        function ucwords($str, $delimiters = " \t\r\n\f\v")
+        {
+            $delims = preg_split('//u', $delimiters, -1, PREG_SPLIT_NO_EMPTY);
+
+            foreach ($delims as $delim) {
+                $str = implode($delim, array_map('ucfirst', explode($delim, $str)));
+            }
+
+            return $str;
+        }
+    } else {
+        function ucwords($str, $delimiters)
+        {
+            return \ucwords($str, $delimiters);
+        }
+    }
 }
diff --git a/tests/Qiniu/Tests/ConfigTest.php b/tests/Qiniu/Tests/ConfigTest.php
@@ -1,6 +1,7 @@
 <?php
 
 namespace Qiniu\Tests {
+
     use PHPUnit\Framework\TestCase;
 
     use Qiniu\Config;
diff --git a/tests/Qiniu/Tests/ResumeUpTest.php b/tests/Qiniu/Tests/ResumeUpTest.php
@@ -162,7 +162,7 @@ public function testResumeUploadWithParams()
             $token,
             $key,
             $tempFile,
-            ["x:var_1" => "val_1", "x:var_2" => "val_2", "x-qn-meta-m1" => "val_1", "x-qn-meta-m2" => "val_2"],
+            array("x:var_1" => "val_1", "x:var_2" => "val_2", "x-qn-meta-m1" => "val_1", "x-qn-meta-m2" => "val_2"),
             'application/octet-stream',
             false,
             $resumeFile
@@ -236,7 +236,7 @@ public function testResumeUploadV2WithParams()
             $token,
             $key,
             $tempFile,
-            ["x:var_1" => "val_1", "x:var_2" => "val_2", "x-qn-meta-m1" => "val_1", "x-qn-meta-m2" => "val_2"],
+            array("x:var_1" => "val_1", "x:var_2" => "val_2", "x-qn-meta-m1" => "val_1", "x-qn-meta-m2" => "val_2"),
             'application/octet-stream',
             false,
             $resumeFile,

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`<?php`
`2`		`-// phpcs:disable`
`3`		`-// PSR1.Classes.ClassDeclaration.MultipleClasses`
	`2`	`+// @codingStandardsIgnoreStart`
	`3`	`+// phpcs:disable PSR1.Classes.ClassDeclaration.MultipleClasses`
`4`	`4`
`5`	`5`	`namespace Qiniu\Enum;`
`6`	`6`
`@@ -9,6 +9,8 @@`
`9`	`9`	`if (method_exists("MyCLabs\\Enum\\Enum", "from")) {`
`10`	`10`	`abstract class QiniuEnum extends Enum`
`11`	`11`	`{`
	`12`	`+ // @codingStandardsIgnoreEnd`
	`13`	`+ // @codingStandardsIgnoreStart`
`12`	`14`	`}`
`13`	`15`	`} else {`
`14`	`16`	`/**`
`@@ -18,6 +20,7 @@ abstract class QiniuEnum extends Enum`
`18`	`20`	`*/`
`19`	`21`	`abstract class QiniuEnum extends Enum`
`20`	`22`	`{`
	`23`	`+ // @codingStandardsIgnoreEnd`
`21`	`24`	`/**`
`22`	`25`	`* @param mixed $value`
`23`	`26`	`* @return static`
`@@ -45,5 +48,6 @@ private static function assertValidValueReturningKey($value)`
`45`	`48`
`46`	`49`	`return $key;`
`47`	`50`	`}`
	`51`	`+ // @codingStandardsIgnoreStart`
`48`	`52`	`}`
`49`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ public static function normalizeKey($key)`
`83`	`83`	`return $key;`
`84`	`84`	`}`
`85`	`85`
`86`		`- return ucwords(strtolower($key), '-');`
	`86`	`+ return \Qiniu\ucwords(strtolower($key), '-');`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`/**`