#7103 closed defect (bug) (fixed)
Embeds: rename `BP_oEmbed_Component` class
| Reported by: | DJPaul | Owned by: | r-a-y |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.6 |
| Component: | Core | Version: | |
| Severity: | normal | 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Thanks for the feedback.
Maybe rename to
BP_Core_oEmbed_Extensionto be similar toBP_Group_Extension?