BREAKING CHANGE: upgrade to PHP 8.0 and PHP-Casbin 4.0

php-casbin · Nov 29, 2024 · 1f66504 · 1f66504
1 parent d3c171a
commit 1f66504
Show file tree

Hide file tree

Showing 7 changed files with 39 additions and 47 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -23,11 +23,9 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [ 7.2, 7.3,7.4,8.0 ]
+        php: [ 8.1, 8.2, 8.3, 8.4 ]        
 
-        stability: [ prefer-lowest, prefer-stable ]            
-
-    name: PHP ${{ matrix.php }} - ${{ matrix.stability }}
+    name: PHP ${{ matrix.php }}
 
     steps:
       - name: Checkout code
@@ -57,7 +55,7 @@ jobs:
           COVERALLS_PARALLEL: true
           COVERALLS_FLAG_NAME: ${{ runner.os }} - ${{ matrix.php }}
         run: |
-          composer global require php-coveralls/php-coveralls:^2.4
+          composer global require php-coveralls/php-coveralls:^2.7
           php-coveralls --coverage_clover=build/logs/clover.xml -v
 
   upload-coverage:

diff --git a/composer.json b/composer.json
@@ -22,19 +22,17 @@
         "test": "vendor/bin/phpunit"
     },
     "require": {
-        "php": ">=7.2",
-        "codeigniter4/framework": "^4",
-        "casbin/casbin": "^3.21.6",
-        "casbin/psr3-bridge": "^1.1"
+        "php": "^8.1",
+        "codeigniter4/framework": "^4.5",
+        "casbin/casbin": "^4.0.2"
     },
     "autoload": {
         "psr-4": {
             "Casbin\\CodeIgniter\\": "src/"
         }
     },
     "require-dev": {
-        "phpunit/phpunit": "^7.0|^8.0|^9.0",
-        "php-coveralls/php-coveralls": "^2.1"
+        "phpunit/phpunit": "^10.0|^11.0"
     },
     "autoload-dev": {
         "psr-4": {

diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -2,9 +2,6 @@
 <phpunit bootstrap="vendor/codeigniter4/framework/system/Test/bootstrap.php"
         backupGlobals="false"
         colors="true"
-        convertErrorsToExceptions="true"
-        convertNoticesToExceptions="true"
-        convertWarningsToExceptions="true"
         stopOnError="false"
         stopOnFailure="false"
         stopOnIncomplete="false"
@@ -14,15 +11,23 @@
             <directory>./tests</directory>
         </testsuite>
     </testsuites>
-    <filter>
-        <whitelist processUncoveredFilesFromWhitelist="true">
-            <directory suffix=".php">./src</directory>
-        </whitelist>
-    </filter>
-    <logging>
-        <log type="coverage-clover" target="build/logs/clover.xml"/>
-        <log type="coverage-html" target="build/html"/>
-    </logging>
+
+    <coverage includeUncoveredFiles="true"
+          pathCoverage="false"
+          ignoreDeprecatedCodeUnits="true"
+          disableCodeCoverageIgnore="true">
+        <report>
+            <clover outputFile="build/logs/clover.xml"/>
+            <html outputDirectory="build/html" lowUpperBound="50" highLowerBound="90"/>
+        </report>
+    </coverage>
+
+    <source>
+        <include>
+            <directory>./src</directory>
+        </include>
+    </source>
+
     <php>
         <env name="app.baseURL" value="http://example.com"/>
         <env name="database.tests.hostname" value="127.0.0.1"/>

diff --git a/src/Config/Enforcer.php b/src/Config/Enforcer.php
@@ -47,8 +47,8 @@ class Enforcer extends BaseConfig
             // changes whether Casbin will log messages to the Logger.
             'enabled' => false,
 
-            // Casbin Logger
-            'logger' => \Casbin\CodeIgniter\Logger::class,
+            // Casbin Logger, Supported: \Psr\Log\LoggerInterface|string
+            'logger' => 'logger',
         ],
 
         'cache' => [

diff --git a/src/EnforcerManager.php b/src/EnforcerManager.php
@@ -5,6 +5,8 @@
 use Casbin\Enforcer;
 use Casbin\Model\Model;
 use Casbin\Log\Log;
+use Casbin\Log\Logger\DefaultLogger;
+use Config\Services;
 use InvalidArgumentException;
 use Casbin\CodeIgniter\Config\Enforcer as EnforcerConfig;
 
@@ -75,7 +77,11 @@ protected function resolve($name)
         }
 
         if ($logger = $config['log']['logger']) {
-            Log::setLogger(new $logger());
+            if (is_string($logger)) {
+                $logger = new DefaultLogger(Services::$logger());
+            }
+
+            Log::setLogger($logger);
         }
 
         $model = new Model();
@@ -92,7 +98,7 @@ protected function resolve($name)
             }
         }
 
-        return new Enforcer($model, $adapter, $config['log']['enabled']);
+        return new Enforcer($model, $adapter, $logger, $config['log']['enabled']);
     }
 
     /**

diff --git a/src/Logger.php b/src/Logger.php
diff --git a/tests/EnforcerManagerTest.php b/tests/EnforcerManagerTest.php
@@ -2,14 +2,17 @@
 
 namespace Casbin\CodeIgniter\Tests;
 
-use CodeIgniter\Test\CIDatabaseTestCase;
+use CodeIgniter\Test\CIUnitTestCase;
+use CodeIgniter\Test\DatabaseTestTrait;
 use Config\Services
 ;
 use Config\Autoload;
 use Config\Modules;
 
-class EnforcerManagerTest extends CIDatabaseTestCase
+class EnforcerManagerTest extends CIUnitTestCase
 {    
+    use DatabaseTestTrait;
+
     protected function createApplication()
     {
         $app = parent::createApplication();