german / french ISSUE

[Lister14]

hi people,

this is a strange case... up until a few days ago everything was fine, www.trainer.lu was available in 2 languages: german and french...

now all of a sudden, without me changing anything at all, not even LOOKING at the code (lol) the page doesn't work in german no more...

any help please, please, pretty please...

here part of my code:

"index.php"

<?php
$thisPage="home";

$id = $HTTP_GET_VARS['id'];

switch ($id) {

    case "fr":
        break;
    case "de":
        break;
}

if ($id == "de") {
 include("inc/header.inc.php");
 include("inc/de/$thisPage.inc.php");
 include("inc/footer.inc.php");
}

else {
 include("inc/header.inc.php");
 include("inc/fr/$thisPage.inc.php");
 include("inc/footer.inc.php");
}
?>

part of the navigation.inc file

 <h3>Personal Trainer&nbsp;&raquo;</h3>
<ul>
 <li><a <?php if ($thisPage=="coach") { echo " class=\"currentpage\""; }?> href="coach.php<?php
   if ($id == "de") { echo "?id=de";}
    else if ($id == "en") { echo "?id=en";}
     else { echo "?id=fr";} ?>" title="Coach">Coach</a></li>

[AnthonySterling]

I would guess your PHP version has been updated.

Change...
$id = $HTTP_GET_VARS['id'];

To...
$id = $_GET['id'];

[Mark Baker]

Is $HTTP_GET_VARS still supported in PHP. I thought it had been completely dropped now, replaced with the $_GET superglobal.
It may be that PHP has been upgraded on your server

You should probably also have a default for your language switch statement. In fact, why do you have the switch statement seeing as it does nothing.

[Lister14]

thanks for the $_GET ... haven't kept up to date with the php thing... lol, but seems much easier like that

the switch statement does nothing?, so the if/else statement is enough in index.php?
... what about the rest of the code, i might have a use for it later on

edited: i am sure there must be a way to be even more efficient, not having to create multiple files i.e. index.php, contact.php etc. ... there must be an if / else for that too... but then i start getting overly confused, and therefore keep some things a part...

[Lister14]

$thisPage="page_name";

$id = $_GET['id'];

 include("inc/header.inc.php");

if ($id == "de") {
 include("inc/de/$thisPage.inc.php");
}
 else if ($id == "gb"){
  include("inc/gb/$thisPage.inc.php");
 }
  else {
   include("inc/fr/$thisPage.inc.php");
  }

 include("inc/footer.inc.php"); 

can this be replaced with

$thisPage="page_name";

$id = $_GET['id'];

 include("inc/header.inc.php");

switch ($id) {
    case "fr":
         include("inc/$id/$thisPage.inc.php");
        break;
    case "de":
         include("inc/$id/$thisPage.inc.php");
        break;
    case "gb":
         include("inc/$id/$thisPage.inc.php");
        break;
}

 include("inc/footer.inc.php"); 

? ... or is there a nicer/better/shorter way do do this?
... and what would be the default index.php?