restrict 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0e119fe0a7f5a3b29548df9ff5d6f6bf77b490ab
-  data.tar.gz: 34a0da6663e3113ec9fcd1dce4b1bdde1afc7ae2
+  metadata.gz: 68a4da331094fb87ec736fbd19bfd69789b49df3
+  data.tar.gz: 57abb1a0b4c96ef18c13783a5ae15ed0d8e59016
 SHA512:
-  metadata.gz: b7d7b6b0e2da3fea9b62fffadd031b464956659a7123d6dbab3c7d66e9fb7a060aa8deef33327af5326ec922985560852a0b38827110fc607c2caee9a89c4c35
-  data.tar.gz: 4dce49238f2706ac0930f2d7a59f859cff47cb61d2709422bb9072fe4b916676fb854d33ba25bbe6536afba23596186ab1542487b5227012d5bb5ab6d4b145b4
+  metadata.gz: 24aaef845f178c795be551c1de4efa38d8cd750041b98d401a1c124483fb85f3e62b75e53d25e799316f232434108211a60fdc851f1f5ab4362563544109eca2
+  data.tar.gz: c85ac99aea25f9a20d7d17def516b27d1702223cd5d14c05586868a2b22881555c149995b26c4c3b4512c331e3690e3021ddf18b9ee88187b94ae314d90590b0

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+[0.0.7] - 2014-08-25
+  * Breaking change part 2: restrict without action names will now implicitly restrict all actions
+  * :all_actions modifier is gone
+
 [0.0.6] - 2014-08-25
   * Breaking change: use :unless instead of :allow_if
 

data/README.md

@@ -44,7 +44,7 @@ What that does:
 ### Restrict all actions
 
 ```ruby
-restrict :all_actions
+restrict
 ```
 
 This one will apply to all actions on this controller. It takes the `unless` option as well.
@@ -58,10 +58,6 @@ Restrict.config.authentication_validation_method = :admin_session_exists?
 
 You may set the method that is used to figure out whether a user is signed in or not to whatever you like, however it's default is `:user_signed_in?` which is the most common (devise) method in use.
 
-## Todo Ideas
-
-* restrict :all_actions, except: [:new], unless: 'dsfsdf'
-
 ## Contributing
 
 You know how this works and bonus points for feature branches!

data/lib/restrict/restriction.rb

@@ -3,10 +3,9 @@ module Restrict
     attr_accessor :actions, :unless
 
     def initialize(*args)
-      options   = args.extract_options!
-      @unless = options[:unless]
-      @actions  = args
-      actions.empty? and raise ArgumentError, "expected actions to restrict, but got #{actions.inspect}"
+      options  = args.extract_options!
+      @unless  = options[:unless]
+      @actions = args
     end
 
     def applies_to?(action)
@@ -16,7 +15,7 @@ module Restrict
     private
 
     def applies_to_all_actions?
-      actions.include?(:all_actions)
+      actions.empty?
     end
 
     def applies_to_action?(name)

data/lib/restrict/version.rb

@@ -1,3 +1,3 @@
 module Restrict
-  VERSION = "0.0.6"
+  VERSION = "0.0.7"
 end

data/spec/lib/restrict/gatekeeper_spec.rb

@@ -65,7 +65,7 @@ describe Restrict::Gatekeeper do
 
     context 'with multiple restrictions' do
       before do
-        controller.class.restrict :all_actions
+        controller.class.restrict
         controller.class.restrict :edit, unless: :falsy
       end
 

data/spec/lib/restrict/restriction_spec.rb

@@ -7,10 +7,6 @@ describe Restrict::Restriction do
     it 'knows about its actions' do
       expect(restriction.actions).to eq [:show, :edit]
     end
-
-    it 'raises an error if no actions were given' do
-      expect { Restrict::Restriction.new }.to raise_error(ArgumentError)
-    end
   end
 
   describe '#applies_to?' do
@@ -26,8 +22,8 @@ describe Restrict::Restriction do
       expect(restriction).not_to be_applies_to(:index)
     end
 
-    it 'returns true if it concerns :all_actions' do
-      restriction = Restrict::Restriction.new(:all_actions)
+    it 'returns true if it concerns all actions' do
+      restriction = Restrict::Restriction.new
       expect(restriction).to be_applies_to(:foo)
       expect(restriction).to be_applies_to(:bar)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: restrict
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Johannes Opper