Array to String convesion in LayoutAttributes.php [#2127711]

On the latest 8.x-1.x-dev for AT and the latest Drupal 8 dev I'm seeing:

Warning: Illegal offset type in isset or empty in Drupal\Core\Extension\ModuleHandler->loadInclude() (line 192 of core\lib\Drupal\Core\Extension\ModuleHandler.php).
Notice: Array to string conversion in Drupal\at_core\Layout\LayoutAttributes->rowAttributes() (line 64 of themes\adaptivetheme\at_core\lib\Drupal\at_core\Layout\LayoutAttributes.php).

All I have done is enable AT Core and generate an AT Standard subtheme which I set as default. I didn't go any further than that.
It's on a XAMPP on Windows 7.

Comments

Comment #1

Jeff Burnz CreditAttribution: Jeff Burnz commented 5 November 2013 at 12:29

What version of PHP are you using?

Can you go to the block configuration and see if any blocks are enabled for your theme? Can you tell me which ones?

What version of Drupal 8 are you using, is it an alpha or dev?

Comment #2

mermentau CreditAttribution: mermentau commented 5 November 2013 at 13:12

PHP 5.4.7
Drupal 8 dev pulled from git yesterday.
Blocks: Syndicate, System Help, Main page content
Sidebar first: Search form, Tools, User login
Sidebar second: Recent content
Footer: Footer, Powered by Drupal

Comment #3

Jeff Burnz CreditAttribution: Jeff Burnz commented 5 November 2013 at 14:28

OK, this is probably the issue: https://bugs.php.net/bug.php?id=60198

I'll have to take a closer look at those arrays and see what gives. Line 64 uses array_intersect()

Comment #4

Jeff Burnz CreditAttribution: Jeff Burnz commented 5 November 2013 at 19:36

Assigned:	Unassigned	» Jeff Burnz
Priority:	Normal	» Major
Status:	Active	» Needs review

I've pushed a patch up to head that should fix the issue, tricky one because I could not reproduce on my system, but I could easily see how this can happen so I have reworked the array_intersect arguments so this should not happen.

Just a heads up that I am massively reworking how layouts work, so you might want to give it a day or two before pulling again, then nuke your themes and start over with testing etc, you won't notice much changes on the front end or in theme settings, but how it works at the backend will be quite different, see this issue: #2128037: Refactor Layouts to be DRY

Comment #5

mermentau CreditAttribution: mermentau commented 12 November 2013 at 17:39

Just seeing the one line of error message now:

Warning: Illegal offset type in isset or empty in Drupal\Core\Extension\ModuleHandler->loadInclude() (line 192 of core\lib\Drupal\Core\Extension\ModuleHandler.php).

I pulled the latest git Drupal 8 dev and AT 8.x-1.x-dev just prior to creating a new sub-theme.