devinsays / portfolio-press

A WordPress theme for artists and designers to showcase their work.
https://wptheming.com/portfolio-theme/
GNU General Public License v2.0
62 stars 30 forks source link

Concerns about standard-posts.php #68

Closed mfields closed 10 years ago

mfields commented 10 years ago

After some thought, I would strongly suggest not including this page template in the theme. The main reason is that using a page to display posts goes against the general flow of how WordPress' query system is designed which can and will lead to conflicts with plugins. One such example would be a plugin that expects is_home() to return true when the most recent posts are displayed.

I would propose using core functionality as much as possible here. You should be able to instruct users to set up a "post page" under Settings -> Reading -> Front page displays. You can then add an option giving user's a choice to have the image and gallery formatted posts removed where is_home() returns true. You should be able to hook into the pre_get_posts action to remove the unwanted posts.

devinsays commented 10 years ago

Okay, that's one option I considered when building this out. I'll go that direction instead.