Merge pull request #8 from basakest/savePolicy

fix: Amend some variables of savePolicy method

Author: leeqvip

composer.lock

@@ -62,13 +62,13 @@ public function loadPolicy($model): void
 
     public function savePolicy($model): void
     {
-        foreach ($model->model['p'] as $ptype => $ast) {
+        foreach ($model['p'] as $ptype => $ast) {
             foreach ($ast->policy as $rule) {
                 $this->savePolicyLine($ptype, $rule);
             }
         }
 
-        foreach ($model->model['g'] as $ptype => $ast) {
+        foreach ($model['g'] as $ptype => $ast) {
             foreach ($ast->policy as $rule) {
                 $this->savePolicyLine($ptype, $rule);
             }

src/Adapter.php

@@ -95,6 +95,20 @@ public function testAddPolicies()
         $this->assertEquals($policies, $e->getPolicy());
     }
 
+    public function testSavePolicy()
+    {
+        $e = $this->getEnforcer();
+        $this->assertFalse($e->enforce('alice', 'data4', 'read'));
+
+        $model = $e->getModel();
+        $model->clearPolicy();
+        $model->addPolicy('p', 'p', ['alice', 'data4', 'read']);
+
+        $adapter = $e->getAdapter();
+        $adapter->savePolicy($model);
+        $this->assertTrue($e->enforce('alice', 'data4', 'read'));
+    }
+
     public function testRemovePolicy()
     {
         $e = $this->getEnforcer();

tests/DatabaseAdapterTest.php

@@ -4,6 +4,9 @@ r = sub, obj, act
 [policy_definition]
 p = sub, obj, act
 
+[role_definition]
+g = _, _
+
 [policy_effect]
 e = some(where (p.eft == allow))