Changeset 293

Show
Ignore:
Timestamp:
03/01/06 13:29:53 (3 years ago)
Author:
lazyatom
Message:

Update UserEngine? to use wrapped table names, added migration to change users_roles table to roles_users

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • user_engine/trunk/lib/user_engine.rb

    r279 r293  
    4545  # Join tables for users <-> roles, and roles <-> permissions 
    4646   
    47   config :user_role_table, "#{config(:raw_role_table)}_#{LoginEngine.config(:user_table)}" 
     47  config :user_role_table, "#{config(:role_table)}_#{LoginEngine.config(:user_table)}" 
    4848  config :permission_role_table, "#{config(:raw_permission_table)}_#{config(:raw_role_table)}" 
    4949 
     
    103103     
    104104    if @warning != nil 
    105       RAILS_DEFAULT_LOGGER.warn @warning 
     105      Engines.logger.warn @warning 
    106106      puts @warning 
    107107    end 
     
    188188     
    189189    if !user? 
    190       RAILS_DEFAULT_LOGGER.debug "checking guest authorisation for #{controller}/#{action}" 
     190      Engines.logger.debug "checking guest authorisation for #{controller}/#{action}" 
    191191      if User.guest_user_authorized?(controller, action) 
    192192        yield block if block != nil 
     
    194194      end 
    195195    else 
    196       RAILS_DEFAULT_LOGGER.debug "checking user:#{session[:user].id} authorisation for #{controller}/#{action}" 
     196      Engines.logger.debug "checking user:#{session[:user].id} authorisation for #{controller}/#{action}" 
    197197      if current_user.authorized?(controller, action) 
    198198        yield block if block != nil 
  • user_engine/trunk/lib/user_engine/authorized_user.rb

    r279 r293  
    4545      base.extend(ClassMethods) 
    4646      base.class_eval { 
    47         has_and_belongs_to_many :roles, :join_table => UserEngine.config(:user_role_table
     47        has_and_belongs_to_many :roles, :join_table => wrapped_table_name(UserEngine.config(:user_role_table)
    4848         
    4949        # ensure that all users recieve the 'user' role 
     
    6868      def guest_user_authorized?(controller, action="index") 
    6969        query = <<-EOS 
    70 SELECT DISTINCT #{UserEngine.config(:permission_table)}.*  
    71 FROM #{UserEngine.config(:permission_table)}, #{UserEngine.config(:role_table)},  
    72      #{UserEngine.config(:permission_role_table)} 
    73 WHERE #{UserEngine.config(:role_table)}.name = :role 
    74 AND #{UserEngine.config(:role_table)}.id = #{UserEngine.config(:permission_role_table)}.role_id 
    75 AND #{UserEngine.config(:permission_role_table)}.permission_id = #{UserEngine.config(:permission_table)}.id 
    76 AND #{UserEngine.config(:permission_table)}.controller = :controller 
    77 AND #{UserEngine.config(:permission_table)}.action = :action 
     70SELECT DISTINCT #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.*  
     71FROM #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))},  
     72     #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table))},  
     73     #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table))} 
     74WHERE #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table))}.name = :role 
     75AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table))}.id = #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table))}.role_id 
     76AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table))}.permission_id = #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.id 
     77AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.controller = :controller 
     78AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.action = :action 
    7879EOS 
    7980 
     
    9596 
    9697      query = <<-EOS 
    97 SELECT DISTINCT #{UserEngine.config(:permission_table)}.*  
    98 FROM #{UserEngine.config(:permission_table)}, #{UserEngine.config(:role_table)},  
    99      #{UserEngine.config(:permission_role_table)}, #{UserEngine.config(:user_role_table)}, 
    100      #{LoginEngine.config(:user_table)} 
    101 WHERE #{LoginEngine.config(:user_table)}.id = :person 
    102 AND #{LoginEngine.config(:user_table)}.id = #{UserEngine.config(:user_role_table)}.user_id 
    103 AND #{UserEngine.config(:user_role_table)}.role_id = #{UserEngine.config(:role_table)}.id 
    104 AND #{UserEngine.config(:role_table)}.id = #{UserEngine.config(:permission_role_table)}.role_id 
    105 AND #{UserEngine.config(:permission_role_table)}.permission_id = #{UserEngine.config(:permission_table)}.id 
    106 AND #{UserEngine.config(:permission_table)}.controller = :controller 
    107 AND #{UserEngine.config(:permission_table)}.action = :action 
     98SELECT DISTINCT #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.*  
     99FROM #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))},  
     100     #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table))},  
     101     #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table))},  
     102     #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:user_role_table))}, 
     103     #{ActiveRecord::Base.wrapped_table_name(LoginEngine.config(:user_table))} 
     104WHERE #{ActiveRecord::Base.wrapped_table_name(LoginEngine.config(:user_table))}.id = :person 
     105AND #{ActiveRecord::Base.wrapped_table_name(LoginEngine.config(:user_table))}.id = #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:user_role_table))}.user_id 
     106AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:user_role_table))}.role_id = #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table))}.id 
     107AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table))}.id = #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table))}.role_id 
     108AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table))}.permission_id = #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.id 
     109AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.controller = :controller 
     110AND #{ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table))}.action = :action 
    108111EOS 
    109112 
  • user_engine/trunk/test/functional/permission_controller_test.rb

    r150 r293  
    1212class PermissionControllerTest < Test::Unit::TestCase 
    1313 
    14   fixture :permissions, :table_name => UserEngine.config(:permission_table), :class_name => 'Permission' 
     14  fixture :permissions, :class_name => 'Permission',  
     15          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table)) 
    1516   
    1617  def setup     
  • user_engine/trunk/test/functional/role_controller_test.rb

    r150 r293  
    1212class RoleControllerTest < Test::Unit::TestCase 
    1313 
    14   fixture :users, :table_name => LoginEngine.config(:user_table), :class_name => 'User' 
    15   fixture :users_roles, :table_name => UserEngine.config(:user_role_table) 
    16   fixture :roles, :table_name => UserEngine.config(:role_table), :class_name => 'Role' 
    17   fixture :permissions_roles, :table_name => UserEngine.config(:permission_role_table) 
    18   fixture :permissions, :table_name => UserEngine.config(:permission_table), :class_name => 'Permission' 
     14  fixture :users, :class_name => 'User', 
     15          :table_name => ActiveRecord::Base.wrapped_table_name(LoginEngine.config(:user_table)) 
     16  fixture :roles_users, :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:user_role_table)) 
     17  fixture :roles, :class_name => 'Role', 
     18          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table)) 
     19  fixture :permissions_roles,  
     20          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table)) 
     21  fixture :permissions, :class_name => 'Permission',  
     22          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table)) 
    1923   
    2024  def setup     
  • user_engine/trunk/test/functional/user_controller_test.rb

    r154 r293  
    77class UserControllerTest < Test::Unit::TestCase 
    88 
    9   fixture :users, :table_name => LoginEngine.config(:user_table), :class_name => 'User' 
    10   fixture :users_roles, :table_name => UserEngine.config(:user_role_table) 
    11   fixture :roles, :table_name => UserEngine.config(:role_table), :class_name => 'Role' 
    12   fixture :permissions_roles, :table_name => UserEngine.config(:permission_role_table) 
    13   fixture :permissions, :table_name => UserEngine.config(:permission_table), :class_name => 'Permission' 
     9  fixture :users, :class_name => 'User', 
     10          :table_name => ActiveRecord::Base.wrapped_table_name(LoginEngine.config(:user_table)) 
     11  fixture :roles_users, :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:user_role_table)) 
     12  fixture :roles, :class_name => 'Role', 
     13          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:role_table)) 
     14  fixture :permissions_roles,  
     15          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_role_table)) 
     16  fixture :permissions, :class_name => 'Permission',  
     17          :table_name => ActiveRecord::Base.wrapped_table_name(UserEngine.config(:permission_table)) 
    1418   
    1519  def setup 
  • user_engine/trunk/test/test_helper.rb

    r154 r293  
    1010  config :permission_table, "permissions", :force 
    1111 
    12   config :user_role_table, "users_#{config(:role_table)}", :force 
     12  config :user_role_table, "#{config(:role_table)}_users", :force 
    1313  config :permission_role_table, "#{config(:permission_table)}_#{config(:role_table)}", :force 
    1414 
     
    2525module LoginEngine 
    2626  config :salt, "test-salt", :force 
     27  config :default_home => {:controller => 'user', :action => 'home'}, :force => true 
    2728end 
    2829