Odoo Help


This community is for beginners and experts willing to share their Odoo knowledge. It's not a forum to discuss ideas, but a knowledge base of questions and their answers.


How to get attribute from another class using the browse method in OpenERP ?

on 7/16/14, 6:54 AM 4,776 views

I've installed the oemedical addon in my OperERP 7. I'm adding some modifications on it.
I have a problem with the function _get_ssn_from_name that I have created in the appointment module, which gets the SSN from the oemedical.patient class to display it in oemedical.appointment view.

Here is the code of the class oemedical.appointment containing the function I've developed

    class OeMedicalAppointment(osv.Model):
        _name = 'oemedical.appointment'
        # The function causing the problem
        def _get_ssn_from_name(self, cr, uid, ids, field_name, arg, context=None):
            res = {}
            ssn = ''
            for record in self.pool.get('oemedical.patient').browse(cr, uid, ids, context=context):
                # When patient is not yet specified
                if not record.lastname:
                    ssn = 'No Patient Specified!'
                # Return the ssn when the caller is the field name
                if field_name == 'alias':
                    if record.ssn:
                        ssn = record.ssn
                        ssn = "Not Specified"
                    print("ssn : %s" % ssn)
                res[record.id] = ssn
            return res
        _columns = {
            'patient_id': fields.many2one('oemedical.patient', string='Patient',
                                       required=True, select=True,
                                       help='Patient Name'),
            'name': fields.char(size=256, string='Appointment ID', readonly=True),
            'appointment_date': fields.datetime(string='Date and Time'),
            'appointment_day': fields.date(string='Date'),
            # some other attributes
            'alias': fields.function(_get_ssn_from_name, type='char', string='SSN', help="Social Security Number", multi=False),
            # other attributes

And here is the code of the class oemedical.patient

    class OeMedicalPatient(osv.Model):
            'res.partner': 'partner_id',
            'partner_id': fields.many2one(
                'res.partner', 'Related Partner', required=True,
                ondelete='cascade', help='Partner-related data of the patient'),
            'first_name': fields.char(size=256, string='Name', required=True),
            'lastname': fields.char(size=256, string='Lastname', required=True),
            # many other attributes..
            'ssn': fields.char(size=256, string='SSN'),
            # some other attributes..
            'deceased': fields.boolean(string='Deceased'),


Here is the interface of creation of a new appointment : http://i.stack.imgur.com/lX9CE.png

When I create a new patient on the Appointment creation form (the field where I can choose the patient in the screenshot), I doesnt't get an error and everything goes well, but when I try to create an appointment for an existing patient I get the following error :

    AttributeError: 'Field lastname not found in browse_record(oemedical.patient, 8)'

Please help me, and thanks in advance!!


René Schuster

--René Schuster--
| 5 5 8
Weinheim, Germany
--René Schuster--

Challenge Everything!

René Schuster
On 7/16/14, 7:18 AM

First of all: The way I would do it (with the tradeoff that the field is empty, if no ssn is given).

Instead of the functional field use a relational field:

'alias': fields.related('patient_id', 'ssn', type='char', string='SSN', store=False)


Now to your code:

The passed ids in the function parameter list are the ids of the records that need to have the field values calculated. In your case, ids contains one or more ids of oemedical.appointment. You shouldn't really use those ids to browse your patients!

def _get_ssn_from_name(self, cr, uid, ids, field_name, arg, context=None):
    res = {};
    for obj in self.browse(cr, uid, ids, context=context):
        if obj.patient_id.ssn:
            res[obj.id] = obj.patient_id.ssn;
            res[obj.id] = 'Not Specified';
        # as alternative to the if-clause one could write: res[obj.id] = obj.patient_id.ssn or 'Not Specified';
    return res;

Also, you should think about renaming the function to: _get_ssn(..)

(EDIT: fixed answer)

Hope this helps.

On 7/16/14, 8:42 AM

Thank you your help. But I'm still getting errors for both suggestions that you have provided.


Here is what I get when trying to use relational field and ommitting the _get_ssn(...) function :

raise KeyError(error_msg)
KeyError: "Field 'oemedical.patient' does not exist in object 'browse_record(oemedical.appointment, 17)'"


And here is what I get by changing the code of the function as you specified :

2014-07-16 12:32:09,993 7597 ERROR demo openerp.netsvc: 15
Traceback (most recent call last):
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/netsvc.py", line 296, in dispatch_rpc
    result = ExportService.getService(service_name).dispatch(method, params)
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/service/web_services.py", line 626, in dispatch
    res = fn(db, uid, *params)
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/osv/osv.py", line 190, in execute_kw
    return self.execute(db, uid, obj, method, *args, **kw or {})
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/osv/osv.py", line 132, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/osv/osv.py", line 199, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/osv/osv.py", line 187, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/osv/orm.py", line 3679, in read
    result = self._read_flat(cr, user, select, fields, context, load)
  File "/home/oussama/PycharmProjects/openerp-7.0/openerp/osv/orm.py", line 3803, in _read_flat
    record[f] = res2[record['id']]
KeyError: 15

EDIT: The two solutions worked well after making the modifications you provided :)
Thank you again !

Sorry.. Minor error in the first solution. Edited my answer. Don't know yet what the issue with the other funtion is...

René Schuster
on 7/16/14, 9:03 AM

For the second solution: The res-keywords are wrong. Edited my answer again.

René Schuster
on 7/16/14, 9:09 AM

Both of the solutions that you provided worked well :) Thanks !

on 7/16/14, 9:15 AM

Your Answer

Please try to give a substantial answer. If you wanted to comment on the question or answer, just use the commenting tool. Please remember that you can always revise your answers - no need to answer the same question twice. Also, please don't forget to vote - it really helps to select the best questions and answers!

About This Community

This community is for professionals and enthusiasts of our products and services. Read Guidelines

Question tools

4 follower(s)


Asked: 7/16/14, 6:54 AM
Seen: 4776 times
Last updated: 3/16/15, 8:10 AM