Problema añadir campos a one2many en account_invoice_layout

[pablogom...@gmail.com]

Buenas a todos, 

estoy intentando agregar en la pantalla que se muestra al seleccionar un cliente en el menú de contabilidad/facturas de cliente (donde se muestran las lineas de la factura), 3 campos. Para ello he heredado la clase account_invoice_line y le he añadido los tres campos (son de tipo funcional).

Pues bien, cuando heredo la vista , si intento mostrar los campos que quiero de esta forma:

<xpath expr="/form/notebook/page/field[@name='abstract_line_ids']" position="replace">

                    <field name="abstract_line_ids" colspan="4" nolabel="1">

<tree>

   <field name= "sequence"/>

   <field name= "name"/>

   <field name= "nserie"/>

   <field name= "nticket"/>

   <field name= "nguia"/>

   <field name= "origin"/>

   <field name= "quantity"/>

   <field name= "uos_id"/>

   <field name= "price_unit"/>

   <field name= "discount"/>

   <field name= "price_subtotal"/> 

   

</tree>

   </field>

</xpath>     

Y si, únicamente pongo el campo one2many sin especificar los campos, los campos míos no los muestra.

Los campos en azul son los que intento añadir. el problema de todo es que en account_invoice_layout hay funciones que hacen cosas con los campos en función de determinados valores. Si solo muestro los valores sequence, nserie,nguia, nticket, y origin, no me da error. Pero en el momento que trato de agregar cualquiera de los otros me salta el error: Uncaught Error: Campo desconocido state en el dominio [["state","in",["break","line"]]] 

El campo one2many al que quiero añadirle los tres campos (que he creado al heredar la clase account_invoice_line) es el llamado abstract_line ids (que se ecnuentra en la clase account_invoice, heredada por el módulo account_invoice_layout). El código correspondiente a a account_invoice layout es el siguiente.

class account_invoice_line(osv.osv):

    def move_line_get_item(self, cr, uid, line, context=None):

        if line.state != 'article':

            return None

        return super(account_invoice_line, self).move_line_get_item(cr, uid, line, context)

    def fields_get(self, cr, uid, fields=None, context=None):

        article = {

            'article': [('readonly', False), ('invisible', False)],

            'text': [('readonly', True), ('invisible', True), ('required', False)],

            'subtotal': [('readonly', True), ('invisible', True), ('required', False)],

            'title': [('readonly', True), ('invisible', True), ('required', False)],

            'break': [('readonly', True), ('invisible', True), ('required', False)],

            'line': [('readonly', True), ('invisible', True), ('required', False)],

        }

        states = {

            'name': {

                'break': [('readonly', True),('required', False),('invisible', True)],

                'line': [('readonly', True),('required', False),('invisible', True)],

                },

            'product_id': article,

            'account_id': article,

            'quantity': article,

            'uos_id': article,

            'price_unit': article,

            'discount': article,

            'invoice_line_tax_id': article,

            'account_analytic_id': article,

        }

        res = super(account_invoice_line, self).fields_get(cr, uid, fields, context)

        for field in res:

            if states.has_key(field):

                for key,value in states[field].items():

                    res[field].setdefault('states',{})

                    res[field]['states'][key] = value

        return res

    def onchange_invoice_line_view(self, cr, uid, id, type, context=None, *args):

        if (not type):

            return {}

        if type != 'article':

            temp = {'value': {

                    'product_id': False,

                    'uos_id': False,

                    'account_id': False,

                    'price_unit': False,

                    'price_subtotal': False,

                    'quantity': 0,

                    'discount': False,

                    'invoice_line_tax_id': False,

                    'account_analytic_id': False,

                    },

                }

            if type == 'line':

                temp['value']['name'] = ' '

            if type == 'break':

                temp['value']['name'] = ' '

            if type == 'subtotal':

                temp['value']['name'] = 'Sub Total'

            return temp

        return {}

    def create(self, cr, user, vals, context=None):

        if vals.has_key('state'):

            if vals['state'] == 'line':

                vals['name'] = ' '

            if vals['state'] == 'break':

                vals['name'] = ' '

            if vals['state'] != 'article':

                vals['quantity']= 0

                vals['account_id']= self._default_account(cr, user, None)

        return super(account_invoice_line, self).create(cr, user, vals, context)

    def write(self, cr, user, ids, vals, context=None):

        if vals.has_key('state'):

            if vals['state'] != 'article':

                vals['product_id']= False

                vals['uos_id']= False

                vals['account_id']= self._default_account(cr, user, None)

                vals['price_unit']= False

                vals['price_subtotal']= False

                vals['quantity']= 0

                vals['discount']= False

                vals['invoice_line_tax_id']= False

                vals['account_analytic_id']= False

            if vals['state'] == 'line':

                vals['name'] = ' '

            if vals['state'] == 'break':

                vals['name'] = ' '

        return super(account_invoice_line, self).write(cr, user, ids, vals, context)

    def copy_data(self, cr, uid, id, default=None, context=None):

        if default is None:

            default = {}

        default['state'] = self.browse(cr, uid, id, context=context).state

        return super(account_invoice_line, self).copy_data(cr, uid, id, default, context)

    def _fnct(self, cr, uid, ids, name, args, context=None):

        res = {}

        lines = self.browse(cr, uid, ids, context=context)

        account_ids = [line.account_id.id for line in lines]

        account_names = dict(self.pool.get('account.account').name_get(cr, uid, account_ids, context=context))

        for line in lines:

            if line.state != 'article':

                if line.state == 'line':

                    res[line.id] = '-----------------------------------------'

                elif line.state == 'break':

                    res[line.id] = 'PAGE BREAK'

                else:

                    res[line.id] = ' '

            else:

                res[line.id] = account_names.get(line.account_id.id, '')

        return res

    _name = "account.invoice.line"

    _order = "invoice_id, sequence asc"

    _description = "Invoice Line"

    _inherit = "account.invoice.line"

    _columns = {

        'state': fields.selection([

                ('article','Product'),

                ('title','Title'),

                ('text','Note'),

                ('subtotal','Sub Total'),

                ('line','Separator Line'),

                ('break','Page Break'),]

            ,'Type', select=True, required=True),

        'sequence': fields.integer('Sequence Number', select=True, help="Gives the sequence order when displaying a list of invoice lines."),

        'functional_field': fields.function(_fnct, arg=None, fnct_inv=None, fnct_inv_arg=None, type='char', fnct_search=None, obj=None, store=False, string="Source Account"),

    }

    def _default_account(self, cr, uid, context=None):

        cr.execute("select id from account_account where parent_id IS NULL LIMIT 1")

        res = cr.fetchone()

        return res[0]

    _defaults = {

        'state': 'article',

        'sequence': 0,

    }

