I'm having a scope issue with a javascript function

Question

The variable checkedones is not passing to the AJAX.Updater.

The item.value is reporting a value so I know that there should be something there, however I'm just left with t pland when I get to the last alert.

How can this be fixed/improved?

    function sendemail(){
        var checkedones='';
        var i= $$('.emailer');
        i.each(function(item)
                {

                   if(item.checked){
                    alert(item.value);
                    alert(item.checked);
                    var checkedones = checkedones + item.value + ',';   
                  }
                }
        )
        alert(checkedones);
        new Ajax.Updater('asdfa', 'pages/domailing.php', {
     method:'post',evalScripts:'true' ,parameters: {
  subject1:$('subject1').value,
subject2:$('subject2').value,
body:tinyMCE.get('emailbody').getContent(),
value:checkedones},
evalJS:'force',
onComplete:function(){}
        });
    }

David Wick · Accepted Answer · 2011-09-06 00:36:41Z

2

try changing: var checkedones = checkedones + item.value + ',';

to: checkedones += (item.value + ',');

you've re-declared by using var inside the loop.

answered Sep 6, 2011 at 0:36

David Wick

7,1152 gold badges38 silver badges38 bronze badges

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

1 Comment

BostonAreaHuman Over a year ago

Brilliant. And much more elegant. thanks for the quick response.

Collectives™ on Stack Overflow

I'm having a scope issue with a javascript function

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related