Skip to content

Commit

Permalink
Renamed from UserSessions to Session controller.
Browse files Browse the repository at this point in the history
  • Loading branch information
hectoregm committed Apr 22, 2009
1 parent df5e164 commit 8fa678b
Show file tree
Hide file tree
Showing 10 changed files with 30 additions and 30 deletions.
6 changes: 3 additions & 3 deletions authlogic.rb
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@

# Get controllers
controller "application_controller.rb"
controller "user_sessions_controller.rb"
controller "sessions_controller.rb"
controller "users_controller.rb"
controller "password_resets_controller.rb"
controller 'home_controller.rb'
Expand All @@ -65,7 +65,7 @@
view "users", "edit.html.#{templating_engine}"
view 'users', "_semantic_form.html.#{templating_engine}"
view "users", "show.html.#{templating_engine}"
view "user_sessions", "new.html.#{templating_engine}"
view "sessions", "new.html.#{templating_engine}"
view "user_mailer", "activation.text.html.#{templating_engine}"
view "user_mailer", "reset_password_instructions.text.html.#{templating_engine}"
view "user_mailer", "signup_notification.text.html.#{templating_engine}"
Expand Down Expand Up @@ -116,7 +116,7 @@
spec 'models/user_spec.rb'
spec 'models/user_mailer_spec.rb'
spec 'controllers/users_controller_spec.rb'
spec 'controllers/user_sessions_controller_spec.rb'
spec 'controllers/sessions_controller_spec.rb'
spec 'controllers/password_resets_controller_spec.rb'
spec 'helpers/layout_helper_spec.rb'
spec "helpers/#{templating_engine}_helper_spec.rb"
Expand Down
2 changes: 1 addition & 1 deletion base/app/controllers/application_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ def require_user
# No redirect if requested page is /logout
store_location if !(request.request_uri =~ /\/logout$/)
flash[:error] = t :require_user
redirect_to new_user_session_url
redirect_to login_path
return false
end
end
Expand Down
Original file line number Diff line number Diff line change
@@ -1,20 +1,20 @@
class UserSessionsController < ApplicationController
class SessionsController < ApplicationController
layout 'single_column'

before_filter :require_no_user, :only => [:new, :create]
before_filter :require_user, :only => :destroy

def new
@user_session = UserSession.new
@session = UserSession.new
end

def create
@user_session = UserSession.new(params[:user_session])
if @user_session.save
@session = UserSession.new(params[:user_session])
if @session.save
flash[:notice] = t :login_successful
redirect_back_or_default account_url
else
message = @user_session.errors.on(:base) ? @user_session.errors.on(:base) : t(:login_unsuccessful)
message = @session.errors.on(:base) ? @session.errors.on(:base) : t(:login_unsuccessful)
flash.now[:error] = message
render :new
end
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
<% title t '.title' -%>

<div id="user_sessions-new" class="block">
<div id="sessions-new" class="block">
<h2><%= yield :title %></h2>
<div class="content form">
<% semantic_form_for @user_session, :url => user_session_path do |f| -%>
<% semantic_form_for @session, :url => session_path do |f| -%>
<% f.inputs do -%>
<%= f.input :login, :required => false%>
<%= f.input :password, :required => false%>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
- title t '.title'

#user_sessions-new.block
#sessions-new.block
%h2= yield :title
.content.form
- semantic_form_for @user_session, :url => user_session_path do |f|
- semantic_form_for @session, :url => session_path do |f|
- f.inputs do
= f.input :login, :required => false
= f.input :password, :required => false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ en:
login_successful: 'Login successful!'
login_unsuccessful: 'Incorrect login or password'
logout_successful: 'Logout successful!'
user_sessions:
sessions:
new:
title: 'Login'
lost_password: 'Lost password?'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ es:
login_successful: 'Inicio de sesión exitoso!'
login_unsuccessful: 'Indentificador y/o contraseña incorrecta.'
logout_successful: 'Salida exitosa!'
user_sessions:
sessions:
new:
title: 'Ingreso'
lost_password: '¿Olvido su contraseña?'
Expand Down
6 changes: 3 additions & 3 deletions base/config/routes.rb
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
ActionController::Routing::Routes.draw do |map|
map.root :controller => "home", :action => "index"
map.login '/login', :controller => "user_sessions", :action => "new"
map.logout '/logout', :controller => "user_sessions", :action => "destroy"
map.login '/login', :controller => "sessions", :action => "new"
map.logout '/logout', :controller => "sessions", :action => "destroy"

map.resource :account, :controller => "users", :member => { :confirm => :get }
map.resource :user_session, :users
map.resource :session, :users
map.resources :users
map.resource :password_reset
end
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
require File.expand_path(File.dirname(__FILE__) + '/../spec_helper' )

describe UserSessionsController do
describe SessionsController do

describe "new:" do

Expand All @@ -11,7 +11,7 @@
UserSession.should_receive(:new).and_return(mock_user_session)

get :new
should assign_to(:user_session, :with => mock_user_session)
should assign_to(:session, :with => mock_user_session)
end

it "should render login form" do
Expand All @@ -38,17 +38,17 @@
it "should create a session" do
mock_user_session.should_receive(:save).and_return(true)

post :create, :user_session => {}
should assign_to(:user_session, :with => mock_user_session)
post :create, :session => {}
should assign_to(:session, :with => mock_user_session)
end

it "should redirect to account page" do
post :create, :user_session => {}
post :create, :session => {}
should redirect_to(account_url)
end

it "should be successful" do
post :create, :user_session => {}
post :create, :session => {}
should set_the_flash(:to => "Login successful!")
end

Expand All @@ -63,14 +63,14 @@
end

it "should re-render login form" do
post :create, :user_session => {}
post :create, :session => {}
should render_template(:new)
end

it "should not be successful" do
mock_user_session.should_receive(:save).and_return(false)

post :create, :user_session => {}
post :create, :session => {}
should_not set_the_flash(:to => 'Login successful!')
end

Expand All @@ -93,16 +93,16 @@
it "should destroy the session" do
mock_user_session.should_receive(:destroy)

post :destroy
delete :destroy
end

it "should redirect to login page" do
post :destroy
delete :destroy
should redirect_to(login_url)
end

it "should be successful" do
post :destroy
delete :destroy
should set_the_flash(:to => 'Logout successful!')
end

Expand Down
2 changes: 1 addition & 1 deletion base/spec/spec_helpers/controller_spec_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def get_action

it "should redirect to login page with logged out user" do
get get_action
should redirect_to(new_user_session_url)
should redirect_to(login_path)
end

it "should have /must be logged in/ in the flash with logged out user" do
Expand Down

0 comments on commit 8fa678b

Please sign in to comment.