[_Sam]

Can't you just put this Code into another file and include this every time? That way you have the Code only once present and if it changes for some reason you only have to do it on one place.

[NoizeMe]

Mo3, on 08 November 2011 - 09:50 AM, said:

This is really annoying me.

I have a couple of tasks that I have to execute every time any request takes place. (Checking if an user is logged in, checking if there are any new messages, etc)

I tried putting the relevant code into application_controller.rb and using a before_filter, but the variables wouldn't be accessible in the view.

I ended up putting that code into every single controller action, but there has to be a nicer way, no?

This screams out for a bootstrapper .
That's what I'm doing when I need to do some tasks.

[TheEmpty]

helper_method :current_user

etc, you should be using lazy loads, ex:

def current_user
  @user ||= User.find_by_id(session[:user_id])
end

[Mo3]

ThatRailsGuy, on 08 November 2011 - 10:35 AM, said:

helper_method :current_user

etc, you should be using lazy loads, ex:

def current_user
  @user ||= User.find_by_id(session[:user_id])
end

That's exactly what I do, but @user isn't accessible in the view. (actually, its empty)

[TheEmpty]

Mo3, on 08 November 2011 - 10:39 AM, said:

That's exactly what I do, but @user isn't accessible in the view. (actually, its empty)

Yes, @user isn't, which is why you use the helper_method ex:

class ApplicationController << ActionController::Base
  helper_method :current_user

  private
  def current_user
    @user ||= User.find_by_id(session[:user_id])
  end
end

Hello <%= current_user.username %>.

[Mo3]

ThatRailsGuy, on 08 November 2011 - 12:36 PM, said:

Yes, @user isn't, which is why you use the helper_method ex:

class ApplicationController << ActionController::Base
  helper_method :current_user

  private
  def current_user
    @user ||= User.find_by_id(session[:user_id])
  end
end

Hello <%= current_user.username %>.

But that calls to the database every time I use current_user in the view.. which is 4-5 times D:

[TheEmpty]

Mo3, on 08 November 2011 - 12:37 PM, said:

But that calls to the database every time I use current_user in the view.. which is 4-5 times D:

No, see the ||= it only assigns if nil, but also probably want something like:

if session[:user_id].present?
  @user ||= ...
end