[SOLUCIONADO] Evitar eliminar registros con php

[detective_jd]

Hola a todos, acá estoy con un dilema verán tengo 2 tablas:

roles(rolId, rolNombre)

usuarios(usuId, usuNick, usuPass, usuMail, usuNombre, usuApellido, usuStatus, rolId)
roles(rolId) => rolId

la cuestión es que los usuarios usan su rol, hasta ahí bien, pero cuando elimino el rol también se borrarían los usuarios de ese rol, quería preguntarles que es mejor usar un trigger en mysql o una función que determine la cantidad de usuarios con ese rol??

este es el código para eliminar:

public function eliminame($rol){
$sql="delete from roles where rolId=?";
$consulta = $this->getBD()->prepare($sql);
$consulta->execute(array($rol->getId()));
return ($consulta->rowCount() > 0) ? $rol->getId() : null;
}

Quiero evitar de que los roles que estan en uso no sean borrados, los que no sí.

Espero sus respuestas y saludos

[x_atrix]

Puedes usar

Código:

NOT IN

en tu query.
Quedaría de esta forma (parametrizalo para poder reutilizar el método como antes).

Código PHP:

  public function eliminame($rol, $notUsed = true) { 
    $sql = "DELETE FROM roles WHERE rolId = ?"; 
 
    if ($notUsed === true) { 
        $sql .= ' AND NOT IN (SELECT DISTINCT rolId FROM usuarios)'; 
    } 
 
    $consulta = $this->getBD()->prepare($sql); 
    $consulta->execute(array($rol->getId())); 
 
    return ($consulta->rowCount() > 0) ? $rol->getId() : null; 
}  
   

[detective_jd]

hola x_atrix probé el código del cual me lo diste pero sale este error:

DELETE FROM roles WHERE rolId = ? AND NOT IN (SELECT DISTINCT rolId FROM usuarios) // esto es un echo

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'IN (SELECT DISTINCT rolId FROM usuarios)' at line 1

¿porque será? saludos y espero respuestas.

[x_atrix]

Le falta el rolId despues del AND

Código PHP:

  public function eliminame($rol, $notUsed = true) {  
    $sql = "DELETE FROM roles WHERE rolId = ?";  
 
    if ($notUsed === true) {  
        $sql .= ' AND rolId NOT IN (SELECT DISTINCT rolId FROM usuarios)';  
    }  
 
    $consulta = $this->getBD()->prepare($sql);  
    $consulta->execute(array($rol->getId()));  
 
    return ($consulta->rowCount() > 0) ? $rol->getId() : null;  
}  
   

Un saludo,

[detective_jd]

Hola x_atrix, lo probé y funcionó a la perfeccción, me validó lo más bien los mensajes q tenía en mi controlador:

public function delete(){
if($this->checkUser()){
if (isset($_GET['p'])){
$rol = $this->modelo->obtenerPorId($_GET['p']);
$id= $this->modelo->eliminame($rol);
Session::set("msg", (isset($id)) ? "Rol Borrado" : "No se pudo borrar el rol");
header("Location:index.php?c=roles&a=index");
}
}
}

Mil gracias, ahora adaptaré esto otros delete q tengo, nuevamente Mil gracias!!

[detective_jd]

Lo adapté al código a otros delete y funciona lo más bien, ahora x_atrix quería preguntarte lo siguiente:

roles(rolId, rolNombre)

Tengo el rolNombre para ingresar y en la base de datos esta como clave única pero quería saber cómo hacer para mostrar un mensaje de: "El rol ya existe" tanto para guardar y editar, sólo que en el editar sucede que si no modifico el nombre y lo demás me puede salir ese mensaje.

Te muestro el código:

RolModel:

Código PHP:

public function guardame($rol){
        $sql="insert into roles(rolNombre) values(?)";
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getNombre()));
        return ($consulta->rowCount() > 0) ? $this->getBD()->lastInsertId() : null;
    }
    public function modificame($rol){
        $sql="update roles set rolNombre=? where rolId=?";
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getNombre(),$rol->getId()));
        return ($consulta->rowCount() > 0) ? $rol->getId() : null;
    }
 

RolesController:

Código PHP:

