Ten en cuenta que cuando mandas un form
cada dato de los input
se asocia en un array cuya clave será lo que tengas en la etiqueta name
de cada input
.
Por tanto, según tu form
actual, esto nunca se cumplirá:
$eliminar = $_POST['eliminar'];
if ($eliminar == TRUE) {
Dado que en $_POST['eliminar']
no hay nada, ningún elemento tiene asociado un name="eliminar"
en tu form
, el botón que hace el submit tiene un name="buscar"
y un valor también igual a buscar
.
Entonces, para que se cumpla debes ponerlo así en PHP:
$eliminar = $_POST['buscar'];
if ($eliminar == TRUE) {
Aunque no es muy lógico esto, otra opción es cambiar el botón en el form
:
<input type="submit" value="eliminar" name="eliminar" />
Si haces esto, el código funcionará como lo tienes inicialmente:
$eliminar = $_POST['eliminar'];
if ($eliminar == TRUE) {
Aunque es un código incómodo, porque comparas una cadena contra un booleano (TRUE
), funciona en PHP, no sabemos hasta cuando. En otro lenguaje estrictamente tipado (y PHP va hacia ello), esa comparación resultará en un error, porque $eliminar
es una cadena y la estás comparando contra un valor booleano que es TRUE
. Aunque PHP sea permisivo, aplica un estilo de programación estricto desde le principio, comparando cadenas contra cadenas, y booleanos contra booleanos (ver mejor sugerencia).
Mejor sugerencia
Pero lo más lógico de todo es que des un nombre genérico al botón (vamos a llamarlo action
) y que verifiques su valor, para decidir según él, que acción debe ejecutarse, si eliminar
, insertar
, actualizar
, etc.
Algo así:
En el formulario
<input type="submit" value="eliminar" name="action" />
En PHP:
#Empezaremos a usar fusión de null
$action = $_POST['action'] ?? NULL;
if ($action === 'eliminar') {
Así tu código es más lógico en todos los sentidos.
OJO: En cualquier caso, tu form
debe estar correctamente cerrado, o no funcionará nunca:
<form>
Aquí todos los inputs y botones del form
</form>
Post-Data
Hay otras técnicas que deberías implementar:
- Usar ternarios para recuperar los datos de
$_POST
- Usar consultas preparadas para dar seguridad a tu código
Simplemente lo señalo, sin ahondar en eso, porque se aleja del propósito de tu pregunta y porque tu código tiene aún más deficiencias que deberías corregir, como te señalé en un comentario (convendría que uses Ajax, y que coloques correctamente el body
en tu HTML).
form
, si hay un elemento conname="eliminar"
. Si la consulta SQL se cumple con ese criterio. ¿Qué muestra unvar_dump($_POST);
en tu contexto?if ($eliminar == TRUE) {
porque en tuform
no hay ningún elemento conname="eliminar"
que tenga un valor. Debes replantear la lógica.form
no se enviará con él. De hecho, tu código sigue siendo incompleto, no se ve elform
completo, no se ve por qué medio se envía, si por un botón, ni qué valores tiene dicho botón. Pulsa en editar y pon el contexto completo, con un código por partes es complicado ayudar. Como te sugerí más arriba pon unvar_dump($_POST);
y dinos qué muestra. Así podrás ver todo lo que se está enviando.