Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PHP Notice: Trying to get property of non-object #2

Closed
gedex opened this issue May 19, 2016 · 2 comments
Closed

PHP Notice: Trying to get property of non-object #2

gedex opened this issue May 19, 2016 · 2 comments
Assignees
@gedex
Labels
priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.

Comments

@gedex
Copy link
Member

gedex commented May 19, 2016

Here's the notice:

PHP Notice:  Trying to get property of non-object in /srv/www/wordpress-default/wp-content/plugins/woocommerce-gateway-stripe/includes/class-wc-gateway-stripe.php on line 488

I believe $source->source is a string, containing token, instead of an object.
@gedex gedex added priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug. labels May 19, 2016
@gedex
Copy link
Member Author

gedex commented May 19, 2016

/cc @mikejolley

@gedex gedex self-assigned this May 23, 2016
@gedex
Copy link
Member Author

gedex commented May 23, 2016

This is happens when "Enable Payment via Saved Cards" is disabled.

gedex added a commit that referenced this issue May 23, 2016
@gedex
Fixed notice trying to get property of non-object.
e0fb70f 
The value of `$souce->source` is a string not an object.

Fixes #2.
@gedex gedex mentioned this issue May 23, 2016
Merged
@gedex gedex closed this as completed in #7 May 23, 2016
gedex referenced this issue May 23, 2016
@gedex
Merge pull request #7 from woothemes/issues/2/fix-notice
c077aaf 
Fixed notice trying to get property of non-object.
@azcomps azcomps mentioned this issue Apr 25, 2017
Closed
@kevin25 kevin25 mentioned this issue Nov 22, 2017
Closed
@poolet08 poolet08 mentioned this issue Dec 20, 2017
Closed
2 tasks
@Tigzy Tigzy mentioned this issue Aug 27, 2018
Closed
@thenbrent thenbrent mentioned this issue Oct 2, 2019
Open
@yukikatayama yukikatayama mentioned this issue Oct 22, 2019
Open
@eldritchgreene eldritchgreene mentioned this issue Nov 17, 2020
Closed
@katinthehatsite katinthehatsite mentioned this issue Nov 18, 2020
Closed
@ablears ablears mentioned this issue Dec 15, 2020
Closed
@seffect seffect mentioned this issue Jul 29, 2022
Closed
@isaklafleur isaklafleur mentioned this issue Nov 21, 2022
Closed
@dineshinau dineshinau mentioned this issue Jan 22, 2023
Closed
@marcus-abrahamsson marcus-abrahamsson mentioned this issue Mar 28, 2024
Closed
@Hans9985 Hans9985 mentioned this issue Dec 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gedex gedex

Labels
priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gedex