Estás cometiendo un gravísimo error aquí:
$sucursal=$_POST['sucursal'];
Y también aquí:
var url='../Modelos/productos.modelo.php';
Haciendo eso, estás capando tu modelo de datos, haciéndolo depender de peticiones vía Ajax.
Las clases del modelo de datos son para construir objetos de ese tipo, sin importar de dónde vienen los datos (de Ajax, de un archivo, de otro servidor, de una API, ¡incluso de esa misma consulta SQL, pues con PDO puedes mapear los resultados de la consulta a una clase de tu modelo de datos!, etc).
Si tú metes el $_POST
dentro de la clase, ya no podrás usarla en ninguna otra parte.
Entonces, tú puedes tener un archivo (que llamaremos ajax-request.php
) que hará de archivo intermediario entre cliente y servidor. Él será el que reciba los datos que pasas desde Ajax y llamará a quien tenga que llamar (en este caso la clase del modelo), le pasará los datos, recibirá la respuesta y la devolverá a Ajax.
En ese archivo (ajax-request.php
) o como quieras llamarle, puedes además gestionar varias peticiones Ajax, identificadas por claves... pero ese es otro asunto. Lo importante es que pienses a ese archivo como aquel que te conecta con el ámbito del servidor.
Los cambios, si aplicas esta forma serían:
1. En el cliente (Ajax)
Vas a dirigir la petición al nuevo archivo, que deberás crear por supuesto. Aquí lo puse en una carpeta Ajax
, tú ponlo donde mejor te convenga.
var sucursal = $('#valorSucursal').val();
var url='../Ajax/ajax-request.php';
console.log(sucursal);
$.ajax({
method: "POST",
dataType: "json",
url:url,
data:{'sucursal':sucursal},
success: function() {
alert("Ha sido ejecutada la acción.");
}
});
2. En el servidor (ajax-request.php
)
/*
Verificamos que el POST tiene los datos
*/
$sucursal=$_POST["sucursal"] ?? null;
if ($sucursal) {
/*
Aquí incluimos la clase, pero
dado que el método es estático, no la instanciamos
*/
include '../Modelos/productos.modelo.php';
$table="AQUÍ-PON-EL-NOMBRE-DE-LA-TABLA";
$mData=ModeloProductos::mdlMostrarProductosBySucursal($table,$sucursal);
} else {
$mData=array('error'=>'No enviaste ninguna sucursal');
}
header("Content-type: application/json; charset=utf-8");
echo json_encode($mData);
3. En el servidor (tu clase, no capada, ni dependiente del contexto)
Voy a crear un método más apropiado. Y voy a implementar consultas preparadas.
require "Conexion.php";
class ModeloProductos{
/*=============================================
MOSTRAR PRODUCTOS
=============================================*/
static public function mdlMostrarProductosBySucursal($tabla, $sucursal){
$stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla WHERE sucursal = :sucursal");
$stmt->bindParam(':sucursal', $sucursal, PDO::PARAM_STR);
$stmt -> execute();
return $stmt -> fetchAll();
}
}
De este modo el código debería funcionar, y no habrás contaminado tu modelo de datos con elementos circunstanciales, como podrían ser datos que vienen de un $_POST
.
Y, como dije antes, ajax-request.php
podría emerger como un gestor de peticiones Ajax, haciendo siempre de intermediario. Imaginemos un CRUD por ejemplo, o una aplicación más amplia... poniendo claves adicionales en la petición, puedes decidir desde ajax-request.php
qué otro archivo de PHP llamar, qué datos pasar, etc.
Si lo que debe hacer ajax-request.php
tú lo esparces por todos lados, no solamente te perderás en el código, sino que estarás estropeando tu modelo de datos, teniendo clases que no servirán para fabricar objetos de ese tipo en contextos distintos.
Json.stringify({'sucursal':sucursal});
#valorSucursal
, no un objeto entero... eso aparentemente está bien... diria que el problema está en que no ha puesto todo el código relacionado, pues su error habla dedatatables
pero yo no veo nada dedatatables
en esta pregunta en si.type: "POST"
pero en la documentacion de jquery de ajax ponemethod: "POST",
y no type$_POST
dentro de tu clase la estás capando ...$_POST
, pero al tener a$_POST
dentro de la clase ya la limitas a un solo contexto. Las clases deben ser pensadas como moldes para construir objetos de ese tipo, y la información que se use para construir instancias de esos objetos puede venir de muchas fuentes, no solamente de un$_POST
. Si no comprendes el funcionamiento básico de Ajax, que traté de explicarte en aquel comentario, irás dando tumbos con tu código, y además estropeando tu modelo de datos, lo cual es una pena.