Fixed exception class visibility and simplified array check in Manager

Optimy · May 19, 2016 · aa934a4 · aa934a4
1 parent 6bd4d42
commit aa934a4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/Drivers/ArrayDriver.php b/src/Drivers/ArrayDriver.php
@@ -154,7 +154,7 @@ public function hasRolePermission(Role $role, Permission $permission)
     */
     public function clearPermissionsForResource(Resource $resource)
     {
-        throw new Exception('Unimplemented method "ArrayDriver::clearPermissionsForResource"');
+        throw new \Exception('Unimplemented method "ArrayDriver::clearPermissionsForResource"');
     }
 
     /**

diff --git a/src/Manager.php b/src/Manager.php
@@ -145,12 +145,10 @@ protected function findRole($role)
     * @param array $resources an array of Resource objects
     * @return void
     */
-    public function clearPermissionsForResources($resources)
+    public function clearPermissionsForResources(array $resources)
     {
-        if(is_array($resources)) {
-            foreach($resources as $resource) {
-                $this->driver->clearPermissionsForResource($resource);
-            }
+        foreach($resources as $resource) {
+            $this->driver->clearPermissionsForResource($resource);
         }
     }
-Original file line number
+Diff line change
@@ Expand Up @@
         */
         public function clearPermissionsForResource(Resource $resource)
         {
-            throw new Exception('Unimplemented method "ArrayDriver::clearPermissionsForResource"');
+            throw new \Exception('Unimplemented method "ArrayDriver::clearPermissionsForResource"');
         }
         /**
@@ Expand Down @@