meta tag 'title' in contactus.php

[frommarcq]

The problem :
in contactus.php (and i think in all the pages.php), we just have the name of the page in the meta tag (the name of the website is missing).
I've made some little changes, and i want you to validate them for me :

In contactus.php, instead of :

PHP Code:

if($rs['pPublish']=="Y")
        {
            $class_tpl->assign('pPageContent', safeStripSlashes($rs['pPageContent']));
        }
        else
        {
            $class_tpl->assign('pPageContent', '');
        }
        $class_tpl->assign('body','contact/contactus.tpl.php');
    } 


i've replaced the code with :

PHP Code:

if($rs['pPublish']=="Y")
            {
                $class_tpl->assign('title', $title .' - ' .safeStripSlashes($rs['pPageTitle']));
                $class_tpl->assign('pPageContent', safeStripSlashes($rs['pPageContent']));
                $class_tpl->assign('pPageTitle', safeStripSlashes($rs['pPageTitle']));
            }
            else
            {
                $class_tpl->assign('title', '');
                $class_tpl->assign('pPageContent', '');
                $class_tpl->assign('pPageTitle', '');
            }
            $class_tpl->assign('body','contact/contactus.tpl.php');
        } 


Is the syntax correct ? do I have used the best way to show AND the name of the website AND the name of the page ?

That is the question of the day ! lol

Pascal

NB : to be complete, and avoid to have 'thenameofthesite - the nameofthepage' in the table title, I've replaced '{$title}' with '{$pPageTitle}' in 'contactus.tpl.php'.