Fixes a regression introduced in 1.1.1

myclabs · Jun 19, 2014 · 8844ae9 · 8844ae9
1 parent 6013a80
commit 8844ae9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/ACL.php b/src/ACL.php
@@ -164,7 +164,7 @@ public function rebuildAuthorizations()
         $roleRepository = $this->entityManager->getRepository('MyCLabs\ACL\Model\Role');
 
         // Clear
-        $this->entityManager->createQuery('DELETE MyCLabs\ACL\Model\Authorization');
+        $this->entityManager->createQuery('DELETE MyCLabs\ACL\Model\Authorization')->execute();
         $this->entityManager->clear('MyCLabs\ACL\Model\Authorization');
 
         // Regenerate

diff --git a/tests/Integration/RebuildAuthorizationsTest.php b/tests/Integration/RebuildAuthorizationsTest.php
@@ -27,11 +27,20 @@ public function testRebuildAuthorizations()
 
         $this->em->clear();
 
+        $qb = $this->em->createQueryBuilder();
+        $qb->select('count(authorization.id)');
+        $qb->from('MyCLabs\ACL\Model\Authorization', 'authorization');
+        $query = $qb->getQuery();
+
+        $initialCount = $query->getSingleScalarResult();
+
         $this->acl->rebuildAuthorizations();
 
         $this->assertFalse($this->acl->isAllowed($user, Actions::VIEW, $article1));
         $this->assertFalse($this->acl->isAllowed($user, Actions::EDIT, $article1));
         $this->assertTrue($this->acl->isAllowed($user, Actions::VIEW, $article2));
         $this->assertTrue($this->acl->isAllowed($user, Actions::EDIT, $article2));
+
+        $this->assertEquals($initialCount, $query->getSingleScalarResult());
     }
 }