Merge lp:~inddiana/sisb/edgar_bug_facturas into lp:sisb

[[SISB] Edgar Rivero]

[FIX] Modificacion para que el boton 'Cambiar numero de control'
no aparezca cuando la factura ya se encuentre pagada.
[FIX] Colocado como requerido los campos de la factura refencia
y fecha del documento.
[FIX] Se hace invisible el boton 'Pagar factura' para todas las
facturas.
[FIX] Modificado para que no sea pueda tildar 'Expediente' y
'Sin Doc. Iva' una vez la factura a cambiado su estado de draft
[FIX] Se hace invisible el boton 'Factura Rectificativa'
para las facturas rectificativas.
[FIX] Validado que las facturas no puedan agregarsele lineas
con precio de unidad con valor 0 o menor.
[FIX] Validado que no se puedan agregar lineas a la factura
sin impuesto.
[FIX] Modificado funcion que genera facturas rectificativas
para generar estas con check_total en 0.

[Aristóbulo Meneses (aristobulo)]

[TODO] Planificar crear un módulo nuevo para parches a la OVL con este tipo de validaciones.

[lp:~inddiana/sisb/edgar_bug_facturas]

191. By [SISB] Edgar Rivero on 2012-06-13

[FIX] Se sobre escriben las vistas que agregan el parent_id, para agregar
un domain para que no se puedan seleccionar una factura rectificativa a
partir de otra factura rectificativa y tampoco aparezca en la vista de
facturas al proveedor/cliente.