public function add(){
        if($this->checkUser()){
            if (isset($_POST['btnaceptar'])) {
                if($this->checkDates()) {                
                    $rol = new Rol(0, $_POST['txtnom']);
                    $this->modelo->guardame($rol);
                    Session::set("msg","Rol Creado");
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('add.php'));
        }
    }   
    public function edit(){        
        if($this->checkUser()){
            Session::set("id",$_GET['p']);
            if (Session::get('id')!=null && isset($_POST['btnaceptar'])){                             
                if($this->checkDates()) {                
                    $rol= new Rol($_POST['hid'],$_POST['txtnom']);
                    $this->modelo->modificame($rol);
                    Session::set("msg","Rol Editado");
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('edit.php'),array(
                "rol" => $this->modelo->obtenerPorId(Session::get('id'))
            ));
        }        
    }
 

Te lo pregunto xq me gustó mucho cómo me ayudaste con el borrar... Saludos y espero respuestas.

[x_atrix]

Necesitas un metodo de validacion del nombre.

Ejemplo (dentro de tu modelo ya que es responsabilidad de un modelo saber si un nombre existe).

Código PHP:

  public function existeNombre($nombre) { 
    $query = 'SELECT rolId FROM roles WHERE rolNombre = ?'; 
 
    $consulta = $this->getBD()->prepare($query); 
    $consulta->execute([$nombre]); 
 
    // Indicar si hay algo en la base de datos con este nombre 
    return $consulta->rowCount() > 0; 
}  
   

Dentro de los dos metodos del controlador deberias comprobar si este nombre ya existe pero con una diferencia. Para el caso de modificar primero comprueba que no tenga el mismo nombre de antes (basicamente no se intenta actualizar con el mismo nombre).

Para el caso de alta:
Si ya existe guardar en sesion el mensaje de que ya existe.

Para el caso de actualizar:
Si el nombre es el mismo que el anterior mostrar que no se modificó nada, si el nombre es diferente, usar el mismo metodo de nombre existente y si ya existe mostrar error de que el nombre ya existe en caso contrario, todo esta ok...

Como lo ves para implementarlo tu esto?

TIP* Segun lo tienes estructurado necesitaras un metodo que te diga el nombre del rol buscando por ID para saber si se ha cambiado o no...

Para mejorar:
1. Piensa un solo método para guardar sin importar si existe o no el Rol e internamente la clase RolModel debe saber si es un insert o es un update. Esta lógica no le pertenece al controlador...
2. Seria mas legible algo asi:

Código PHP:

  $rol = new Rol(0, $_POST['txtnom']); 
$rol->guardar();  
   

que esto otro:

Código PHP:

  $rol = new Rol(0, $_POST['txtnom']); 
$this->modelo->guardame($rol);  
   

[detective_jd]

Hola x_atrix ¿cómo estás? mira, hice lo que me dijiste y anduvo, te muestro a ver cómo lo ves:

RolesController.php

Código PHP:

public function add(){
        if($this->checkUser()){
            if (isset($_POST['btnaceptar'])) {
                if($this->checkDates()) {                
                    $rol = new Rol(0, $_POST['txtnom']);
                    $id= $this->modelo->guardame($rol);
                    Session::set("msg",(isset($id)) ? "Rol Creado" : Session::get('msg')); 
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('add.php'));
        }
    }   
    public function edit(){        
        if($this->checkUser()){
            Session::set("id",$_GET['p']);
            if (Session::get('id')!=null && isset($_POST['btnaceptar'])){                             
                if($this->checkDates()) {                
                    $rol= new Rol($_POST['hid'],$_POST['txtnom']);
                    $id= $this->modelo->modificame($rol);
                    Session::set("msg",(isset($id)) ? "Rol Editado" : Session::get('msg'));
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('edit.php'),array(
                "rol" => $this->modelo->obtenerPorId(Session::get('id'))
            ));
        }        
    }
 

RolModel.php

Código PHP:

public function guardame($rol){
        if($this->checkRol($rol->getNombre())){
            Session::set('msg', 'El rol ya existe');
            return null;
        }
        $sql="insert into roles(rolNombre) values(?)";
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getNombre()));
        return ($consulta->rowCount() > 0) ? $this->getBD()->lastInsertId() : null;
    }
    public function modificame($rol){
        $aux = $this->obtenerPorId($rol->getId()); 
        if(!$rol->equals($aux)){
            if($this->checkRol($rol->getNombre())){
                Session::set('msg', 'El rol ya existe');
                return null;
            }        
        }
        $sql="update roles set rolNombre=? where rolId=?";
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getNombre(),$rol->getId()));
        return ($consulta->rowCount() > 0) ? $rol->getId() : null;
    }
    private function checkRol($nombre) { 
        $query = 'SELECT rolId FROM roles WHERE rolNombre = ?'; 
        $consulta = $this->getBD()->prepare($query); 
        $consulta->execute([$nombre]); 
        // Indicar si hay algo en la base de datos con este nombre 
        return $consulta->rowCount() > 0; 
    }
 

