flexible_accessibility 0.1.6.pre → 0.1.8.pre

data/Rakefile

@@ -15,7 +15,7 @@ require 'jeweler'
 Jeweler::Tasks.new do |gem|
   # gem is a Gem::Specification... see http://docs.rubygems.org/read/chapter/20 for more options
   gem.name = "flexible_accessibility"
-  gem.version = "0.1.6.pre"
+  gem.version = "0.1.8.pre"
   gem.homepage = "http://github.com/mochnatiy/flexible_accessibility"
   gem.license = "MIT"
   gem.summary = %Q{Flexible access control system}

data/flexible_accessibility.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "flexible_accessibility"
-  s.version = "0.1.6.pre"
+  s.version = "0.1.8.pre"
 
   s.required_rubygems_version = Gem::Requirement.new("> 1.3.1") if s.respond_to? :required_rubygems_version=
   s.authors = ["Sergey Awanesov"]
@@ -30,7 +30,9 @@ Gem::Specification.new do |s|
     "lib/flexible_accessibility.rb",
     "lib/flexible_accessibility/controller_methods.rb",
     "lib/flexible_accessibility/exceptions.rb",
-    "lib/flexible_accessibility/permissions.rb",
+    "lib/flexible_accessibility/filters.rb",
+    "lib/flexible_accessibility/permission.rb",
+    "lib/flexible_accessibility/utils.rb",
     "test/helper.rb",
     "test/test_flexible_accessibility.rb"
   ]

data/lib/flexible_accessibility/controller_methods.rb

@@ -4,44 +4,25 @@ module FlexibleAccessibility
 
       # Macro for skip authorization
       def skip_authorization_here
-        self.instance_variable_set :@route_permitted, true
-        self.instance_variable_set :@checkable_routes, []
-        self.send :before_filter, :check_if_route_permitted
+        self.instance_variable_set :@_route_permitted, true
+        self.instance_variable_set :@_checkable_routes, []
+        #self.send :before_filter, :check_if_route_permitted
       end
 
       # Macro for define authorization
   	  def authorize args={}
-        self.instance_variable_set :@route_permitted, false
-  	  	self.send :before_filter, :check_permission_to_route
-        self.send :before_filter, :check_if_route_permitted
+        self.instance_variable_set :@_route_permitted, false
+  	  	#self.send :before_filter, :check_permission_to_route
+        #self.send :before_filter, :check_if_route_permitted
         set_actions_to_authorize args
   	  end
       
       private
-      #
+      # Set actions for authorize as instance variable
       def set_actions_to_authorize args={}
-        self.instance_variable_set :@checkable_routes, args[:only] unless args[:only].nil?
+        self.instance_variable_set :@_checkable_routes, args[:only] unless args[:only].nil?
         # TODO: understand and fix it
-        self.instance_variable_set :@checkable_routes, self.action_methods - args[:except] unless args[:except].nil?  
-      end
-
-      #
-      def current_route
-        path = ActionController::Routing::Routes.recognize_path request.env["PATH_INFO"]
-        [path[:controller], path[:action]]
-      end
-
-      # We checks access to route
-      # And we expected the existing of current_user helper
-      def check_permission_to_route
-        if self.instance_variable_get(:@checkable_routes).include? current_route[1].to_sym
-          self.instance_variable_set(:@route_permitted, true) unless Permissions.is_action_permitted_for_user? "#{current_route[0]}##{current_route[1]}", current_user
-        end
-      end
-
-      # We checks @authorized variable state
-      def check_if_route_permitted
-        raise FlexibleAccessibility::AccessDeniedException unless self.instance_variable_get :@route_permitted
+        self.instance_variable_set :@_checkable_routes, self.action_methods - args[:except] unless args[:except].nil?  
       end
     end
  
@@ -53,7 +34,7 @@ module FlexibleAccessibility
     
     # We checks url for each link in view to show it
     def has_access? controller, action
-      Permissions.is_action_permitted_for_user? "#{controller}##{action}", current_user
+      Permission.is_action_permitted_for_user? "#{controller}##{action}", current_user
     end
   end
 end

data/lib/flexible_accessibility/exceptions.rb

@@ -1,4 +1,4 @@
-module FlexibleAccessibilty
+module FlexibleAccessibility
 	class AccessDeniedException < StandardError
 		attr_reader :action, :subject
     attr_writer :default_message
@@ -7,7 +7,7 @@ module FlexibleAccessibilty
       @message = message
       @action = action
       @subject = subject
-      @default_message = I18n.t('errors.access_denied')
+      @default_message = I18n.t('flexible_accessibility.errors.access_denied', :action => @action)
     end
 
     def to_s

data/lib/flexible_accessibility/filters.rb