=== modified file 'diana_fixes/__openerp__.py'
--- diana_fixes/__openerp__.py	2012-01-11 15:05:33 +0000
+++ diana_fixes/__openerp__.py	2012-06-13 18:01:24 +0000
@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 {
     'name' : 'Diana FIXES',
-    'version' : '0.1',
+    'version' : '0.2',
     'author' : 'Industrias Diana, C.A. <[e-mail]>',
     'description' : """
                     Conjunto de sobreescrituras para corregir
@@ -10,9 +10,15 @@
     'category' : 'other',
     'website' : 'www.industriasdiana.gob.ve',
     'depends' : [
-        'base_vat'
+        'base_vat',
+        'account',
+        'account_voucher',
+        'l10n_ve_fiscal_requirements',
+        'l10n_ve_fiscal_reports',
+        'l10n_ve_withholding_iva',
         ],
     'update_xml' : [
+        'view/factura_fixes.xml',
         ],
     'active' : False,
     'installable' : True,

=== modified file 'diana_fixes/model/fixes.py'
--- diana_fixes/model/fixes.py	2012-01-11 15:47:40 +0000
+++ diana_fixes/model/fixes.py	2012-06-13 18:01:24 +0000
@@ -1,5 +1,6 @@
 # -*- coding: utf-8 -*-
 from osv import osv, fields
+import decimal_precision as dp
 import netsvc
 import pooler
 
@@ -29,3 +30,80 @@
     }
 
 account_aged_trial_balance()
+
+class account_invoice_line(osv.osv):
+    _inherit='account.invoice.line'
+
+    def write(self, cr, uid, ids, vals, context=None):
+        """Sobre escribir ORM write para validar que la factura no se le agreguen
+        lineas con precio unitario menor a 0, y que se hayan agregado impuestos"""
+        if context is None:
+            context = {}
+        
+        if vals.get('price_unit', 1) <= 0.0:
+            raise osv.except_osv('Error con Precio Unidad',"El precio unidad debe tener un valor mayor a 0")
+                
+        if vals.get('invoice_line_tax_id', False):
+            if vals['invoice_line_tax_id'][0][2] == []:
+                raise osv.except_osv('Error con Impuestos',"Se debe asignar una cuenta de impuesto a la línea, asi se encuentre exenta")
+        return super(account_invoice_line, self).write(cr, uid, ids, vals, context=context)
+    
+    def create(self, cr, uid, vals, context=None):
+        """Sobre escribir ORM create para validar que la factura no se le agreguen
+        lineas con precio unitario menor a 0, y que se hayan agregado impuestos"""
+        if context is None:
+            context = {}
+            
+        if float(vals['price_unit']) <= 0.0:
+            raise osv.except_osv('Error con Precio Unidad',"El precio unidad debe tener un valor mayor a 0")
+            
+        if vals['invoice_line_tax_id'][0][2] == []:
+            raise osv.except_osv('Error con Impuestos',"Se debe asignar una cuenta de impuesto a la línea, asi se encuentre exenta")
+        return super(account_invoice_line, self).create(cr, uid, vals, context=context)
+
+account_invoice_line()
+
+class account_invoice_refund(osv.osv_memory):
+
+    """Refunds invoice"""
+    
+    """Para sobre escribir el wizard de facturas rectificativas y poder usar 
+    siempre por defecto 'Facturas rectificativas'.
+    
+    Tambien se hereda para colocar en 0.0 el Total al generar una factura
+    rectificativa."""
+
+    _inherit = "account.invoice.refund"
+    _description = "Sobreescribir Factura Rectificativas"
+    
+    _defaults = {
+        'filter_refund': 'refund',
+    }
+
+    def compute_refund(self, cr, uid, ids, mode='refund', context=None):
+        """
+        Super de compute_refund para colocar las facturas rectificativas con el
+        monto check_total en 0
+        """
+        result = super(account_invoice_refund, self).compute_refund(cr, uid, ids, mode, context)
+        inv_obj = self.pool.get('account.invoice')
+        if result.get('domain', False):
+            for res in result['domain']:
+                if res[0] == 'id':
+                    inv_obj.write(cr, uid, res[2], {'check_total':0.0}, context)
+                    
+        return result
+
+account_invoice_refund()
+
+class account_invoice(osv.osv):
+    """Se hereda account.invoice para modificar el campo wh_iva_rate, creado 
+    por l10n_ve_withholding_iva y colocarlo siempre en readonly, esto porque 
+    ese campo se debe llenar por defecto segun el proveedor/cliente"""
+    _inherit = "account.invoice"
+    
+    _columns = {
+        'wh_iva_rate': fields.float('Wh rate', digits_compute= dp.get_precision('Withhold'), readonly=True, help="Withholding vat rate"),
+    }
+
+account_invoice()

=== added directory 'diana_fixes/view'
=== added file 'diana_fixes/view/factura_fixes.xml'
--- diana_fixes/view/factura_fixes.xml	1970-01-01 00:00:00 +0000
+++ diana_fixes/view/factura_fixes.xml	2012-06-13 18:01:24 +0000
@@ -0,0 +1,223 @@
+<?xml version="1.0" encoding="utf-8"?>
+<openerp>
+    <data>
+
+<!--
+        En facturas al proveedor se hace requerido nro_ctrl (Numero de control)
+        y este solo puedo ser modificado cuando la factura este en borrador.
+        Tambien se modifica el boton 'Cambiar numero de control' para solo se
+        pueda usar cuando la factura este en borrador o abierto.
+        Y se hace requerido y que solo modificable en borrador o abierto, el 
+        campo reference, que es usado para la 'Referencia libre'.
+-->
+        <record id="l10n_ve_fiscal_requirements.account_invoice_nro_control_supplier_form" model="ir.ui.view">
+            <field name="name">account.invoice.nro.control.supplier.form</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_supplier_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/notebook/page/field[@name='reference']" position="replace">
+                        <field name='reference' nolabel="1" required='True' attrs="{'readonly':[('state','not in',('open','draft'))]}"/>
+                </xpath>
+                <xpath expr="/form/notebook/page/field[@name='check_total']" position="after">
+                    <group colspan="2">
+                        <field name="nro_ctrl" required="1" attrs="{'readonly':[('state','&lt;&gt;','draft')]}"/>
+                        <button name="%(l10n_ve_fiscal_requirements.action_wiz_nroctrl)d" type='action' string='Change control number' icon="gtk-execute" states='draft,open'/>
+                    </group>
+                </xpath>
+            </field>
+        </record>
+        
+<!--
+        En facturas al cliente se hace requerido nro_ctrl (Numero de control)
+        y este solo puedo ser modificado cuando la factura este en borrador.
+        Tambien se modifica el boton 'Cambiar numero de control' para solo se
+        pueda usar cuando la factura este en borrador o abierto.
+-->
+        <record id="l10n_ve_fiscal_requirements.account_invoice_nro_control_customer_form" model="ir.ui.view">
+            <field name="name">account.invoice.nro.control.customer.form</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/notebook/page/field[@name='payment_term']" position="after">
+                    <group colspan="2" >
+                        <field  name="nro_ctrl" required="1" attrs="{'readonly':[('state','&lt;&gt;','draft')]}"/>
+                        <button name="%(l10n_ve_fiscal_requirements.action_wiz_nroctrl)d" type='action' string='Change control number' icon="gtk-execute" states='draft,open'/>
+                    </group>
+                </xpath>
+            </field>
+        </record>
+
+<!--
+        Se modifica en facturas a clientes, date_document (Fecha del Documento) 
+        para que sea modificado solo cuando este se encuentre en borrador o 
+        abierto y es requerido.
+-->
+        <record model="ir.ui.view" id="l10n_ve_fiscal_reports.inherited_account_invoice_form_fiscal">
+            <field name="name">account.invoice.inherit.fiscal</field>
+            <field name="model">account.invoice</field>
+            <field name="inherit_id" ref="account.invoice_form"/>
+            <field name="arch" type="xml">
+                <data>
+                    <field name="date_invoice" position="after">
+                        <field name="date_document" required="True" attrs="{'readonly':[('state','not in',('open','draft'))]}"/>
+                    </field>
+                </data>
+            </field>
+        </record>
+        
+<!--
+        Se modifica el campo boleano expedence (Expediente) para que solo
+        pueda ser modificado cuando la factura este en borrador o abierto.
+        Tambien se modifica date_document (Fecha del Documento) para que sea
+        modificado solo cuando este se encuentre en borrador o abierto y es 
+        requerido en las facturas al proveedor.
+-->
+        <record model="ir.ui.view" id="l10n_ve_fiscal_reports.inherited_account_invoice_form_fiscal2">
+            <field name="name">account.invoice.inherit.fiscal2</field>
+            <field name="model">account.invoice</field>
+            <field name="inherit_id" ref="account.invoice_supplier_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/group/group" position="after">
+                    <field name="expedient" attrs="{'readonly':['|',('type','in',['out_invoice','out_refund']),('state','&lt;&gt;','draft')]}"/>
+                </xpath>
+                <xpath expr='//field[@name="date_due"]' position="after">
+                    <field name="date_document" required="True" attrs="{'readonly':[('state','not in',('open','draft'))]}"/>
+                </xpath>
+            </field>
+        </record>
+        
+
+<!--
+        Se coloca invisible el boton de 'Pagar factura', para facturas del
+        cliente. Esto se hace porque el boton no es necesario.
+-->
+        <record id="account_voucher.view_invoice_customer" model="ir.ui.view">
+            <field name="name">account.invoice.customer.pay</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_form"/>
+            <field name="arch" type="xml">
+                <button name="invoice_open" position="after">
+                    <button name="invoice_pay_customer" type="object" string="Payment" states="open" icon="gtk-go-forward" invisible='True'/>
+                </button>
+            </field>
+        </record>
+
+<!--
+        Se coloca invisible el boton de 'Pagar factura', para facturas del
+        proveedor. Esto se hace porque el boton no es necesario.
+-->
+        <record id="account_voucher.view_invoice_supplier" model="ir.ui.view">
+            <field name="name">account.invoice.supplier.pay</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_supplier_form"/>
+            <field name="arch" type="xml">
+                <button name="invoice_open" position="after">
+                    <button name="invoice_pay_customer" type="object" string="Pay Invoice" states="open" icon="gtk-go-forward" invisible='True'/>
+                </button>
+            </field>
+        </record>
+        
+<!--
+        Se modifica el campo boleano vat_apply (Sin Doc. Iva) para que solo
+        pueda ser modificado cuando la factura este en borrador.
+-->
+        <record id="account_invoice_wh_iva_supplier_inherit" model="ir.ui.view">
+            <field name="name">account.invoice.wh.iva.supplier.inherit</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="l10n_ve_withholding_iva.account_invoice_wh_iva_supplier"/>
+            <field name="arch" type="xml">
+                <xpath expr="//field[@name='vat_apply']" position="replace">
+                    <field name="vat_apply" attrs="{'readonly':[('state','&lt;&gt;','draft')]}"/>
+                </xpath>
+            </field>
+        </record>
+        
+<!--
+        Se coloca invisible el boton de 'Factura Rectificativa', para facturas del
+        proveedor, cuando la factura sea una factura rectificativa.
+-->
+        <record id="l10n_ve_fiscal_requirements.account_invoice_refund_supplier_form" model="ir.ui.view">
+            <field name="name">account.invoice.refund.supplier.form</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_supplier_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/notebook/page/group/group/button[@string='Refund']" position="replace">
+                    <button name="%(account.action_account_invoice_refund)d" type='action' string='Refund' icon="gtk-execute" attrs="{'invisible':['|',('type','in',('out_refund','in_refund')),('state','not in',('open','paid'))]}"/>
+                </xpath>
+            </field>
+        </record>
+        
+<!--
+        Se coloca invisible el boton de 'Factura Rectificativa', para facturas del
+        cliente, cuando la factura sea una factura rectificativa.
+-->
+        <record id="l10n_ve_fiscal_requirements.account_invoice_refund_customer_form" model="ir.ui.view">
+            <field name="name">account.invoice.refund.customer.form</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/notebook/page/group/group/button[@string='Refund']" position="replace">
+                    <button name="%(account.action_account_invoice_refund)d" type='action' string='Refund' icon="gtk-execute" attrs="{'invisible':['|',('type','in',('out_refund','in_refund')),('state','not in',('open','paid'))]}"/>
+                </xpath>
+            </field>
+        </record>
+
+<!--
+        Se sobre escribe la vista para agregar un domain a parent_id para que no 
+        se puedan seleccionar una factura rectificativa a partir de otra factura
+        rectificativa y tampoco aparezca en la vista de facturas al cliente.
+        Para las facturas al cliente.
+-->
+        <record id="l10n_ve_fiscal_requirements.account_inv_ext_form_crdrc" model="ir.ui.view">
+            <field name="name">invoice.ext.form.crdr.customer</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/group/group" position="after">
+                    <newline/>
+                    <field name="parent_id" attrs="{'required':[('type','=','out_refund')], 'invisible':[('type','!=','out_refund')]}" domain="[('partner_id','=',partner_id),('id','!=',active_id),('type','in',['out_invoice'])]"/>
+                </xpath>
+                <xpath expr="//button[@string='Refund']" position="after">
+                    <button name="%(l10n_ve_fiscal_requirements.action_account_invoice_debit)d" type='action' string='Debit Note' icon="gtk-execute" attrs="{'invisible':['|',('state','in',['proforma2','proforma','cancel', 'draft']),('type','in',['out_refund', 'in_refund'])]}"/>
+                </xpath>
+                <xpath expr="//button[@string='Refund']" position="after">
+                    <button name="%(l10n_ve_fiscal_requirements.action_account_invoice_parent)d" type='action' string='Parent Invoice' states='open,paid' icon="gtk-execute" attrs="{'invisible':[('type','!=','out_invoice')]}"/>
+                </xpath>
+            </field>
+        </record>
+        
+<!--
+        Se sobre escribe la vista para agregar un domain a parent_id para que no 
+        se puedan seleccionar una factura rectificativa a partir de otra factura
+        rectificativa y tampoco aparezca en la vista de facturas al proveedor.
+        Para las facturas al proveedor.
+-->
+        <record id="l10n_ve_fiscal_requirements.account_inv_ext_form_crdrs" model="ir.ui.view">
+            <field name="name">invoice.ext.form.crdr.supplier</field>
+            <field name="model">account.invoice</field>
+            <field name="type">form</field>
+            <field name="inherit_id" ref="account.invoice_supplier_form"/>
+            <field name="arch" type="xml">
+                <xpath expr="/form/group/group" position="after">
+                    <newline/>
+                    <field name="parent_id" attrs="{'required':[('type','=','in_refund')], 'invisible':[('type','!=','in_refund')]}" domain="[('partner_id','=',partner_id),('id','!=',active_id),('type','in',['in_invoice'])]"/>
+                </xpath>
+                <xpath expr="//button[@string='Refund']" position="after">
+                    <button name="%(l10n_ve_fiscal_requirements.action_account_invoice_debit)d" type='action' string='Debit Note' icon="gtk-execute" attrs="{'invisible':['|',('state','in',['proforma2','proforma','cancel', 'draft']),('type','in',['out_refund', 'in_refund'])]}"/>
+                </xpath>
+                <xpath expr="//button[@string='Refund']" position="after">
+                    <button name="%(l10n_ve_fiscal_requirements.action_account_invoice_parent)d" type='action' string='Parent Invoice' states='open,paid' icon="gtk-execute" attrs="{'invisible':[('type','!=','in_refund')]}"/>
+                </xpath>
+            </field>
+        </record>
+
+    </data>
+</openerp>

=== modified file 'sisb_account_invoice_wizard/account_invoice.xml'
--- sisb_account_invoice_wizard/account_invoice.xml	2012-01-25 16:20:51 +0000
+++ sisb_account_invoice_wizard/account_invoice.xml	2012-06-13 18:01:24 +0000
@@ -13,6 +13,6 @@
                     </group>
                 </xpath>
             </field>
-	</record>
+        </record>
     </data>
 </openerp>