[fix] email digest periods for weekly and monthly
diff --git a/erpnext/accounts/doctype/journal_entry/journal_entry.py b/erpnext/accounts/doctype/journal_entry/journal_entry.py
index 835679c..c5035ad 100644
--- a/erpnext/accounts/doctype/journal_entry/journal_entry.py
+++ b/erpnext/accounts/doctype/journal_entry/journal_entry.py
@@ -176,6 +176,9 @@
against_voucher = frappe.db.get_value(d.reference_type, d.reference_name,
[scrub(dt) for dt in field_dict.get(d.reference_type)])
+ if not against_voucher:
+ frappe.throw(_("Row {0}: Invalid reference {1}").format(d.idx, d.reference_name))
+
# check if party and account match
if d.reference_type in ("Sales Invoice", "Purchase Invoice"):
if (against_voucher[0] != d.party or against_voucher[1] != d.account):
@@ -500,16 +503,16 @@
if voucher_type=="Bank Entry":
account = frappe.db.get_value("Company", company, "default_bank_account")
if not account:
- account = frappe.db.get_value("Account",
+ account = frappe.db.get_value("Account",
{"company": company, "account_type": "Bank", "is_group": 0})
elif voucher_type=="Cash Entry":
account = frappe.db.get_value("Company", company, "default_cash_account")
if not account:
- account = frappe.db.get_value("Account",
+ account = frappe.db.get_value("Account",
{"company": company, "account_type": "Cash", "is_group": 0})
if account:
- account_details = frappe.db.get_value("Account", account,
+ account_details = frappe.db.get_value("Account", account,
["account_currency", "account_type"], as_dict=1)
return {
"account": account,
@@ -731,15 +734,15 @@
def get_exchange_rate(account, account_currency=None, company=None,
reference_type=None, reference_name=None, debit=None, credit=None, exchange_rate=None):
from erpnext.setup.utils import get_exchange_rate
- account_details = frappe.db.get_value("Account", account,
+ account_details = frappe.db.get_value("Account", account,
["account_type", "root_type", "account_currency", "company"], as_dict=1)
-
+
if not company:
company = account_details.company
-
+
if not account_currency:
account_currency = account_details.account_currency
-
+
company_currency = get_company_currency(company)
if account_currency != company_currency:
diff --git a/erpnext/setup/doctype/email_digest/email_digest.py b/erpnext/setup/doctype/email_digest/email_digest.py
index e2b0de0..2053fc0 100644
--- a/erpnext/setup/doctype/email_digest/email_digest.py
+++ b/erpnext/setup/doctype/email_digest/email_digest.py
@@ -111,7 +111,8 @@
"""Set standard digest style"""
context.text_muted = '#8D99A6'
context.text_color = '#36414C'
- context.h1 = 'margin-bottom: 30px; margin-bottom: 0; margin-top: 40px; font-weight: 400;'
+ context.h1 = 'margin-bottom: 30px; margin-top: 40px; font-weight: 400; font-size: 30px;'
+ context.h2 = 'margin-bottom: 30px; margin-top: -20px; font-weight: 400; font-size: 20px;'
context.label_css = '''display: inline-block; color: {text_muted};
padding: 3px 7px; margin-right: 7px;'''.format(text_muted = context.text_muted)
context.section_head = 'margin-top: 60px; font-size: 16px;'
@@ -152,7 +153,7 @@
todo_list = frappe.db.sql("""select *
from `tabToDo` where (owner=%s or assigned_by=%s) and status="Open"
- order by field(priority, 'High', 'Medium', 'Low') asc, date asc""",
+ order by field(priority, 'High', 'Medium', 'Low') asc, date asc limit 20""",
(user_id, user_id), as_dict=True)
for t in todo_list:
@@ -289,6 +290,7 @@
elif self.frequency == "Weekly":
# from date is the previous week's monday
from_date = today - timedelta(days=today.weekday(), weeks=1)
+
# to date is sunday i.e. the previous day
to_date = from_date + timedelta(days=6)
else:
@@ -300,32 +302,18 @@
return from_date, to_date
def set_dates(self):
- today = now_datetime().date()
+ self.future_from_date, self.future_to_date = self.from_date, self.to_date
# decide from date based on email digest frequency
if self.frequency == "Daily":
- # from date, to_date is today
- self.future_from_date = self.future_to_date = today
- self.past_from_date = self.past_to_date = today - relativedelta(days = 1)
+ self.past_from_date = self.past_to_date = self.future_from_date - relativedelta(days = 1)
elif self.frequency == "Weekly":
- # from date is the current week's monday
- self.future_from_date = today - relativedelta(days=today.weekday())
-
- # to date is the current week's sunday
- self.future_to_date = self.future_from_date + relativedelta(days=6)
-
- self.past_from_date = self.future_from_date - relativedelta(days=7)
- self.past_to_date = self.future_to_date - relativedelta(days=7)
+ self.past_from_date = self.future_from_date - relativedelta(weeks=1)
+ self.past_to_date = self.future_from_date - relativedelta(days=1)
else:
- # from date is the 1st day of the current month
- self.future_from_date = today - relativedelta(days=today.day-1)
-
- # to date is the last day of the current month
- self.future_to_date = self.future_from_date + relativedelta(days=-1, months=1)
-
- self.past_from_date = self.future_from_date - relativedelta(month=1)
- self.past_to_date = self.future_to_date - relativedelta(month=1)
+ self.past_from_date = self.future_from_date - relativedelta(months=1)
+ self.past_to_date = self.future_from_date - relativedelta(days=1)
def get_next_sending(self):
from_date, to_date = self.get_from_to_date()
diff --git a/erpnext/setup/doctype/email_digest/templates/default.html b/erpnext/setup/doctype/email_digest/templates/default.html
index d0bd13a..bd88baf 100644
--- a/erpnext/setup/doctype/email_digest/templates/default.html
+++ b/erpnext/setup/doctype/email_digest/templates/default.html
@@ -11,6 +11,7 @@
<div style="max-width: 500px; margin: auto; padding: 20px 0 40px 0">
<h1 style="{{ h1 }}">{{ title }}</h1>
+<h2 style="{{ h2 }}">{{ company }}</h2>
<h4 style="font-weight: normal; color: {{ text_muted }}; margin-top: 7px; font-size: 16px; margin-top: 7px;">
<p>{% if frequency == "Daily" %}
{{ frappe.format_date(future_from_date) }}