Merge lp://staging/~vauxoo/openerp-mexico-localization/7.0-l10n_mx_diot_report-dev_ernesto into lp://staging/openerp-mexico-localization

Status: Merged
Merged at revision: 234
Proposed branch: lp://staging/~vauxoo/openerp-mexico-localization/7.0-l10n_mx_diot_report-dev_ernesto
Merge into: lp://staging/openerp-mexico-localization
Diff against target: 263 lines (+127/-96)
2 files modified
l10n_mx_account_tax_category/data/account_tax_category_data.xml (+9/-0)
l10n_mx_diot_report/wizard/wizard_diot_report.py (+118/-96)
To merge this branch: bzr merge lp://staging/~vauxoo/openerp-mexico-localization/7.0-l10n_mx_diot_report-dev_ernesto
Reviewer Review Type Date Requested Status
Luis Ernesto García Medina - http://www.vauxoo.com (community) Needs Resubmitting
Moisés López - http://www.vauxoo.com Pending
Review via email: mp+173381@code.staging.launchpad.net

Description of the change

se comprueba que los partner tengan RFC, en caso de no tener lanza la vista de proveedores (mostrando solo los que no tienen RFC), comprueba espacios en blanco y "_" (guion bajo en el RFC), tambien agrega la compañia del usuario al nombre del archivo

To post a comment you must log in.
Revision history for this message
Luis Ernesto García Medina - http://www.vauxoo.com (ernesto-gm) wrote :

Moy dice:
Omitir el periodo de apertura o cierre en el domain del period_id y en el default.
Para asignar la compañía, usar esta funcion self.pool.get('res.company')._company_default_get(cr, uid, 'MODELO', context=context),

Cuando detecte un campo requerido para el diot, entonces agregue a la lista de "ToFix partner", comentado que hace falta capturarle su RFC o DIOT Info. (como type of third, type of operation)

El strip, solo deberá hacerlo en el campo de res_partner.vat (en ningun otro campo)
Y los caracteres a omitir, deberan ser, los mismos que se consideran validos en el modulo del rfc mx:
Espacio, guion medio y guion bajo

review: Needs Fixing
Revision history for this message
Luis Ernesto García Medina - http://www.vauxoo.com (ernesto-gm) wrote :

Moy dice:
Que solo te muestre los partners que estan en la lista del DIOT y que ademas, no cumplen con los datos requeridos.

review: Needs Fixing
231. By Moisés López - http://www.vauxoo.com

[MERGE][IMP][l10n_mx_diot_report] show partner without RFC in view of suppliers & set default period & company & remove print

Revision history for this message
Luis Ernesto García Medina - http://www.vauxoo.com (ernesto-gm) wrote :

se comprueba que los partners cumplan con los campos de type_of_third, type_of_operation and VAT, en caso de que algunos de los campos requeridos no se cumplan, se manda a la vista de proveedores mostrando solo lo que no cumplen con algún dato requerido, se cambio la forma de escritura del archivo (usando un archivo CSV temporal), se agrego el nombre de la compañía seleccionada en el wizard al nombre del archivo generado se quitaron los caracteres "_ -" y espacios del campo VAT

review: Needs Resubmitting

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.