[buddypress-trac] [BuddyPress Trac] #5241: Notices occur when BP_ENABLE_ROOT_PROFILES is enabled

buddypress-trac noreply at wordpress.org
Tue Jul 22 17:26:37 UTC 2014


#5241: Notices occur when BP_ENABLE_ROOT_PROFILES is enabled
------------------------------------+-----------------------
 Reporter:  henrywright             |       Owner:
     Type:  defect (bug)            |      Status:  reopened
 Priority:  normal                  |   Milestone:  2.1
Component:  Core                    |     Version:  1.7
 Severity:  minor                   |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+-----------------------

Comment (by r-a-y):

 > 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?

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5241#comment:19>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list