WooCommerce Analytics: a fix to check if the WC session is a valid object

[greenafrican]

Fixes #10533

Changes proposed in this Pull Request:

• Two checks to ensure WC()->session exists before calling get

Testing instructions:

• I'm not clear on how to replicate the issue as reported in Uncaught Error: Call to a member function get() on null in wp-woocommerce-analytics-universal.php:82 #10533

Proposed changelog entry for your changes:

• WooCommerce Analytics: ensure WC()->session exists before calling get method

[jetpackbot]

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

Scheduled Jetpack release: December 4, 2018.
Scheduled code freeze: November 27, 2018

Generated by 🚫 dangerJS

[kraftbj]

It looks good and worked for me, but unable to repo the original issue either.

@hr-automation -- Would you be able to test this on the site where you originally reported the issue? You can download the Jetpack Beta Plugin and then click the Jetpack Beta button on the admin bar once activated. On that page, you can search for the fix/wc-session-is-null branch to grab a copy of this version of Jetpack to test.

[jeherve]

@hr-automation Did you have the chance to give that patch a try yet?

[hr-automation]

@hr-automation Did you have the chance to give that patch a try yet?

Sry for late response - the fix worked like a charm! Thanks for that!