From 5bb7b5b6da046df68278e655918f8cfa8eb00839 Mon Sep 17 00:00:00 2001 From: Tristan Cavelier <tristan.cavelier@nexedi.com> Date: Wed, 19 Oct 2016 11:24:31 +0200 Subject: [PATCH] erp5_payroll: simplify Person_getPayrollEnrollmentRecord --- .../erp5_payroll/Person_getPayrollEnrollmentRecord.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/bt5/erp5_payroll/SkinTemplateItem/portal_skins/erp5_payroll/Person_getPayrollEnrollmentRecord.py b/bt5/erp5_payroll/SkinTemplateItem/portal_skins/erp5_payroll/Person_getPayrollEnrollmentRecord.py index 2ca5a628c8..089d75b1dd 100644 --- a/bt5/erp5_payroll/SkinTemplateItem/portal_skins/erp5_payroll/Person_getPayrollEnrollmentRecord.py +++ b/bt5/erp5_payroll/SkinTemplateItem/portal_skins/erp5_payroll/Person_getPayrollEnrollmentRecord.py @@ -16,9 +16,9 @@ if isinstance(organisation, str): organisation = getObjectOrRaise(organisation_module, organisation) # If no id, let's expect that the parameters are objects -career_step_list = context.objectValues(portal_type="Career", - subordination_uid=organisation.getUid(), - validation_state='open') +career_step_list = context.objectValues(portal_type="Career") +career_step_list = [career for career in career_step_list if career.getValidationState() == 'open'] + if len(career_step_list) <= 0: raise ValueError("No open Career for employee %s" % context.getRelativeUrl()) else: -- 2.30.9