[SOLUCIONADO] El siguiente script de php debería ingresar datos y no lo hace

[AngelMarine]

Este código debería tomar los datos de un formulario e ingresarlos en una base de datos, no lo hace, ¿Sabe alguien por qué?, supongo que porque está mal, jejeje, bueno me gustaría saber dónde está el fallo.

<?php
session_start();
error_reporting (E_ALL ^ E_NOTICE);
if (mysqli_connect_errno()) {
printf("Connect failed: %s\n", mysqli_connect_error());
exit();
}
if($_SESSION['logged'] == 'yes')
{
$usuario_comentario = $_POST['usuario'];
$comentario = $_POST['comentario'];
$reqlen = strlen ($usuario_comentario) * strlen($comentario);

if ($reqlen > 0) {
require("conectar_db.php");
$meter = @mysql_query('INSERT INTO comments (usuario_comentario, comentario) values ("'.mysql_real_escape_string($usuario_comentario). '", "'.mysql_real_escape_string($comentario).'")') ;
}
header("location: perfil.php");
exit();
}
?>

[Alexis88]

Quita el @ que colocas antes de mysql_query para que veas el error. Y decídete a usar mysqli o mysql, no ambas al mismo tiempo.

Saludos

[AngelMarine]

Solucionado..... el asunto estaba en que puse en la línea 10: '$usuario_comentario = $_POST['usuario'];' y debería ser: '$usuario_comentario = $_SESSION['usuario'];'.

Esto debido a que 'usuario' no está en el formulario sino que ya está definido previamente en la sesión de usuario.

[AngelMarine]

Iniciado por Alexis88

Quita el @ que colocas antes de mysql_query para que veas el error. Y decídete a usar mysqli o mysql, no ambas al mismo tiempo.

Saludos

Si, es una manía que tengo, bueno 'manías' en plural, ya me han dicho varias veces que quite el '@' y definitivamente voy a hacerlo con todas las páginas de la web (aunque en el fondo me de algo de miedo por lo que va salir a la superficie) y lo otro, el mysqli, hace tiempo empecé a 'modernizar' la web, y he tenido que actualizar bastantes códigos de php, este se me había pasado. Gracias por tu ayuda. Saludos.