In panelizer_update_7115()

Update script clones pane

$new_pane = clone $display->content[$pid];

And then adds panes for revisions using db_insert

 $new_pane->pid = db_insert('panels_pane')
            ->fields(array(
              'did' => $new_pane->did,
              'panel' => $new_pane->panel,
              'type' => $new_pane->type,
              'subtype' => $new_pane->subtype,
              'shown' => $new_pane->shown,
              'access' => serialize($new_pane->access),
              'configuration' => serialize($new_pane->configuration),
              'cache' => serialize($new_pane->cache),
              'style' => serialize($new_pane->style),
              'css' => serialize($new_pane->css),
              'extras' => serialize($new_pane->extras),
              'locks' => serialize($new_pane->locks),
              'uuid' => $new_pane->uuid
            ))
            ->execute();

So, position field is not recorded.

I suggest to add

'position' => $new_pane->position,

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Yury_Kaminski created an issue. See original summary.

azinck’s picture

Status: Active » Needs review
FileSize
441 bytes

This does seem quite important.

  • DamienMcKenna committed ffae911 on 7.x-3.x authored by azinck
    Issue #2795735 by azinck: Missing {panels_pane}.position value in update...
DamienMcKenna’s picture

Status: Needs review » Fixed
Parent issue: » #2787147: Plan for Panelizer 7.x-3.5 release

Oh goodness! Thanks for catching that. And for the patch. Committed.

Status: Fixed » Closed (fixed)

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