account_invoice_line()

class one2many_mod2(fields.one2many):

    def get(self, cr, obj, ids, name, user=None, offset=0, context=None, values=None):

        if context is None:

            context = {}

        if not values:

            values = {}

        res = {}

        for id in ids:

            res[id] = []

        ids2 = obj.pool.get(self._obj).search(cr, user, [(self._fields_id,'in',ids),('state','=','article')], limit=self._limit)

        for r in obj.pool.get(self._obj)._read_flat(cr, user, ids2, [self._fields_id], context=context, load='_classic_write'):

            res[r[self._fields_id]].append( r['id'] )

        return res

class account_invoice(osv.osv):

    def copy(self, cr, uid, id, default=None, context=None):

        if default is None:

            default = {}

        default['invoice_line'] = False

        return super(account_invoice, self).copy(cr, uid, id, default, context)

    _inherit = "account.invoice"

    _columns = {

        'abstract_line_ids': fields.one2many('account.invoice.line', 'invoice_id', 'Invoice Lines',readonly=True, states={'draft':[('readonly',False)]}),

        'invoice_line': one2many_mod2('account.invoice.line', 'invoice_id', 'Invoice Lines',readonly=True, states={'draft':[('readonly',False)]}),

    }

account_invoice()

Muchas gracias por vuestro tiempo!

 

[Pedro Manuel Baeza Romero]

Buenas, Pablo,

La verdad es que el account_invoice_layout es un módulo que está programado bastante mal, y de hecho, ha sido discontinuado en la última versión, Tratar de ver el problema se hace bastante difícil, pero sí que te recomendaría que probaras a poner otro xpath distinto en lugar de uno que reemplace entero el abstract_line_ids. Con uno que referencie a name y de position pongas after bastaría.

Te envío el enlace a una excelente explicación de cómo funciona xpath por Roberto Lizana:

https://groups.google.com/forum/#!msg/openerp-spain-users/b6_f94e4DzQ/BfoD4V8URFAJ

Un saludo.

--
Has recibido este mensaje porque estás suscrito al grupo "Usuarios OpenERP en España" de Grupos de Google.
Para anular la suscripción a este grupo y dejar de recibir sus correos electrónicos, envía un correo electrónico a openerp-spain-u...@googlegroups.com.
Para obtener más opciones, visita https://groups.google.com/groups/opt_out.

[pablogom...@gmail.com]

Si, al final desinstalé el módulo y sin problemas. Gracias igualmente Pedro.