fix!: require sender and message for contact us page (#34707)
fix!: require sender and message for contact us page (#34707)
* fix: require sender and message for contact us page
* refactor: dont override frappe.send_message from client side
used override_whitelisted_method hook for the same
(cherry picked from commit f193393f57)
Co-authored-by: Ritwik Puri <ritwikpuri5678@gmail.com>
This commit is contained in:
@@ -6,13 +6,12 @@ import frappe
|
||||
|
||||
|
||||
@frappe.whitelist(allow_guest=True)
|
||||
def send_message(subject="Website Query", message="", sender="", status="Open"):
|
||||
def send_message(sender, message, subject="Website Query"):
|
||||
from frappe.www.contact import send_message as website_send_message
|
||||
|
||||
website_send_message(sender, message, subject)
|
||||
|
||||
lead = customer = None
|
||||
|
||||
website_send_message(subject, message, sender)
|
||||
|
||||
customer = frappe.db.sql(
|
||||
"""select distinct dl.link_name from `tabDynamic Link` dl
|
||||
left join `tabContact` c on dl.parent=c.name where dl.link_doctype='Customer'
|
||||
@@ -59,5 +58,3 @@ def send_message(subject="Website Query", message="", sender="", status="Open"):
|
||||
}
|
||||
)
|
||||
comm.insert(ignore_permissions=True)
|
||||
|
||||
return "okay"
|
||||
|
||||
Reference in New Issue
Block a user