javascript - Syntax error, unrecognized expression: , $(selector).before(","); - Stack Overflow

I've been using this one line of code for a couple of years now with jQuery 1.2.6.

$("#acListTemp div.amenitiesDiv label").before(",");

I just upgraded to jQuery 1.6.1 and now it is giving me this error:

Syntax error, unrecognized expression: ,

I also tried this, but it yielded the same error:

theChar = ",";
$("#acListTemp div.amenitiesDiv label").before(theChar);

I checked the jQuery API page for the before mand, but I'm still stumped. Any help is greatly appreciated!

I've been using this one line of code for a couple of years now with jQuery 1.2.6.

$("#acListTemp div.amenitiesDiv label").before(",");

I just upgraded to jQuery 1.6.1 and now it is giving me this error:

Syntax error, unrecognized expression: ,

I also tried this, but it yielded the same error:

theChar = ",";
$("#acListTemp div.amenitiesDiv label").before(theChar);

I checked the jQuery API page for the before mand, but I'm still stumped. Any help is greatly appreciated!

• javascript
• jquery

Share Improve this question Follow edited Jun 9, 2011 at 15:48 gen_Eric 227k4242 gold badges303303 silver badges342342 bronze badges asked Jun 9, 2011 at 15:43 DanDan 11111 gold badge11 silver badge33 bronze badges 9

• 6 Seems fine to me: jsfiddle. Are you sure it's this line causing the issue? Using the @ symbol in 1.3+ causes that error (source). – Town Commented Jun 9, 2011 at 15:46
• 4 Are you using the fancy quotes, like in the question title? – Jeremy Commented Jun 9, 2011 at 15:49
• 2 @Pointy: He is getting a syntax error. – Jeremy Commented Jun 9, 2011 at 15:51
• 2 @Jeremy Heiler: That was my first thought, but they're added automatically in titles on SO. Interestingly (or not) they cause Syntax error: Unexpected token ILLEGAL in Chrome. – Town Commented Jun 9, 2011 at 15:52
• 3 I thought that the error was ing from inside jQuery, as if it were trying to parse the string as a selector. If not, well, then you could very well be right. – Pointy Commented Jun 9, 2011 at 15:53

 |  Show 4 more ments

1 Answer 1

Sorted by: Reset to default 11

If you're using .insertBefore() instead of .before(), you'll get that error.

The error doesn't appear using insertBefore() in jQuery 1.2.6

---

Looks like the error es specifically from Sizzle:

https://github./jquery/sizzle/blob/master/sizzle.js#L325-327

Sizzle.error = function( msg ) {
    throw "Syntax error, unrecognized expression: " + msg;
};

...so I'd guess that somewhere in your code you're using "," as a selector, perhaps in insertBefore() or some other method that expects a selector.

---

EDIT:

From the ments below, it has been determined that jQuery does not fail silently as expected when calling .before() against a jQuery object for which no matches are found for its selector.

Seems like a bug to me, but I'm sure the jQuery guys will e up with some reason why this is the desired behavior.

You can see the issue here: https://github./jquery/jquery/blob/1.6.1/src/manipulation.js#L130-140

before: function() {
    if ( this[0] && this[0].parentNode ) {
        return this.domManip(arguments, false, function( elem ) {
            this.parentNode.insertBefore( elem, this );
        });
    } else if ( arguments.length ) {
        var set = jQuery(arguments[0]);
        set.push.apply( set, this.toArray() );
        return this.pushStack( set, "before", arguments );
    }
},

As you can see, there are only 2 tests.

• The first checks to see if there is at least one element in the jQuery object (and if it has a parentNode). If so, it inserts the content before the matched elements.

• If there were no elements, it just checks to see if any arguments were passed, and if so, it seems to assume that the first one is a selector (or perhaps a tag), and it goes ahead and sends that to the jQuery function.

var set = jQuery(arguments[0]);

So if you're trying to insert a ",", but your selector didn't find any matches, jQuery will end up doing:

var set = jQuery(",");

...which is an invalid expression for Sizzle to process.