Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[fix] email digest periods for weekly and monthly

Browse Source
This commit is contained in:
Rushabh Mehta 2015-11-02 17:47:10 +05:30
parent a9dda232b2
commit bd4814fbb7
3 changed files with 21 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
erpnext/accounts/doctype/journal_entry/journal_entry.py
View File

@ -176,6 +176,9 @@ class JournalEntry(AccountsController):
against_voucher = frappe.db.get_value(d.reference_type, d.reference_name, against_voucher = frappe.db.get_value(d.reference_type, d.reference_name,
[scrub(dt) for dt in field_dict.get(d.reference_type)]) [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 # check if party and account match
if d.reference_type in ("Sales Invoice", "Purchase Invoice"): if d.reference_type in ("Sales Invoice", "Purchase Invoice"):
if (against_voucher[0] != d.party or against_voucher[1] != d.account): if (against_voucher[0] != d.party or against_voucher[1] != d.account):
@ -500,16 +503,16 @@ def get_default_bank_cash_account(company, voucher_type, mode_of_payment=None):
if voucher_type=="Bank Entry": if voucher_type=="Bank Entry":
account = frappe.db.get_value("Company", company, "default_bank_account") account = frappe.db.get_value("Company", company, "default_bank_account")
if not account: if not account:
account = frappe.db.get_value("Account", account = frappe.db.get_value("Account",
{"company": company, "account_type": "Bank", "is_group": 0}) {"company": company, "account_type": "Bank", "is_group": 0})
elif voucher_type=="Cash Entry": elif voucher_type=="Cash Entry":
account = frappe.db.get_value("Company", company, "default_cash_account") account = frappe.db.get_value("Company", company, "default_cash_account")
if not account: if not account:
account = frappe.db.get_value("Account", account = frappe.db.get_value("Account",
{"company": company, "account_type": "Cash", "is_group": 0}) {"company": company, "account_type": "Cash", "is_group": 0})
if account: 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) ["account_currency", "account_type"], as_dict=1)
return { return {
"account": account, "account": account,
@ -731,15 +734,15 @@ def get_account_balance_and_party_type(account, date, company, debit=None, credi
def get_exchange_rate(account, account_currency=None, company=None, def get_exchange_rate(account, account_currency=None, company=None,
reference_type=None, reference_name=None, debit=None, credit=None, exchange_rate=None): reference_type=None, reference_name=None, debit=None, credit=None, exchange_rate=None):
from erpnext.setup.utils import get_exchange_rate 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) ["account_type", "root_type", "account_currency", "company"], as_dict=1)
if not company: if not company:
company = account_details.company company = account_details.company
if not account_currency: if not account_currency:
account_currency = account_details.account_currency account_currency = account_details.account_currency
company_currency = get_company_currency(company) company_currency = get_company_currency(company)
if account_currency != company_currency: if account_currency != company_currency:

32
erpnext/setup/doctype/email_digest/email_digest.py
View File

@ -111,7 +111,8 @@ class EmailDigest(Document):
"""Set standard digest style""" """Set standard digest style"""
context.text_muted = '#8D99A6' context.text_muted = '#8D99A6'
context.text_color = '#36414C' 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}; context.label_css = '''display: inline-block; color: {text_muted};
padding: 3px 7px; margin-right: 7px;'''.format(text_muted = context.text_muted) padding: 3px 7px; margin-right: 7px;'''.format(text_muted = context.text_muted)
context.section_head = 'margin-top: 60px; font-size: 16px;' context.section_head = 'margin-top: 60px; font-size: 16px;'
@ -152,7 +153,7 @@ class EmailDigest(Document):
todo_list = frappe.db.sql("""select * todo_list = frappe.db.sql("""select *
from `tabToDo` where (owner=%s or assigned_by=%s) and status="Open" 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) (user_id, user_id), as_dict=True)
for t in todo_list: for t in todo_list:
@ -289,6 +290,7 @@ class EmailDigest(Document):
elif self.frequency == "Weekly": elif self.frequency == "Weekly":
# from date is the previous week's monday # from date is the previous week's monday
from_date = today - timedelta(days=today.weekday(), weeks=1) from_date = today - timedelta(days=today.weekday(), weeks=1)
# to date is sunday i.e. the previous day # to date is sunday i.e. the previous day
to_date = from_date + timedelta(days=6) to_date = from_date + timedelta(days=6)
else: else:
@ -300,32 +302,18 @@ class EmailDigest(Document):
return from_date, to_date return from_date, to_date
def set_dates(self): 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 # decide from date based on email digest frequency
if self.frequency == "Daily": if self.frequency == "Daily":
# from date, to_date is today self.past_from_date = self.past_to_date = self.future_from_date - relativedelta(days = 1)
self.future_from_date = self.future_to_date = today
self.past_from_date = self.past_to_date = today - relativedelta(days = 1)
elif self.frequency == "Weekly": elif self.frequency == "Weekly":
# from date is the current week's monday self.past_from_date = self.future_from_date - relativedelta(weeks=1)
self.future_from_date = today - relativedelta(days=today.weekday()) self.past_to_date = self.future_from_date - relativedelta(days=1)
# 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)
else: else:
# from date is the 1st day of the current month self.past_from_date = self.future_from_date - relativedelta(months=1)
self.future_from_date = today - relativedelta(days=today.day-1) self.past_to_date = self.future_from_date - relativedelta(days=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)
def get_next_sending(self): def get_next_sending(self):
from_date, to_date = self.get_from_to_date() from_date, to_date = self.get_from_to_date()

1
erpnext/setup/doctype/email_digest/templates/default.html
View File

@ -11,6 +11,7 @@
<div style="max-width: 500px; margin: auto; padding: 20px 0 40px 0"> <div style="max-width: 500px; margin: auto; padding: 20px 0 40px 0">
<h1 style="{{ h1 }}">{{ title }}</h1> <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;"> <h4 style="font-weight: normal; color: {{ text_muted }}; margin-top: 7px; font-size: 16px; margin-top: 7px;">
<p>{% if frequency == "Daily" %} <p>{% if frequency == "Daily" %}
{{ frappe.format_date(future_from_date) }} {{ frappe.format_date(future_from_date) }}