Du hast da einen Konstruktionsfehler. Die Funktionen lassen sich zu einer zusammenfassen, bei denen Du 2 Werte übergibst: Das Link-Ziel und den Link-Text.
<?php
function getNavLink( $LinkTarget, $Linktext ) {
$current = '';
if ( isset( $_GET['subpage'] ) ) {
if ( $LinkTarget == $_GET['subpage'] ) {
$current = 'class="current" ';
}
} else {
if ( $LinkTarget == 'game' ) {
$current = 'class="current" ';
}
}
return '<a '
. $current
. 'href="'
. $LinkTarget
. '">'
. $Linktext
. '</a>';
}
?>
Im Template-Teil dann:
<?=getNavLink('game', 'Game' ); ?>
<?=getNavLink('releases', 'Releases' ); ?>
<?=getNavLink('merchandise', 'Me<span style="letter-spacing: .115em;">r</span>cha<span style="letter-spacing: .105em;">n</span>dise' ); ?>
Das ist doch viel einfacher - oder?
Ich habe hier mal die wichtigen Anmerkungen von Gunnar außer Acht gelassen… Aber von der wilden Durchmischung von HTML, CSS und PHP musst Du dringend weg, weil Du dabei das jeweilige Ziel aus den Augen verlierst und es Dir selbst schwer machst.