Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tarea #2970 - agregar worker actualizar numero productos en familias #1584

Conversation

elguitarraverde
Copy link
Contributor

Descripción

  • Añadir un worker para actualizar el número de productos de las familias, cuando se crea, modifica o elimina un producto.
  • He usado 'Model.Producto.Insert', 'Model.Producto.Update' y 'Model.Producto.Delete' porque usando 'Model.Producto.*' duplica las llamadas, ya que, por ejemplo, al actualizar un producto llama dos veces al worker: 'Model.Producto.Update' y tambien a 'Model.Producto.Save'
  • He añadido al metodo saveUpdate() de la clase Core/Model/Base/ModelClass.php que se envia al Worker los datos antes de actualizar, ya que si no es así cuando se desvincula un producto de una familia llega el codfamilia a null y no hay forma de saber a que familia pertenecía antes. Lo he puesto en esta clase porque creo que es útil y se podrá usar en muchas ocasiones y para los Plugins también creo que puede venir muy bien disponer de estos datos en el worker.

¿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.

@NeoRazorX
Copy link
Owner

No me vale la modificación en ModelClass. esta tarea no debe tocar nada más que el propio worker.

@elguitarraverde
Copy link
Contributor Author

@NeoRazorX cuando se desvincula un producto de una familia llega el codfamilia a null y no hay forma de saber a que familia pertenecía antes, para poder actualizar los totales de esa familia.

He añadido al metodo saveUpdate() de la clase Core/Model/Base/ModelClass.php que se envia al Worker los datos antes de actualizar, ya que si no es así cuando se desvincula un producto de una familia llega el codfamilia a null y no hay forma de saber a que familia pertenecía antes.

Lo he puesto en esta clase porque creo que es útil y se podrá usar en muchas ocasiones y para los Plugins también creo que puede venir muy bien disponer de estos datos en el worker. Datos antes de actualizar y datos actualizados.

no obstante dime donde prodria obtaner el codfamilia cuando se desvincula un producto de una familia en el worker.

@NeoRazorX
Copy link
Owner

NeoRazorX commented Jul 1, 2024 via email

@elguitarraverde
Copy link
Contributor Author

OK, dejo entonces esta PR en borrador hasta que se rediseñen los modelos.

@elguitarraverde elguitarraverde marked this pull request as draft July 1, 2024 16:36
@NeoRazorX
Copy link
Owner

Por ahora he añadido 2 procesos al cron para actualizar estos contadores.

@NeoRazorX NeoRazorX closed this Sep 18, 2024
@elguitarraverde elguitarraverde deleted the Tarea-#2970---agregar-worker-actualizar-numero-productos-en-familias branch September 18, 2024 16:45
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.

2 participants