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 #3302 - agregar codpais, provincia y ciudad a clientes y provee… #1576

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

Descripción

  • Tanto en clientes como en proveedores deberíamos tener los campos de codpais, provincia y ciudad replicados en el modelo, para simplificar las consultas. Si no lo tenemos, nos toca combinar con la tabla de contactos para sacar esa información.
  • Se añaden a los modelos Cliente y Proveedor las nuevas propiedades.
  • Se replican los datos del contacto al modelo cuando se crea alguno de estos dos modelos o se actualiza el Contacto por defecto.
  • Se actualizan las tablas correspondientes.
  • Se agregan tests para comprobar estas circunstancias y se modifican las llamadas innecesarias a getDefaultAddress() en los tests.
  • Se cambia en el resto del codigo la llamda a getDefaultAddress() cuando no es necesaria.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant