add validation and new value taxType

[developmentOYR]

He realizado unas modificaciones en el código para añadir una validación de todos los campos que envías a AEAT. También he ampliado la funcionalidad añadiendo nuevas etiquetas o valores ( Ejemplo la fecha de operación entre otros o he ampliado el tipo de operaciones (S1, S2, N1, N2) y otros campos más). He comentado cada uno de los cambios para ayudarte, a repasar cada uno de los cambios realizados. Aun así, si tienes alguna duda, no dudes en preduntarme.

Por otro lado, comentarte que nos gusta mucho el enfoque que le has dado al módulo y compartimos tu filosofía y estamos dispuestos a colaborar con el proyecto en todo lo que podamos. Quedo a tu disposición
Un saludo
Yamil - OYR Solutions

[orzocogorzo]

Hola Yamil,

Muchas gracias por la contribución, que bien contar con vuestro apoyo!

Puesto que la contribución es extensa, quizás tarde unos días en sacar una nueva release con ella, quiero darle una buena revisada, armonizar un poco el formato y estilo de código, y explorar algunas ideas nuevas e interesantes que planteas, como los extraFields a nivel de línea o facturedet.

En cuanto a la capa de validación de datos que has añadido, dos cosas:

1. No acabo de tener claro lo de las llamadas a la API para validar NIFs. Las APIs de Veri*Factu ya aplican esta validación durante la remisión del registro de factura. En caso de haber algun error, el registro será rechazado y se detendrá la validación de la factura, dejando un registro con la respuesta de error con información para el usuario para que peuda subsanar los errores. Realizar una llamada adicional a otra API no añade tiempo y consumo al proceso de forma algo redundante?
2. No crees que los ficheros validatio.lib.php y validate.lib.php se podrían juntar?

Intento dedicarle tiempo estos días y me quedo con este canal para compartir dudas.

[developmentOYR]

Hola orzocogorzo:

Pensándolo bien, tienes razón. Tiene más sentido hacer solo una petición a la API y mostrar el error en caso de rechazo, evitando así llamadas redundantes.

Con respecto a los ficheros validation.lib.php y validate.lib.php , mi idea inicial era separarlos para aislar la lógica de validación de datos puros de la lógica propia de Veri*Factu. El nombre tampoco me convencía del todo. De todas formas, si consideras que es más limpio e integrado unificarlos en un solo fichero, me parece perfecto; lo cambiamos.

Si quieres indicarme por dónde podemos seguir avanzando para no pisarnos el trabajo, te lo agradecería mucho.

Un saludo,

Yamil
Equipo OYR Solutions