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
  • Issues
  • #23

Closed
Open
Opened 6 years ago by Sotero Toribio Campistrano@sotero.toribioContributor
  • New issue

  • Report abuse

  • New issue

  • Report abuse

Numero de cuenta para complemento de pago

Closed

Numero de cuenta para complemento de pago

Cuando se tiene un complemento de pago y quererlo firmar, el numero de cuenta del cliente es opcional. Tener una mejor estabilidad en esta parte del sistema, es poner las validaciones correspondientes a este campo dependiendo la forma de pago seleccionada como lo establece el SAT. Cuenta_Ordenante

Please solve the reCAPTCHA

We want to be sure it is you, please confirm you are not a robot.

  1. Oh no!

    You are trying to upload something other than an image. Please upload a .png, .jpg, .jpeg, .gif, .bmp, .tiff or .ico.

    Incoming!

    Drop your designs to start your upload.

Linked issues
0

Related merge requests

  • Sotero Toribio Campistrano
    Sotero Toribio Campistrano @sotero.toribio · 6 years ago
    Contributor

    @OpenPymeMx

  • Salvador Martínez
    Salvador Martínez @chavamm.83 · 6 years ago
    Developer

    Saludos @OpenPymeMx

    Una propuesta, es la siguiente:

    Implementar una validación local que sea transparente al usuario.

    • El la configuración de la forma de pago, agregar un campo con el patrón del Numero de Cuenta Ordenante, tal y como lo presenta el SAT en su catalogo.

    • Se crea un método que ejecuta dicha validación, para el momento del Timbrado del Pago. Este metodo podria bien, agregarse al modelo payment.type, y

    • En donde, se debera de pasarse los siguientes datos que servirán como referencia para ejecutar la validación:

      • Número de cuenta de Cliente
    • La Validación, en base al Patrón designado en la Forma de Pago, deberá de validar:

      • Identificando la longitud del numero de cuenta.
      • Caracteres Validos para la estructura de la misma.
    • Si cumple, entonces el método retorna el Número de Cuenta

    • De lo contrario, retorna False

    Se podrá usar en cualquier parte del sistema de la siguiente manera, por ejemplo:

        
        acc_num = payment_type_id.validate_acc(
                partner_bank_id.acc_number
        )
    

    El pedazo de código, puede tener errores y estar totalmente mal =) pero es para que se den una idea.

    Ya usandolo para la generación del XML, entonces,

    • que se valide el número de cuenta,
    • y si no es valida, se omitirá ese atributo en el XML para que no sea agregado

    Con esto evitar que truene por la validación del PAC cuando va esta info en el XML.

    Saludos!

    Edited by Salvador Martínez 6 years ago
  • Agustín Cruz
    Agustín Cruz @agb80 · 6 years ago
    Owner

    Muevo entonces este issue al repositorio de facturación para que se implemente por ahí.

  • Agustín Cruz @agb80 moved from facturacion#97 (moved) 6 years ago

    moved from facturacion#97 (moved)

  • Agustín Cruz @agb80 assigned to @agb80 6 years ago

    assigned to @agb80

  • Agustín Cruz @agb80 mentioned in issue facturacion#97 (moved) 6 years ago

    mentioned in issue facturacion#97 (moved)

  • Agustín Cruz @agb80 mentioned in merge request !60 (merged) 6 years ago

    mentioned in merge request !60 (merged)

  • Agustín Cruz @agb80 closed via commit 6a351e87 6 years ago

    closed via commit 6a351e87

  • 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
None
Milestone
None
Assign milestone
None
Time tracking
No estimate or time spent
None
Due date
None
1
Labels
Feature
Assign labels
  • No matching results
  • Manage project labels
Confidentiality
Not confidential
Lock issue
Unlocked
participants
Reference: l10n_mx/l10n_mx_facturae#23