Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
L
l10n_mx_facturae
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 6
    • Issues 6
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 4
    • Merge Requests 4
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • l10n_mx
  • l10n_mx_facturae
  • Merge Requests
  • !139

Merged
Opened 3 years ago by Agustín Cruz@agb80Owner

fix(account.invoice): corrección definitiva al cálculo de la fecha de emisión

  • Overview 1
  • Commits 1
  • Changes 2
  • Loading...

El cálculo de la fecha de emisión había sucitado algunos problemas ya que no consideraba correctamente el rango de tiempo que proporciona el SAT para realizar el timbrado de la factura.

Originalmente se había creado la función que calculaba la hora de timbrado sumando 5 minutos para que se pudiera timbrar en el pasado, esto ocasionaba que varias facturas no se pudieran timbrar de manera automática cuando se creaban porque la hora de generación quedaba fuera de rango.

Luego se modificó la función para que se restara un minuto a la hora actual. Esto permitía timbrar correctamente las facturas recién creadas pero volvía a introducir el problema con las facturas creadas en el pasado.

Adicionalmente había un problema con el guardado de la hora ya que no se convertía la fecha calculada en la zona horaria de UTC ocasionando que a la hora de timbrar se generara una conversión de zona horaria adicional que alteraba por completo la fecha de emisión de la factura llegando a haber casos en que se generaba la factura con una fecha de emisión de un periodo anterior. En especial cuando la emisión de la factura se hacía el primer día del periodo siguiente.

Para corregir esto lo que estamos haciendo ahora es considerar la fecha que se seleccionó para la emisión de la factura y comparándola contra la fecha actual. Si es una fecha en el pasado le aumentamos 1 minuto a la fecha calculada para estar en el rango de las 72 horas y si no le restamos 1 minuto para quedar en el rango de las 72 horas.

Adicionalmente agregamos la conversión de la zona horaria antes de guardar la fecha y con eso evitamos la doble conversión para asegurar que siempre se respeta la fecha seleccionada por el usuario.

Request to merge agb80:fix_2_19_1 into master
  • Download as
  • Email patches

  • Plain diff

Checking pipeline status.

Checking approval status

Merged by Agustín Cruz 3 years ago (Mar 18, 2022 2:23am GMT+0000) 3 years ago

The changes were merged into master with d7272eb3

Could not retrieve the pipeline status. For troubleshooting steps, read the documentation.


  • Agustín Cruz
    Agustín Cruz @agb80 · 3 years ago
    Owner

    @noe.izquierdo me regalas una revisión aquí por favor. No es algo que me hayan reportado en soporte pero si es algo que detecté y que estoy corrigiendo.

    Quedo pendiente de tus comentarios

  • Agustín Cruz @agb80 added 1 commit 3 years ago

    added 1 commit

    • 7ac5a50d - fix(account.invoice): corrección definitiva al cálculo de la fecha de emisión

    Compare with previous version

  • Agustín Cruz @agb80 merged 3 years ago

    merged

  • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
Please register or sign in to reply
0 Assignees
Assign to
0 Reviewers
Request review from
None
Milestone
None
Assign milestone
None
Time tracking
No estimate or time spent
0
Labels
None
Assign labels
  • No matching results
  • Manage project labels
Lock merge request
Unlocked
participants
Reference: l10n_mx/l10n_mx_facturae!139
Source branch: fix_2_19_1

    0 pending comments