Merge pull request #41006 from frappe/mergify/bp/version-14-hotfix/pr-40858

fix: Don't set delivery date as today while making SO from Quotation (backport #40858)
This commit is contained in:
Nabin Hait
2024-04-15 13:05:21 +05:30
committed by GitHub
2 changed files with 2 additions and 1 deletions

View File

@@ -302,7 +302,6 @@ def _make_sales_order(source_name, target_doc=None, ignore_permissions=False):
)
target.flags.ignore_permissions = ignore_permissions
target.delivery_date = nowdate()
target.run_method("set_missing_values")
target.run_method("calculate_taxes_and_totals")

View File

@@ -122,6 +122,7 @@ class TestQuotation(FrappeTestCase):
sales_order.naming_series = "_T-Quotation-"
sales_order.transaction_date = nowdate()
sales_order.delivery_date = nowdate()
sales_order.insert()
def test_make_sales_order_with_terms(self):
@@ -152,6 +153,7 @@ class TestQuotation(FrappeTestCase):
sales_order.naming_series = "_T-Quotation-"
sales_order.transaction_date = nowdate()
sales_order.delivery_date = nowdate()
sales_order.insert()
# Remove any unknown taxes if applied