@@ -0,0 +1,42 @@
+module FlexibleAccessibility
+  module Filters
+  	extend ActiveSupport::Concern
+
+  	included do
+  	  append_before_filter :check_permission_to_route
+  	  append_before_filter :check_if_route_permitted
+  	end
+	  
+	private
+    # Detect current controller and action and return a permission
+    def current_resource
+      # ActionController::Routing::Routes.recognize_path request.env["PATH_INFO"][:controller]
+      params[:controller]
+    end
+
+    def current_action
+      # ActionController::Routing::Routes.recognize_path request.env["PATH_INFO"][:action]
+      params[:action]
+    end
+
+    def current_route
+      "#{current_resource}##{current_action}"
+    end
+	# We checks access to route
+	# And we expected the existing of current_user helper
+	def check_permission_to_route
+	  if self.class.instance_variable_get(:@_checkable_routes).include? current_action.to_sym
+	    self.class.instance_variable_set :@_route_permitted, Permission.is_action_permitted_for_user?(current_route, current_user)
+	  end
+	end
+
+	# We checks @authorized variable state
+	def check_if_route_permitted
+	  raise AccessDeniedException.new(nil, current_route, nil) unless self.class.instance_variable_get :@_route_permitted
+	end
+  end
+
+  ActiveSupport.on_load(:action_controller) do
+    ActionController::Base.send(:include, Filters)
+  end
+end

data/lib/flexible_accessibility/permission.rb

@@ -0,0 +1,22 @@
+module FlexibleAccessibility
+  class Permission
+  	class << self
+	  def all
+		permissions = {}
+		Utils.new.get_controllers.each do |klass|
+		  permissions[klass.to_sym] = klass.camelize.constantize.instance_variable_get(:@_checkable_routes).collect{ |a| a.to_s }.join(', ')
+		end
+		permissions
+	  end
+
+      # Stub methods
+	  def is_action_permitted? action
+	  	false
+	  end
+
+	  def is_action_permitted_for_user? action, user
+	  	false 
+	  end
+	end
+  end
+end

data/lib/flexible_accessibility/utils.rb

@@ -0,0 +1,25 @@
+module FlexibleAccessibility
+  class Utils
+    def initialize
+      @path = "#{::Rails.root}/app/controllers/"
+      @controllers = []
+    end
+
+    def get_controllers
+      get_controllers_recursive @path
+    end
+
+    def get_controllers_recursive path
+      (Dir.new(path).entries - ["..", "."]).each do |entry|
+        if File.directory? path + entry
+          # TODO: Add namespace handling here
+          # get_controllers_recursive path + entry + '/'
+          next
+        else
+          @controllers << File.basename(path + entry, ".*") unless File.basename(path + entry, ".*") == "application_controller"
+        end
+      end
+      @controllers
+    end
+  end
+end

data/lib/flexible_accessibility.rb

@@ -1,3 +1,5 @@
 require 'flexible_accessibility/controller_methods.rb'
 require 'flexible_accessibility/exceptions.rb'
-require 'flexible_accessibility/permissions.rb'
+require 'flexible_accessibility/permission.rb'
+require 'flexible_accessibility/filters.rb'
+require 'flexible_accessibility/utils.rb'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: flexible_accessibility
 version: !ruby/object:Gem::Version 
-  hash: 961915988
+  hash: 961916012
   prerelease: 6
   segments: 
   - 0
   - 1
-  - 6
+  - 8
   - pre
-  version: 0.1.6.pre
+  version: 0.1.8.pre
 platform: ruby
 authors: 
 - Sergey Awanesov
@@ -116,7 +116,9 @@ files:
 - lib/flexible_accessibility.rb
 - lib/flexible_accessibility/controller_methods.rb
 - lib/flexible_accessibility/exceptions.rb
-- lib/flexible_accessibility/permissions.rb
+- lib/flexible_accessibility/filters.rb
+- lib/flexible_accessibility/permission.rb
+- lib/flexible_accessibility/utils.rb
 - test/helper.rb
 - test/test_flexible_accessibility.rb
 homepage: http://github.com/mochnatiy/flexible_accessibility

data/lib/flexible_accessibility/permissions.rb

@@ -1,20 +0,0 @@
-module FlexibleAccessibility
-  class Permissions
-  	class << self
-	  def get_permissions
-		permissions = {}
-		ApplicationController.subclasses.each do |klass|
-		  permissions[klass.to_s.tableize.singularize.to_sym] = klass.instance_variable_get(:@checkable_routes).collect{ |a| a.to_s }.join(', ')
-		end
-		permissions
-	  end
-
-	  def is_action_permitted? action
-	  end
-
-	  def is_action_permitted_for_user? action, user
-	  	!self.where(["action = ? and user_id = ?", action, user.id]).empty?
-	  end
-	end
-  end
-end