Warning messages - Panopoly seems OK. Though I had installed the Panopoly News App and this is not showing up as listed as an app (assuming it is an app)
(Issue about this:
Perhaps the warnings are something to do with why Panopoly News is not listed as an App.

The warnings are shown at the end of this issue, in the section required by the issue template.

Proposed resolution

Fix so that these warnings don't get displayed.

Remaining tasks

As above.

User interface changes

Not applicable - don't think UI changes are the issue.

API changes

Perhaps it is an issue with using an API incorrectly.

Original report by [username]

Warning: Illegal string offset '#theme' in apps_preprocess_apps_install_page() (line 137 of ..profiles/panopoly/modules/contrib/apps/theme/
Warning: Invalid argument supplied for foreach() in element_children() (line 6300 of ../includes/
Warning: Illegal string offset '#children' in drupal_render() (line 5767 of ../includes/
Warning: Illegal string offset '#children' in drupal_render() (line 5777 of ../includes/
Warning: Illegal string offset '#children' in drupal_render() (line 5815 of ../includes/
Warning: Illegal string offset '#printed' in drupal_render() (line 5822 of ../includes/


therobyouknow’s picture

Running on MAMP PRO on MacOS. PHP 5.4.4, MySQL 5.5.25
Drupal 7.16. Updated to latest using drush.

Although these messages appeared, Panopoly appears to run OK. It is disconcerting to see them though. Not fixing this issue may be storing up problems for later.


populist’s picture

I will look into these, but think they are probably PHP 5.4 issues and glad it isn't causing you too much trouble.

In terms of Panopoly News, it is currently not ready for RC1 (boo) so it isn't there but it shall be soon and then it will appear in the Apps section.

therobyouknow’s picture

Thanks @populist for your quick follow up - most appreciated!

populist’s picture

Title: Warning messages on admin page admin/apps/panopoly/manage - various "Warning: Illegal string offset" in drupal_render() etc » PHP 5.4 Issues with Featured App Logic
Project: Panopoly » Apps
Version: 7.x-1.0-beta6 » 7.x-1.x-dev
Component: Admin » Code
Status: Active » Needs review

I did some more digging here and believe this is an issue with Apps. The problem here is that the logic that defines a #theme callback is doing an isset() check on $featured_apps in apps_preprocess_apps_install_page() to add its callback. However, in the case where there are no featured apps, the $featured_apps variable *IS* set but it contains the string "No applications currently available." instead of information around Featured Apps.

I made a patch which I think resolves the problem mentioned in #1, but I don't use the featured apps section so not sure. However, this should fix the error so wanted to throw this out.

therobyouknow’s picture

Thanks @populist for the further follow up. I noticed that you changed the title of the issue to be more precise about the problem - thanks.

febbraro’s picture

Status: Needs review » Active

@populist I think you forgot to attach the patch.

therobyouknow’s picture

Just out of interest, how do you "dig", @populist - do you use a debugger to find problems like these?

I'm currently evaluating the following PHP debuggers, welcome a quick vote from you as to which you use, if any, or something else?

Anything with xdebug pre-compiled would be a bonus (or even better, doesn't rely upon it), it seems to be a pain to setup, having to compile it, find dependencies, etc.

populist’s picture

#7 - I did not use a debugger, but just saw some error reports from Panopoly so I figured the code manually.

#5 - I apologize for not attaching the patch and it is now gone. Apps might need a larger PHP 5.4 review.

InternetDevels’s picture

Status: Active » Needs review
1.3 KB

I have created the patch, which is supposed to fix the issue for 7.x-1.0-beta7 version of the module.Yet, it seems the output was changed in the last release of dev version of the module, thus it's not working for this version at the moment (not displaying featured app, ratings etc). That's why I don't understand the idea of the plans concerning Apps Install page. The logic of this patch is irrelevant for 7.x-1.x-dev version.

hefox’s picture

Status: Needs review » Needs work

Needs work based on this being a patch against beta

hefox’s picture

Issue summary: View changes