I am just trying resolve a random bug so decided to run some modules through the Coder module to eliminate that as a possible issue. Saw few issues with the D7 version that would be great to get into an upcoming release.

CommentFileSizeAuthor
#1 wysiwyg.coder_.1.patch10.21 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Priority: Normal » Minor
Status: Active » Needs review
FileSize
wysiwyg.coder_.1.patch10.21 KB
mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

It applies well.

When I went here /admin/config/content/wysiwyg to test it though I got:
Fatal error: Class name must be a valid object or a string in /home/dm7/includes/common.inc on line 7394

I brought the old wysiwyg code back and it worked fine.

Thanks for doing this sun. I know it's a lot of work!

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented

Thanks Sun!

Status: Fixed » Closed (fixed)

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