@@ -55,6 +55,7 @@ class AccountController < ApplicationController protected def default_url(user) - admin? ? url_for(:controller => '/admin/overview', :action => 'index') : dispatch_url(:path => []) + # If the user can log in then they have permission to act in the admin section (non-admins can post, admins can admin the site) + logged_in? ? url_for(:controller => '/admin/overview', :action => 'index') : dispatch_url(:path => []) end end app/controllers/account_controller.rb @@ -38,8 +38,8 @@ class AccountControllerLoginTest < ActiveSupport::TestCase test "should login as site user" do post :login, :login => 'ben', :password => 'test' assert session[:user] - # ben is not an admin so should be redirected to the front page - assert_redirected_to :controller => 'mephisto', :action => 'dispatch' + # ben is not a site admin but as a user ben has the ability to post so direct ben to the admin overview + assert_redirected_to :controller => 'admin/overview', :action => 'index' post :logout assert !session[:user] end test/functional/account_controller_test.rb 3700f8d9e7aeefaf5d175d2ea4e412c2927ec229 Chris Cummer chris@postal-code.com http://github.com/emk/mephisto/commit/e6ef40f48b4d37f18381e8a0ed2dd5a551a7d82b e6ef40f48b4d37f18381e8a0ed2dd5a551a7d82b 2009-01-09T04:28:52-08:00 2009-01-04T12:36:51-08:00 Changed user login to send user to admin section on succesful login instead of the blog homepage since users have the ability to post to the blog f48310c10b955f3eacf2eea4017212f38fd66984 Eric Kidd git@randomhacks.net