Returning Ruby Array from database query to javascript

Question

I'm trying to return an array of dates from my database through my rails controller, which is then used by Javascript while rendering a calendar. It seems to be working when I pull up the rails console for testing but not in the view. Any ideas? My code is below.

Gear has_many line_items

LineItem belongs_to Gear

Javascript Variable

var $myBadDates = new Array("<%= @gear.line_items.rented %>");

View that is being returned.

var $myBadDates = new Array("[]");

Line Item Model (shortened)

class LineItem < ActiveRecord::Base
  belongs_to :gear

  scope :available, where(:cart_id => nil)

  def self.rented
    LineItem.available.collect {|x| (x.rentstart..x.rentend).to_a}
  end

end

Array from Rails Console

1.9.3-p194 :007 > g.line_items.rented
  LineItem Load (0.7ms)  SELECT `line_items`.* FROM `line_items` WHERE `line_items`.`gear_id` = 4 AND `line_items`.`cart_id` IS NULL
 => [[Tue, 12 Feb 2013, Wed, 13 Feb 2013, Thu, 14 Feb 2013, Fri, 15 Feb 2013, Sat, 16 Feb 2013, Sun, 17 Feb 2013, Mon, 18 Feb 2013, Tue, 19 Feb 2013, Wed, 20 Feb 2013, Thu, 21 Feb 2013], [Tue, 05 Feb 2013, Wed, 06 Feb 2013, Thu, 07 Feb 2013, Fri, 08 Feb 2013, Sat, 09 Feb 2013, Sun, 10 Feb 2013, Mon, 11 Feb 2013, Tue, 12 Feb 2013, Wed, 13 Feb 2013, Thu, 14 Feb 2013, Fri, 15 Feb 2013]]

UPDATED Working javascript code from accepted answer

var $myBadDates = <%= @gear.line_items.rented.flatten.to_json.html_safe %>;

I think @gear.line_items will return list of line_items not single line_item. So @ gear.line_items.rented wo'nt work. — Sachin R
– Sachin R, Commented Mar 7, 2013 at 5:44

jvnill · Accepted Answer · 2013-03-07 05:42:42Z

2

try using to_json

var $myBadDates = <%= @gear.line_items.rented.to_json %>;

answered Mar 7, 2013 at 5:42

jvnill

29.6k5 gold badges86 silver badges86 bronze badges

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

6 Comments

DaveG Over a year ago

That pulled the data but getting strange return values new Array("[["2013-02-12","2013-02-13","2013-02-14","2013-02-15","2013-02-16","2013-02-17","2013-02-18","2013-02-19","2013-02-20","2013-02-21"],["2013-02-05","2013-02-06","2013-02-07","2013-02-08","2013-02-09","2013-02-10","2013-02-11","2013-02-12","2013-02-13","2013-02-14","2013-02-15"]]")

mu is too short Over a year ago

@DaveG: Perhaps tacking a .html_safe on the end would help.

railsdog Over a year ago

Looks like you just stuck a to_json onto the end of your existing code. If you look at jvnill's response, that's not quite what he suggested. You're getting a json array as a response, quoting it, and making an array of 1 item (the quoted json response).

user166390 Over a year ago

Is to_json vulnerable to injecting </script> or ]]> in the data? Some implementations, like that found in PHP will prevent it by using appropriate escapes, but if it is not done, it's not suitable for JavaScript value injection even though "it will almost always work".

DaveG Over a year ago

jvnil & @muistooshort Thanks got it working from your suggestions.

|

Collectives™ on Stack Overflow

Returning Ruby Array from database query to javascript

1 Answer 1

6 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Linked

Related