This patch fixes the error "get_option() on null in view->init_query()"

The fix below on line :946 on includes/view.inc is included in this patch:

if (empty($this->display_handler)) {
  return FALSE;
}
CommentFileSizeAuthor
#6 3111384-2.patch437 bytesvgardner
#5 Screen Shot 2020-02-07 at 8.19.53 AM.png24.18 KBrenatog
#2 3111384-1.patch441 bytesvgardner

Comments

vgardner created an issue. See original summary.

vgardner’s picture

vgardner
Location London
commented
StatusFileSize
new 3111384-1.patch441 bytes
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-3.23 » 7.x-3.x-dev

I'm sorry you've run into that problem, and I appreciate you taking the time to build a patch for it.

Have you tested to see if it's still a problem with the current dev version, or even 3.24-rc1?

vgardner’s picture

vgardner
Location London
commented

I appreciate the quick response. Yes, I'm seeing the same issue on both dev and 7.x-3.24-rc1.

renatog’s picture

renatog commented
Issue summary: View changes
Status: Active » Reviewed & tested by the community
StatusFileSize
new Screen Shot 2020-02-07 at 8.19.53 AM.png24.18 KB

So really makes sense IMHO

P.s. We have extra spaces on patch #2 (line 950):

But we can fix it easily

Thank you so much

vgardner’s picture

vgardner
Location London
commented
StatusFileSize
new 3111384-2.patch437 bytes

Great, thanks! Here's a second patch without the spaces.

renatog’s picture

renatog commented

Wow, great!

Thank you so much!

  • DamienMcKenna committed afb6d5c on 7.x-3.x authored by vgardner 
    Issue #3111384 by vgardner, RenatoG, DamienMcKenna: Error: get_option()...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed
Parent issue: » #3118500: Plan for Views 7.x-3.25

Committed. Thank you.

Status: Fixed » Closed (fixed)

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