Line 1003 (or so) of dxmpp.js uses the .unwrap() function of jquery, which wasn't introduced until jquery 1.4. This is problematic because the latest dev version of jquery_update, which is noted as required on the project page, only goes up to 1.3.2.

CommentFileSizeAuthor
#3 dxmppremoveupdate.js_.patch606 bytessrjosh

Comments

ManyNancy’s picture

ManyNancy commented

You can replace .unwrap with:

.parent().each(function(){
    $(this).replaceWith( this.childNodes );
});

That should fix it.

srjosh’s picture

srjosh
he/him
commented

well, you can also simply upgrade your version of jquery, which is what I did, but the issue needs to be addressed all the same.

srjosh’s picture

srjosh
he/him
commented
Status: Active » Needs review
StatusFileSize
new dxmppremoveupdate.js_.patch606 bytes

Updating the jquery stuff doesn't seem to play well with Panels; here's a patch that rolls in ManyNancy's change.

aaron’s picture

aaron commented
Priority: Normal » Critical

bumping this... would definitely be nice to have this not require jquery ui 1.4 for a beta...