Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

Status: Active » Needs review
FileSize
4.75 KB

Here's the patch

dsnopek’s picture

Status: Needs review » Fixed

Committed :-)

dsnopek’s picture

Status: Fixed » Needs work

Blergh. I just noticed #3120637: ckeditor backport e40827c which gets some additional instances of eval() that we should probably get too. I'm going to look at merging those changes into this patch.

dsnopek’s picture

Status: Needs work » Needs review
FileSize
7.42 KB

Here's a new version of this patch that includes the additional changes from #3120637: ckeditor backport e40827c

dsnopek’s picture

Status: Needs review » Fixed

Committed!

nhoeller’s picture

I upgraded to ckeditor-6.x-1.17 this morning. The only minor issue I noticed is that the Available Updates report no longer shows CKEditor, but rather "projectname 6.x-1.17 / Includes: CKEditor". It looks like the update to "ckeditor.info" clobbered the "project =" line.

Regards, Norbert

dsnopek’s picture

@nhoeller that was my mistake when running the script to update the .info file. It'll get fixed in the next release!

Status: Fixed » Closed (fixed)

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