I am experiencing the same issue as described in this issue, but only in specific circumstances.

https://www.drupal.org/project/yoast_seo/issues/2701091

Specifically, when clicking a button in CKEditor that causes a dialog to be opened (e.g. the link button), this error is generated:

Uncaught TypeError: Drupal.ckeditor.openDialog is not a function

To reproduce the problem that I have encountered, do the following:

  1. Enable this module
  2. Assign to a bundle (e.g. a content type)
  3. Add a new formatted text field to that content type.
  4. Move the Realtime SEO field immediately above the newly added field in the form display
  5. Try to use the Link button in any formatted text field in that content type. No dialog appears, and the error is generated.

Moving the Realtime SEO below the new text field resolves the problem, as does unassigning the module from this content type.

CommentFileSizeAuthor
#10 yoast_seo-ckeditor-function-errors-2934515-10.patch352 bytesandreyks
#4 yoast_seo-ckeditor-function-errors-2934515-4.patch357 byteskingdutch

Comments

millionleaves created an issue. See original summary.

millionleaves’s picture

millionleaves commented
Issue summary: View changes
millionleaves’s picture

millionleaves commented
Title: CKEditor openDialog break if Yoast module is enabled with multiple text fields. » CKEditor openDialog error if Yoast module is enabled with multiple text fields.
kingdutch’s picture

kingdutch
he/him
Primary language English
commented
Version: 8.x-1.3 » 8.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new yoast_seo-ckeditor-function-errors-2934515-4.patch357 bytes

Changing the version because I was able to replicate this in the development branch of 2.x which is actively developed.

It seems the attached one line change fixes the issue.

millionleaves’s picture

millionleaves commented

Thanks. I'm on v1.3. The patch wouldn't apply to my version, but I removed the same line from that version (line 18 in yoast_seo.libraries.yml), cleared cache and the problem was fixed.

  • Kingdutch committed d63177d on 8.x-2.x 
    Issue #2934515 by Kingdutch, millionleaves: CKEditor openDialog error if...

  • Kingdutch committed fe5a2e3 on 8.x-1.x 
    Issue #2934515 by Kingdutch, millionleaves: CKEditor openDialog error if...
kingdutch’s picture

kingdutch
he/him
Primary language English
commented
Status: Needs review » Fixed

As it's a one line fix I didn't see a reason not to take 30 seconds to fix this in the 1.x branch too :)

Thanks!

Status: Fixed » Closed (fixed)

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

andreyks’s picture

andreyks commented
StatusFileSize
new yoast_seo-ckeditor-function-errors-2934515-10.patch352 bytes

Reroll patch for tag 8.x-1.3