backdrop-contrib / ckeditor

NOW IN BACKDROP CORE 🎉 Rich text editor integration using the CKeditor library.
GNU General Public License v2.0
0 stars 0 forks source link

Setting Editor CSS to Use theme CSS generates a fatal error #2

Closed vtad closed 9 years ago

vtad commented 9 years ago

Fatal error: Call to a member function load() on a non-object in /home/www/backdrop/core/includes/theme.inc on line 1415

and then calling config generates Warning: Illegal offset type in isset or empty in theme_get_setting() (line 1413 of /home/www/backdrop/core/includes/theme.inc). ... more notices and warnings ...

quicksketch commented 9 years ago

It sounds like the $theme variable that is being passed into this function is not a string. Perhaps CKEditor is calling theme_get_setting() with an array or object for the $theme parameter?

quicksketch commented 9 years ago

This is probably the issue:

// Grab stylesheets from color module
$color_paths = theme_get_setting('color_' . $current_theme . '_stylesheets', array());

That second parameter should just be dropped. In Drupal 7, the second parameter was the default and the third parameter was the theme name. In Backdrop, the default no longer needs to be provided (it's always populated by the .info file), so the second parameter was dropped.

Graham-72 commented 9 years ago

Thanks @quicksketch. I have done this on my test site and the problem is fixed. Great!

quicksketch commented 9 years ago

You're welcome! Thanks for doing this port. :smile:

klonos commented 9 years ago

Fixed. Right?