schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: remove unused imports, sort imports, fix sider

Browse Source
This commit is contained in:
Rucha Mahabal
2022-02-28 10:19:16 +05:30
parent d14a5c0f64
commit c7d594984a
2 changed files with 26 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

30
erpnext/hr/report/employee_leave_balance/employee_leave_balance.py
View File

@@ -6,7 +6,7 @@ from itertools import groupby
import frappe import frappe
from frappe import _ from frappe import _
from frappe.utils import add_days, date_diff, getdate from frappe.utils import add_days, getdate
from erpnext.hr.doctype.leave_allocation.leave_allocation import get_previous_allocation from erpnext.hr.doctype.leave_allocation.leave_allocation import get_previous_allocation
from erpnext.hr.doctype.leave_application.leave_application import ( from erpnext.hr.doctype.leave_application.leave_application import (
@@ -223,21 +223,21 @@ def get_leave_ledger_entries(from_date, to_date, employee, leave_type):
ledger = frappe.qb.DocType('Leave Ledger Entry') ledger = frappe.qb.DocType('Leave Ledger Entry')
records = ( records = (
frappe.qb.from_(ledger) frappe.qb.from_(ledger)
.select( .select(
ledger.employee, ledger.leave_type, ledger.from_date, ledger.to_date, ledger.employee, ledger.leave_type, ledger.from_date, ledger.to_date,
ledger.leaves, ledger.transaction_name, ledger.transaction_type, ledger.leaves, ledger.transaction_name, ledger.transaction_type,
ledger.is_carry_forward, ledger.is_expired ledger.is_carry_forward, ledger.is_expired
).where( ).where(
(ledger.docstatus == 1) (ledger.docstatus == 1)
& (ledger.transaction_type == 'Leave Allocation') & (ledger.transaction_type == 'Leave Allocation')
& (ledger.employee == employee) & (ledger.employee == employee)
& (ledger.leave_type == leave_type) & (ledger.leave_type == leave_type)
& ( & (
(ledger.from_date[from_date: to_date]) (ledger.from_date[from_date: to_date])
| (ledger.to_date[from_date: to_date]) | (ledger.to_date[from_date: to_date])
| ((ledger.from_date < from_date) & (ledger.to_date > to_date)) | ((ledger.from_date < from_date) & (ledger.to_date > to_date))
)
) )
)
).run(as_dict=True) ).run(as_dict=True)
return records return records

23
erpnext/hr/report/employee_leave_balance/test_employee_leave_balance.py
View File

@@ -5,22 +5,21 @@
import unittest import unittest
import frappe import frappe
from frappe.utils import ( from frappe.utils import add_days, add_months, flt, get_year_ending, get_year_start, getdate
add_days,
add_months,
get_year_ending,
get_year_start,
getdate,
flt,
)
from erpnext.hr.doctype.employee.test_employee import make_employee from erpnext.hr.doctype.employee.test_employee import make_employee
from erpnext.hr.report.employee_leave_balance.employee_leave_balance import execute
from erpnext.hr.doctype.holiday_list.test_holiday_list import set_holiday_list from erpnext.hr.doctype.holiday_list.test_holiday_list import set_holiday_list
from erpnext.hr.doctype.leave_type.test_leave_type import create_leave_type from erpnext.hr.doctype.leave_application.test_leave_application import (
from erpnext.hr.doctype.leave_application.test_leave_application import get_first_sunday, make_allocation_record get_first_sunday,
from erpnext.payroll.doctype.salary_slip.test_salary_slip import make_holiday_list, make_leave_application make_allocation_record,
)
from erpnext.hr.doctype.leave_ledger_entry.leave_ledger_entry import process_expired_allocation from erpnext.hr.doctype.leave_ledger_entry.leave_ledger_entry import process_expired_allocation
from erpnext.hr.doctype.leave_type.test_leave_type import create_leave_type
from erpnext.hr.report.employee_leave_balance.employee_leave_balance import execute
from erpnext.payroll.doctype.salary_slip.test_salary_slip import (
make_holiday_list,
make_leave_application,
)
test_records = frappe.get_test_records('Leave Type') test_records = frappe.get_test_records('Leave Type')