From 0d98d34c1244dc567203d70f330f5190039ada8f Mon Sep 17 00:00:00 2001
From: Antonio <elguitarraverde@hotmail.com>
Date: Tue, 13 Aug 2024 22:23:45 +0200
Subject: [PATCH] Tarea #710 - Al borrar contacto desvincularlo de clientes y
 proveedores

---
 Core/Model/Contacto.php         | 15 +++++++
 Test/Core/Model/ClienteTest.php | 69 +++++++++++++++++++++++++++++++++
 2 files changed, 84 insertions(+)

diff --git a/Core/Model/Contacto.php b/Core/Model/Contacto.php
index 052ad319f..17a972106 100644
--- a/Core/Model/Contacto.php
+++ b/Core/Model/Contacto.php
@@ -289,4 +289,19 @@ public function url(string $type = 'auto', string $list = 'ListCliente?activetab
     {
         return parent::url($type, $list);
     }
+
+    public function delete(): bool
+    {
+        $deleted = parent::delete();
+
+        if ($deleted){
+            $sql = 'UPDATE ' . Cliente::tableName() . ' SET idcontactoenv = NULL WHERE idcontactoenv = ' . $this->idcontacto . ';';
+            $sql .= 'UPDATE ' . Cliente::tableName() . ' SET idcontactofact = NULL WHERE idcontactofact = ' . $this->idcontacto . ';';
+            $sql .= 'UPDATE ' . Proveedor::tableName() . ' SET idcontacto = NULL WHERE idcontacto = ' . $this->idcontacto;
+
+            return self::$dataBase->exec($sql);
+        }
+
+        return $deleted;
+    }
 }
diff --git a/Test/Core/Model/ClienteTest.php b/Test/Core/Model/ClienteTest.php
index 323c050ba..3e73dc5df 100644
--- a/Test/Core/Model/ClienteTest.php
+++ b/Test/Core/Model/ClienteTest.php
@@ -21,6 +21,8 @@
 
 use FacturaScripts\Core\Lib\Vies;
 use FacturaScripts\Core\Model\Cliente;
+use FacturaScripts\Core\Model\Contacto;
+use FacturaScripts\Core\Model\Proveedor;
 use FacturaScripts\Test\Traits\LogErrorsTrait;
 use PHPUnit\Framework\TestCase;
 
@@ -210,6 +212,73 @@ public function testVies(): void
         $this->assertTrue($cliente->delete());
     }
 
+    /**
+     * Comprobamos que al eliminar un contacto,
+     * se desvincula de todos los clientes y proveedores.
+     */
+    public function testUnlinkCustomerContact()
+    {
+        // creamos un contacto
+        $contacto = new Contacto();
+        $contacto->nombre = 'Test Contacto';
+        $this->assertTrue($contacto->save(), 'contact-cant-save');
+
+        // creamos un cliente y vinculamos al contacto
+        $cliente = new Cliente();
+        $cliente->nombre = 'Test';
+        $cliente->cifnif = '12345677A';
+        $cliente->idcontactoenv = $contacto->idcontacto;
+        $cliente->idcontactofact = $contacto->idcontacto;
+        $this->assertTrue($cliente->save(), 'cliente-cant-save');
+
+        // creamos otro cliente y vinculamos al contacto
+        $cliente2 = new Cliente();
+        $cliente2->nombre = 'Test 2';
+        $cliente2->cifnif = '12345678A';
+        $cliente2->idcontactoenv = $contacto->idcontacto;
+        $cliente2->idcontactofact = $contacto->idcontacto;
+        $this->assertTrue($cliente2->save(), 'cliente-cant-save');
+
+        // creamos un proveedor y vinculamos al contacto
+        $proveedor = new Proveedor();
+        $proveedor->nombre = 'Test 3';
+        $proveedor->cifnif = '12345679A';
+        $proveedor->idcontacto = $contacto->idcontacto;
+        $this->assertTrue($proveedor->save(), 'proveedor-cant-save');
+
+
+        // comprobamos que el contacto creado se encuentra asociado
+        $this->assertEquals($cliente->idcontactoenv, $contacto->idcontacto);
+        $this->assertEquals($cliente->idcontactofact, $contacto->idcontacto);
+
+        $this->assertEquals($cliente2->idcontactoenv, $contacto->idcontacto);
+        $this->assertEquals($cliente2->idcontactofact, $contacto->idcontacto);
+
+        $this->assertEquals($proveedor->idcontacto, $contacto->idcontacto);
+
+        // eliminamos contacto
+        $this->assertTrue($contacto->delete(), 'contact-cant-delete');
+
+        // refrescamos los datos desde la base de datos.
+        $cliente->loadFromCode($cliente->codcliente);
+        $cliente2->loadFromCode($cliente2->codcliente);
+        $proveedor->loadFromCode($proveedor->codproveedor);
+
+        // comprobamos que el contacto creado, NO se encuentra asociado
+        $this->assertNotEquals($cliente->idcontactoenv, $contacto->idcontacto);
+        $this->assertNotEquals($cliente->idcontactofact, $contacto->idcontacto);
+
+        $this->assertNotEquals($cliente2->idcontactoenv, $contacto->idcontacto);
+        $this->assertNotEquals($cliente2->idcontactofact, $contacto->idcontacto);
+
+        $this->assertNotEquals($proveedor->idcontacto, $contacto->idcontacto);
+
+        // eliminamos
+        $this->assertTrue($cliente->delete());
+        $this->assertTrue($cliente2->delete());
+        $this->assertTrue($proveedor->delete());
+    }
+
     protected function tearDown(): void
     {
         $this->logErrors();