Purnendu Singh (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-payroll-yml-fix-psi into 
lp:~openerp-dev/openobject-addons/trunk-payroll.

Requested reviews:
  OpenERP R&D Team (openerp-dev)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-yml-fix-psi/+merge/66421

Hello,

As the struct_id field on hr.contract object is set as required from the py 
file, we were getting a trace back while installing hr_payroll module.

To fix the problem i set the struct_id field as required from the view xml.

Thanks
Purnendu Singh
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-yml-fix-psi/+merge/66421
Your team OpenERP R&D Team is requested to review the proposed merge of 
lp:~openerp-dev/openobject-addons/trunk-payroll-yml-fix-psi into 
lp:~openerp-dev/openobject-addons/trunk-payroll.
=== modified file 'hr_payroll/hr_payroll.py'
--- hr_payroll/hr_payroll.py	2011-06-16 10:49:18 +0000
+++ hr_payroll/hr_payroll.py	2011-06-30 09:21:13 +0000
@@ -119,7 +119,7 @@
     _inherit = 'hr.contract'
     _description = 'Employee Contract'
     _columns = {
-        'struct_id': fields.many2one('hr.payroll.structure', 'Salary Structure', required=True),
+        'struct_id': fields.many2one('hr.payroll.structure', 'Salary Structure'),
         'schedule_pay': fields.selection([
             ('monthly', 'Monthly'),
             ('quarterly', 'Quarterly'),

=== modified file 'hr_payroll/hr_payroll_view.xml'
--- hr_payroll/hr_payroll_view.xml	2011-06-16 10:30:51 +0000
+++ hr_payroll/hr_payroll_view.xml	2011-06-30 09:21:13 +0000
@@ -45,7 +45,7 @@
             <field name="arch" type="xml">
             <data>
                 <xpath expr="/form/notebook/page/group/field[@name='advantages']" position="before">
-                    <field name="struct_id" select="1"/>
+                    <field name="struct_id" required="1"/>
                 </xpath>
                  <xpath expr="/form/notebook/page/group/field[@name='working_hours']" position="after">
                     <field name="schedule_pay"/>

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to