y lo único que agregue fue el equals a la clase Rol:

Código PHP:

<?php
namespace Clases;
class Rol {
    private $id;
    private $nombre;
    function getId() {
        return $this->id;
    }
    function getNombre() {
        return $this->nombre;
    }    
    function setNombre($nombre) {
        $this->nombre = $nombre;
    }
    function __construct($xid, $xnombre) {
        $this->id = $xid;
        $this->nombre = $xnombre;
    }
    public function equals(Rol $obj){
        return $this->nombre == $obj->nombre;                
    }
}
 

Con respecto a la mejora que me dijiste lo tengo muy presente además eso me serviría muy bien para el guardame, modificame y eliminame, pero te pregunto: ¿cómo haría para el obtenerPorId o los obtenerDatos? lo único que se me ocurre es poner un constructor vacío, a las clases conozcan a sus modelos, pero quiero una opinión tuya con respecto a eso, por otro lado Mil gracias, luego adaptare lo que me dijiste.

PD: lástima que php no tenga para sobreescribir el equals como java

[x_atrix]

Hola,

Según lo que veo en añadir el ID lo pones a 0 y en editar sabes cual es su id real.

Comprueba este ID para saber si ejecutas el guardame o modificame. Ejemplo:

Código PHP:

public function guardar() {
    if ($this->id === 0) {
        return $this->guardame();
    } else {
        return $this->modificame();
    }
}
 

Los métodos guardame y modificame los cambias a protegidos para no poder acceder a ellos directamente desde fuera y estar obligado a usar el guardar().

[detective_jd]

Hola x_atrix, adapté lo que me dijiste y funciona, gracias:

Rol.php

Código PHP:

<?php
namespace Clases;
use \Model\RolModel;
class Rol 
{
    private $id;
    private $nombre;
    private $mod;
    function getId() {
        return $this->id;
    }
    function getNombre() {
        return $this->nombre;
    }    
    function setNombre($nombre) {
        $this->nombre = strtoupper($nombre);
    }
    function __construct($xid, $xnombre) {
        $this->id = $xid;
        $this->nombre = strtoupper($xnombre);
    }
    public function equals(Rol $obj){
        return $this->nombre == $obj->nombre;                
    }
    public function save(){
        $this->mod = new RolModel();
        return ($this->id == 0) ? $this->mod->guardame($this) : $this->mod->modificame($this); 
    }
    public function del(){
        $this->mod = new RolModel();
        return $this->mod->eliminame($this);
    }
}
 

RolesController.php

Código PHP:

