Warum funktioniert das Script nicht ? ? ?

123kartoffelbrei

hallööchen,
also ich habe ein script gefunden:

PHP

<?php


$file = "pics.php";


echo "<form action=\"$PHP_SELF\" method=\"POST\">";
echo "<textarea name=\"text\" cols=\"50\" rows=\"5\">";
readfile($file);
echo "</textarea>
";
echo "<input type=\"submit\" name=\"gesendet\" value=\"speichern\">";
echo "</form>
";


if ($gesendet)
{
		$datei_speichern = fopen($file, "w");
		fwrite ($datei_speichern, $_POST['text']);
		fclose($datei_speichern);
		echo "Änderung erfolgreich";
}


?>

Alles anzeigen

bloß irgendwie speichert es nicht!?
Was habe ich das falsch gemacht??

phpfan

Das Script ist einfach Schrott bzw. veraltet. Stichwort register_globals.

123kartoffelbrei

kennt dann jemand da einen aktuellen script?

yngvar

PHP

<?php


$file = "pics.php";


echo "<form action=\"?seite=gesendet\" method=\"POST\">";
echo "<textarea name=\"text\" cols=\"50\" rows=\"5\">";
readfile($file);
echo "</textarea>
";
echo "<input type=\"submit\" name=\"gesendet\" value=\"speichern\">";
echo "</form>
";


if ($_GET["seite"] == "gesendet")
{
      $datei_speichern = fopen($file, "w");
      fwrite ($datei_speichern, $_POST['text']);
      fclose($datei_speichern);
      echo "Änderung erfolgreich";
}


?>

Alles anzeigen

Is nur schnell gemacht bin mir nicht ganz sicher ob es funktioniert

phpfan

Zitat

bin mir nicht ganz sicher ob es funktioniert

Da wäre ich mir auch nicht so sicher. Da sind nämlich Fehler drin.

1. Wozu ?seite=gesendet
2. Wozu \", ' ist doch einfacher
3. if ($_GET['seite'] == "gesendet") sollte wohl besser if (isset($_POST['seite'])) heißen

No0ob

Zitat von phpfan

3. if ($_GET['seite'] == "gesendet") sollte wohl besser if (isset($_POST['seite'])) heißen

Du meinst glaub ich if (isset($_POST['gesendet'])) =)

GreenRover

nee läute was was im action ist währe in dem falle schon get...

PHP

<?php


$file = 'pics.php';


echo '<form action="'.$_SERVER['PHP_SELF'].'" method="POST">';
echo '<textarea name="text" cols="50" rows="5">';
readfile($file);
echo '</textarea>
';
echo '<input type="submit" name="gesendet" value="speichern">';
echo '</form>
';


if (!empty($_POST['gesendet']))
{
      $datei_speichern = fopen($file, 'w+');
      fwrite ($datei_speichern, $_POST['text']);
      fclose($datei_speichern);
      echo 'Änderung erfolgreich';
}


?>

Alles anzeigen

123kartoffelbrei

soo ich hab mal rumgebastelt, leider geht mein xampp grad ned also hier ist der quellcode:

PHP

<?php
$datei = 'pics.php';


if (isset($_POST['speichern']))
{
		$fp = fopen($datei, 'w');


		$news_neu = str_replace("\n", "", $_POST['news_neu']);
		$news_neu = stripslashes($news_neu);
		fputs($fp, $news_neu, strlen($news_neu));
		fclose($fp);
}


$news = file($datei);


echo '<form action="' . $_SERVER['PHP_SELF'] . '" method="post">';
echo '<textarea cols="60" rows="30" name="news_neu">';
echo implode('', $news);
echo '</textarea>';
echo '
';
echo '<input type="submit" name="speichern" value="speichern">';
echo '</form>';


?>

Alles anzeigen

würde dies funktionieren???

phpfan

Zitat

würde dies funktionieren???

Vielleicht. Sind wir hier deine kostenlosen Scripttester?

$_SERVER['PHP_SELF'] solltest du nicht verwenden, füttere google mit $_SERVER['PHP_SELF'] und xss

Warum funktioniert das Script nicht ? ? ?

Tags

Benutzer online in diesem Thema