When resetting an entity's display, it wasn't actually deleting the entity's display. The code in reset_entity_panelizer() is at fault.

CommentFileSizeAuthor
#4 panelizer-n2757277-4.patch3.91 KBDamienMcKenna
#2 panelizer-n2757277-2.patch3.3 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
panelizer-n2757277-2.patch3.3 KB

This should cover it. Might need some tests.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

That doesn't work as the {panelizer_entity} record is still present. Poop.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2757277-4.patch3.91 KB

Forgot to delete the actual {panelizer_entity} record(s).

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Title: Resetting a display doesn't work unless a new revision is created » Resetting a display doesn't work correctly
Issue summary: View changes

Clarifying the title.

  • DamienMcKenna committed 0fe7bfb on 7.x-3.x 
    Issue #2757277 by DamienMcKenna: Resetting a display didn't work...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Assigned: DamienMcKenna » Unassigned