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

CommentFileSizeAuthor
#4 ckeditor-user-profile-fix-1350998-4.patch746 bytesmegachriz
#1 ckeditor_1350998_01.patch895 bytesdjg_tram

Comments

djg_tram’s picture

djg_tram commented
StatusFileSize
new ckeditor_1350998_01.patch895 bytes

The patch.

rooby’s picture

rooby commented

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

djg_tram commented

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

megachriz’s picture

megachriz
Primary language Dutch
commented
StatusFileSize
new ckeditor-user-profile-fix-1350998-4.patch746 bytes

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

megachriz
Primary language Dutch
commented

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

mkesicki’s picture

mkesicki commented

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

dczepierga’s picture

dczepierga commented
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.