Hi

i m am using common distribution with commerce module. i just install backoffice and after enabling it,below is the warning message displaying in /admin/structure/types

Warning: Illegal string offset 'colspan' in PanelizerEntityNode->hook_page_alter() (line 142 of /Applications/MAMP/htdocs/semaphore/profiles/commons/modules/contrib/panelizer/plugins/entity/PanelizerEntityNode.class.php).

Comments

bojanz’s picture

Title: Warning message after enabling backoffice module. » Warning: Illegal string offset 'colspan' in PanelizerEntityNode->hook_page_alter()
Project: Commerce Backoffice » Panelizer (obsolete)
Version: 7.x-1.2 » 7.x-3.x-dev
Category: support » bug

Well, panelizer is not a part of backoffice ;)

merlinofchaos’s picture

Hmm. Something else there is modifying the node types list at admin/structure/types and Panelizer seems to be conflicting with some other change. I've no way of knowing what that change is, however.

bojanz’s picture

Ah, that's us.
We are overriding the page callback completely.
Any advice on how I should proceed here?

merlinofchaos’s picture

My code is trying to add a column to the table.

I could add more verification that the table actually exists, or I could try to figure out what you're doing differently so I can handle both?

Take a look at the function that's dying? It's not terribly complex, it is just trying to make life easier on users so that panelizer info is available at admin/structure/types.

In the worst case we can add validation that the table actually exists and disable that code. In the best case we can make our paths close together so they can cooperate.

kimwes’s picture

Status: Active » Needs review
StatusFileSize
new1.07 KB

The problem is that commerce backorder is adding product display column as the second column in content types listing. Panelizer is setting the colspan of the second column because it's relying that it's the operations column. Here's a patch that changes panelizer to increase the colspan of the last column which should be the operations column.

delacosta456’s picture

i confim, patch #5 fix. i will wait a little days for more test before closing. thanks kimwes

damienmckenna’s picture

Issue summary: View changes
Status: Needs review » Fixed

This worked just fine in my testing. Thanks kimee!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Thanks @kimwes

Your patch works.