Farid Ghanchi (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-improve-addons31-hr-ggh into lp:~openerp-dev/openobject-addons/trunk-improve_addons31.
Requested reviews: OpenERP R&D Team (openerp-dev) For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-improve-addons31-hr-ggh/+merge/139891 Hello, I had improved in HR Module as you suggest.. Thanks... -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-improve-addons31-hr-ggh/+merge/139891 Your team OpenERP R&D Team is requested to review the proposed merge of lp:~openerp-dev/openobject-addons/trunk-improve-addons31-hr-ggh into lp:~openerp-dev/openobject-addons/trunk-improve_addons31.
=== modified file 'hr/hr.py' --- hr/hr.py 2012-12-08 12:43:19 +0000 +++ hr/hr.py 2012-12-14 13:32:41 +0000 @@ -46,9 +46,9 @@ _name = "hr.employee.category" _description = "Employee Category" _columns = { - 'name': fields.char("Category", size=64, required=True), + 'name': fields.char("Employee Tag", size=64, required=True), 'complete_name': fields.function(_name_get_fnc, type="char", string='Name'), - 'parent_id': fields.many2one('hr.employee.category', 'Parent Category', select=True), + 'parent_id': fields.many2one('hr.employee.category', 'Parent Employee Tag', select=True), 'child_ids': fields.one2many('hr.employee.category', 'parent_id', 'Child Categories'), 'employee_ids': fields.many2many('hr.employee', 'employee_category_rel', 'category_id', 'emp_id', 'Employees'), } @@ -121,7 +121,7 @@ } _sql_constraints = [ - ('name_company_uniq', 'unique(name, company_id)', 'The name of the job position must be unique per company!'), + ('name_company_uniq', 'unique(name, company_id, department_id)', 'The name of the job position must be unique per department in company!'), ] === modified file 'hr_holidays/hr_holidays.py' --- hr_holidays/hr_holidays.py 2012-12-08 12:43:19 +0000 +++ hr_holidays/hr_holidays.py 2012-12-14 13:32:41 +0000 @@ -79,7 +79,7 @@ 'categ_id': fields.many2one('crm.meeting.type', 'Meeting Type', help='Once a leave is validated, OpenERP will create a corresponding meeting of this type in the calendar.'), 'color_name': fields.selection([('red', 'Red'),('blue','Blue'), ('lightgreen', 'Light Green'), ('lightblue','Light Blue'), ('lightyellow', 'Light Yellow'), ('magenta', 'Magenta'),('lightcyan', 'Light Cyan'),('black', 'Black'),('lightpink', 'Light Pink'),('brown', 'Brown'),('violet', 'Violet'),('lightcoral', 'Light Coral'),('lightsalmon', 'Light Salmon'),('lavender', 'Lavender'),('wheat', 'Wheat'),('ivory', 'Ivory')],'Color in Report', required=True, help='This color will be used in the leaves summary located in Reporting\Leaves by Department.'), - 'limit': fields.boolean('Allow to Override Limit', help='If you select this check box, the system allows the employees to take more leaves than the available ones for this type and take them into account for the "Remaining Legal Leaves" defined on the employee form.'), + 'limit': fields.boolean('Allow to Override Limit', help='If you select this check box, the system allows the employees to take more leaves than the available ones for this type and will not take them into account for the "Remaining Legal Leaves" defined on the employee form.'), 'active': fields.boolean('Active', help="If the active field is set to false, it will allow you to hide the leave type without removing it."), 'max_leaves': fields.function(_user_left_days, string='Maximum Allowed', help='This value is given by the sum of all holidays requests with a positive value.', multi='user_left_days'), 'leaves_taken': fields.function(_user_left_days, string='Leaves Already Taken', help='This value is given by the sum of all holidays requests with a negative value.', multi='user_left_days'),
_______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : openerp-dev-gtk@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp