Tienes un código que mezcla la API mysqli_
con la API mysql_
, esta última declarada obsoleta.
La lógica también es confusa y poco óptima (como podrás verificar en comentarios del código).
Te aporto esta respuesta para ayudar a corregir todas esas anomalías. Considera prestar atención a los comentarios e implementar un estilo de programación controlado y claro, de modo que te sea más fácil seguir las eventualidades, depurar errores, etc.
He puesto varios comentarios para que sea más fácil de entender. Si tienes dudas lo puedes decir en comentarios.
<?php
/*
Usamos un orden lógico:
sin esta condición no haría falta la conexión
por eso incluimos todo lo propio si se cumple este if
*/
if(isset($_POST['enviar'])) {
include ("configuracion.php");
$conexion = new mysqli ($server,$user,$pass,$bd);
/*
Aunque el Manual de PHP lo pone como ejemplo
no conviene mostrar mensajes internos de la base de datos
es mejor verificar la conexión y mostrar un mensaje personalizado
Con if (!$conexion) preguntamos "si no se pudo conectar"
*/
if (!$conexion) {
/*
Usaremos una variable que recoja lo que haya ocurrido
y al final imprimimos esa variable
*/
$msg="Conexión imposible. Revise las credenciales de conexión";
} else {
/*
Vamos a optar por un poco de claridad:
mediante operadores ternarios verificaremos los datos del POST
y al mismo tiempo los guardaremos en variables
luego usaremos esas variables
*/
$usuario=!empty($_POST['usuario']) ? $_POST['usuario'] : NULL;
$password=!empty($_POST['password']) ? $_POST['password'] : NULL;
$repassword=!empty($_POST['repassword']) ? $_POST['repassword'] : NULL;
/*
Intentas insertar una variable $nombre
que no parece estar declarada
imagino que está en el POST, por eso la defino aquí
Considera si esta variable es también condición o no para el if
que verifica si los datos fueron llenados
*/
$nombre=!empty($_POST['nombre']) ? $_POST['nombre'] : NULL;
if($usuario && $password && $repassword && $nombre) {
$sql = 'SELECT * FROM usuarios';
/*
Usaremos el estilo orientado a objetos de mysqli
que es el que usaste para crear la conexión
Además es un estilo más claro que el procedural
Puedes revisar las diferencias en el Manual de PHP
*/
$rec = $conexion->query($sql);
/*
Es mejor usar un booleano directamente
*/
$verificar_usuario = FALSE;
/*
No es correcta la lógica que estás siguiendo,
porque obtienes todos los usuarios para luego ver si
alguno de ellos es igual a $_POST['usuario']
sería mejor hacer una consulta del tipo
SELECT * FROM usuarios WHERE usuario = $_POST['usuario']
Dado que esto implicaría aplicar consultas preparadas y demás
te lo pongo en este comentario para que consideres modificar tu lógica
Uno de los problemas aquí es que si hay un millón de usuarios
tu consulta traería ese millón de usuarios a memoria
para luego buscar entre ese millón si está el usuario buscado
cuando lo puedes traer sólo a él con un WHERE ....
*/
while($result = $rec->fetch_object()) {
if($result->usuario == $usuario) {
$verificar_usuario = TRUE;
/*
En caso de seguir con esta lógica, lo cual es ilógico
habría que salir del bucle con break cuando la condición se cumpla
sino seguirá leyendo las otras filas sin necesidad
*/
break;
}
}
/*
Esto también es un poco raro ...
las claves en PHP deberían verificarse de otro modo,
pero ese es otro capítulo
*/
if($verificar_usuario) {
if($password == $repassword) {
/*
NOTA SOBRE LA SEGURIDAD: Esta consulta
es vulnerable a ataques de Inyección SQL
considera informarte sobre el tema y usar consultas preparadas
*/
$sql = "INSERT INTO usuarios (nombre,correo,usuario,password) VALUES
('$nombre','$correo','$usuario','$password')";
/*
Verificamos si la consulta es exitosa
y si el número de filas afectadas es mayor que 0
condición que verifica una inserción real
*/
if ($conexion->query($sql) && $conexion->affected_rows > 0) {
$msg="Usted se ha registrado correctamente.";
} else {
$msg="Error en la inserción";
}
} else {
$msg="Las claves no son iguales, intente nuevamente.";
}
} else {
/*
No se entiende mucho esta parte de la lógica ¿?
*/
$msg="Este usuario ya ha sido registrado anteriormente.";
}
} else {
$msg="Por favor llene todos los campos. Faltan datos en el POST";
}
}
} else {
$msg="No hay POST enviar";
}
/*
Al final imprimimos $msg
*/
echo $msg;