Skip to content

Added versions to prevent warning on WooCoomerce #6

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RubenMartins
Copy link
Contributor

No description provided.

@cimplicity cimplicity mentioned this pull request Apr 16, 2021
@zanman69
Copy link

zanman69 commented Apr 1, 2022

Hi, can someone please point me in the right direction to get this plugin working? I've been struggling for two days now :-) I downloaded the latest version and I did use composer to update and install dependencies, compressed it to zip again, uploaded it to Woocommere and activated it and when I go to settings / Sage One Integration it says the following:
" ThinkingLogic WooCommerce / Sage One Integration
There has been a critical error on this website. Please check your site admin email inbox for instructions. "

When seeking help with this issue, you may be asked for some of the following information:
WordPress version 5.9.2
Current theme: Storefront (version 4.0.0) Current plugin: ThinkingLogic WooCommerce / Sage One Integration (version 0.2) PHP version 8.1.3

Error Details
wc-sage-one-integration.zip

=============
An error of type E_ERROR was caused in line 34 of the file /home/zjzti3ezmv7d/public_html/wp-content/plugins/wc-sage-one-integration-master/options.php. Error message: Uncaught Error: Call to a member function isRefreshTokenExpiringSoon() on bool in /home/zjzti3ezmv7d/public_html/wp-content/plugins/wc-sage-one-integration-master/options.php:34
Stack trace:
#0 /home/zjzti3ezmv7d/public_html/wp-content/plugins/wc-sage-one-integration-master/thinkinglogic-wc-sage-one-integration.php(71): include_once()
#1 /home/zjzti3ezmv7d/public_html/wp-includes/class-wp-hook.php(307): tl_wc_sage_plugin_options()
#2 /home/zjzti3ezmv7d/public_html/wp-includes/class-wp-hook.php(331): WP_Hook->apply_filters()
#3 /home/zjzti3ezmv7d/public_html/wp-includes/plugin.php(474): WP_Hook->do_action()
#4 /home/zjzti3ezmv7d/public_html/wp-admin/admin.php(259): do_action()
#5 /home/zjzti3ezmv7d/public_html/wp-admin/options-general.php(10): require_once('/home/zjzti3ezm...')
#6 {main}
thrown

@YetAnotherMatt
Copy link
Member

Hi @zanman69, I've had a go at fixing the error you were seeing. I'm afraid I don't offer any form of support, it's definitely a case of use it at your own risk.

@zanman69
Copy link

zanman69 commented Apr 5, 2022

Hi @YetAnotherMatt thank you for your reply, I will check if I can fix it on my side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants