Rails: two attributes diffrent tables in view

Question

I have been trying trying to use two attributes from two different models so they can be used with the calendar_for method provided by the table_builder plugin in the index view. I have been through the Rails guides for http://guides.rubyonrails.org/active_record_querying.html#retrieving-multiple-objects and posts such as Ruby on Rails: How to join two tables however i must be doing something wrong.

My models are as follows:

Class Event < ActiveRecord::Base
belongs_to :user

and

class User < ActiveRecord::Base
User has_many :events

The different ways I have tried in the controller (not all at once) are:

@event.user.name
@users = User.joins(:event).where(:event => {:event_date => true})
@users = User.where(:event => :event_date)

Amoung others, my view looks like:

my view code:

<% calendar_for @users, :year => @date.year, :month => @date.month do |calendar| %>
<%= calendar.head('Sunday', 'Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday',   'Saturday') %>
<% calendar.day(:day_method => :created_at) do |date, users| %>
<%= date.day %>
  <ul>
    <% for user in users %>
      <li><%= link_to h(user.name), user %></li>
    <% end %>
  </ul>
<% end %>
<% end %>
</div>

I have tried changing the variables in the view accordingly however to no avail. I would like to show the users name and a link to the user on the specific day that their event is booked.

Answer 1

Two issues I see, though I'm not familiar with the calendar library.

First, make sure your query in the controller returns something useful. Of the three lines you gave us, the first doesn't even search, it calls methods on an undefined variable. The second is close to working, but you are searching for a date and matching it to true... How about:

@users = User.joins(:event).where('events.event_date is not null')

Furthermore, if you have a date range, you might include that in the search:

@users = User.joins(:event).where('events.event_date > ? and events.event_date < ?', start_date, end_date)

Next in the view, you aren't consistent with your variable naming. The controller sets up the @users variable, which you access once, but then later you are missing the @ in front of it, which is not the same thing. I don't know what the calendar part wants as input, but at least the for loop should be:

for user in @users

That said, for loops are not very rubyish. The ruby way is to use each:

@users.each do |user|
  ...
end

or even better, to make all of your links:

<ul>
<%= @users.collect {|user| content_tag(:li, link_to h(user.name), user).join } %>
</ul>

edit

Based on more information, I think you are starting at the wrong place. Let's start with events.

Event.joins(:users).where('events.event_date is not null')


<% calendar_for @events, :year => @date.year, :month => @date.month do |calendar| %>
  <%= calendar.head('Sunday', 'Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday',   'Saturday') %>
  <% calendar.day(:day_method => :event_date) do |date, events| %>
    <%= date.day %>
    <ul>
      <% for event in events %>
        <li><%= link_to h(event.user.name), event.user %></li>
      <% end %>
    </ul>
  <% end %>
<% end %>