Merge lp:~vauxoo/openerp-venezuela-localization/rodolfo-l10n_ve_wh_islr-logs into lp:~openerp-venezuela/openerp-venezuela-localization/6.0-trunk

[Rodolfo Alvarez(Vauxoo)]

log messages added for withholding documents, documents vouchers and xml lines generation, cancel option pending.

[Nhomar - Vauxoo (nhomar)]

Verifica la linea 51

EL mensaje original esta en espa;ol y la cadena traducida tambie

+msgid "DEMO Cualquier otra Comisión distintas a Remuneraciones accesorias de los sueldos, salarios y demás remuneraciones similares"

Por favor. Verifica eso....

[Nhomar - Vauxoo (nhomar)]

Es de SUMA importancia que verifiques eso ANTES de la propuesta de merge, y revises en INGLES todas las cadenas en ingles, en Español todas las cadenas en español.

[Nhomar - Vauxoo (nhomar)]

Saludos.

Por favor, antes de hacer una propuesta de merge de traducciones, se debe verifica TODO el funcionamiento a nivel de cadenas en los modulos, tanto en codigo como en mensajes, el asunto u objetivo de delegar estas tareas es poder pasar por un proceso de auditoria en cadenas, no podemos permitirnos que el 20% de las traducciones esten incorrectas y que adicionalemnte se dañe otro grupo en correcto funcionamiento, por consiguiente, durante el proceso de traduccion y ajuste se debe solicitar por parte del traductor una revision de las funcionalidades y ventanas del modulo.

Como hago esto:

Te vas a la ventana de instalacion del modulo, verificas que OBJETOS - VISTAS Y MENUES crea de tal forma que podamos hacer estas verficaciones previo montaje del merge proposal!

review: Needs Fixing (queality comments.)

[Rodolfo Alvarez(Vauxoo) (augusto2507)]

> Saludos.
>
> Por favor, antes de hacer una propuesta de merge de traducciones, se debe
> verifica TODO el funcionamiento a nivel de cadenas en los modulos, tanto en
> codigo como en mensajes, el asunto u objetivo de delegar estas tareas es poder
> pasar por un proceso de auditoria en cadenas, no podemos permitirnos que el
> 20% de las traducciones esten incorrectas y que adicionalemnte se dañe otro
> grupo en correcto funcionamiento, por consiguiente, durante el proceso de
> traduccion y ajuste se debe solicitar por parte del traductor una revision de
> las funcionalidades y ventanas del modulo.
>
> Como hago esto:
>
> Te vas a la ventana de instalacion del modulo, verificas que OBJETOS - VISTAS
> Y MENUES crea de tal forma que podamos hacer estas verficaciones previo
> montaje del merge proposal!

###

Entiendo, el problema fue que exporte el archivo es_VE.po de nuevo y lo que debi hacer fue solo hacer las modificaciones que yo realice en el módulo en el archivo .po que ya existia, por eso el diff es tan grande, el objetivo de este branch era agregar logs en las acciones mencionadas en el modulo, no la traduccion, la traduccion ya estaba hecha, yo solo intente agregar mis cambios a la traduccion que ya existia, cosa que hice de manera incorrecta pero ya estoy arreglando eso. En cuanto a las cadenas que estan en español hable con gaby y me dijo que estan asi por que son de data demo, de todos modos revisare a ver por si acaso cualquier otro error

[lp:~vauxoo/openerp-venezuela-localization/rodolfo-l10n_ve_wh_islr-logs]

480. By <email address hidden> on 2011-11-18

[IMP]The file es_VE.po has been modified to improve the spanish translation option

Unmerged revisions

=== modified file 'l10n_ve_withholding_islr/i18n/es_VE.po'
--- l10n_ve_withholding_islr/i18n/es_VE.po	2011-11-17 23:15:16 +0000
+++ l10n_ve_withholding_islr/i18n/es_VE.po	2011-11-18 16:38:24 +0000
@@ -1423,6 +1423,7 @@
 msgid "Impossible withholding income, because the lines of the invoice has not concept withholding!"
 msgstr "Imposible realizar retención de ISLR, porque las lineas de la factura no tienen concepto de retención asociado!"
 
