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

fix: version diff for empty datetime fields (#33605)

Browse source 
* fix: version diff for empty datetime fields

* fix: remove debug print
This commit is contained in:
Hussain Nagaria 2025-08-11 17:19:21 +05:30 • committed by GitHub
parent acb7f82732
commit 6e880bdcfe
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
frappe/core/doctype/version/version.py
View file

@ -4,7 +4,7 @@
import json
import frappe
from frappe.model import no_value_fields, table_fields
from frappe.model import datetime_fields, no_value_fields, table_fields
from frappe.model.document import Document
from frappe.utils import cstr
@ -120,6 +120,10 @@ def get_diff(old, new, for_child=False, compare_cancelled=False):
if df.fieldtype in ("Link", "Dynamic Link"):
old_value, new_value = cstr(old_value), cstr(new_value)
if df.fieldtype in datetime_fields:
if old_value is None and new_value == "":
new_value = None
if not for_child and df.fieldtype in table_fields:
old_rows_by_name = {}
for d in old_value: