Skip to content

Tarea #4012#1868

Open
DeathPaul1125 wants to merge 3 commits intoNeoRazorX:masterfrom
DeathPaul1125:master
Open

Tarea #4012#1868
DeathPaul1125 wants to merge 3 commits intoNeoRazorX:masterfrom
DeathPaul1125:master

Conversation

@DeathPaul1125
Copy link
Contributor

Nuevo Worker
Worker que escucha los eventos Model.ProductoProveedor.Insert, Model.ProductoProveedor.Update y Model.ProductoProveedor.Delete para actualizar los precios de coste de forma asíncrona.

Modificado: Core/Model/ProductoProveedor.php
Eliminado el use de CostPriceTools
Eliminados los métodos onDelete(), onInsert() y onUpdate() que llamaban directamente a CostPriceTools::update()
Ahora la actualización de precios de coste se hace a través del worker

Modificado: Core/Kernel.php

WorkQueue::addWorker('CostPriceWorker', 'Model.ProductoProveedor.Delete');
WorkQueue::addWorker('CostPriceWorker', 'Model.ProductoProveedor.Insert');
WorkQueue::addWorker('CostPriceWorker', 'Model.ProductoProveedor.Update');

Nuevo archivo: Test/Core/Worker/CostPriceWorkerTest.php
Tests unitarios para verificar:

Actualización de precio de coste al insertar ProductoProveedor
Actualización de precio de coste al actualizar ProductoProveedor
Actualización de precio de coste al eliminar ProductoProveedor
Actualización de precio de coste desde documentos de compra

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

@DeathPaul1125
Copy link
Contributor Author

Tarea #4012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant