acts_permissive 0.3.0 → 0.3.1

data/lib/acts_permissive/permissive_user.rb

@@ -136,7 +136,7 @@ module ActsPermissive
             return perm.mask & bits == bits
           end
         else
-          warn "You haven't given an object or circle to search on. Failing silently. Argument list: #{args.inspect}"
+          warn "You haven't given an object or circle to search on. Failing silently. (Perhaps a model does not call 'is_used_permissively', check that. Argument list: #{args.inspect}"
         end
         # Failsafe to false
         false
@@ -162,7 +162,7 @@ module ActsPermissive
         end
 
         # Remove from circle and destroy permissions if the mask is zero
-        perm.destroy! if perm.mask == 0
+        perm.destroy if perm.mask == 0
       end
     end
   end

data/lib/acts_permissive/version.rb

@@ -1,3 +1,3 @@
 module ActsPermissive
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 end

data/spec/permissive_user_spec.rb

@@ -92,7 +92,9 @@ describe ActsPermissive::PermissiveUser do
       it "should allow using the can? method on a model" do
         @new_user.can?(:read, @admin_circle.items.first).should be_true
         @admin_circle.items.each{|i| @new_user.can?(:read, i).should be_true}
-        @admin_circle.items.each{|i| @new_user.can?(:write, i).should be_false}
+        @admin_circle.items.each do |item|
+          @new_user.can?(:write, item).should be_false
+        end
       end
 
       it "should work with work with more than one object, but throw a warning" do
@@ -151,6 +153,14 @@ describe ActsPermissive::PermissiveUser do
         @user.can?(p, :in => @admin_circle).should be_false
       end
     end
+
+    it "should correctly revoke the lowest permission" do
+      user = FactoryGirl.create :user
+      user.can!(:see, :in => @user_circle)
+      user.can?(:see, :in => @user_circle).should be_true
+      user.revoke!(:see, :in => @user_circle)
+      user.can?(:see, :in => @user_circle).should be_false
+    end
   end
 
   describe "query methods" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: acts_permissive
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-02 00:00:00.000000000 Z
+date: 2012-04-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &70238536645680 !ruby/object:Gem::Requirement
+  requirement: &70253321798780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 3.1.1
   type: :runtime
   prerelease: false
-  version_requirements: *70238536645680
+  version_requirements: *70253321798780
 - !ruby/object:Gem::Dependency
   name: uuidtools
-  requirement: &70238536644760 !ruby/object:Gem::Requirement
+  requirement: &70253321797800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70238536644760
+  version_requirements: *70253321797800
 - !ruby/object:Gem::Dependency
   name: rspec-rails
-  requirement: &70238536643540 !ruby/object:Gem::Requirement
+  requirement: &70253321796620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536643540
+  version_requirements: *70253321796620
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &70238536642380 !ruby/object:Gem::Requirement
+  requirement: &70253321795440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536642380
+  version_requirements: *70253321795440
 - !ruby/object:Gem::Dependency
   name: pg
-  requirement: &70238536640340 !ruby/object:Gem::Requirement
+  requirement: &70253321793220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536640340
+  version_requirements: *70253321793220
 - !ruby/object:Gem::Dependency
   name: capybara
-  requirement: &70238536655020 !ruby/object:Gem::Requirement
+  requirement: &70253321807900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536655020
+  version_requirements: *70253321807900
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70238536652880 !ruby/object:Gem::Requirement
+  requirement: &70253321805980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536652880
+  version_requirements: *70253321805980
 - !ruby/object:Gem::Dependency
   name: spork
-  requirement: &70238536651460 !ruby/object:Gem::Requirement
+  requirement: &70253321804480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -98,10 +98,10 @@ dependencies:
         version: 0.9.0
   type: :development
   prerelease: false
-  version_requirements: *70238536651460
+  version_requirements: *70253321804480
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: &70238536650340 !ruby/object:Gem::Requirement
+  requirement: &70253321803460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,10 +109,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536650340
+  version_requirements: *70253321803460
 - !ruby/object:Gem::Dependency
   name: guard-spork
-  requirement: &70238536649120 !ruby/object:Gem::Requirement
+  requirement: &70253321802140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536649120
+  version_requirements: *70253321802140
 - !ruby/object:Gem::Dependency
   name: factory_girl_rails
-  requirement: &70238536661520 !ruby/object:Gem::Requirement
+  requirement: &70253321812940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,10 +131,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536661520
+  version_requirements: *70253321812940
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &70238536655920 !ruby/object:Gem::Requirement
+  requirement: &70253321808940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -142,10 +142,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536655920
+  version_requirements: *70253321808940
 - !ruby/object:Gem::Dependency
   name: growl
-  requirement: &70238536671300 !ruby/object:Gem::Requirement
+  requirement: &70253321823400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -153,10 +153,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536671300
+  version_requirements: *70253321823400
 - !ruby/object:Gem::Dependency
   name: fakeweb
-  requirement: &70238536669560 !ruby/object:Gem::Requirement
+  requirement: &70253321822640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -164,10 +164,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536669560
+  version_requirements: *70253321822640
 - !ruby/object:Gem::Dependency
   name: rb-fsevent
-  requirement: &70238536668660 !ruby/object:Gem::Requirement
+  requirement: &70253321821480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -175,7 +175,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70238536668660
+  version_requirements: *70253321821480
 description: Allows for arbitrary permissions on arbitrary collections of objects,
   rather than system-wide role based management
 email: