Merge lp://staging/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri into lp://staging/openerp-mexico-localization

[REM] Remove _name of partner object
[ADD] wizard files

[IMP] Improve required fields when supplier field = True ( type_of_third and type_of_operation)

Federico,
IMHO no deberia ser requerido dicho campo con tan solo ser proveedor.
Imaginate el caso multi-company de diferentes países.

Tal vez sea mejor opción marcar error a la hora de generar dicho reporte,
indicando qué proveedores no tiene dichos campos capturados (es como un
seudo-requerido).

Otra puede ser que si el vat de la company es MX entonces que si haga la
validación de required, pero el problema se da cuando quieres generas una
cotización de compra y aun no tienes todos los datos de el proveedor, como
este.
El may 9, 2013 11:06 p.m., "Federico Manuel Echeverri Choux - Conectel" <
<email address hidden>> escribió:

> [IMP] Improve required fields when supplier field = True ( type_of_third
> and type_of_operation)
> --
>
> https://code.launchpad.net/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri/+merge/163166
> You are requested to review the proposed merge of
> lp:~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri
> into lp:openerp-mexico-localization.
>

Hola como estas????

Ya lo quito tienea toda la razon

Saludos
On May 10, 2013 1:52 AM, "Moisés López - http://www.vauxoo.com" <
<email address hidden>> wrote:

> Federico,
> IMHO no deberia ser requerido dicho campo con tan solo ser proveedor.
> Imaginate el caso multi-company de diferentes países.
>
> Tal vez sea mejor opción marcar error a la hora de generar dicho reporte,
> indicando qué proveedores no tiene dichos campos capturados (es como un
> seudo-requerido).
>
> Otra puede ser que si el vat de la company es MX entonces que si haga la
> validación de required, pero el problema se da cuando quieres generas una
> cotización de compra y aun no tienes todos los datos de el proveedor, como
> este.
> El may 9, 2013 11:06 p.m., "Federico Manuel Echeverri Choux - Conectel" <
> <email address hidden>> escribió:
>
> > [IMP] Improve required fields when supplier field = True ( type_of_third
> > and type_of_operation)
> > --
> >
> >
> https://code.launchpad.net/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri/+merge/163166
> > You are requested to review the proposed merge of
> >
> lp:~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri
> > into lp:openerp-mexico-localization.
> >
>
> --
>
> https://code.launchpad.net/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri/+merge/163166
> You proposed
> lp:~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri
> for merging.
>

[REM] Remove fields attrs ( type_of_third and type_of_operation )

[ADD] Add view of wizard report

[IMP] Improve DIOT Wizard view

Echeverri,

En el proyecto ya hay un módulo que te categoriza los impuestos por su código conocido por el SAT.
IVA, ISR, IETU...

Así no dependería del susceptible nombre del impuesto para poderlo identificar, solo se tendría que asignar al impuesto su categoría, y de aquí tomar el código.

Lo puedes revisar en la siguiente sección
l10n_mx_account_tax_category/data

Que tal Federico, tengo una duda, pues al ejecutar el wizard, me arroja el siguiente error en el log:

POST /web/dataset/call_kw/account.diot.report:read HTTP/1.0" 200 -
Periodo elegido 87
date_period [browse_record(account.period, 87)]
2013-06-01 2013-06-30
account_invoice [1590, 1593, 1591, 1594, 1595, 1597, 1618, 1596, 1622, 1621, 1592] 11
[browse_record(account.invoice.tax, 11)]
Tax DIOT ID ----->>>> None
2013-06-06 00:21:21,307 29447 ERROR IT_contpaq_20130530-20130605_231024 openerp.osv.osv: Uncaught exception
Traceback (most recent call last):
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 131, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 197, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 185, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/mnt/home/testing/openerp7_testing_AMX/com_addons/l10n_mx_diot_report/wizard/wizard_diot_report.py", line 89, in create_diot
    if tax.tax_id.tax_category_id.code == 'IVA' and tax.tax_id.amount == 0.16:
AttributeError: 'NoneType' object has no attribute 'code'
2013-06-06 00:21:21,307 29447 ERROR IT_contpaq_20130530-20130605_231024 openerp.netsvc: 'NoneType' object has no attribute 'code'
Traceback (most recent call last):
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/netsvc.py", line 292, in dispatch_rpc
    result = ExportService.getService(service_name).dispatch(method, params)
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/service/web_services.py", line 626, in dispatch
    res = fn(db, uid, *params)
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 188, in execute_kw
    return self.execute(db, uid, obj, method, *args, **kw or {})
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 131, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 197, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/mnt/home/testing/openerp7_testing_AMX/server/openerp/osv/osv.py", line 185, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/mnt/home/testing/openerp7_testing_AMX/com_addons/l10n_mx_diot_report/wizard/wizard_diot_report.py", line 89, in create_diot
    if tax.tax_id.tax_category_id.code == 'IVA' and tax.tax_id.amount == 0.16:
AttributeError: 'NoneType' object has no attribute 'code'
2013-06-06 00:21:21,309 29447 INFO IT_contpaq_20130530-20130605_231024 werkzeug: 127.0.0.1 - - [06/Jun/2013 00:21:21] "POST /web/dataset/call_button HTTP/1.0" 200 -

Estuve buscando el campo: "tax_diot_id", sin embargo no lo encontré en tu módulo pero el wizard si lo manda llamar, por lo que no se si hay que instalar algún otro módulo que agrega este campo o que es lo que me falta hacer.

Saludos.

Que tal Federico buen día, oye en el archivo __openerp__.py
en la llave "depends" : ["base", "base_vat","account", "account_tax_category", "account_invoice_tax" ],
el modulo account_invoice_tax lo marca como desconocido, dado que en la localización ese modulo se llama l10n_mx_account_invoice_tax.
si te refieres al mismo modulo? o es otro?
En caso de ser el mismo solo es cosa de cambiar el nombre

Perdon el modulo que cambio de nombre es: l10n_mx_account_tax_category

si Rodolfo lo vi ayer que hice el video y no me funcionaba, ya te lo cambio
y lo subo

Saludos

El 8 de junio de 2013 12:03, Rodolfo Lopez - http://www.vauxoo.com <
<email address hidden>> escribió:

> Que tal Federico buen día, oye en el archivo __openerp__.py
> en la llave "depends" : ["base", "base_vat","account",
> "account_tax_category", "account_invoice_tax" ],
> el modulo account_invoice_tax lo marca como desconocido, dado que en la
> localización ese modulo se llama l10n_mx_account_invoice_tax.
> si te refieres al mismo modulo? o es otro?
> En caso de ser el mismo solo es cosa de cambiar el nombre
> --
>
> https://code.launchpad.net/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri/+merge/163166
> You proposed
> lp:~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri
> for merging.
>

listo Rodolfo ya quedo lo de las dependencias

saludos

El 8 de junio de 2013 12:13, Federico Manuel Echeverri Choux - ( Vauxoo ) <
<email address hidden>> escribió:

> si Rodolfo lo vi ayer que hice el video y no me funcionaba, ya te lo cambio
> y lo subo
>
> Saludos
>
>
> El 8 de junio de 2013 12:03, Rodolfo Lopez - http://www.vauxoo.com <
> <email address hidden>> escribió:
>
> > Que tal Federico buen día, oye en el archivo __openerp__.py
> > en la llave "depends" : ["base", "base_vat","account",
> > "account_tax_category", "account_invoice_tax" ],
> > el modulo account_invoice_tax lo marca como desconocido, dado que en la
> > localización ese modulo se llama l10n_mx_account_invoice_tax.
> > si te refieres al mismo modulo? o es otro?
> > En caso de ser el mismo solo es cosa de cambiar el nombre
> > --
> >
> >
> https://code.launchpad.net/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri/+merge/163166
> > You proposed
> >
> lp:~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri
> > for merging.
> >
>
> --
>
> https://code.launchpad.net/~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri/+merge/163166
> You proposed
> lp:~openerp-mexico-team-activo/openerp-mexico-localization/7.0-l10n_mx_diot-dev-echeverri
> for merging.
>