How to pass a parameter to an inline javascript function

Question

I'm trying to pass story_id into the innermost inline function, but its always 0 even though its not 0 in the outer function. How do I pass this value in?

function activateSearch(){
  if($('story_filter')) Event.observe('story_filter', 'keyup',
    function(event) {
      $('stories_table').select(".story").each(
        function(story) {
          story_id = story.id.split('_')[1];
          story.select('.tableCell', '.indexCardContent').each(
            function(task, story_id) {
              hideStoryRow(story_id);
              task_filter = new RegExp($F('story_filter'), "i");
              if (task.innerHTML.match( task_filter ))
              {
                  showStoryRow(story_id);
                  throw $break;
              }
            }
          );
        }
      );
    }
  );
}

Jason Orendorff · Accepted Answer · 2009-04-23 14:39:24Z

7

All you have to do to make this work is change this line:

    function(task, story_id) {

to this:

    function(task) {

You don't have to pass story_id anywhere -- it's already in scope in the inner function!

answered Apr 23, 2009 at 14:39

Jason Orendorff

45.4k6 gold badges69 silver badges103 bronze badges

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

Comments

BenMorel · Accepted Answer · 2014-01-08 10:27:27Z

0

You don't. The each method takes the function definition and then calls that function for each element in the Array. You'd need to write your own custom each method if you wanted to use the inline function definition.

For both readability and functionality, I would break out the inner most inline function and then for each element in the list, call out to your function.

edited Jan 8, 2014 at 10:27

BenMorel

37k52 gold badges208 silver badges339 bronze badges

answered Apr 23, 2009 at 14:02

Justin Niessner

246k40 gold badges416 silver badges546 bronze badges

Comments

Chad Grant · Accepted Answer · 2009-04-23 14:01:24Z

-1

you've got a scoping issue.

quick fix: define story_id as a global.

var story_id;

function activateSearch() { ....

answered Apr 23, 2009 at 14:01

Chad Grant

45.6k10 gold badges67 silver badges80 bronze badges

2 Comments

viam0Zah Over a year ago

Variable story_id is already global, as it was introduced here without the var statement.

Chad Grant Over a year ago

Would have been a better question if you had show that in your example code or explained that.

Collectives™ on Stack Overflow

How to pass a parameter to an inline javascript function

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related