The secure CMS.

Welcome to Geeklog, Anonymous Saturday, December 06 2025 @ 11:58 pm EST

Geeklog Forums

Forum plugin SEO

 New Topic  Post Reply
 12/05/14 07:06am (Read 2,634 times)  
Status: offline

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1582
For a better SEO of your forum plugin 2.8.0

public_html/forum/index.php line 109

Text Formatted Code

//SEO Title
$title = $LANG_GF01['FORUM'];
$forum_id = COM_applyFilter($_REQUEST['forum'],true);
if (isset($_REQUEST['forum'])) $title = stripslashes(DB_getItem($_TABLES['forum_forums'],'forum_name',"forum_id='{$forum_id}'")) ;
// Display Common headers
$display .= gf_siteHeader($title);


public_html/forum/viewtopic.php line 63

Text Formatted Code
//SEO title
$result = DB_query("SELECT forum, pid, subject, forum_name FROM {$_TABLES['forum_topic']} LEFT JOIN {$_TABLES['forum_forums']} ON forum = forum_id WHERE id = '$showtopic'"); // <- new
list($forum, $topic_pid, $subject, $forum_name) = DB_fetchArray($result); // <- new
$subject = stripslashes($forum_name) . ' | '. $subject;


and for your forum plugin 2.9.0

public_html/forum/index.php line 1026

Text Formatted Code

//SEO Title
$title = $LANG_GF01['FORUM'];
$forum_id = COM_applyFilter($_REQUEST['forum'],true);
if (isset($_REQUEST['forum'])) $title = stripslashes(DB_getItem($_TABLES['forum_forums'],'forum_name',"forum_id='{$forum_id}'")) ;
$display = gf_createHTMLDocument($display, $title);


public_html/forum/viewtopic.php line 66

Text Formatted Code
//SEO title
$result = DB_query("SELECT forum, pid, subject, forum_name FROM {$_TABLES['forum_topic']} LEFT JOIN {$_TABLES['forum_forums']} ON forum = forum_id WHERE id = '$showtopic'"); // <- new
list($forum, $topic_pid, $subject, $forum_name) = DB_fetchArray($result); // <- new
$subject = stripslashes($forum_name) . ' | '. $subject;
 Quote
 12/05/14 09:07pm  
Status: offline

Laugh

Site Admin
Admin
Registered: 09/27/05
Posts: 1482
Location:Canada
Good catch Ben. I added your changes to the forum repository so they will be included in the next version of the forum.
One of the Geeklog Core Developers.
 Quote
 12/06/14 03:20am  
Status: offline

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1582
Maybe we could also add a h1 tag to topic_navbar.thtml

Text Formatted Code
 <td colspan="2" style="padding-bottom:5px;"><h1>{navtopicimg}&nbsp;<b><a href="viewtopic.php?showtopic={topic_id}">{subject}</a></b></h1></td>
 Quote
 12/06/14 10:41am  
Status: offline

Laugh

Site Admin
Admin
Registered: 09/27/05
Posts: 1482
Location:Canada
Both Modern Curve and Denim use an H1 tag in the header. We should really only have 1 h1 tag per page.

I think we should leave this one and if the person requires it, then they will have to manually add it.

BTW I am hoping at some point soon that the forum templates will receive an update to make them responsive.

Tom
One of the Geeklog Core Developers.
 Quote
 12/06/14 11:30am  
Status: offline

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1582
Maybe the site name or a site logo with an h1 tag on every page is not the most SEO optimisation Wink Anyway, a responsive theme will be a very nice feature.
 Quote
 12/06/14 11:59am  
Status: offline

Laugh

Site Admin
Admin
Registered: 09/27/05
Posts: 1482
Location:Canada
Like with most SEO it is largely a matter of opinion. I have seen cases brought forth for both (and I am not sure which is best).

With the way the themes are setup that come with Geeklog H1 is the title of the site (or logo) and the Article titles are H2 (along with other things like the Directory). Since the article titles are H2 then any H tags in the actual article should be H3 or higher.
One of the Geeklog Core Developers.
 Quote
 New Topic  Post Reply
All times are EST. The time is now 11:58 pm.
  • Normal Topic
  • Sticky Topic
  • Locked Topic
  • New Post
  • Sticky Topic W/ New Post
  • Locked Topic W/ New Post
  •  View Anonymous Posts
  •  Able to post
  •  Filtered HTML Allowed
  •  Censored Content