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.
The patch in the next reply will limit the settings to the main account page only.
Comment | File | Size | Author |
---|---|---|---|
#4 | ckeditor-user-profile-fix-1350998-4.patch | 746 bytes | MegaChriz |
#1 | ckeditor_1350998_01.patch | 895 bytes | djg_tram |
Comments
Comment #1
djg_tram CreditAttribution: djg_tram commentedThe patch.
Comment #2
rooby CreditAttribution: rooby commentedOne note - The patch in the duplicate issue #1371466-1: Show Rich text editor settings fieldset appears on all profile forms, not just the main account one also moves the module_load_include() call inside the if statement so it isn't loaded unnecessarily.
I think that would be a good idea for this patch.
Comment #3
djg_tram CreditAttribution: djg_tram commentedJust post that slightly different patch here, I guess that's the easiest solution.
Comment #4
MegaChriz CreditAttribution: MegaChriz commentedI also came across this problem. The attached patch includes the change from rooby from that other issue, but also changes the hook implementation. Instead of a general hook_form_alter(), the hook hook_form_FORM_ID_alter() is now used, since only one form is altered. This gains a little bit of performance, since implementations of hook_form_alter() are called for every form.
Regarding the module_load_include(): it looks unnecessary to call that, as in line 48 of the module file, the file 'includes/ckeditor.user.inc' is already included:
Comment #5
MegaChriz CreditAttribution: MegaChriz commentedMarked issue #1170120: Cleaning up user interface as a duplicate of this one.
Comment #6
mkesicki CreditAttribution: mkesicki commentedThank you for patch we try check this as soon as possible.
Comment #7
dczepierga CreditAttribution: dczepierga commented@MegaChriz, really thx for patch.
I commit it to GIT (diff).
Pls check last DEV version and write if it doesn't work.
Greetings