Closed (fixed)
Project:
DraggableViews
Version:
6.x-3.2-beta6
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Aug 2009 at 18:40 UTC
Updated:
14 Apr 2010 at 20:53 UTC
Jump to comment: Most recent file
Comments
Comment #1
seanrThis was from beta5 to beta6 BTW.
Comment #2
seanrEven after removing the module, changing the views to use a different style plugin and then redownloading and enabling the module, I still get the error. All of my modules (views, cck, etc.) are completely up to date. As soon as I enable this module, it hoses my site.
Comment #3
sevi commentedI'm shocked.
Unfortunately I cannot reproduce this error message. It seems as if a display handler is broken.
Try the attached patch to find out what's wrong. It will output an error message for the broken view/display.
Greetings,
sevi
Comment #4
seanrMind redoing that as a unified diff? (cvs diff -up > example.patch)
Comment #5
sevi commentedOh. Oups, sure.
Greetings,
sevi
PS: Just to be sure the BETA7-version checks for a valid handler object (like this patch does).
But this patch provides additional debug output.
Comment #6
seanrbeta7 fixes the fatal error. Your patch returns the following output:
error => View news, display panel_pane_1 is broken.
error => View videos, display panel_pane_1 is broken.
error => View press, display panel_pane_1 is broken.
error => View events, display panel_pane_1 is broken.
All but videos are default views defined by the ngpcampaign module, though I think they've all been tweaked a bit.
Comment #7
sevi commented..but not by DraggableViews, I suppose. This module doesn't change other views on its own.
I installed the ngpcampaign module. I haven't spent much time for this but I couldn't notice any problems. Probably there's another module tweaking your views..?
Greetings,
sevi
Comment #8
seanrI meant that I had tweaked the views myself after the module defined them (you can edit module-defined views).
Comment #9
seanrHere's the video view export, if that helps any:
Comment #10
sevi commentedConcerned code has been removed due to some theme-related refactoring (see #541130: Theme registery issues).
Your error should be gone too. The development snapshot contains the changes.
Setting status to "closed",
greetings,
sevi