The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Hi
Working on upgrading av D6 site to 7 and bumped into this issue and I'm not able to resolve without some assistance. Trying to edit/clone/export some of the views give me a WSOD and:
PHP Fatal error: Call to a member function get_option() on a non-object in /drupal-7.2/sites/all/modules/views/includes/admin.inc on line 1063
- Drupal 6.22 -> Drupal 7.2
- Views latest dev.
cTools - beta 1
PHP 5.3.x
I think the problem is related to Views Attach (http://drupal.org/project/views_attach) used back in views 6.3.x before EVA came along, at least these views are the one blowing up on edit/clone/export.
As a non OOP ninja (PHP objects, patterns and Practice is still gathering dust in my shelf), do I have a little problem working out, how where to start digging. Is the problem that $handler is undefined? Yeah, I'm a bit lot, but willing to learn :)
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 1175286-missing-display-fix-with-unset.patch | 2.06 KB | dawehner |
| #10 | 1175286-missing-display-fix.patch | 2.06 KB | dawehner |
| #10 | error.png | 11.01 KB | dawehner |
| #7 | 1175286-missing-display-fix.patch | 1.06 KB | dawehner |
Comments
Comment #1
dawehnerCan you give us some help to reproduce the problem? This would be great.
Comment #2
steinmb commentedHi, thanx for your quick reply :)
Went back to D6 and removed the node attach displays from the exploding views, and migrated one more time. No they do not blow up so I think to reproduce you could:
- Use D6 and CCK, Views 6.2.x + views attach
- Create a view that picks up a few CCK fields, I used tables layout (prob. not important)
- Create a display that use, views attach and attach this view to a node on full node render.
- Verify that this is working
- Turn off Views, CCK, views attach etc. and run your migration to D7 and Views 7.3.x
- Edit the view that used views attach and boom!
Comment #3
dawehnerWell you probably have to wait until views_attach is ported.
Not sure whether views should try to work around the problem, because that's actually something which shows you clearly that you have somehow a problem on your site.
Comment #4
swentel commentedAlso, Views attach is deprecated for http://drupal.org/project/eva
Comment #5
steinmb commentedYeah, that is the core of the problem. There will never be a D7 version of views attach and, from what I know, never a migration path. I'm OK with recreating views attach in EVA, but it is hard to do, when Views blows up when we try to do just that.
Comment #6
merlinofchaos commentedWell, in any case we shoudln't WSOD. I suspect this means that for invalid displays the new admin is crashing. I know the old UI checked for that stuff, but some of those checks probably got lost in updates. WSODs like this are at least major.
Comment #7
dawehnerHere is a patch which adds a little help text.
Comment #8
merlinofchaos commentedThere's a typo in the text, initialize is misspelled.
The text we usually use is something like "Invalid/missing display handler" -- might want to search and model it to be the same.
Comment #9
merlinofchaos commentedThis looks like the wording we're currently using:
Comment #10
dawehnerHere is a new patch and a screenshot attached.
The -with-unset is for testing.
Comment #11
dawehnerMh okay, the default one also has the unset. So takes care when you commit.
Comment #12
dawehnerCommited.