Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#6 | wysiwyg-enable_summary-2843308-7-6.patch | 578 bytes | afinnarn |
wysiwyg-enable_summary_settting_not_applied.patch | 682 bytes | dapulli |
Comments
Comment #2
Proteo CreditAttribution: Proteo as a volunteer commentedI can confirm the issue after updating to 7.x-2.3. The patch fixed the problem for me, thanks!
Comment #4
TwoDThanks for patching and testing this!
Comment #6
afinnarn CreditAttribution: afinnarn at University of Colorado Boulder commentedI 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.