Skip to:
Content

bbPress.org

Opened 7 years ago

Closed 7 years ago

#2239 closed defect (invalid)

Get rid of some template files...

Reported by: alexvorn2 Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.1
Component: General Keywords:
Cc:

Description

For custom pages we have:

archive-forum.php
archive-topic.php
single-forum.php
single-forum-edit.php
single-reply.php
single-reply-edit.php
single-reply-move.php
single-topic.php
single-topic-edit.php
single-topic-edit.php
single-topic-split.php
single-user.php
single-user-edit.php
single-view.php
taxonomy-topic-tag.php
taxonomy-topic-tag-edit.php

They all have the same layout, just a different class, why not just remove them all and use single page.php file instead ? (as I see bbPress already is using it if some of these files is missed)

So in the page.php file we will have:

if ( function_exists( 'is_bbpress' ) and is_bbpress() ) {
  // all bbpress templates here
} else {
  // page template
}

Change History (2)

#1 @alexvorn2
7 years ago

so this will help increase the speed of the custom modification...

#2 @johnjamesjacoby
7 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Confused; these templates aren't the same, and have very little overlap. They exist in the /extras folder, as an indication of possible extra root template files someone can create on their own, or copy over.

Closing as invalid. Feel free to reopen if there's something I'm misunderstanding.

Note: See TracTickets for help on using tickets.