The patch in the next reply will limit the settings to the main account page only.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

djg_tram’s picture

FileSize
895 bytes

The patch.

rooby’s picture

One 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.

djg_tram’s picture

Just post that slightly different patch here, I guess that's the easiest solution.

MegaChriz’s picture

I 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:

require_once('includes/ckeditor.user.inc');
MegaChriz’s picture

Marked issue #1170120: Cleaning up user interface as a duplicate of this one.

mkesicki’s picture

Thank you for patch we try check this as soon as possible.

dczepierga’s picture

Title: CKEditor settings appear on all user profile edit pages » [D7] CKEditor settings appear on all user profile edit pages
Status: Needs review » Fixed

@MegaChriz, really thx for patch.

I commit it to GIT (diff).
Pls check last DEV version and write if it doesn't work.

Greetings

Status: Fixed » Closed (fixed)

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