$page['content'] key not being validated in page_build hook = Unsupported operand types error [#1291338]

On line 651 (both in 7.x-1.x-dev and -alpha2) the presence of the 'content' key in the page array needs to be validated in the page_build hook:

change
$page['content'] = array('metatags' => array()) + $page['content'];

to:
$page['content'] = (array_key_exists('content', $page)) ? array('metatags' => array()) + $page['content'] : array('metatags' => array());

This creates the $page['content'] key if it does not exist in the $page array, and sets the empty 'metatags' array to the index.

This error appears to have been reported in the following issue, but not resolved in the dev release:

http://drupal.org/node/1282452

I notice it on the CKEditor admin page.

Comments

Comment #1

ciniko CreditAttribution: ciniko commented 27 September 2011 at 14:11

I had the same problem, which is solved after i used your code, thanks...

Comment #2

Dave Reid

he/him

English

Nebraska USA

CreditAttribution: Dave Reid commented 30 September 2011 at 01:15

Status:

Active

» Fixed

The current code has this above the described lines:

  // For some reason with Overlay enabled we get an empty $page, so just fail
  // this case.
  if (!isset($page['content'])) {
    return;
  }

This is already fixed in the latest release.

Comment #3

davmorr CreditAttribution: davmorr commented 30 September 2011 at 19:58

I just upgraded to the alpha2 release and the error message no longer displays. I wasn't sure if the "$page['content'] => array ( [metatags] => array ( ) ) )" was expected elsewhere and would cause less obvious process issue down the line - I didn't have time to step through the code as much as I wanted to - so I went ahead and created placeholders, just in case. Nice module, thanks for contributing!