<?php
namespace Controller;
use \App\Controller;
use \App\Session;
use \Model\RolModel;
use \Clases\Rol;
class RolesController extends Controller 
{
    private $modelo;
    function __construct() {
        parent::__construct();
        $this->modelo= new RolModel();
    }
    public function index(){
        if($this->checkUser()){
            $this->redirect(array("index.php"),array(
                "roles" => $this->modelo->obtenerTodos()
            )); 
        }   
    }
    public function add(){
        if($this->checkUser()){
            if (isset($_POST['btnaceptar'])) {
                if($this->checkDates()) {                
                    $rol = new Rol(0, $_POST['txtnom']);
                    //$id= $this->modelo->guardame($rol);
                    $id = $rol->save();
                    Session::set("msg",(isset($id)) ? "Rol Creado" : Session::get('msg')); 
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('add.php'));
        }
    }   
    public function edit(){        
        if($this->checkUser()){
            Session::set("id",$_GET['p']);
            if (Session::get('id')!=null && isset($_POST['btnaceptar'])){                             
                if($this->checkDates()) {                
                    $rol= new Rol($_POST['hid'],$_POST['txtnom']);
                    //$id= $this->modelo->modificame($rol);
                    $id = $rol->save();
                    Session::set("msg",(isset($id)) ? "Rol Editado" : Session::get('msg'));
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('edit.php'),array(
                "rol" => $this->modelo->obtenerPorId(Session::get('id'))
            ));
        }        
    }
    public function delete(){
        if($this->checkUser()){
            if (isset($_GET['p'])){
                $rol = $this->modelo->obtenerPorId($_GET['p']);
                //$id= $this->modelo->eliminame($rol);
                $id= $rol->del();
                Session::set("msg", (isset($id)) ? "Rol Borrado" : "No se pudo borrar el rol");
                header("Location:index.php?c=roles&a=index");
            }                           
        }
    }
    private function checkDates(){
        if(empty($_POST['txtnom'])){
            Session::set("msg","Ingrese los datos obligatorios (*) para continuar.");
            return false;
        }
        else{
            return true;
        }
    }
    private function checkUser(){
        if(Session::get("log_in")!= null and Session::get("log_in")->getRol()->getNombre() == "ADMIN"){
            return true;
        }
        else {
            Session::set("msg","Debe ser administrador para acceder.");
            $this->redirect(array('Main','index.php'));
        }
    }
}
 

Pero tuve el problema que en la clase Rol no podía instanciar el modelo en el constructor xq si lo hacía parte del sistema me dejaba de funcionar, ¿porque será?
En el controlador te pregunto ¿cómo hacer algo parecido a lo que me dijiste pero con obtenerTodos y obtenerPorId? porque según lo puesto estaríamos haciendo que la clase persista sus propios datos, probé agregarle a la clase Rol otro constructor vacío para instanciar el modelo pero php no deja hacer más de un controlador ¿que debo hacer?

Saludos y espero respuestas.

[x_atrix]

Cual es la diferencia entre la clase Rol y la clase RolModel? No deberías tener dos. Es la misma clase. Todo el código para tratar con los roles debería ir en RolModel o Rol como lo quieras llamar pero solo una no dos.

En PHP para simular la sobrecarga como en java (método sobrecargado) hay que darle valores por defecto a los parámetros.

Código PHP:

  public function __construct($id = 0; $nombre = '') { 
    // Codigo ...  
}  
   

En este caso te funcionará new Rol() o bien new Rol(22) o bien new Rol(22, 'Admin') - parecido a tener definidos tres constructores. Esto funciona para cualquier tipo de método no solo para __construct.

Que código tienes en la clase RolModel?

[detective_jd]

Hola x_atrix, digamos que en Rol tengo los atributos de la clase en sí, pero en RolModel tengo lo necesario para persistir los datos del rol

Rol.php

Código PHP:

<?php
namespace Clases;
use \Model\RolModel;
class Rol 
{
    private $id;
    private $nombre;
    private $mod;
    function getId() {
        return $this->id;
    }
    function getNombre() {
        return $this->nombre;
    }    
    function setNombre($nombre) {
        $this->nombre = strtoupper($nombre);
    }
    function __construct($xid, $xnombre) {
        $this->id = $xid;
        $this->nombre = strtoupper($xnombre);
    }
    public function equals(Rol $obj) {
        return $this->nombre == $obj->nombre;                
    }
    public function save(){
        $this->mod = new RolModel();
        return ($this->id == 0) ? $this->mod->guardame($this) : $this->mod->modificame($this); 
    }
    public function del(){
        $this->mod = new RolModel();
        return $this->mod->eliminame($this);
    }
}
 

RolModel.php

Código PHP:

<?php
namespace Model;
use \PDO;
use \App\Model;
use \App\Session;
use \Clases\Rol;
class RolModel extends Model
{
    function __construct() {
        parent::__construct();
    }
    public function obtenerTodos(){
        $sql="select * from roles";
        $datos= array();
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute();
        foreach($consulta->fetchAll(PDO::FETCH_ASSOC) as $row){
            $rol = new Rol($row['rolId'], $row['rolNombre']);
            array_push($datos, $rol);
        }
        return $datos;
    }
    public function guardame($rol){
        if($this->check($rol->getNombre())){
            Session::set('msg', 'El rol ya existe');
            return null;
        }
        $sql="insert into roles(rolNombre) values(?)";
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getNombre()));
        return ($consulta->rowCount() > 0) ? $this->getBD()->lastInsertId() : null;
    }
    public function modificame($rol){
        $aux = $this->obtenerPorId($rol->getId()); 
        if(!$rol->equals($aux)){
            if($this->check($rol->getNombre())){
                Session::set('msg', 'El rol ya existe');
                return null;
            }        
        }
        $sql="update roles set rolNombre=? where rolId=?";
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getNombre(),$rol->getId()));
        return ($consulta->rowCount() > 0) ? $rol->getId() : null;
    }
    private function check($unique) { 
        $query = 'SELECT rolId FROM roles WHERE rolNombre = ?'; 
        $consulta = $this->getBD()->prepare($query); 
        $consulta->execute([$unique]); 
        // Indicar si hay algo en la base de datos con este nombre 
        return $consulta->rowCount() > 0; 
    }
    public function eliminame($rol, $notUsed = true) {
        $sql = "DELETE FROM roles WHERE rolId = ?";
        if ($notUsed === true) {
            $sql .= ' AND rolId NOT IN (SELECT DISTINCT rolId FROM usuarios)';
        }
        $consulta = $this->getBD()->prepare($sql);
        $consulta->execute(array($rol->getId()));
        return ($consulta->rowCount() > 0) ? $rol->getId() : null;
    }
    public function obtenerPorId($id) {
        $consulta = $this->getBD()->prepare("SELECT * FROM roles WHERE rolId = ?");
        $consulta->execute(array($id));
        if($consulta->rowCount() > 0) {
            $res= $consulta->fetchAll(PDO::FETCH_ASSOC)[0];
            return new Rol($res['rolId'], $res['rolNombre']);
        }
        else {
            return null;
        }
    }
}
 

