Hallo Leute,

ich benutze das folgende Skript als Eingabeform in meinem Forum nur leider geht er nach dem Speichern in der DB nicht auf die Bestätigungsseite.

Was hab ich jetzt schon wieder falsch gemacht?

PHP-Code:

<? include '../../logged_in.php'; ?>
<?php require_once('../../Connections/db.php'); ?><?php
function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") 
{
  $theValue = (!get_magic_quotes_gpc()) ? addslashes($theValue) : $theValue;

  switch ($theType) {
    case "text":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;    
    case "long":
    case "int":
      $theValue = ($theValue != "") ? intval($theValue) : "NULL";
      break;
    case "double":
      $theValue = ($theValue != "") ? "'" . doubleval($theValue) . "'" : "NULL";
      break;
    case "date":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;
    case "defined":
      $theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue;
      break;
  }
  return $theValue;
}

$editFormAction = $_SERVER['PHP_SELF'];
if (isset($_SERVER['QUERY_STRING'])) {
  $editFormAction .= "?" . htmlentities($_SERVER['QUERY_STRING']);
}

if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form1")) {
  $insertSQL = sprintf("INSERT INTO for_beitrag (for_thema_ID, beitrag_bezeichnung, beitrag_text, beitrag_user_ID, beitrag_user_IP, beitrag_time) VALUES (%s, %s, %s, %s, %s, %s)",
                       GetSQLValueString($_POST['for_thema_ID'], "int"),
                       GetSQLValueString($_POST['beitrag_bezeichnung'], "text"),
                       GetSQLValueString($_POST['beitrag_text'], "text"),
                       GetSQLValueString($_POST['beitrag_user_ID'], "int"),
                       GetSQLValueString($_POST['beitrag_user_IP'], "text"),
                       GetSQLValueString($_POST['beitrag_time'], "text"));

  mysql_select_db($database_db, $db);
  $Result1 = mysql_query($insertSQL, $db) or die(mysql_error());

  $insertGoTo = "for_gespeichert.php";
  if (isset($_SERVER['QUERY_STRING'])) {
    $insertGoTo .= (strpos($insertGoTo, '?')) ? "&" : "?";
    $insertGoTo .= $_SERVER['QUERY_STRING'];
  }
}
?>

Ich finde in deinem Code nichts, was irgendwie auf eine Bestätigungsseite/-stelle weiterleitet!?

greetz
daddz

PHP-Code:

 $insertGoTo = "for_gespeichert.php";
  if (isset($_SERVER['QUERY_STRING'])) {
    $insertGoTo .= (strpos($insertGoTo, '?')) ? "&" : "?";
    $insertGoTo .= $_SERVER['QUERY_STRING']; 


Dieser Teil soll das bewirken, nur leider hab ichs nicht selbst geschrieben und weiß nicht ob das stimmt.

Der Teil bewirkt nur die Veränderung der Variable $insertGoTo aber mehr auch nicht!

greetz
daddz

Und wenn ich mal ganz blöd fragen darf: wir mach ich es das er weiterleitet?

PHP-Code:

header("Location: $insertGoTo"); 


greetz
daddz

Den Teil hatte ich schon drin.Dann kommt leider der Fehler das der Header nicht geändert werden kann, da er beriets übertragen wurde.

Glaub nicht dass das unbedingt schön gelöst ist, aber ich setz mir ans Ende einfach ein kleines JavaScript. Das wird zumindest garantiert am Ende ausgeführt und macht das was ich will. Aber keine Ahnung ob das auch das macht was du willst.

PHP-Code:

echo "<script language=\"javascript\" type=\"text/javascript\">";
echo "document.location.href('bla.php')";
echo "</script>"; 


Wenn ich das einbauen würde, würde er doch bei jedem Aufruf der Seite auf eine andere umleiten. Das soll er aber nur wenn etwas in die DB geschrieben wurde.

Beispiel:

PHP-Code:

if ($_POST["insert"])
 {
  mysql_query("insert ...");
  header("Location:...");
 } 


Raff ich nicht so ganz, ist das nicht das selbe wie der Teil den ich ganz oben als meinen Code eingegeben habe? Der prüft doch auch ob etwas gesendet wurde und soll wenn ja auf eine ander Seite wechseln. Und da kommt eben diese Fehler das der Header schon gesendet wurde.

Zum Thema, dass Header schon gesendet wurden gibt's schon zig Millionen Threads.
Schau Dir mal die Infos zu diesen beiden Befehlen an:
ob_start()
ob_end_flush()
Wenn Du ohne die beiden, kleinen Funktionen auskommen willst, was meiner Meinung nach aber nicht zwingend noetig ist, dann darfst Du halt vor header() keine Ausgabe haben.

Zitat von ArFiBa

Wenn ich das einbauen würde, würde er doch bei jedem Aufruf der Seite auf eine andere umleiten. Das soll er aber nur wenn etwas in die DB geschrieben wurde.

Wenn das auf mein Vorschlag bezogen war, du kannst das Script ja unmittelbar nachdem etwas in die DB geschrieben wurde reinsetzen und evtl. danach mit exit() aussteigen.