When resetting the display for a given entity view mode, it currently deletes all records for the entity - regardless of view mode. What it instead should do is update the entity_type/entity_id/revision_id/view_mode record to the default panelizer display, or at least just delete that one record, not everything.

CommentFileSizeAuthor
#4 panelizer-n2218275-4.patch3.9 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Parent issue: » #2155813: Plan for Panelizer 7.x-3.2 release

This needs to be fixed in the next release.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Related issues: +#2209083: 'Reset' should have option to create new revision

It's related to #2209083: 'Reset' should have option to create new revision.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: 'Reset' should not delete all {panelizer_entity} records » 'Reset' should not delete all {panelizer_entity} records for the entity
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
panelizer-n2218275-4.patch3.9 KB

This adds a new reset_entity_panelizer() method to PanelizerEntityDefault that updates the {panelizer_entity} records to the current default display, rather than just deleting records.

  • Commit 6cf81f7 on 7.x-3.x by DamienMcKenna: 
    Issue #2218275 by DamienMcKenna: 'Reset' action should reset to the...
DamienMcKenna’s picture

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

Committed.

Status: Fixed » Closed (fixed)

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