+<<<<<<< TREE
 #. module: l10n_ve_withholding_islr
 #: view:wh.islr.config:0
 msgid "Withholding income Configuration"
@@ -1446,3 +1447,22 @@
 #: model:ir.actions.act_window,name:l10n_ve_withholding_islr.action_config_wh_islr
 msgid "Configure Income Withholding"
 msgstr "Configuración de la Retención de ISLR"
+=======
+#. module: l10n_ve_withholding_islr
+#: code:addons/l10n_ve_withholding_islr/invoice.py:305
+#, python-format
+msgid "Withholding income xml line generated."
+msgstr "Línea xml de retención de ISLR generada."
+
+#. module: l10n_ve_withholding_islr
+#: code:addons/l10n_ve_withholding_islr/invoice.py:574
+#, python-format
+msgid "Withholding income voucher %s generated."
+msgstr "Comprobante de retención de ISLR %s generado."
+
+#. module: l10n_ve_withholding_islr
+#: code:addons/l10n_ve_withholding_islr/islr_wh_doc.py:318
+#, python-format
+msgid "Withholding income voucher %s validated and accounting entry generated."
+msgstr "Comprobante de retención de ISLR %s validado y asiento contable generado."
+>>>>>>> MERGE-SOURCE

=== modified file 'l10n_ve_withholding_islr/invoice.py'
--- l10n_ve_withholding_islr/invoice.py	2011-11-17 22:24:35 +0000
+++ l10n_ve_withholding_islr/invoice.py	2011-11-18 16:38:24 +0000
@@ -302,6 +302,8 @@
                 self.pool.get('account.invoice.line').write(cr, uid, line, {'apply_wh': apply})
             else:
                 self.pool.get('account.invoice.line').write(cr, uid, line, {'apply_wh': apply,'wh_xml_id':self._create_islr_xml_wh_line(cr, uid,line,dict)})
+                message = _("Withholding income xml line generated.")
+                self.log(cr, uid, line, message)
                 
     def _create_islr_xml_wh_line(self,cr, uid, line, dict):
         '''
@@ -567,6 +569,11 @@
                     self._create_doc_invoices(cr,uid,key,islr_wh_doc_id)
                         
                 self.pool.get('account.invoice').write(cr,uid,inv_brw.invoice_id.id,{'islr_wh_doc_id':islr_wh_doc_id})
+                
+                account_obj = self.pool.get('account.invoice')
+                account_brw = account_obj.browse(cr, uid, inv_brw.invoice_id.id).islr_wh_doc_id.name
+                message = _("Withholding income voucher %s generated.") % account_brw
+                self.log(cr, uid, islr_wh_doc_id, message)
             else:
                 pass
         else:

=== modified file 'l10n_ve_withholding_islr/islr_wh_doc.py'
--- l10n_ve_withholding_islr/islr_wh_doc.py	2011-10-25 20:33:16 +0000
+++ l10n_ve_withholding_islr/islr_wh_doc.py	2011-11-18 16:38:24 +0000
@@ -141,7 +141,7 @@
         for wh_doc in wh_doc_brw:
             for wh_doc_line in wh_doc.islr_wh_doc_id: 
                 inv_ids.append(wh_doc_line.id)
-
+               
         context["wh_doc_id"]=ids[0]
         inv_obj.action_ret_islr(cr, uid, inv_ids,context)
         return True
@@ -254,6 +254,7 @@
         wh_doc_obj = self.pool.get('islr.wh.doc.line')
         context = {}
         inv_id = None
+        doc_brw = None
         
         for ret in self.browse(cr, uid, ids):
             if not ret.date_uid:
@@ -312,6 +313,10 @@
                                 raise osv.except_osv(_('Invalid action !'),_("Impossible change the period accountig to a withholding that has already been declared."))
                             
 #                    inv_obj.write(cr, uid, line.invoice_id.id, {'retention':True}, context=context)
+        doc_brw = self.browse(cr, uid, ids[0]).name
+        
+        message = _("Withholding income voucher %s validated and accounting entry generated.") % doc_brw
+        self.log(cr, uid, ids[0], message) 
         return True