For sure the shorter the better, we'll see what suffix will end up the best when we release the update
I recoded sessions.php and .htaccess without the html suffix and it seems
to work, except for this rewrite rule:
- Code: Select all
# FORUM PROTECTION RULE
# RewriteRule ^forums/.+/([^/]+)$ forums/ [R=301,L]
What is this protecting? Incoming referrals to topics or posts that have been deleted?
I handle deleted topics / posts by redirecting to a "not found" page with pointers to the search choices. The robots.txt includes the "not found" page, so presumably the deleted URL will be removed from the search engine's cache someday. Would have been better to return a 301 to the forum's index? Not that forum topics are deleted that
As an aside, I noticed this board doesn't define a custom 404 page, e.g.:
http://www.phpbb-seo.com/boards/phpbb-f ... sions.html
Returns the standard Apache 404 page. Bleech.
Back to my testing, I also recoded viewtopic.php and viewtopic_body.tpl to follow the # convention, i.e., viewtopic?p= becomes topic-subject-vtxxx
with the mini post's alt/title text = topic subject. Probably similar to what you're working on for the next version, except of course I hope your handling of pagination was cleaner. Well, I followed your lead with $seo_topic_name and declared globals $seo_topic_id and $seo_topic_start. Not pretty, but works.