Hello again.

     I still cant get it to work, I have tried differen ways but dont know how to distinguish between old api and new api.

This code gets me:

ValueError: dictionary update sequence element #0 has length 1; 2 is required

The code as simple as I could:

'make_invisible':fields.function(get_user, string="User Visible", readonly=0)

@api.multi  
def get_user(self ):  
    _logger.debug("This is GET_USER method ")  
    self.make_invisible=self.env['res.users'].has_group('hr_employee.GM')

The objective of this method is to define is logged user belongs to a certain group, because then I will use make_invisible flag to hide some fields to a certain group. NOTE: making the fields visible to certains groups is no option because it is a small group the one who should not see these fields.

@api.multi :- Self will be the current RecordSet without iteration

@api.depends:- This decorator will trigger the call to the decorated function if any of the fields specified in the decorator is altered by ORM or changed in the form.

reference

You are using old API for defining the computed field. seems to be wrong. look here

from openerp import api
total = fields.Float(compute='_compute_total')

@api.depends('value', 'tax')
def _compute_total(self):
    for record in self:
        record.total = record.value + record.value * record.tax

This is the new guideline to make a computed field.

reference