Las líneas var_dump() son para poder ver lo que tienes y sí, son las causantes del error en el header(), una vez que resuelvas el problema del insert, las eliminas y listo!
Veamos tu consulta:
Código PHP:
Ver original$insertSQL = sprintf("INSERT INTO tblcomentarios (nombre, correo, telefono, mensaje, fecha, hora, idnoticia) VALUES (%s, %s, %s, %s, NOW(), CURRENT_TIME(), '". $id_not ."')", GetSQLValueString($_POST['nombre'], "text"), // Primer %s
GetSQLValueString($_POST['correo'], "text"), // segundo %s
GetSQLValueString($_POST['telefono'], "text"), // tercer %s
GetSQLValueString($_POST['mensaje'], "text"), // cuarto %s
GetSQLValueString($_POST['fecha'], "date"), // No se necesita
GetSQLValueString($_POST['hora'], "date"), // Tampoco se necesita
GetSQLValueString($_POST['idnoticia'], "int")); // Deberías tener otro %s
// Deberías tener:
$insertSQL = sprintf("INSERT INTO tblcomentarios (nombre, correo, telefono, mensaje, fecha, hora, idnoticia) VALUES (%s, %s, %s, %s, NOW(), CURRENT_TIME(), %s)", GetSQLValueString($_POST['nombre'], "text"),
GetSQLValueString($_POST['correo'], "text"),
GetSQLValueString($_POST['telefono'], "text"),
GetSQLValueString($_POST['mensaje'], "text"),
GetSQLValueString($_POST['idnoticia'], "int")); // Quinto %s
Como ves, el error original es que estabas poniendo
$id_not (variable no definida) directamente en la consulta y no era reemplazada con el contenido de $_POST['idnoticia'].
Deberías leer un poco acerca de la función
sprintf() y no usarla a ciegas.