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
  • !158

Merged
Opened 2 years ago by Agustín Cruz@agb80Owner0 of 7 tasks completed0/7 tasks

fix(account.invoice.line): altera el precio únicamente en caso de que el producto lleve IEPS

  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 1
  • Loading...

¿Qué hace este MR?

Cuando el producto no lleva IEPS y el cliente tampoco era sujeto de IEPS se estaba alterando el precio unitario porque al pasar por la función que calcula el total de impuestos redondeaba la base. Dado que en dicha función se está redondeando la base de impuestos para que coincida con el número de decimales que maneja la moneda, en caso de enviar un producto con una unidad y un precio por unicad con una precisión mayor a la precisión de la moneda, inmediatamente se altera el precio por unidad enviado al SAT en el XML.

Ejemplos:

Cantidad | Precio Unitario | Importe | IVA | Total 393 | 3.7183 | 1461.29 | 233.81 | 1695.10 1000 | 1.2922 | 1292.20 | 206.75 | 1498.95

¿Cumple los criterios de aceptación?

  • Agrega pruebas unitarias y/o escenarios de negocio
  • Documentación creada/actualizada
  • Sigue las guías de evolución del C4
  • Sigue las guías de estilo de código

Checklist de revisión

  • Your team's review (required)
  • PM's review (recommended, but not a blocker)
  • Technical writer's review (required)

cc @OpenPymeMx

Request to merge fix_3_3_2 into master
  • Download as
  • Email patches

  • Plain diff

Pipeline #17475 passed for a5de2df5 on fix_3_3_2
Checking approval status

Merged by Agustín Cruz 2 years ago (Sep 2, 2022 11:14pm GMT+0000) 2 years ago

The changes were merged into master with 8672a0f6

The source branch has been deleted

Pipeline #17489 passed for 8672a0f6 on master

  • Agustín Cruz @agb80 added Bug Feature labels 2 years ago

    added Bug Feature labels

  • Agustín Cruz @agb80 removed Feature label 2 years ago

    removed Feature label

  • Noe Fernando Izquierdo Hernandez @noe.izquierdo approved this merge request 2 years ago

    approved this merge request

  • Agustín Cruz @agb80 merged 2 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
1
Labels
Bug
Assign labels
  • No matching results
  • Manage project labels
Lock merge request
Unlocked
participants
Reference: l10n_mx/l10n_mx_facturae!158
Source branch: fix_3_3_2

    0 pending comments