The isNode() callback in Javascript WYSIWYG plugins is supposed to allow the plugin to say if it's active or not. It's not working in the latest version of WYSIWYG, because the plugin names for Drupal plugins have changed to include the 'drupal_' prefix which isn't being used when looking up the plugin in TinyMCE 3. I don't know if this bug also affects TinyMCE 4, which I didn't test, but it definitely affects 3!

CommentFileSizeAuthor
#2 wysiwyg-isnode-2890066-2.patch601 bytesdsnopek
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Active » Needs review
FileSize
wysiwyg-isnode-2890066-2.patch601 bytes

Here's a patch that fixes it in my testing!

  • TwoD committed 555ca20 on 6.x-2.x authored by dsnopek 
    - #2890066 by dsnopek: Fixed regression isNode() callback not working...
  • TwoD committed feb0b56 on 7.x-2.x authored by dsnopek
    - #2890066 by dsnopek: Fixed regression isNode() callback not working...
TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD as a volunteer commented
Status: Needs review » Fixed

Thank you very much!

Status: Fixed » Closed (fixed)

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