Compare commits

...

3 Commits

Author SHA1 Message Date
Ankush Menat
0255642deb Merge branch 'develop' into fix-get_single_value 2024-01-10 10:42:12 +05:30
barredterra
d4d9e6c6fa test: fix wrong fieldname 2024-01-10 01:25:24 +01:00
barredterra
2eb318a5ce fix: wrong usage of get_single_value 2024-01-10 00:53:05 +01:00
3 changed files with 7 additions and 13 deletions

View File

@@ -1238,10 +1238,10 @@ class TestPurchaseInvoice(FrappeTestCase, StockTestMixin):
@change_settings("Accounts Settings", {"unlink_payment_on_cancellation_of_invoice": 1}) @change_settings("Accounts Settings", {"unlink_payment_on_cancellation_of_invoice": 1})
def test_gain_loss_with_advance_entry(self): def test_gain_loss_with_advance_entry(self):
unlink_enabled = frappe.db.get_single_value( unlink_enabled = frappe.db.get_single_value(
"Accounts Settings", "unlink_payment_on_cancel_of_invoice" "Accounts Settings", "unlink_payment_on_cancellation_of_invoice"
) )
frappe.db.set_single_value("Accounts Settings", "unlink_payment_on_cancel_of_invoice", 1) frappe.db.set_single_value("Accounts Settings", "unlink_payment_on_cancellation_of_invoice", 1)
original_account = frappe.db.get_value("Company", "_Test Company", "exchange_gain_loss_account") original_account = frappe.db.get_value("Company", "_Test Company", "exchange_gain_loss_account")
frappe.db.set_value( frappe.db.set_value(
@@ -1432,7 +1432,7 @@ class TestPurchaseInvoice(FrappeTestCase, StockTestMixin):
pay.cancel() pay.cancel()
frappe.db.set_single_value( frappe.db.set_single_value(
"Accounts Settings", "unlink_payment_on_cancel_of_invoice", unlink_enabled "Accounts Settings", "unlink_payment_on_cancellation_of_invoice", unlink_enabled
) )
frappe.db.set_value("Company", "_Test Company", "exchange_gain_loss_account", original_account) frappe.db.set_value("Company", "_Test Company", "exchange_gain_loss_account", original_account)

View File

@@ -2,11 +2,7 @@ import frappe
def execute(): def execute():
settings = frappe.db.get_single_value( settings = frappe.db.get_singles_dict("Selling Settings", cast=True)
"Selling Settings",
["campaign_naming_by", "close_opportunity_after_days", "default_valid_till"],
as_dict=True,
)
frappe.reload_doc("crm", "doctype", "crm_settings") frappe.reload_doc("crm", "doctype", "crm_settings")
if settings: if settings:

View File

@@ -78,13 +78,11 @@ def get_frequency(value):
def update_youtube_data(): def update_youtube_data():
# Called every 30 minutes via hooks # Called every 30 minutes via hooks
enable_youtube_tracking = frappe.db.get_single_value("Video Settings", "enable_youtube_tracking") video_settings = frappe.get_cached_doc("Video Settings")
frequency = frappe.db.get_single_value("Video Settings", "frequency") if not video_settings.enable_youtube_tracking:
if not cint(enable_youtube_tracking):
return return
frequency = get_frequency(frequency) frequency = get_frequency(video_settings.frequency)
time = datetime.now() time = datetime.now()
timezone = pytz.timezone(get_system_timezone()) timezone = pytz.timezone(get_system_timezone())
site_time = time.astimezone(timezone) site_time = time.astimezone(timezone)