Hi,



Right now you are replacing the META keywords with this expression:

Code:
$newtext = preg_replace('#(<meta name="keywords".*?content=)"#is',
                            '$1"' . str_replace('$', '\$' , $kw_content) . ',',
                            $newtext);
which puts a lot of unnecessary stuff in the keywords META. Example of unnecessary stuff are the META keywords that Vbulletin is placing in each and single page that it generates. The keywords for each page should be "localized", meaning that they should be generated based on the page content. Including the same keywords again and again and again in each single page equals to SPAM not SEO.

IMO, a better version would be this:

$newtext = preg_replace('#(<meta name="keywords".*?content=)"[^"]*#is',
'$1"' . $kw_content,
$newtext);
Which only puts in the META keywords the words found in the title, minus the stop words.

Let's take a random VBSEO page:
vBSEO Functions for Extensibility

The current META keywords are this:

HTML Code:
<meta name="keywords" content="vBSEO,Functions,Extensibility, vBSEO Functions for Extensibility, SEO, vbulletin search engine optimization, search engine optimisation, search engine friendly forums, vbulletin seo" />
And it would be better if they would be like this:
HTML Code:
<meta name="keywords" content="vBSEO,Functions,Extensibility" />
Anyway, whether you agree or not, please implement a more flexible META generation so that people will not be forced to hack VBSEO just for some trivial functionality.

In your case (vbseo.com), this problem is not very big because all of your forums have very specific targets: VBSEO, SEO, optimizations aso. The problem is that not all forums are that specialized. For most forums, keeping the same set of keywords in *all* pages is inappropriate.



Regards,
Razvan