The fix posted for #2493443: The ajax_html_ids list hits server limit. does not account for the case when there are no selectors to exclude, and in jQuery 1.7 at least :not() causes an error. just adding a test on the array length fixed it for me.

CommentFileSizeAuthor
#1 wysiwyg-excludeSelectors-2522462.patch558 bytesbrad.bulger

Comments

brad.bulger’s picture

brad.bulger commented
Status: Active » Needs work
StatusFileSize
new wysiwyg-excludeSelectors-2522462.patch558 bytes

  • TwoD committed 727a208 on 7.x-2.x authored by brad.bulger 
    - #2522462 by brad.bulger: Fixed empty excludeSelectors array causes JS...
twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Needs work » Fixed

Nice catch, forgot to test that because I always had CKEditor enabled.
The if-statement also needs to wrap the clearing of the list when there are no exclusions to perform. Other than that the patch looks great!
I made the minor changed and pushed it to the 7.x-2.x branch.

Thanks a lot!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.