I'm so happy about this module. You guys knocked it out of the park, but I'm receiving this error message.

warning: array_keys() [function.array-keys]: The first argument should be an array in /home/****/public_html/modules/color/color.module on line 138.
warning: array_shift() [function.array-shift]: The argument should be an array in /home/****/public_html/modules/color/color.module on line 139.

Seems like everything's working. Should we be concerned about this?

Thanks again for your outstanding work.

CommentFileSizeAuthor
#7 ckeditor-652274.patch1.64 KBmephir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

timotter9’s picture

I'm seeing that exact same errors.

sutch’s picture

To get rid of these warnings, I re-enabled the site's theme (somehow it was set as the default but not enabled) and reselected the administration theme. A similar problem is noted at http://drupal.org/node/290009.

Update: I spoke too soon. These warnings are still being displayed.

Duplika’s picture

Same problem here.

MBroberg’s picture

Me too, using a Marinelli-based theme.

mephir’s picture

I need more infrmation, i cannot confirm this bug.
Could you provide me a list of installed modules with their version numbers?
Could you give a name and version of theme which were used ehwn these warnings occur?
You can also give me helping hand with write about Drupal version.

wwalc’s picture

Ok, it seems we can reproduce this bug.
1. Start with Garland theme
2. In CKEditor profile change "User Interface color" to "Color module: base"
3. Save CKEditor profile
4. Change theme to Bluemarine (doesn't support color module)
5. Go to node/add/page
6. Result:

Warning: array_keys() [function.array-keys]: The first argument should be an array in D:\xampp\htdocs\drupalck\modules\color\color.module on line 138
Warning: array_shift() [function.array-shift]: The argument should be an array in D:\xampp\htdocs\drupalck\modules\color\color.module on line 139
mephir’s picture

Status: Active » Needs review
FileSize
1.64 KB

It looks like color module developers overlooked something in color.module. Reported on Drupal project issue page (#654266: color_get_palette warnings).
Patch harden ckeditor.module for this bug.

wwalc’s picture

Status: Needs review » Fixed
Anonymous’s picture

Issue tags: +p

I confirm the same problem with Marinelli theme, how can i apply the patch?
Thanks

Anonymous’s picture

Issue tags: -p

I confirm the same problem with Marinelli theme, how can i apply the patch?
Thanks

wwalc’s picture

Use the dev release until the next stable version of CKEditor module is available.

Status: Fixed » Closed (fixed)

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

brulain’s picture

Hello,
Thanks for this dev release : it works fine !
And congratulations for this new wonderfull module : strong, fast and very nice !!! Its father FCKEditor can rest in peace.
Happy new year!