openerp-l10n-ar-localization team mailing list archive

[Darío De Giacomo <dariokdg@xxxxxxxxx>]

Darío De Giacomo has proposed merging lp:~dariokdg/openerp-l10n-ar-localization/7.0 into lp:openerp-l10n-ar-localization.

Requested reviews:
  OpenERP - Team de Localización Argentina (openerp-l10n-ar-localization)

For more details, see:
https://code.launchpad.net/~dariokdg/openerp-l10n-ar-localization/7.0/+merge/193118

[MOD] Orden de los comprobantes en Invoice: para que el primero que aparezca sea Factura de Venta A (por ejemplo)
al ser emitido de un Responsable Inscripto a otro Responsable Inscripto, y no sea una Nota de Debito M, ya que es
raramente utilizada. (se ordeno la lista de comprobantes mediante un sort())

[MOD] Los campos afip_cae; afip_cae_due; afip_batch_number y afip_error_code de la invoice_view.xml deberian ser
READONLY, ya que el usuario no debe tener acceso a modificaciones de esa clase, porque esos datos vuelven desde AFIP y son inmutables
-- 
https://code.launchpad.net/~dariokdg/openerp-l10n-ar-localization/7.0/+merge/193118
Your team OpenERP - Team de Localización Argentina is requested to review the proposed merge of lp:~dariokdg/openerp-l10n-ar-localization/7.0 into lp:openerp-l10n-ar-localization.

=== modified file 'l10n_ar_invoice/invoice.py'
--- l10n_ar_invoice/invoice.py	2013-10-26 14:38:47 +0000
+++ l10n_ar_invoice/invoice.py	2013-10-29 18:25:28 +0000
@@ -196,6 +196,7 @@
             if 'value' not in result: result['value'] = {}
 
             if accepted_journal_ids:
+                accepted_journal_ids.sort()
                 result['domain'].update({
                     'journal_id': [('id','in', accepted_journal_ids)],
                 })

=== modified file 'l10n_ar_wsafip_fe/data/invoice_view.xml'
--- l10n_ar_wsafip_fe/data/invoice_view.xml	2013-10-07 15:16:52 +0000
+++ l10n_ar_wsafip_fe/data/invoice_view.xml	2013-10-29 18:25:28 +0000
@@ -31,10 +31,10 @@
                     </group>
                 </notebook>
                 <field name="move_id" position="after">
-                    <field name='afip_batch_number' invisible="1"/>
-                    <field name='afip_cae' attrs="{'readonly': [('state','!=','draft')]}"/>
-                    <field name='afip_cae_due' attrs="{'readonly': [('state','!=','draft')]}"/>
-                    <field name='afip_error_id'/>
+                    <field name='afip_batch_number' invisible="1" readonly="1"/>
+                    <field name='afip_cae' readonly="1"/>
+                    <field name='afip_cae_due' readonly="1"/>
+                    <field name='afip_error_id' readonly="1"/>
                 </field>
             </field>
         </record>