CODEWITHSUNDEEP
ruby-on-railsrubyruby-on-rails-3devise
John
John

Reputation: 171

Rails devise: user_signed_in? not working

I have this in my view:

<% if user_signed_in? %>
<%= current_user.email %>
<% else %>
<%= link_to "Sign in", new_user_session_path %><br />
<%= link_to "Opret", new_user_session_path %><br />
<% end %>

But when have signed in as a user: I still get the links in view: 

<%= link_to "Sign in", new_user_session_path %><br />
<%= link_to "Opret", new_user_session_path %><br />

Why is the helper not working?

Upvotes: 17

Views: 41668

Answers (7)

epicrato
epicrato

Reputation: 8428

If you are using it within an action, make sure you add the colon before:

before_action :set_global_variables, if: :user_signed_in?

Instead of

before_action :set_global_variables, if: user_signed_in?

Upvotes: 0

Brian
Brian

Reputation: 4930

Did you use devise's before action in your controller?

before_action :authenticate_user!

Upvotes: 22

Mayank Singh
Mayank Singh

Reputation: 53

This could be a possible explanation.

You might have a pre-existing current_user method which stops Devise's current_user called within user_signed_in? from returning expected values.

Upvotes: 4

cat
cat

Reputation: 1

I had encountered exactly the same issue and solved it by doing $bundle exec rails g devise user again. My problem was devise_for users was somehow missing in the routing file.

Upvotes: 0

ramez emad
ramez emad

Reputation: 53

Make sure that you are using the right model. For example if your model is member then you should use member_sign_in instead of user_sign_in.

Upvotes: 5

user133012
user133012

Reputation:

If login is working properly, it could be because of some kind of caching going on?

Upvotes: 1

Scott
Scott

Reputation: 1127

Not sure what is behind the user_signed_in? method, either your login is not working correctly or your method is broken.

Maybe try this, if that doesn't work, I would take a look at whats going wrong with the actual login.

<%unless current_user.blank? -%>
signed in
<%else -%>
not signed in
<%end-%>

Upvotes: 8

Related Questions