accessible_for 0.3.1 → 0.4.0

data/README.markdown

@@ -37,11 +37,11 @@ accessible_for API (to avoid name conflicts).
       # and specify a single value instead of an array
       accessible_for :manager => :promotion
 
-      # If that's not DRY enough can compose access lists from other roles
-      # using the class method accessible_attributes
+      # If that's not DRY enough you can compose access lists from other roles
+      # using the class method accessible_by
       accessible_for :common   => [ :filling, :topping ]
-      accessible_for :customer => accessible_attributes(:common) + [ :rating ]
-      accessible_for :manager  => accessible_attributes(:common) + [ :price, :promotion ]
+      accessible_for :customer => accessible_by(:common) + [ :rating ]
+      accessible_for :manager  => accessible_by(:common) + [ :price, :promotion ]
 
       def update
         Taco.find(params[:id]).update_attributes!(taco_params)

data/lib/accessible_for.rb

@@ -1,37 +1,34 @@
 require 'mass_assignment_backport'
 
 module AccessibleFor
-  VERSION = "0.3.1"
+  VERSION = "0.4.0"
 
   def self.included(mod)
     mod.extend ClassMethods
   end
 
   module ClassMethods
-    attr_accessor :_accessible_attributes
+    attr_accessor :_accessible_by
 
     def accessible_for params
       params.each do |role, attrs|
-        self._accessible_attributes ||= {}
+        self._accessible_by ||= {}
         [role].flatten.each do |name|
-          self._accessible_attributes[name] = accessible_attributes(name) + [attrs].flatten
+          self._accessible_by[name] = accessible_by(name) + [attrs].flatten
         end
       end
     end
 
-    def accessible_attributes role
-      _accessible_attributes[role] || []
+    def accessible_by role
+      _accessible_by && _accessible_by[role] || []
     end
   end
 
   def sanitize_for role, values
     return nil if values.nil?
-    if !self.class._accessible_attributes || self.class._accessible_attributes[role].nil?
-      return {}
-    end
     result = block_given? ? nil : {}
     values.each do |k, v|
-      if self.class._accessible_attributes[role].include?(k.to_sym)
+      if self.class.accessible_by(role).include?(k.to_sym)
         if block_given?
           yield k, v
         else

data/lib/mass_assignment_backport.rb

@@ -12,7 +12,7 @@ module MassAssignmentBackport
     end
 
     def accessible_attributes role=:default
-      super role
+      accessible_by role
     end
   end
 

data/test/accessible_for_test.rb

@@ -3,7 +3,7 @@ require 'accessible_for'
 class AccessibleForTest < MiniTest::Unit::TestCase
   include AccessibleFor
   accessible_for :default => :topping
-  accessible_for :manager => accessible_attributes(:default) + [:price]
+  accessible_for :manager => accessible_by(:default) + [:price]
 
   def test_nil_params
     assert_nil sanitize_for(:default, nil)
@@ -33,7 +33,7 @@ class AccessibleForTest < MiniTest::Unit::TestCase
   end
 
   class UnspecifiedTest
-    include MassAssignmentBackport
+    include AccessibleFor
   end
 
   def test_unspecified_strips_values

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: accessible_for
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-07 00:00:00.000000000Z
+date: 2012-03-08 00:00:00.000000000Z
 dependencies: []
 description: ! "    This is a simple mass-assignment security module loosely based
   on\n    ActiveModel::MassAssignmentSecurity. It attempts to steal the good ideas\n