lo que no entiendo es porque en el constructor dónde le paso los parámetros del rol, no puedo instanciar el modelo y deja de funcionar parte del sistema??

Espero respuestas y saludos.

[x_atrix]

Por los parámetros que necesita ... si le defines parámetros de entrada pero no se los pasas falla.

[detective_jd]

Hola x_atrix adapté un poco las cosas en Rol al final me quedó así:

Rol.php

Código PHP:

<?php
namespace Clases;
use \Model\RolModel;
class Rol 
{
    private $id;
    private $nombre;
    private $modelo;
    function getId() {
        return $this->id;
    }
    function getNombre() {
        return $this->nombre;
    }    
    function setNombre($nombre) {
        $this->nombre = strtoupper($nombre);
    }
    function __construct() {
        $args = func_get_args();
        $nargs = func_num_args();
        switch($nargs){ 
            case 1:
                self::__construct0();
                break;
            case 2:
                self::__construct1($args[0], $args[1]);
                break;
        }
    }
    function __construct0() { }
    function __construct1($xid, $xnombre) {
        $this->id = $xid;
        $this->nombre = strtoupper($xnombre);
    }
    public function equals(Rol $obj) {
        return $this->nombre == $obj->nombre;                
    }
    public function save(){
        $this->modelo = new RolModel();
        return ($this->id == 0) ? $this->modelo->guardame($this) : $this->modelo->modificame($this); 
    }
    public function del(){
        $this->modelo = new RolModel();
        return $this->modelo->eliminame($this);
    }
    public function obtenerTodos(){
        $this->modelo = new RolModel();
        return $this->modelo->obtenerTodos();
    }
    public function obtenerPorId($id){
        $this->modelo = new RolModel();
        return $this->modelo->obtenerPorId($id);
    }
}
 

RolesController.php

Código PHP:

