0042962: Error en la selección de facturas a enviar por proceso SII

0042962

Localization Pack: Spain

SII

public

2020-01-22 14:06

2020-06-22 14:19

Reporter

qualitic

Assigned To

Mery Anelo

Priority

high

Severity

major

Reproducibility

N/A

Status

closed

Resolution

fixed

Platform

OS

30

OS Version

Openbravo Appliance 14.04

Product Version

Merge Request Status

Regression date

Regression introduced by commit

Regression level

Review Assigned To

Support ticket

OBNetwork customer

Regression introduced in release

Summary

0042962: Error en la selección de facturas a enviar por proceso SII

Description

Analizando los datos de la aplicación del cliente, nos hemos percatado de que algunas de las facturas seleccionadas por el proceso de envío de facturas al SII no deberían serlo.
Específicamente en los siguientes casos:

    - Dada una cadencia de envió de facturas de dos días, si se ejecuta el proceso de envió de facturas, las facturas generadas y completadas hoy serán seleccionadas.
    - Dada una fecha de acogida al SII del 01-01-2019, si se ejecuta el proceso de envío de facturas, este seleccionará facturas con fecha anterior a la fecha de acogida al SII.

La consulta de selección presente en la clase SendInvoicesGroupsToAEAT.java es la siguiente:

    select i.id, i.documentNo, i.salesTransaction
            from Invoice i
            where i.aeatsiiIssent = false
            and i.organization.id = (:orgId)
            and i.processed = true
            and i.aeatsiiErrorRegistral = false
            and (
                    (i.salesTransaction = true
                     and ((i.invoiceDate <= :todayMinusCadenceSales
                          and i.invoiceDate >= :sinceSii)
                          or (month(i.aeatsiiFechaOperacion) - month(DATE(:today)) = 0
                              and (i.invoiceDate < :today
                                    or (i.invoiceDate <= :dateLimitSII
                                        and i.invoiceDate > :salesLimitCadence
                                        and DATE(i.invoiceDate) = DATE(:today))
                                  )
                         )
                         or (month(i.aeatsiiFechaOperacion) <> month(i.invoiceDate)
                           and DATE(i.invoiceDate) = DATE(:today))
                    )
                  )
                 or (i.salesTransaction = false
                        and i.posted = true
                        and ((i.aeatsiiFechaRegCont <= :todayMinusCadencePurchase
                        and i.aeatsiiFechaRegCont >= :sinceSii)
                            or (month(i.aeatsiiFechaOperacion) - month(DATE(:today)) = 0
                                and (i.aeatsiiFechaRegCont < :today
                                      or (i.aeatsiiFechaRegCont <= :dateLimitSII
                                          and i.aeatsiiFechaRegCont > :purchaseLimitCadence
                                          and DATE(i.aeatsiiFechaRegCont) = DATE(:today))
                                    )
                                )
                            )
                            or (month(i.aeatsiiFechaRegCont) <> month(i.aeatsiiFechaOperacion)
                                and DATE(i.aeatsiiFechaRegCont) = DATE(:today))
                       )
                   )

Tras analizar la consulta, hemos comprobado como existe un problema con los operadores OR y AND, que posibilita la selección de facturas que dan lugar a los dos problemas mencionados arriba.
Concretamente, creemos que el siguiente trozo de código es el causante del problema (para facturas de venta en nuestro caso):

                        (i.invoiceDate <= :todayMinusCadenceSales
                          and i.invoiceDate >= :sinceSii)
                          or (month(i.aeatsiiFechaOperacion) - month(DATE(:today)) = 0
                              and (i.invoiceDate < :today
                                    or (i.invoiceDate <= :dateLimitSII
                                        and i.invoiceDate > :salesLimitCadence
                                        and DATE(i.invoiceDate) = DATE(:today))
                                  )
                         )
                         or (month(i.aeatsiiFechaOperacion) <> month(i.invoiceDate)
                           and DATE(i.invoiceDate) = DATE(:today))

Dicho trozo de código, queda dividido por dos OR en tres sentencias distintas.
La sentencia central, la cual se detalla a continuación, posibilita la selección de facturas que no deberían ser seleccionadas dada la cadencia de envío o la fecha de acogida al SII.

                      (month(i.aeatsiiFechaOperacion) - month(DATE(:today)) = 0
                          and (i.invoiceDate < :today
                                or (i.invoiceDate <= :dateLimitSII
                                    and i.invoiceDate > :salesLimitCadence
                                    and DATE(i.invoiceDate) = DATE(:today))
                              )
                     )

Steps To Reproduce

Alta en el SII

Proposed Solution

Tras analizar la consulta, hemos comprobado como existe un problema con los operadores OR y AND, que posibilita la selección de facturas que dan lugar a los dos problemas mencionados arriba.
Concretamente, creemos que el siguiente trozo de código es el causante del problema (para facturas de venta en nuestro caso):

                        (i.invoiceDate <= :todayMinusCadenceSales
                          and i.invoiceDate >= :sinceSii)
                          or (month(i.aeatsiiFechaOperacion) - month(DATE(:today)) = 0
                              and (i.invoiceDate < :today
                                    or (i.invoiceDate <= :dateLimitSII
                                        and i.invoiceDate > :salesLimitCadence
                                        and DATE(i.invoiceDate) = DATE(:today))
                                  )
                         )
                         or (month(i.aeatsiiFechaOperacion) <> month(i.invoiceDate)
                           and DATE(i.invoiceDate) = DATE(:today))

Dicho trozo de código, queda dividido por dos OR en tres sentencias distintas.
La sentencia central, la cual se detalla a continuación, posibilita la selección de facturas que no deberían ser seleccionadas dada la cadencia de envío o la fecha de acogida al SII.

                      (month(i.aeatsiiFechaOperacion) - month(DATE(:today)) = 0
                          and (i.invoiceDate < :today
                                or (i.invoiceDate <= :dateLimitSII
                                    and i.invoiceDate > :salesLimitCadence
                                    and DATE(i.invoiceDate) = DATE(:today))
                              )
                     )

Additional Information