It's great to be able to customize FCKeditor through fckeditor.config.js, but the placement of the file seems to me to make it very easy to overwrite by mistake (like when updating the module to a new version). Apparently Drupal's settings.php file had that problem. I know I overwrote a few of those with the default settings.php in Drupal 5. Now in Drupal 6 the file is called default.settings.php and renamed settings.php by the installer. That way you can't overwrite your server settings by uploading a fresh distribution of core. I wonder if a similar approach couldn't be adopted with fckeditor.config.js. Call it default.fckeditor.config.js in the download package and have the installation routine rename it. Or--an option I'm even fonder of--what if the module checked for the presence of fckeditor.config.js in the theme folder and used it if it existed the way the templating system does for tpl.php files? That would keep things all nicely packaged and portable. Whatever the final solution, I think it would be good to do something.

Comments

Jorrit’s picture

Nice suggestion, I'll add it to our todo list!

JohnAlbin’s picture

Yay! Drupal.org hit 300,000 nodes! Happy nodeday, Drupal! ;-)

Jorrit’s picture

I would fix this bug just so I could put this nice number in our changelogs.

TravisCarden’s picture

Haha. Sweet! :-P I'm glad you like the suggestion, too. Thanks for adding it to the todo list. :-)

kenorb’s picture

Gratulation of 300k!;)

chrisschaub’s picture

You can change the location of the config file. You set the location under the advanced options in the fckeditor drupal config area. This way it can be in your theme directory or wherever. I create a fckeditor directory in my theme directory and locate other fckeditor custom files and plugins there. You can localize all fckeditor settings this way.

Under advanced options:

CustomConfigurationsPath = '/sites/all/themes/sometheme/fckeditor/fckeditor.config.js';

Hope it helps.

TravisCarden’s picture

Version: 5.x-2.2-rc1 » 6.x-1.3
Status: Active » Closed (duplicate)