<?php
namespace Controller;
use \App\Controller;
use \App\Session;
use \Clases\Rol;
class RolesController extends Controller 
{
    function __construct() {
        parent::__construct();
    }
    public function index(){
        if($this->checkUser()){
            $this->redirect(array("index.php"),array(
                "roles" => (new Rol())->obtenerTodos()
            )); 
        }   
    }
    public function add(){
        if($this->checkUser()){
            if (isset($_POST['btnaceptar'])) {
                if($this->checkDates()) {                
                    $rol = new Rol(0, $_POST['txtnom']);
                    $id = $rol->save();
                    Session::set("msg",(isset($id)) ? "Rol Creado" : Session::get('msg')); 
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('add.php'));
        }
    }   
    public function edit(){        
        if($this->checkUser()){
            Session::set("id",$_GET['p']);
            if (Session::get('id')!=null && isset($_POST['btnaceptar'])){                             
                if($this->checkDates()) {                
                    $rol= new Rol($_POST['hid'],$_POST['txtnom']);
                    $id = $rol->save();
                    Session::set("msg",(isset($id)) ? "Rol Editado" : Session::get('msg'));
                    header("Location:index.php?c=roles&a=index");
                    exit();
                }
            }
            $this->redirect(array('edit.php'),array(
                "rol" => (new Rol())->obtenerPorId(Session::get('id'))
            ));
        }        
    }
    public function delete(){
        if($this->checkUser()){
            if (isset($_GET['p'])){
                $rol = (new Rol())->obtenerPorId($_GET['p']);
                $id= $rol->del();
                Session::set("msg", (isset($id)) ? "Rol Borrado" : "No se pudo borrar el rol");
                header("Location:index.php?c=roles&a=index");
            }                           
        }
    }
    private function checkDates(){
        if(empty($_POST['txtnom'])){
            Session::set("msg","Ingrese los datos obligatorios (*) para continuar.");
            return false;
        }
        else{
            return true;
        }
    }
    private function checkUser(){
        if(Session::get("log_in")!= null and Session::get("log_in")->getRol()->getNombre() == "ADMIN"){
            return true;
        }
        else {
            Session::set("msg","Debe ser administrador para acceder.");
            $this->redirect(array('Main','index.php'));
        }
    }
}
 

[detective_jd]

Pero cuando quiero hacer parecido con Modelo me da este error:

Fatal error: Call to a member function getId() on a non-object in /var/www/html/amnesia_1/Model/ModeloModel.php on line 57

No me toma la marca que le paso por parámetro, te paso el código, puede que sea una tontería y no me doy cuenta:

Modelo.php

Código PHP:

<?php
namespace Clases;
use \Model\ModeloModel;
class Modelo {
    private $id;
    private $nombre;
    private $marca;
    private $modelo;
    function getId() {
        return $this->id;
    }
    function getNombre() {
        return $this->nombre;
    }
    function getMarca() {
        return $this->marca;
    }
    function setId($id) {
        $this->id = $id;
    }
    function setNombre($nombre) {
        $this->nombre = strtoupper($nombre);
    }
    function setMarca($marca) {
        $this->marca = $marca;
    }
    function __construct() {
        $args = func_get_args();
        $nargs = func_num_args();
        switch($nargs){ 
            case 1:
                self::__construct0();
                break;
            case 2:
                self::__construct1($args[0], $args[1],$args[2]);
                break;
        }
    }
    function __construct0() { }
    function __construct1($xid, $xnombre, $xmarca) {
        $this->id = $xid;
        $this->nombre = strtoupper($xnombre);
        $this->marca = $xmarca;
    }
    public function equals(Modelo $obj){
        return $this->nombre == $obj->nombre;                
    }
    public function save(){
        $this->modelo = new ModeloModel();
        return ($this->id == 0) ? $this->modelo->guardame($this) : $this->modelo->modificame($this); 
    }
    public function del(){
        $this->modelo = new ModeloModel();
        return $this->modelo->eliminame($this);
    }
    public function obtenerXBus($criterio){
        $this->modelo = new ModeloModel();
        return $this->modelo->buscador($criterio); 
    }
    public function obtenerXMarcas($criterio){
        $this->modelo = new ModeloModel();
        return $this->modelo->obtenerXDataList($criterio); 
    }
    public function obtenerPorId($id){
        $this->modelo = new ModeloModel();
        return $this->modelo->obtenerPorId($id);
    }
}
 

ModelosController.php

Código PHP:

<?php
namespace Controller;
use \App\Controller;
use \App\Session;
use \Clases\Marca;
use \Clases\Modelo;
class ModelosController extends Controller
{
    function __construct() {
        parent::__construct();
    }
    public function index(){
        if($this->checkUser()){
            Session::set('mar', '');
            Session::set('p', isset($_GET['p']) ? $_GET['p'] : 1);
            Session::set('b',(isset($_POST['txtbuscador'])) ? $_POST['txtbuscador'] : Session::get('b'));
            $modelos =(Session::get('b')!="") ? $this->getPaginator()->paginar((new Modelo)->obtenerXBus(Session::get('b')), Session::get('p')) : array();
            $this->redirect(array("index.php"),array(
                "modelos" => $modelos,
                "paginador" => $this->getPaginator()->getPages()
            ));         
        }
    }
    public function add(){
        if($this->checkUser()){ 
            Session::set('mar', isset($_POST['txtmar']) ? $_POST['txtmar'] : Session::get('mar'));
            $marcas = (Session::get('mar')!="") ? (new Modelo)->obtenerXMarcas(Session::get('mar')) : array();
            if (isset($_POST['btnaceptar'])) {
                if($this->checkDates()) {
                    $marca = (new Marca())->obtenerPorId($_POST['txtmar']);
                    $modelo = new Modelo(0,$_POST['txtnom'] , $marca);
                    $id = $modelo->save();
                    Session::set("msg",(isset($id)) ? "Modelo Creado" : Session::get('msg')); 
                    header("Location:index.php?c=modelos&a=index");
                    exit();
                }
            }
            $this->redirect(array('add.php'),array(
                'marcas' => $marcas
            ));
        }
    }
    public function edit(){
        if($this->checkUser()){           
            Session::set("id",$_GET['p']);
            Session::set('mar', isset($_POST['txtmar']) ? $_POST['txtmar'] : Session::get('mar'));
            $marcas = (Session::get('mar')!="") ? (new Modelo)->obtenerXMarcas(Session::get('mar')) : array();
            if (Session::get('id')!=null && isset($_POST['btnaceptar'])){                                         
                if($this->checkDates()) { 
                    $marca = (new Marca())->obtenerPorId($_POST['txtmar']);
                    $modelo = new Modelo($_POST['hid'],$_POST['txtnom'] , $marca);
                    $id = $modelo->save();
                    Session::set("msg",(isset($id)) ? "Modelo Editado" : Session::get('msg'));
                    header("Location:index.php?c=modelos&a=index");
                    exit();
                }
            }
            $this->redirect(array('edit.php'),array(
                "modelo" => (new Modelo)->obtenerPorId(Session::get('id')),
                'marcas' => $marcas
            ));
        }
    }
    public function delete(){
        if($this->checkUser()){
            if (isset($_GET['p'])){
                $modelo= (new Modelo)->obtenerPorId($_GET['p']);
                $id = $modelo->del();
                Session::set("msg", (isset($id)) ? "Modelo Borrado" : "No se pudo borrar el modelo");
                header("Location:index.php?c=modelos&a=index"); 
            }                           
        }
    }
    private function checkDates(){
        if(empty($_POST['txtnom']) || empty($_POST['txtmar'])){
            Session::set("msg","Ingrese los datos obligatorios (*) para continuar.");
            return false;
        }
        else {
            return true;
        }
    }
    private function checkUser(){
        if(Session::get("log_in")!= null and Session::get("log_in")->getRol()->getNombre() == "ADMIN"){
            return true;
        }
        else {
            Session::set("msg","Debe ser administrador para acceder.");
            $this->redirect(array('Main','index.php'));
        }
    }
}
 

Pareciera que el problema esta en el constructor de Modelo.php pero no sé x_atrix

Espero respuestas y saludos

[x_atrix]

Hola,

El error que te da es bastante claro ... estas llamando al metodo getId() en una variable que no es un objeto ...
Ejemplo:

Código PHP:

