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 #3330 - idempresa y codalmacen opcionales en usuarios #1573

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

Conversation

elguitarraverde
Copy link
Contributor

@elguitarraverde elguitarraverde commented Apr 22, 2024

Descripción

  • Necesitamos que en los usuarios el codalmacen e idempresa sean opcionales, de forma que si no se selecciona nada, se asumen los del panel de control. De esta forma cuando creamos una nueva empresa (al almacén) y queremos que sea la predeterminada, solamente hay que cambiarlo en el panel de control.

  • Se asigna null a idempresa y codalmacen en el método clear() de la clase User para que inicialmente el usuario no tenga ninguna configuración respecto a empresa o almacén y tome los datos de la configuración por defecto.

  • Eliminamos el testWarehouse que se encargaba de asignar la configuración por defecto a estos campos. Con esta implementación estos campos son null inicialmente.

  • Configuramos los campos como opcionales en el xml de la tabla.

  • Realizamos tests en PresupuestoCliente como se indica en la tarea.

  • AL ÚNICA PEGA QUE LE VEO ES QUE ESTOS CAMBIOS SOLO VAN A AFECTAR A USUARIOS NUEVOS YA QUE LOS QUE YA ESTÁN CREADOS YA TIENEN LOS VALORES POR DEFECTO ASIGNADOS Y AUNQUE SE CAMBIE EN LA CONFIGURACION POR DEFECTO SEGUIRAN CON EL VALOR ANTERIOR. SE DEBERÍAN SETEAR LOS DOS CAMPOS A NULL EN TODOS LOS USUARIOS.

¿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