function statement requires a name error in JQuery

Question

I have this function but it gives me function statement require name onStop function

<script type="text/javascript">
 jQuery( function($) {
    $('#Hnav, #Nnav').NestedSortable(
            {
                accept: 'sort',
                noNestingClass: "no-children",
                helperclass: 'helper',
                autoScroll: true,
                onChange: function(serialized) {
                onStop : function(){
                    $('#output').html($(this).id);
                },
                    nestingPxSpace : '0'
            }
    );
});
</script>

You didn't close the onChange function correctly so there is a parentheses mismatch — devnull69
– devnull69, Commented Jun 28, 2012 at 10:03

Florian Margaine · Accepted Answer · 2012-06-28 10:16:24Z

4

In this context, this refers to the DOM element on which the onStop event happens. A DOM element is not a jQuery object.

jQuery $(this) object doesn't have id property, while the DOM elements do. So, use either:

$('#output').html(this.id);

or:

$('#output').html($(this).attr("id"));

And don't forget to close the bracket in onChange handler function.

edited Jun 28, 2012 at 10:16

Florian Margaine

61.2k15 gold badges94 silver badges120 bronze badges

answered Jun 28, 2012 at 10:04

VisioN

146k35 gold badges287 silver badges291 bronze badges

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

3 Comments

Florian Margaine Over a year ago

The answer would be better if you explained the why

VisioN Over a year ago

@FlorianMargaine Can't do it better :)

Florian Margaine Over a year ago

May I? :-p I'd just add what this is, so the why one works while another doesn't.

Umesh Aawte · Accepted Answer · 2012-06-28 10:20:50Z

1

Collectively there are two problems in the code,

The inner function was not closed properly.
Used id property of element that is not supported by the jQuery selector.

The modified code is,

jQuery( function($) {
    $('#Hnav, #Nnav').NestedSortable(
            {
                accept: 'sort',
                noNestingClass: "no-children",
                helperclass: 'helper',
                autoScroll: true,
                onChange: function(serialized) {
                    //A empty function
                },
                onStop : function(){
                    $('#output').html($(this).attr("id"));
                },
                nestingPxSpace : '0'
            }
    );
});

answered Jun 28, 2012 at 10:20

Umesh Aawte

4,7007 gold badges44 silver badges52 bronze badges

6 Comments

Zoha Ali Khan Over a year ago

there is a problem in JS still when I load the page it says jQuery("#" + s).get(0) is undefined

Umesh Aawte Over a year ago

As far as this code is considered there is no problem now this problem is with the NestedSortable javasciprt file you are using in your code. Also check the HTML #Hnav, #Nnav this could be a problems also.

Zoha Ali Khan Over a year ago

Thanks for your help i fixed it. I have an issue with toggeling can you please help me?

Umesh Aawte Over a year ago

Why not share your problem. If you are not able to toggle things. I need HTML code and JavaScript code also

Zoha Ali Khan Over a year ago

wait let me create a question

|

Adil · Accepted Answer · 2012-06-28 10:30:33Z

1

There was a bracket missing and you use wrong syntax getting id try this,

$('#output').html($(this).id);

should be

$('#output').html(this.id);

or

$('#output').html($(this).attr(id));

  jQuery(function($) {
    $('#Hnav, #Nnav').NestedSortable({
        accept: 'sort',
        noNestingClass: "no-children",
        helperclass: 'helper',
        autoScroll: true,
        onChange: function(serialized) {
            alert("changed");// Changed to fix
        },               
        onStop: function() {
             $('#output').html(this.id);
        },
        nestingPxSpace: '0'       
    });
});

edited Jun 28, 2012 at 10:30

answered Jun 28, 2012 at 10:03

Adil

148k25 gold badges217 silver badges207 bronze badges

9 Comments

devnull69 Over a year ago

no it's not ... bracket is missing right before the onStop parameter

devnull69 Over a year ago

Try again ... onStop won't be inside onChange

Florian Margaine Over a year ago

God, you don't even understand what this does does, do you?

devnull69 Over a year ago

Please ignore the -2 answer and concentrate on the +2 one and the comments

Adil Over a year ago

I have changed the answer check it again @Zoha Ali Khan and all.

|

Collectives™ on Stack Overflow

function statement requires a name error in JQuery

3 Answers 3

3 Comments

6 Comments

9 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

6 Comments

9 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related