Sæt værdier baseret på link

hvis det bare er samme side, så
<li><a href="?valg=hvem">Hvem er vi</a></li>
og så tester du på valg - er den ikke kendt, så tag startsiden.

Som Claes er inde på mener jeg også at GET parametre er vejen frem for dig. Det gør det ligeledes muligt for dine brugere at linke direkte til enkelte sider via unikt link.

Et lille kode eksempel fra et af mine egne projekter:

$menu = new Menu();

if(isset($_GET['page'])) {
  echo $menu->publicMenu();
  $page = new Page($_GET['page']);
  echo $page->showPage();
} else {
    echo Page::startPage();
}

Er ikke helt med på det sidste du skriver - gider du lave et eksempel?

ovenstående var til claes

I den form jeg bruger til at rette i teksten bruger jeg $_SERVER['PHP_SELF'] som action, men det betyder at ?valg=... forsvinder ved gem - hvordan får jeg det til at blive?

Har fundet ud af blot at lave .?valg=... på den og det virker fint :o)

Du bliver nødt til at vise os den kode der ikke virker, og vise os konkret hvad der er fejlen.

Den her kode fungerer efter hensigten:
<?
if (isset($_GET['site'])) {
    $menu = "menu".$_GET['site'];   
}

if(isset($_POST['submit']))
{
    $text = $_POST['text'];
       
    $sql = "UPDATE menubasen SET text = '$text' WHERE name='$menu'";
    mysql_query($sql)or die("MySQL fejl: " . mysql_error());
}
?>
<?
error_reporting (E_ALL ^ E_NOTICE);
$sql = "SELECT text FROM menubasen WHERE name='$menu'";
$result = mysql_query($sql)or die("MySQL fejl: " . mysql_error());
$array = mysql_fetch_row($result);
$text = $array[0];
?>
<? include("header1.php"); ?>
<tr>
<td class="left"><img src="../images/waterpump.jpg" /></td>
<td class="indholdscelle" colspan="5">
<p><i style="color:red;">BEMÆRK DU ER I MENU REDIGERINGS MODE - FOR AT KOMME TILBAGE <a href="index.php">KLIK HER</a></i></p>
<form method="post" action="<? echo htmlentities($_SERVER['PHP_SELF']).'?site='.$_GET['site']; ?>" style="width:100%">
<? echo $text; ?>: <input type="text" name="text" />
<input name="submit" type="submit" value="Gem"/>
</form>
</td>
</tr>

Vil gerne give point, men jeg skal lige bruge et svar fra dig Claes

det, jeg lavede var så lidt, så du må selv lukke...

ok jamen det gør jeg så, med mindre danco vil have dem?

"UPDATE menubasen SET text = '$text' WHERE name='$menu'"

Det der er skidefarligt. Du tager noget brugerinput og sender strengen direkte mod din database, der kan stå alverdens ting i det. Du burde læse lidt om SQL injections. Sørg for at du escaper al data du arbejder med som brugeren har haft mellem fingrene.

For at undgå SQL injections her skal du omskrive

 $menu = "menu".$_GET['site'];

til

 $menu = "menu". mysql_real_escape_string($_GET['site']);

Jeg er enig i at man altid bør escape de input man får fra brugere, dog antog jeg, måske forkert, at den fremviste kode herinde udelukkende var beskrivende og ikke udtryk for den konkrete kode.

Men jeg smider et svar.

<ole>

wanze >> - og det der er skidefarligt minus 0,0000001!

Før PHP kom med DB-API'er med parametriserede kald, var den slags 'hacks' rigtig nok eneste løsning. I dag, hvor vi har både PDO og mysqli til rådighed, bør man bruge de sikre metoder i stedet  *o)

/mvh
</bole>

olebole:
Det er rigtigt, men jeg tænker bare jeg har mere held med at få ham til at bruge mysql_real_escape_string(), fremfor at få ham til at sætte sig ind i det nye API.

-- probably right  *o)