Skip to:
Content

BuddyPress.org

Opened 10 years ago

Closed 10 years ago

#7124 closed defect (bug) (fixed)

bp_locate_template_asset() should use the theme compat dir and theme compat url

Reported by: imath Owned by: imath
Priority: normal Milestone: 2.6
Component: Templates Version:
Severity: normal Keywords: has-patch commit
Cc:

Description

@r-a-y when doing the replace thing, the $data['uri'] is not good for a symlinked/custom template pack. Just tested with BP Nouveau.

I think using bp_get_theme_compat_dir() & bp_get_theme_compat_url() would work for any template pack (including BP Legacy).

It's working for BP Nouveau ;)

As you only use $data['file'] to include the embed css inline there's no big deal, but it makes the bp_locate_template_asset() unusable for other assets.

See attached patch.

Attachments (1)

7124.patch (542 bytes ) - added by imath 10 years ago.

Download all attachments as: .zip

Change History (4)

@imath
10 years ago

#1 @r-a-y
10 years ago

  • Keywords commit added

+1.

Commit at will! :)

#2 @imath
10 years ago

  • Milestone Under Consideration2.6

I'll do it asap :)

#3 @imath
10 years ago

  • Owner set to imath
  • Resolutionfixed
  • Status newclosed

In 10889:

Make sure any template pack can use bp_locate_template_asset()

Fixes #7124

Note: See TracTickets for help on using tickets.