fixes #4927 supplier.fiscal-data permitir 'Razón social' duplicada #1230
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1230
Loading…
Reference in New Issue
No description provided.
Delete Branch "4927-permitir-razonSocial-duplicada"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -131,0 +127,4 @@
if (nameChanged || countryChanged) {
if (!isAlpha(name)) throw new UserError('The social name has an invalid format');
const supplier = await Self.app.models.Supplier.findOne({where: {name, countryFk}, fields: ['id']});
Esta duplicada la validació, que confirme Juan açò
Si, en principio con la clave UNIQUE que se crea en la tabla es suficiente
@ -130,1 +121,3 @@
throw new UserError('The social name has an invalid format');
if (hasChanges) {
const name = changes.name || orgData.name;
const nameChanged = hasChanges && orgData.name != name;
hasChanges
ya se esta evaluando en el if, no es necesario repetirlo aquí@ -131,0 +122,4 @@
const name = changes.name || orgData.name;
const nameChanged = hasChanges && orgData.name != name;
const countryFk = changes.countryFk || orgData.countryFk;
const countryChanged = hasChanges && orgData.countryFk != countryFk;
No es necesario evaluar
hasChanges
@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed
- (Reclamaciones -> Descriptor) Cambiado el campo Agencia por Zona
- (Tickets -> Líneas preparadas) Actualizada sección para que sea más visual
- (Proveedores -> Crear/Editar) Permite añadir Proveedores con la misma razón social pero con países distintos
Este cambio debería ir en la versión actual