Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

#954 closed defect (bug) (fixed)

Load plugin template files without requiring extra installation step

Reported by: djpaul's profile DJPaul Owned by: djpaul's profile DJPaul
Milestone: 1.1 Priority: minor
Severity: Version:
Component: Keywords: has-patch dev-feedback
Cc: djpaul@…

Description

Any non-basic plugin will introduce new pages for its component into a BP site. This requires end-users to move the template files out of the folder. This sucks because after the one-click plugin install introduced in WP 2.7, you still have to log into your server to finish installing the plugin.

The proposed patch will call an action when it tries to load a template file which doesn't exist in the currently active site theme. This will allow BuddyPress plugins to hook in to the chain of events and remove the extra installation step that otherwise has to happen.

If a user does choose to move the plugin template files into their site theme, those will be loaded instead allowing them to be easily customised if so desired.

Attachments (3)

ticket_954.patch (597 bytes) - added by DJPaul 13 years ago.
ticket_954-2.patch (604 bytes) - added by DJPaul 13 years ago.
ticket_954-3.patch (642 bytes) - added by DJPaul 13 years ago.

Download all attachments as: .zip

Change History (9)

@DJPaul
13 years ago

#1 @DJPaul
13 years ago

Second patch is less rubbish.

#2 @DJPaul
13 years ago

  • Owner set to DJPaul
  • Status changed from new to assigned

ticket_954-3.patch works. A plugin author would implement the action as per this example which I use in my Achievements plugin: http://buddypress.pastebin.com/f1c094f38

#3 @DJPaul
13 years ago

To put the pastebin snippet above into context, I am loading the screen file with the following (as usual):

bp_core_load_template( 'achievements-member-theme/summary' );

#4 @DJPaul
13 years ago

  • Keywords dev-feedback added

#5 @apeatling
13 years ago

  • Type changed from enhancement to defect

#6 @apeatling
13 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [1930]) Fixes #954

Note: See TracTickets for help on using tickets.