Skip to content

Commit 3dc0c4d

Browse files
authored
Revert "BREAKING CHANGE: upgrade to PHP 8.0 and PHP-Casbin 4.0 (#75)"
This reverts commit a050a9c.
1 parent a050a9c commit 3dc0c4d

File tree

4 files changed

+82
-13
lines changed

4 files changed

+82
-13
lines changed

.github/workflows/build.yml

+66
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,73 @@ jobs:
2727
# laravel: [ ]
2828
# stability: [ prefer-lowest, prefer-stable ]
2929
include:
30+
# Laravel 5.5
31+
- php: 7.1
32+
laravel: 5.5.*
33+
phpunit: ~6.0
34+
- php: 7.2
35+
laravel: 5.5.*
36+
phpunit: ~6.0
37+
- php: 7.3
38+
laravel: 5.5.*
39+
phpunit: ~6.0
40+
41+
# Laravel 5.6
42+
- php: 7.1
43+
laravel: 5.6.*
44+
phpunit: ~7.0
45+
- php: 7.2
46+
laravel: 5.6.*
47+
phpunit: ~7.0
48+
- php: 7.3
49+
laravel: 5.6.*
50+
phpunit: ~7.0
51+
52+
# Laravel 5.7
53+
- php: 7.1
54+
laravel: 5.7.*
55+
phpunit: ~7.5
56+
- php: 7.2
57+
laravel: 5.7.*
58+
phpunit: ~7.5
59+
- php: 7.3
60+
laravel: 5.7.*
61+
phpunit: ~7.5
62+
63+
# Laravel 5.8
64+
- php: 7.1
65+
laravel: 5.8.*
66+
phpunit: ~7.5
67+
- php: 7.2
68+
laravel: 5.8.*
69+
phpunit: ~8.0
70+
- php: 7.3
71+
laravel: 5.8.*
72+
phpunit: ~8.0
73+
74+
# Laravel 6.x
75+
- php: 7.2
76+
laravel: 6.*
77+
phpunit: ~8.0
78+
- php: 7.3
79+
laravel: 6.*
80+
phpunit: ~8.0
81+
82+
# Laravel 7.x
83+
- php: 7.3
84+
laravel: 7.*
85+
phpunit: ~9.0
86+
- php: 7.4
87+
laravel: 7.*
88+
phpunit: ~9.0
89+
3090
# Laravel 8.x
91+
- php: 7.3
92+
laravel: 8.*
93+
phpunit: ~9.0
94+
- php: 7.4
95+
laravel: 8.*
96+
phpunit: ~9.0
3197
- php: 8.0
3298
laravel: 8.*
3399
phpunit: ~9.0

composer.json

+9-8
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,18 @@
2020
],
2121
"license": "Apache-2.0",
2222
"require": {
23-
"php": ">=8.0",
24-
"illuminate/support": "~8.0|~9.0|~10.0|~11.0",
25-
"illuminate/database": "~8.0|~9.0|~10.0|~11.0",
26-
"illuminate/console": "~8.0|~9.0|~10.0|~11.0",
27-
"casbin/casbin": "~4.0"
23+
"php": ">=7.1.0",
24+
"illuminate/support": "~5.5|~6.0|~7.0|~8.0|~9.0|~10.0|~11.0",
25+
"illuminate/database": "~5.5|~6.0|~7.0|~8.0|~9.0|~10.0|~11.0",
26+
"illuminate/console": "~5.5|~6.0|~7.0|~8.0|~9.0|~10.0|~11.0",
27+
"casbin/casbin": "~3.1",
28+
"casbin/psr3-bridge": "^1.1"
2829
},
2930
"require-dev": {
30-
"phpunit/phpunit": "~9.0|~10.5",
31-
"php-coveralls/php-coveralls": "^2.7",
31+
"phpunit/phpunit": "~7.0|~8.0|~9.0|~10.5",
32+
"php-coveralls/php-coveralls": "^2.4",
3233
"mockery/mockery": "^1.0",
33-
"laravel/laravel": "~9.0|~10.0|~11.0"
34+
"laravel/laravel": "~5.5|~6.0|~7.0|~8.0|~9.0|~10.0|~11.0"
3435
},
3536
"autoload": {
3637
"psr-4": {

src/EnforcerManager.php

+4-4
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@
22

33
namespace Lauthz;
44

5+
use Casbin\Bridge\Logger\LoggerBridge;
56
use Casbin\Enforcer;
67
use Casbin\Model\Model;
78
use Casbin\Log\Log;
8-
use Casbin\Log\Logger\DefaultLogger;
99
use Lauthz\Contracts\Factory;
1010
use Lauthz\Models\Rule;
1111
use Illuminate\Support\Arr;
@@ -80,10 +80,10 @@ protected function resolve($name)
8080

8181
if ($logger = Arr::get($config, 'log.logger')) {
8282
if (is_string($logger)) {
83-
$logger = new DefaultLogger($this->app->make($logger));
83+
$logger = $this->app->make($logger);
8484
}
8585

86-
Log::setLogger($logger);
86+
Log::setLogger(new LoggerBridge($logger));
8787
}
8888

8989
$model = new Model();
@@ -98,7 +98,7 @@ protected function resolve($name)
9898
]);
9999
}
100100

101-
return new Enforcer($model, $adapter, $logger, Arr::get($config, 'log.enabled', false));
101+
return new Enforcer($model, $adapter, Arr::get($config, 'log.enabled', false));
102102
}
103103

104104
/**

tests/DatabaseAdapterForCacheTest.php

+3-1
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,11 @@
22

33
namespace Lauthz\Tests;
44

5+
use Enforcer;
56
use Lauthz\Models\Rule;
67
use Illuminate\Foundation\Testing\DatabaseMigrations;
7-
use Lauthz\Facades\Enforcer;
8+
use Casbin\Persist\Adapters\Filter;
9+
use Casbin\Exceptions\InvalidFilterTypeException;
810

911
class DatabaseAdapterForCacheTest extends TestCase
1012
{

0 commit comments

Comments
 (0)