mirror of
https://github.com/frappe/erpnext.git
synced 2026-02-16 08:05:00 +00:00
* fix: SO delivery Date not getting set via data import
* fix: logic to add delivery dates
* fix: linting issue
Co-authored-by: Afshan <33727827+AfshanKhan@users.noreply.github.com>
Co-authored-by: Afshan <afshan13k@gmail.com>
(cherry picked from commit d9a219850a)
Co-authored-by: Deepesh Garg <42651287+deepeshgarg007@users.noreply.github.com>
This commit is contained in:
@@ -110,7 +110,7 @@ class SalesOrder(SellingController):
|
||||
if self.order_type == 'Sales' and not self.skip_delivery_note:
|
||||
delivery_date_list = [d.delivery_date for d in self.get("items") if d.delivery_date]
|
||||
max_delivery_date = max(delivery_date_list) if delivery_date_list else None
|
||||
if not self.delivery_date:
|
||||
if (max_delivery_date and not self.delivery_date) or (max_delivery_date and getdate(self.delivery_date) != getdate(max_delivery_date)):
|
||||
self.delivery_date = max_delivery_date
|
||||
if self.delivery_date:
|
||||
for d in self.get("items"):
|
||||
@@ -119,8 +119,6 @@ class SalesOrder(SellingController):
|
||||
if getdate(self.transaction_date) > getdate(d.delivery_date):
|
||||
frappe.msgprint(_("Expected Delivery Date should be after Sales Order Date"),
|
||||
indicator='orange', title=_('Warning'))
|
||||
if getdate(self.delivery_date) != getdate(max_delivery_date):
|
||||
self.delivery_date = max_delivery_date
|
||||
else:
|
||||
frappe.throw(_("Please enter Delivery Date"))
|
||||
|
||||
|
||||
Reference in New Issue
Block a user