Rails 3 appending object to an array

Question

I have a view where I am trying to display all the objects in an array. The objects are tickets.

<tbody>
    <% @tickets.each do |ticket| %>
        <tr>
            <td><%= ticket.ticket_number %></td>
            <td><%= ticket.title %></td>
            <td><%= ticket.product_name %></td>
            <td><%= truncate(ticket.description) %></td>
            <td><%= ticket.status %></td>
            <td><%= link_to "Show", users_ticket_path(ticket) %>
        </tr>
    <% end %>
</tbody>

My controller is finding all Users that have the same 'reseller_id' as the current User. So say I have 3 Users all under the same Reseller. The 3 Users all have the same 'reseller_id' Here is my controller that doesn't work.

def index
  if admin_user?
    @test = "HELLO"
    all_users = User.find_all_by_reseller_id(current_user.reseller_id)
    all_users.each do |u|
      @tickets = u.tickets
    end
  #@tickets = @tickets.paginate(page: params[:page])
  else
    @test = "WORLD"
    @tickets = current_user.tickets.paginate(page: params[:page])
end
end

This saves a blank array. How do I make an array with all the ticket objects?

Thanks

Would anyone know why this works in development mode and not in production mode? — DDDD
– DDDD, Commented Sep 19, 2013 at 23:01

tihom · Accepted Answer · 2013-09-19 21:54:10Z

2

Currently @tickets is equal to tickets of last user found. Use something like this

def index
  @tickets = []
  if admin_user?
    @test = "HELLO"
    all_users = User.find_all_by_reseller_id(current_user.reseller_id)
    all_users.each do |u|
      @tickets += u.tickets
    end
  #@tickets = @tickets.paginate(page: params[:page])
  else
    @test = "WORLD"
    @tickets = current_user.tickets.paginate(page: params[:page])
end
end

answered Sep 19, 2013 at 21:54

tihom

8,0032 gold badges27 silver badges30 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

DDDD Over a year ago

I did not know I could use += awesome

DDDD Over a year ago

Do you know why this works in development mode but now in production mode?

tihom Over a year ago

what error are you getting in production? If you get empty @tickets in production while non-empty @tickets in development then it could be due to lack of data in production db.

DDDD Over a year ago

I do not get an error. The array is displaying only the current_user and not the other User's tickets in production. All three users have tickets.

tihom Over a year ago

seems like the production app has not been restarted to use the new code.

|

Ayman · Accepted Answer · 2013-09-19 22:34:00Z

0

Or you can do it in one line

def index
  if admin_user?
    @test = "HELLO"
    @tickets = Ticket.where("user_id IN (?)", User.find_all_by_reseller_id(current_user.reseller_id).map(&:id))
    @tickets = @tickets.paginate(page: params[:page])
  else
    @test = "WORLD"
    @tickets = current_user.tickets.paginate(page: params[:page])
  end
end

answered Sep 19, 2013 at 22:34

Ayman

3062 silver badges4 bronze badges

Collectives™ on Stack Overflow

Rails 3 appending object to an array

2 Answers 2

9 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

9 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related