Changing an entity's default display may not be working correctly [#2758569]

It was reported in #2404999: Don't save records for default values that when an entity has a non-default, non-custom display (i.e. there are multiple displays available, but it's not using the one configured as the default for that view mode) changing the value to the default doesn't make the change take effect.

Comment	File	Size	Author
#5	panelizer-n2758569-5.patch	5.75 KB	DamienMcKenna
#5	7.x-3.x: PHP 5.3 & MySQL 5.5, D7 33 pass
#3	panelizer-n2758569-3.patch	2.55 KB	DamienMcKenna
#3	7.x-3.x: PHP 5.3 & MySQL 5.5, D7 32 pass, 1 fail
#2	panelizer-n2758569-2.patch	2.02 KB	DamienMcKenna
#2	7.x-3.x: PHP 5.3 & MySQL 5.5, D7 33 pass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

30 June 2016 at 12:36

DamienMcKenna created an issue. See original summary.

Comment #2

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 30 June 2016 at 20:30

Status:

Active

» Needs review

File	Size
panelizer-n2758569-2.patch	2.02 KB
7.x-3.x: PHP 5.3 & MySQL 5.5, D7 33 pass

This test shows it appears to be working correctly after all.

Comment #3

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 30 June 2016 at 20:32

File	Size
panelizer-n2758569-3.patch	2.55 KB
7.x-3.x: PHP 5.3 & MySQL 5.5, D7 32 pass, 1 fail

An updated test to see if the display can be switched back to the default display.

Comment #4

30 June 2016 at 20:35

Status:

Needs review

» Needs work

The last submitted patch, 3: panelizer-n2758569-3.patch, failed testing.

Comment #5

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 30 June 2016 at 21:11

Status:

Needs work

» Needs review

File	Size
panelizer-n2758569-5.patch	5.75 KB
7.x-3.x: PHP 5.3 & MySQL 5.5, D7 33 pass

2 files were hidden/shown/deleted

File	Size
panelizer-n2758569-2.patch	2.02 KB
7.x-3.x: PHP 5.3 & MySQL 5.5, D7 33 pass
panelizer-n2758569-3.patch	2.55 KB
7.x-3.x: PHP 5.3 & MySQL 5.5, D7 32 pass, 1 fail

Lets see if this fixes the bug.

Comment #6

30 June 2016 at 21:49

DamienMcKenna committed a9d771a on 7.x-3.x

Issue #2758569 by DamienMcKenna: When changing a node from a non-default...

Comment #7

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 30 June 2016 at 21:49

Status:

Needs review

» Fixed

*phew*. Committed.

Comment #8

FeyP CreditAttribution: FeyP at werk21 commented 4 July 2016 at 04:46

@DamienMcKenna: Thanks a lot for the quick fix! It works like a charm now. I'm so glad, that this made it into the new release :). Keep up the good work, it is much appreciated!