Skip to:
Content

Opened 10 years ago

Closed 9 years ago

#954 closed defect (bug) (fixed)

Load plugin template files without requiring extra installation step

Reported by: DJPaul Owned by: 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 10 years ago.
ticket_954-2.patch (604 bytes) - added by DJPaul 10 years ago.
ticket_954-3.patch (642 bytes) - added by DJPaul 10 years ago.

Download all attachments as: .zip

Change History (9)

@DJPaul
10 years ago

#1 @DJPaul
10 years ago

Second patch is less rubbish.

#2 @DJPaul
10 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
10 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
10 years ago

  • Keywords dev-feedback added

#5 @apeatling
9 years ago

  • Type changed from enhancement to defect

#6 @apeatling
9 years ago

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

(In [1930]) Fixes #954

Note: See TracTickets for help on using tickets.