Function python for odoo 10

Question

I have a function for filling in many fields automatically with an api.onchange() decorator.

@api.onchange('nursery_morning', 'nursery_evening', 'responsible_partner')
def retrieve_responsible_nursery(self):
    if self.nursery_morning or self.nursery_evening:
        if self.responsible_partner:
            if not self.resp_civility1 and not self.resp_name1 and not self.resp_cp1 and not self.resp_num1 \
                    and not self.resp_address1 and not self.resp_town1 \
                    and not self.resp_phone1 and not self.resp_phonemobile1:
                self.resp_civility1 = self.responsible_partner.title.name
                self.resp_name1 = self.responsible_partner.name
                self.resp_cp1 = self.responsible_partner.zip_id.name
                self.resp_num1 = self.responsible_partner.street_number_id.name
                self.resp_address1 = self.responsible_partner.street_id.name
                self.resp_town1 = self.responsible_partner.city_id.name
                self.resp_phone1 = self.responsible_partner.phone
                self.resp_phonemobile1 = self.responsible_partner.mobile

This function works, but I do not want the fields to change until the fields are saved in the database and not before.

Currently, the fields do not change once one of the fields listed are filled but not saved in database

Answer 1

Two ways you can approach this, it seems that you could change these fields to be related fields if they are only ever going to hold the values from a related record (responsible_partner).

resp_civility1 = fields.Char("Field Label", related="responsible_partner.title.name")

Editing this field will edit the linked record as well, so it will not be a good fit if you want to change this value at all, or you could set it as read-only if you don't want it to be edited from here.

Another way to do it would be to override the write method, this will write the values at the time of saving the record and would look like this:

@api.multi
def write(self, vals):
    res = super(ModelName, self).write(vals)

    if vals.get('nursery_morning') or vals.get('nursery_evening') or vals.get('responsible_partner'):
        if res.nursery_morning or res.nursery_evening:
            if res.responsible_partner:
                if not res.resp_civility1 and not res.resp_name1 and not res.resp_cp1 and not res.resp_num1 \
                        and not res.resp_address1 and not res.resp_town1 \
                        and not res.resp_phone1 and not res.resp_phonemobile1:
                    res.resp_civility1 = res.responsible_partner.title.name
                    res.resp_name1 = res.responsible_partner.name
                    res.resp_cp1 = res.responsible_partner.zip_id.name
                    res.resp_num1 = res.responsible_partner.street_number_id.name
                    res.resp_address1 = res.responsible_partner.street_id.name
                    res.resp_town1 = res.responsible_partner.city_id.name
                    res.resp_phone1 = res.responsible_partner.phone
                    res.resp_phonemobile1 = res.responsible_partner.mobile
    return res

This will first call the write method of the model, then it will check if the three fields are being written to, if so it will run the same logic from your onchange method. Replace ModelName with whatever you have named your model class.