#5241 closed defect (bug) (fixed)
Notices occur when BP_ENABLE_ROOT_PROFILES is enabled
Reported by: | henrywright | Owned by: | r-a-y |
---|---|---|---|
Milestone: | 2.1 | Priority: | normal |
Severity: | minor | Version: | 1.7 |
Component: | Core | Keywords: | needs-refresh has-patch |
Cc: | mpa4hu@… |
Description
I am getting the same 3 PHP notices at various times when using BP with WordPress SEO:
- On successful avatar upload
- After clicking the submit button when searching private messages
The notices are:
Notice: Trying to get property of non-object in /htdocs/wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 548
Notice: Trying to get property of non-object in /htdocs/wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 840
Notice: Trying to get property of non-object in /htdocs/wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 843
After speaking with one of the plugin's contributors in the WP forum see the WP forum thread, they suggested I raise a BP ticket due to the notices being caused by BuddyPress doing something to the $post
object which causes it to not be an object.
Attachments (1)
Change History (25)
#2
@
11 years ago
I was using a child theme of bp-default.
Considering the attempts to get the $post property are all wrapped in a is_singular()
check (see lines 548, 840 and 843 in class-frontend.php
), should that particular piece of plugin code be running at all at the point of change-avatar or private message search? The change-avatar and private message pages are not any of is_single()
, is_page()
or is_attachment()
are they?
UPDATE: I've been doing some investigation and can see an is_singular()
check is being used around the code attempting to get the $post property. is_page()
seems to return true for the change-avatar and private message pages (where there notices are displayed) but the $post
variable seems to be null here. Could this be the reason why toys are being thrown out of the push buggy?
#3
@
11 years ago
- Keywords reporter-feedback removed
- Milestone Awaiting Review deleted
- Resolution set to wontfix
- Status changed from new to closed
: I've been doing some investigation and can see an is_singular() check is being used around the code attempting to get the $post property. is_page() seems to return true for the change-avatar and private message pages (where there notices are displayed) but the $post variable seems to be null here. Could this be the reason why toys are being thrown out of the push buggy?
Thanks for the additional investigation. This is related to the fact that you're using a bp-default descendant. In a nutshell: Because BP content is not WP content, it's not found by WP's initial parse_request() -> query chain. So there is a portion of the load process where WP thinks that requests for BP content are 404s. BP manually unsets this 404 before loading its template, and where/how this happens depends on whether you're using theme compat. The pre-theme-compat switch happens here http://buddypress.trac.wordpress.org/browser/tags/1.8.1/bp-core/bp-core-catchuri.php#L378. We manually flip is_page and is_singular to true, as well as turning off the 404. But we don't do anything to the $post
global.
In contrast, theme compat fakes the entire content of a "dummy" post. See http://buddypress.trac.wordpress.org/browser/tags/1.8.1/bp-members/bp-members-screens.php#L407. So if you switch to a theme that utilizes theme compat, the notices should go away.
(FWIW, I switched to bp-default to test and I *still* didn't see the notices. So this is mostly an educated guess.)
Given that the old method is being abandoned, and given that the notices shouldn't actually cause any real problems (aside from some non-optimized robots/meta tags in wordpress-seo), I'm going to close this as wontfix. My apologies to the wordpress-seo devs for the inconvenience - this issue will disappear as people stop using the old BP themes.
#4
@
11 years ago
boonebgorges, thanks for the detailed explanation of what is happening, you've made sense out of a situation that was confusing me all morning. The guys at wordpress-seo suggested they can add some validation at their end - which will resolve the issue until people migrate away from bp-default.
#5
@
11 years ago
- Resolution wontfix deleted
- Status changed from closed to reopened
Please feel free to re-close this ticket but I wanted to make you aware that notices have increased 11-fold after upgrading to 1.9 beta 1. This is the set of notices displayed when visiting a member's profile page:
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 548
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 638
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 840
Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 843
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3034
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3402
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3404
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3406
#6
@
11 years ago
I can see 3 of the notices that seem to be repeated lots are due to the use of the is_page()
function (see lines 3402, 3404, 3406 in wp-includes/query.php).
#7
@
11 years ago
henrywright: I can't duplicate this. Can you list the plugins that you are running?
#8
@
11 years ago
r-a-y - Those notices came from a customised theme which had a few plugins activated. To make things less complicated, it might be worth looking at the PHP notices on http://jobtain.co.uk/henrywright/
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3276
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3651
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3653
Notice: Trying to get property of non-object in /wp-includes/query.php on line 3655
Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3276
The site has no plugins activated (aside from BP 1.9 beta 1). Uses Twenty Thirteen as it comes out of the box.
#9
@
11 years ago
henrywright: I still can't duplicate this locally. Are you sure you're not running any additional plugins / code snippets anywhere?
I'm on WP 3.7.1, BP 1.9-bleeding. Tested with both Twenty Twelve and Twenty Thirteen.
Can you try reinstalling WordPress from the admin dashboard ("Dashboard > Updates") to see what happens?
#11
@
11 years ago
- Component changed from Core to Rewrite Rules
- Milestone set to 2.0
- Severity changed from normal to minor
- Summary changed from $post object doesn't seem to be an object to Notices occur when BP_ENABLE_ROOT_PROFILES is enabled
- Version set to 1.7
This will hopefully resolve itself when rewrite rules are implemented in 2.0 (see #4954).
#12
@
11 years ago
Some more feedback on this since upgrading to 1.9. I'm now getting a huge number of debug notices when root profiles are enabled.
Steps:
a) Enable root profiles
b) visit the notifications page example.com/username/notifications
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3675
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 552
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 642
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3671
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3673
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3675
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 848
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 851
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3671
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3673
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3675
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 154
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 159
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3671
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3673
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3675
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-frontend.php on line 642
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-opengraph.php on line 124
- Notice: Trying to get property of non-object in /wp-includes/general-template.php on line 1442
- Notice: Trying to get property of non-object in /wp-includes/general-template.php on line 1583
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3671
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3673
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3675
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-opengraph.php on line 327
- Notice: Trying to get property of non-object in /wp-includes/post-template.php on line 29
- Notice: Trying to get property of non-object in /wp-content/plugins/wordpress-seo/frontend/class-opengraph.php on line 332
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3671
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3673
- Notice: Trying to get property of non-object in /wp-includes/query.php on line 3675
- Notice: Trying to get property of non-object in /wp-includes/comment-template.php on line 939
- Notice: Trying to get property of non-object in /wp-includes/comment-template.php on line 964
- Notice: Trying to get property of non-object in /wp-includes/general-template.php on line 1704
- Notice: Undefined property: WP_Query::$post in /wp-includes/query.php on line 3296
These notices don't happen when root profiles are disabled.
#13
@
11 years ago
- Component changed from Rewrite Rules to Core
- Keywords has-patch added
While looking into another ticket, I came across this bug again.
Thought I might as well patch this since rewrite rules isn't going to be implemented until the next release cycle.
Attached patch:
- Removes the query to look up the page based on the BP page offset. I don't think this is necessary. When we move to rewrite rules, we'll be using virtual pages anyway.
- Resets the post with
bp_theme_compat_reset_post()
since it's a great function to add a dummy post - Sets the theme compat flag to false since bp_theme_compat_reset_post() sets that flag to true
- Tweaks the
locate_template()
call so it is only run whenbp_use_theme_compat_with_current_theme()
is false. This should probably be moved to a separate commit altogether, but thought I'd throw it in here since I was touchingbp_core_load_template()
!
Notices should disappear now when root profiles is enabled. I've tested this in various scenarios and it works. Also passes the existing root profile unit tests.
Feedback appreciated.
#14
@
11 years ago
Is this going to cause problems for people doing checks like is_page( 'members' )
, since we're switching from the bp_page to a dummy page?
#16
@
11 years ago
someway related:
in the "dummy" post the post ID is set to int 0, what about themes that in their pages may use functions like get_post_thumbnail_id() that expects a "real" ID? On failure, it will return false instead of an empty string (see the Codex) and I bet lots of themes are checking this against an empty string.
#17
@
11 years ago
- Cc mpa4hu@… added
@r-a-y if I only have found this ticket earlier :)
I can confirm that patch works
The only thing that changed in my case (I use classes to determine current page) was that body classes on homepage now are
directory activity buddypress page page-id-0 page-parent page-template-default logged-in no-js row
earlier it also had home-page
but this is something I can work on, just reporting.
#18
@
10 years ago
- Keywords dev-feedback added
I think I would like to bump this to 2.2 because it sounds a pretty big change to make this late in the cycle, we might miss something subtle.
#19
follow-up:
↓ 20
@
10 years ago
in the "dummy" post the post ID is set to int 0, what about themes that in their pages may use functions like get_post_thumbnail_id() that expects a "real" ID?
get_post_thumbnail_id()
doesn't work right now on BuddyPress pages because theme compatibility overrides the WP post anyway with or without this patch. It only works if you pass the correct post ID.
The only thing that changed in my case (I use classes to determine current page) was that body classes on homepage now
I can confirm this. It's missing the home-page
class if you're using a BP page as the homepage and the page-id-X
class is now always page-id-0
. Once we move to rewrite rules, there will not be a page ID anyway.
I think I would like to bump this to 2.2
If I address the missing <body>
home-page
class, would it be doable for 2.1?
#20
in reply to:
↑ 19
@
10 years ago
- Keywords dev-feedback removed
If I address the missing
<body>
home-page
class, would it be doable for 2.1?
Sure, as long as you're confident that the type of concern mentioned by Boone earlier isn't going to be a problem. 2.1's not scheduled to hit beta for almost 2 more weeks, so there's enough time. I bumped the ticket purely because it hadn't had any activity for about 3 months, and I wasn't sure if anyone wants going to finish up this last bit.
Replying to boonebgorges:
Is this going to cause problems for people doing checks like
is_page( 'members' )
, since we're switching from the bp_page to a dummy page?
henrywright - What theme are you using? Is it using BP's theme compat?
I'm dubious that BP is wiping out the
$post
global - we use it in a number of important ways. I've just done a bit of testing using Twenty Twelve as a theme, and was not able to reproduce the notices. Are there any special settings I need to activate in WordPress SEO?