vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

fix: delete cron jobs when switching server script type

Browse source
This commit is contained in:
Ankush Menat 2024-02-19 13:08:42 +05:30
parent 30fcf354e5
commit 68d1a9ec07
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
frappe/core/doctype/server_script/server_script.py
View file

@ -121,11 +121,13 @@ class ServerScript(Document):
def clear_scheduled_events(self):
"""Deletes existing scheduled jobs by Server Script if self.event_frequency or self.cron_format has changed"""
if self.script_type == "Scheduler Event" and (
self.has_value_changed("event_frequency") or self.has_value_changed("cron_format")
if (
self.script_type == "Scheduler Event"
and (self.has_value_changed("event_frequency") or self.has_value_changed("cron_format"))
or (self.has_value_changed("script_type") and self.script_type != "Scheduler Event")
):
for scheduled_job in self.scheduled_jobs:
frappe.delete_doc("Scheduled Job Type", scheduled_job.name)
frappe.delete_doc("Scheduled Job Type", scheduled_job.name, delete_permanently=1)
def check_if_compilable_in_restricted_context(self):
"""Check compilation errors and send them back as warnings."""