Skip to content
This repository has been archived by the owner on Mar 17, 2022. It is now read-only.

Catchable fatal error: Order List on Dashboard getProductTranslationByObject() is being given a Boolean instead of an WC_Product Object #16

Closed
Nathaniel401 opened this issue Dec 6, 2015 · 5 comments
Assignees
Labels

Comments

@Nathaniel401
Copy link

First,

Let me say thanks for taking the time to make this plugin; it's greatly appreciated!

I am having a problem after upgrading my WooCommerce and WordPress cores. I am getting the following error when looking on my Orders list in the dashboard:

Catchable fatal error: Argument 1 passed to Hyyan\WPI\Utilities::getProductTranslationByObject() must be an instance of WC_Product, boolean given, called in /wp-content/plugins/woo-poly-integration/src/Hyyan/WPI/Order.php on line 105 and defined in /wp-content/plugins/woo-poly-integration/src/Hyyan/WPI/Utilities.php on line 88

Cheers man!

Nathaniel

ebelrose added a commit to ebelrose/woo-poly-integration that referenced this issue Apr 21, 2016
@ebelrose
Copy link
Contributor

ebelrose commented Apr 21, 2016

Just pushed corrections for this.
Let's see if the plugin author accepts it.

@razcakappa
Copy link

@ebelrose Did your pull request accept? I'm having the same error which @Nathaniel401 has.

@decarvalhoaa
Copy link
Contributor

I have been patching and develop further this plugin. I have included ebelrose pull in my fork. Can I suggest you use my fork at https://github.com/decarvalhoaa/woo-poly-integration/releases/tag/v0.27

Note that I have not test it yet with WC 2.6. I don't usually upgrade immediately after a release to field, since in the first months there will be plenty of issues being raised and solved, and testing all this releases is time consuming.

@ebelrose
Copy link
Contributor

No, there was no commit on this project in 11 months so I presume it's dead

hyyan added a commit that referenced this issue Jul 30, 2016
@hyyan hyyan added the bug label Jul 30, 2016
@hyyan hyyan self-assigned this Jul 30, 2016
@hyyan
Copy link
Owner

hyyan commented Jul 30, 2016

PR is accepted , thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
5 participants