Javascript: return html element from a function

Question

I am trying to filter certain html element using the datetime attributes. My html elements looks like

<time title="Sat Dec 22 16:29:21 2012 UTC" datetime="2012-12-22T16:29:21+00:00">4 hours</time>

Now, what I am doing is

var x = $("p time").map(function() { filter(this);})

and my filter function looks like:

function filter(var1){

  var now = new Date();
  var time = $(var1).attr("datetime");

  var time = new Date(time);
  var diff = now - time;

  if( diff < 7200000){

   console.log("yes");
   return $(var1).parent().parent();

  }
}

When I run the above code I get x as an empty array and yes is printed 9 times.

So, my question is why my filter function is not returning the parent html tag.

you are missing return from the function you pass to map() — Ilia G
– Ilia G, Commented Dec 22, 2012 at 22:05
If you're trying to compare dates, you're also missing getTime() to get the timestamp. And you're redeclearing the time variable. — adeneo
– adeneo, Commented Dec 22, 2012 at 22:09

David G · Accepted Answer · 2012-12-22 22:07:58Z

2

var x = $("p time").map(function() { filter(this); });

Should be

var x = $("p time").map(function() { return filter(this); });

You're missing the return statement.

answered Dec 22, 2012 at 22:07

David G

97.6k41 gold badges173 silver badges258 bronze badges

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

1 Comment

ronnie Over a year ago

Thanks. I believe I should sleep now.

charlietfl · Accepted Answer · 2012-12-22 22:21:58Z

1

Use jQuery filter() method

var p_parents_I_want = $("p time").filter(function() {
    var now = new Date();
    var time = $(this).attr("datetime");
    time = new Date(time);/* don't declare same var twice*/
    var diff = now - time;

    return diff < 7200000

}).parent().parent()

Can usually replace parent().parent() using closest(selector)

API reference: http://api.jquery.com/filter/

edited Dec 22, 2012 at 22:21

answered Dec 22, 2012 at 22:12

charlietfl

172k14 gold badges125 silver badges154 bronze badges

2 Comments

ronnie Over a year ago

Cool, didn't knew about this.

charlietfl Over a year ago

map() likely uses filter() internally in jQuery, but what you want is the elements so this method makes most sense to me

Collectives™ on Stack Overflow

Javascript: return html element from a function

2 Answers 2

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related