First attempt at an upgrade path from Panel Nodes to Panelizer.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

merlinofchaos’s picture

Changes in the variables have made this require a little rework. Also, 2,x being entity-aware has renamed the table used to panelizer_entity -- we probably need to detect which table to use (we can use db_table_exists) and behave appropriately.

Also, panel_node should be able to detect that its table no longer exists and remove its menu functions so that even if left enabled, it will cease doing anything.

merlinofchaos’s picture

Oh well the stuff I just said about 2.x only actually applies to the 7.x branch. But in 6.x I believe we're still changing the variables for features.

Letharion’s picture

Assigned: merlinofchaos » Letharion
Status: Needs review » Needs work
Letharion’s picture

Project: Panels » Panelizer
Version: 6.x-3.x-dev » 7.x-2.x-dev
Component: Panel nodes » Code

We did agree to put the code for this in the Panelizer module instead, correct?

DamienMcKenna’s picture

@Letharion: that'd be the best place for it, IMHO.

lotyrin’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev
Assigned: Letharion » lotyrin
Status: Needs work » Needs review
FileSize
2.98 KB

I need this for a project. This is against 7.x-3.x, if we like the approach I'll backport to 7.x-2.x.

Letharion’s picture

Assigned: lotyrin » Unassigned

Great that you are picking this up. The reason I haven't is that I don't actually need this anywhere, so my tests were completely artificial.

Un-assigning it since you're not the one doing review. I would assign it to Merlin, but I don't have such powers in this queue. :)

lotyrin’s picture

Status: Needs review » Needs work

Doesn't quite work. I forgot to even execute one of the queries... too large of a coffee:sleep ratio I guess.

codewatson’s picture

Any progress on the upgrade path? If panels is really going to depreciate panel nodes in the future then our site will really need this to work by then?

DamienMcKenna’s picture

FYI I added a corresponding issue in the Panels issue queue to deprecate Panel Nodes once this is resolved: #1833984: Deprecate Panel Nodes in favor of Panelizer

sonicthoughts’s picture

So it looks like there is no longer any way to display a panel node format in a view. Has anyone been using this script against 3.1?