#7103 closed defect (bug) (fixed)
Embeds: rename `BP_oEmbed_Component` class
Reported by: | DJPaul | Owned by: | r-a-y |
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Core | Keywords: | |
Cc: |
Description
Please can the BP_oEmbed_Component
class be renamed to BP_Core_oEmbed_Component
?
That way, it'll get picked up by the autoloader, then we can remove the require_once
hack at the top of class-bp-activity-oembed-component.php
.
I really dislike the oEmbed
capitalisation but will let it pass because it's more readable this way.
I also dislike the fact this class is called a "component" which has a very specific meaning in BuddyPress, but I don't have a better suggestion.
Change History (4)
Note: See
TracTickets for help on using
tickets.
Thanks for the feedback.
Maybe rename to
BP_Core_oEmbed_Extension
to be similar toBP_Group_Extension
?