New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix Uncaught Error: Call to a member function load_reports() on null #5839

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@Mte90

Mte90 commented Oct 2, 2018

PR Checklist

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

Summary

I got an error, every week probably because of a wp cli script executed by the system cron.

Description

An explanation of what is done in this PR

  • If the code generate an issue don't rise the exception
[30-Sep-2018 12:40:06 UTC] PHP Fatal error:  Uncaught Error: Call to a member function load_reports() on null in /home/domain/wp-content/plugins/elementor/includes/tracker.php:335
Stack trace:
#0 /home/domain/wp-content/plugins/elementor/includes/tracker.php(136): Elementor\Tracker::get_system_reports_data()
#1 /home/domain/wp-includes/class-wp-hook.php(286): Elementor\Tracker::send_tracking_data()
#2 /home/domain/wp-includes/class-wp-hook.php(310): WP_Hook->apply_filters('', Array)
#3 /home/domain/wp-includes/plugin.php(515): WP_Hook->do_action(Array)
#4 /home/domain/wp-cron.php(126): do_action_ref_array('elementor/track...', Array)
#5 {main}
  thrown in /home/domain/wp-content/plugins/elementor/includes/tracker.php on line 335

Quality assurance

  • I have tested this code to the best of my abilities
  • I have added unittests to verify the code works as intended
  • Docs have been added / updated (for bug fixes / features)

Fixes #

@CLAassistant

This comment has been minimized.

CLAassistant commented Oct 2, 2018

CLA assistant check
All committers have signed the CLA.

KingYes added a commit that referenced this pull request Oct 3, 2018

@KingYes

This comment has been minimized.

Member

KingYes commented Oct 3, 2018

This issue fixed by this commit: 0469518

Thanks!

@KingYes KingYes closed this Oct 3, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment