I create a test content type, page
I enabled full content
I panelized it for page content type
I disabled full content
I enabled default
all content was still laid out with the disabled full content layout

Comments

socialnicheguru’s picture

Title: Disabling panelizer mode does not change layout of node or new nodes created » Disabling panelizer layout does not change layout of node or new nodes created

changed title

merlinofchaos’s picture

Are you sure that 'default' was actually different than 'full' at that time?

rancas’s picture

The same for me using 7.x-3.0-rc1+6-dev on panoply distribution.

merlinofchaos’s picture

Status: Active » Fixed

Indeed, the old panelizer object was still being found and used. I committed and pushed a fix.

astutonet’s picture

Status: Fixed » Active

I have this issue too, using 7.x-3.0+0-dev (2013-jan-30) version.

In my case, I create a teaser panelizer. When I try to disable or remove it, no chance.

The teaser panelized is showed with the same layout, as if nothing had happened.

Even with the settings disabled in the teaser display, the teasers that were worked by Panelizer are displayed with the settings that are disabled.

I can say this with certainty because I have other teasers that were not affected by Panelizer, which are displayed side by side on the homepage.

We needs to have an option to remove changes that are no longer desired made by Panelizer. This also can be understood as a feature request.

Edit:

The first code block below is responsible for displaying teasers which were not treated by Panelizer.

<div class="views-row views-row-1 views-row-odd views-row-first views-row-last">
<article id="node-11" class="node node-typeone node-teaser contextual-links-region article odd node-lang-pt-br iat-c clearfix" about="/mysite/content/article-11" typeof="sioc:Item foaf:Document" role="article">

The second block of code is responsible for displaying teasers that were treated by Panelizer but are disabled.

<div class="views-row views-row-1 views-row-odd views-row-first views-row-last">
<div class="panelizer-view-mode node node-teaser node-typetwo node-10 contextual-links-region">
merlinofchaos’s picture

Status: Active » Needs review
StatusFileSize
new493 bytes

Ahh, that same check needs to also go in the entity view alter. I should've checked that one too. :(

This patch should fix it. Can you verify it for me?

astutonet’s picture

StatusFileSize
new20.74 KB

Verified. The patch returns an error, as in the image.

Tks.

astutonet’s picture

Status: Needs review » Active

Hi @merlinofchaos,

I forgot to change the status. Can you help me?

merlinofchaos’s picture

Status: Active » Needs review
StatusFileSize
new494 bytes

Ahh, just a missing parens. I'd hoped whoever tested this would have enough PHP knowledge to fix small things like that, but that's okay, Here's an update.

Also, when a patch fails testing, 'needs work' is an appropriate status.

merlinofchaos’s picture

StatusFileSize
new484 bytes

Last patch also doesn't work. Try this one.

astutonet’s picture

Status: Needs review » Reviewed & tested by the community

Hi Merlin.

The patch in #10 worked perfectly.

About the lack of parenthesis, I apologize, it was a lack of attention on my part.

Sorry & Thank you.

merlinofchaos’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed!

Status: Fixed » Closed (fixed)

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