Repeatable: Always
Steps to repeat:
1. Update to Wysiwyg 7.x-2.3
2. Change the setting "Enable on summary fields" in a wysiwyg profile (CKEditor in this instance)
3. Visit a node add form that has a longtext field with summary the same profile, the summary will always have the CKEditor applied.

Expected Results:
User expected the CKEditor to be removed from the field

Actual Results:
CKEditor is applied to the summary part of the field

Fix:
A look at the module code shows this setting is attached to the form within the lines

if (isset($profile->settings['add_to_summaries']) && !$profile->settings['add_to_summaries']) {
     $settings[$format]['skip_summary'] = 1;
}

The "add_to_summaries" key is found in the $profile->preferences array.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dapulli created an issue. See original summary.

Proteo’s picture

I can confirm the issue after updating to 7.x-2.3. The patch fixed the problem for me, thanks!

  • TwoD committed 1b3c3d9 on 7.x-2.x authored by dapulli
    - #2843308 by dapulli: Fixed regression with attaching to summary fields...
TwoD’s picture

Title: "Enable on summary fields" setting in CKEditor profile for full HTML not being applied » Regression with attaching to summary fields
Status: Active » Fixed

Thanks for patching and testing this!

Status: Fixed » Closed (fixed)

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

afinnarn’s picture

I tried to apply this patch locally but saw that the patch was made from a project root instead of the module's root. I'm uploading the patch since we use a tool to apply patches from d.o.