diff --git a/odex25_ensan/odex_benefit/models/visit.py b/odex25_ensan/odex_benefit/models/visit.py
index 070270359..488d01621 100644
--- a/odex25_ensan/odex_benefit/models/visit.py
+++ b/odex25_ensan/odex_benefit/models/visit.py
@@ -159,8 +159,8 @@ class Visit(models.Model):
                             _("The email template 'Visit Location OTP Email' is missing. Please contact your administrator."))
                     template.write({'email_to': self.benefit_id.email,
                                     'email_cc': self.env.user.company_id.hr_email or self.env.user.company_id.email,})
-                    email_values = {"email_from": self.env.user.company_id.hr_email or self.env.user.company_id.email}
-                    template.with_context(lang=self.env.user.lang,tracking_disable=True).send_mail(self.id, force_send=True,
+                    email_values = {"email_from": self.env.user.company_id.hr_email or self.env.user.company_id.email,'res_id': None}
+                    template.with_context({'lang':self.env.user.lang,'tracking_disable':True,'mail_create_nolog': True,'mail_notrack': True,}).send_mail(self.id, force_send=True,
                                                                             raise_exception=False,email_values=email_values)
                 elif self.benefit_id.contact_type == 'sms':
                     if not self.benefit_id.sms_phone:
diff --git a/odex25_ensan/odex_benefit/security/ir.model.access.csv b/odex25_ensan/odex_benefit/security/ir.model.access.csv
index 4423d8913..878cb04af 100644
--- a/odex25_ensan/odex_benefit/security/ir.model.access.csv
+++ b/odex25_ensan/odex_benefit/security/ir.model.access.csv
@@ -150,8 +150,6 @@ access_service_refuse_reason,access_service_refuse_reason,model_service_refuse_r
 access_service_refuse_reason_wizard,access_service_refuse_reason_wizard,model_service_refuse_reason_wizard,base.group_user,1,1,1,1
 access_reason_for_return_wizard,access_reason_for_return_wizard,model_reason_for_return_wizard,base.group_user,1,1,1,1
 access_transportation_insurance,access_transportation_insurance,model_transportation_insurance,base.group_user,1,1,1,1
-access_res_city,access_res_city,model_res_city,base.group_user,1,0,0,0
-access_res_city,access_res_city,model_res_city,odex_benefit.group_benefit_manager,1,1,1,1
 access_job_settings,access_job_settings,model_job_settings,base.group_user,1,0,0,0
 access_job_settings_manager,access_job_settings.manager,model_job_settings,odex_benefit.group_benefit_manager,1,1,1,1
 access_death_reason_settings,access_death_reason_settings,model_death_reason_settings,base.group_user,1,0,0,0
diff --git a/odex25_ensan/odex_benefit/views/benefit_view.xml b/odex25_ensan/odex_benefit/views/benefit_view.xml
index e9e6776d9..772c3f0c1 100644
--- a/odex25_ensan/odex_benefit/views/benefit_view.xml
+++ b/odex25_ensan/odex_benefit/views/benefit_view.xml
@@ -305,6 +305,9 @@
                                        readonly="1"
                                        options="{'no_create': True, 'no_create_edit': True}"
                                        groups="!odex_benefit.group_benefit_manager"/>
+                                <field name="member_income" widget="monetary"
+                                       options="{'currency_field': 'currency_id'}" readonly="1" force_save="1"/>
+                                <field name="benefit_category_id" readonly="1" force_save="1"/>
                                 <field name="company_id" groups="base.group_multi_company"/>
                                 <field name="currency_id" groups="base.group_multi_currency"/>
                                 <field name="mother_family_member_id" invisible="1" readonly="1"/>
@@ -381,9 +384,12 @@
                                                options="{'no_create': True, 'no_create_edit': True,'no_quick_create': True, 'no_open': True}"
                                                groups="!odex_benefit.group_benefit_manager"/>
                                         <field name="father_dead_city_id" invisible="1"/>
-                                        <field name="father_dead_res_city_id" attrs="{'invisible': [('father_dead_country_id_code', '!=', 'SA')]}" options="{'no_create': True, 'no_create_edit': True,'no_quick_create': True, 'no_open': True}"/>
+                                        <field name="father_dead_res_city_id"
+                                               attrs="{'invisible': [('father_dead_country_id_code', '!=', 'SA')]}"
+                                               options="{'no_create': True, 'no_create_edit': True,'no_quick_create': True, 'no_open': True}"/>
                                         <field name="father_dead_country_id_code" invisible="1"/>
-                                        <field name="father_dead_city" attrs="{'invisible': [('father_dead_country_id_code', '=', 'SA')]}"/>
+                                        <field name="father_dead_city"
+                                               attrs="{'invisible': [('father_dead_country_id_code', '=', 'SA')]}"/>
                                         <field name="father_dead_certificate"
                                                attrs="{'readonly':[('state','not in',['draft','new','complete_info'])]}"
                                                widget="many2many_attachment_preview"/>