  $object = 'hola'; 
$object->getId(); // Esto falla con el error que indicas ...  
   

No hagas esto: self::metodoEstatico(). Self se utiliza para llamar a metodo estaticos y siempre se refiere a la clase donde estas. No es tu caso.

Código PHP:

      function __construct() { 
        $args = func_get_args(); 
        $nargs = func_num_args(); 
        switch($nargs){  
            case 1: 
                self::__construct0(); 
                break; 
            case 2: 
                self::__construct1($args[0], $args[1]); 
                break; 
        } 
    } 
    function __construct0() { } 
    function __construct1($xid, $xnombre) { 
        $this->id = $xid; 
        $this->nombre = strtoupper($xnombre); 
    }  
   

Definicion basica de una funcion o un metodo:

Código PHP:

  function hazAlgo($parametro = ''){ 
    // Código para hacer algo ...  
}  
   

Ahora, la funcion de arriba la puedes llamar asi:

Código PHP:

  hazAlgo(); // Funciona sin problemas  
   

O asi:

Código PHP:

  hazAlgo('Un parametro'); // Tambien funciona  
   

Aplicandolo a un constructor tuyo este concepto de parametros con valores por defecto quedaria asi:

Código PHP:

      function __construct($xid = 0, $xnombre = null, $xmarca = null) { 
        $this->id = $xid; 
        $this->nombre = strtoupper($xnombre); 
        $this->marca = $xmarca; 
    }  
   

[detective_jd]

Hola x_atrix, hice lo que me dijiste y funcionó a la perfección, te pongo el repositorio de github:

https://github.com/detectivejd/amnesia_php_mvc

Por otro lado, me quedé curioso porque para que la clase se pueda persistir a si mismo tenía que instanciarla según su modelo correspondiente y no me dejaba instanciarla en el constructor:

Código PHP:

<?php
namespace Clases;
use \App\IPersiste;
use \Model\RolModel;
class Rol implements IPersiste 
{
    private $id;
    private $nombre;
    private $modelo;
    function getId() {
        return $this->id;
    }
    function getNombre() {
        return $this->nombre;
    }    
    function setNombre($nombre) {
        $this->nombre = strtoupper($nombre);
    }
    function __construct($xid = 0, $xnombre = null) {
        $this->id = $xid;
        $this->nombre = strtoupper($xnombre);
    }
    public function equals(Rol $obj) {
        return $this->nombre == $obj->nombre;                
    }
    public function save(){
        $this->modelo = new RolModel();
        return ($this->id == 0) ? $this->modelo->create($this) : $this->modelo->update($this); 
    }
    public function del(){
        $this->modelo = new RolModel();
        return $this->modelo->delete($this);
    }
    public function find($criterio = null){
        $this->modelo = new RolModel();
        return $this->modelo->find();
    }
    public function findById($id){
        $this->modelo = new RolModel();
        return $this->modelo->findById($id);
    }
}
 

Lo otro que no pude hacer bien es que al chequear la fecha de vencimiento del pago no me hace bien la condición, es decir muestra el mensaje con y sin cumplir la condición, en el repositorio esta en:

Clases/Compra.php ->función checkFecVenc()
Clases/Pago.php
Controller/PagosController.php -> función index()

Porque lo demás ante bien de bien, gracias a tu ayuda.

Espero tus respuestas y saludos

[x_atrix]

Hola,
var_dump aqui:

Código PHP:

