Print

[Makc666]

For example I have added to keywords to some picture (separate with spaces) as gallery says.

And when I look at HTML source I see this one:

Code: [Select]

<meta name="keywords" content="help tree"/>
But meta name="keywords" must be separated with commas like:

Code: [Select]

<meta name="keywords" content="help,tree"/>
For example you can look at this page's HTML code:

Code: [Select]

<meta name="keywords" content="coppermine,gallery,picture,forum" />

[Makc666]

Open:
displayimage.php

Find:

Code: [Select]

if ($CURRENT_PIC_DATA['keywords']) { $meta_keywords = "<meta name=\"keywords\" content=\"".$CURRENT_PIC_DATA['keywords']."\"/>"; }
Replace with:

Code: [Select]

if ($CURRENT_PIC_DATA['keywords']) { $meta_keywords = "<meta name=\"keywords\" content=\"".preg_replace('/\s+/', ',', $CURRENT_PIC_DATA['keywords'])."\"/>"; }

[Nibbler]

Done for 1.5.

[stunning]

but i think meta tags are no more useful. from SEO point of view content is the main thing.

[Joachim Müller]

Really? Great to hear that. Welcome to the bugs board, where we talk about bugs and not about usefullness of features. Please keep out of the discussion here.

[Joachim Müller]

The resources-consuming preg_replace should not be used if you can use the straightforward str_replace

I have replaced

Code: [Select]

if ($CURRENT_PIC_DATA['keywords']) { $meta_keywords = "<meta name=\"keywords\" content=\"".$CURRENT_PIC_DATA['keywords']."\"/>"; }with

Code: [Select]

    if ($CURRENT_PIC_DATA['keywords']) {
    $meta_keywords = "<meta name=\"keywords\" content=\"".rtrim(str_replace(' ', ',', $CURRENT_PIC_DATA['keywords']),',')."\"/>";
    }in the SVN repository for cpg1.4.x. Marking thread as "fixed" and moving accordingly.