Skip to content
This repository was 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
Closed

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

Nathaniel401 opened this issue Dec 6, 2015 · 5 comments
Assignees
@hyyan
Labels
bug

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
Resolving issue hyyan#16
26c612c 
hyyan#16
@ebelrose ebelrose mentioned this issue Apr 21, 2016
Merged
@ebelrose
Copy link
Contributor

ebelrose commented Apr 21, 2016 •
edited
Loading

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
Merge pull request #28 from ebelrose/patch-1
e0b14ce 
Resolving issue #16
@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

@hyyan hyyan closed this as completed Jul 30, 2016
hyyan added a commit that referenced this issue Jul 30, 2016
@hyyan
Fixed code standard after merging PR #16
34b87ff
@JosefinBerger JosefinBerger mentioned this issue Mar 13, 2022
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@hyyan hyyan

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Nathaniel401 @ebelrose @razcakappa @hyyan @decarvalhoaa