  <?php 
namespace Controller; 
use AppController; 
use AppSession; 
use ClasesCompra; 
use ClasesPago; 
class PagosController extends Controller 
{ 
    function __construct() { 
        parent::__construct(); 
    } 
    public function index(){ 
        if($this->checkUser()){ 
            Session::set("id",$_GET['p']); 
            $com = (new Compra())->obtenerPorId(Session::get('id')); 
 
var_dump($com); 
 
            Session::set('pg', isset($_GET['pg']) ? $_GET['pg'] : 1); 
            if($com->getCuotas() == $com->obtenerNroPago() -1){ 
                Session::set("msg","Deuda Saldada..."); 
            } 
            else if($com->checkFecVenc() == true){  
                Session::set("msg","Pago atrasado...");             
            }            
            $this->redirect(array('index.php'),array( 
                'pagos' => $this->getPaginator()->paginar($com->getPagos()), 
                "paginador" => $this->getPaginator()->getPages() 
            )); 
        } 
    }   

Para ver que valores tienes y en que formato.

[detective_jd]

Hola x_atrix con respecto al verificar la fecha de vencimiento en los pagos, la cambié por sql:

CompraModel.php

Código PHP:

public function checkFecVenc($compra){
        $consulta = $this->getBD()->prepare("SELECT pagFecVenc as vence from pagos where comId = ? order by pagId desc limit 0,1");
        $consulta->execute(array($compra->getId()));
        return date("Y/m/d") > $consulta->fetch(PDO::FETCH_ASSOC)['vence'];
    }
 

Compra.php

Código PHP:

public function checkFecVenc(){  
        $this->modelo = new CompraModel();
        return $this->modelo->checkFecVenc($this);
    }
 

PagosController.php

Código PHP:

public function index(){
        if($this->checkUser()){
            Session::set("id",$_GET['p']);
            $com = (new Compra())->findById(Session::get('id'));
            Session::set('pg', isset($_GET['pg']) ? $_GET['pg'] : 1);
            if($com->getCuotas() == $com->obtenerNroPago() -1){
                Session::set("msg","Deuda Saldada...");
            }
            else if($com->checkFecVenc()){ 
                Session::set("msg","Pago atrasado...");            
            }           
            $this->redirect(array('index.php'),array(
                'pagos' => $this->getPaginator()->paginar($com->getPagos()),
                "paginador" => $this->getPaginator()->getPages()
            ));
        }
    }
 

Pero el mensaje de Pago atrasado me sigue dando al cumplir o no la condición... ¿porque será?

Espero respuestas y saludos.

[detective_jd]

Hola x_atrix, te cuento que solucioné el problema: lo único que tenía hacer era esto

CompraModel.php

Código PHP:

public function checkFecVenc($compra){
        $consulta = $this->getBD()->prepare("SELECT pagFecVenc as vence from pagos where comId = ? order by pagId desc limit 0,1");
        $consulta->execute([$compra->getId()]);
        return count($this->findPagosByCompra($compra))>0 and date("Y-m-d") > $consulta->fetch(PDO::FETCH_ASSOC)['vence'];
    }
 

Cambiar el formato del date xq sino no me hacia la comparación....

Gracias x_atrix