Skip to content

Refactor sql#10

Merged
NeoRazorX merged 13 commits into
FacturaScripts:masterfrom
Yopli2k:refactor_sql
Jul 21, 2023
Merged

Refactor sql#10
NeoRazorX merged 13 commits into
FacturaScripts:masterfrom
Yopli2k:refactor_sql

Conversation

@Yopli2k
Copy link
Copy Markdown
Contributor

@Yopli2k Yopli2k commented Jun 22, 2023

Cuando se obtienen los totales de la factura, ahora se utiliza un método protected para el sql general y otro para la cláusula where. Esto permite a otros programadores poder personalizar la obtención del cálculo de totales.

Este PR depende del PR anterior donde se añade el campo excluir347.
#9

Yopli2k and others added 13 commits June 22, 2023 09:51
Se añade mediante extensión el campo excluir347 a las facturas de cliente y proveedor.
Añadido un selector de tipo select en el modal de las cabeceras de documentos para que el usuario pueda seleccionar el valor del nuevo campo excluir347.
Añadido el filtro del nuevo campo excluir347 a la cláusula where del informe, cuando se calcula desde facturas.
Pendiente añadir a otros lenguajes
Cuando se obtienen los totales de la factura, ahora se utiliza un método protected para el sql general y otro para la cláusula where. Esto permite a otros programadores poder personalizar la obtención del cálculo de totales.
Se añade mediante extensión el campo excluir347 a las facturas de cliente y proveedor.
Añadido un selector de tipo select en el modal de las cabeceras de documentos para que el usuario pueda seleccionar el valor del nuevo campo excluir347.
Añadido el filtro del nuevo campo excluir347 a la cláusula where del informe, cuando se calcula desde facturas.
Pendiente añadir a otros lenguajes
Cuando se obtienen los totales de la factura, ahora se utiliza un método protected para el sql general y otro para la cláusula where. Esto permite a otros programadores poder personalizar la obtención del cálculo de totales.
@NeoRazorX NeoRazorX merged commit 4a19f8f into FacturaScripts:master Jul 21, 2023
@Yopli2k Yopli2k deleted the refactor_sql branch July 22, 2023 06:19
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