Merge pull request #252 from alyf-de/change-log-wb-rules

landa/water_body_management/doctype/water_body_rules/water_body_rules.json

@@ -49,7 +49,7 @@
  ],
  "issingle": 1,
  "links": [],
- "modified": "2023-09-22 14:26:40.377433",
+ "modified": "2023-11-27 18:59:08.307363",
  "modified_by": "Administrator",
  "module": "Water Body Management",
  "name": "Water Body Rules",
@@ -80,5 +80,6 @@
   }
  ],
  "sort_field": "modified",
- "sort_order": "DESC"
+ "sort_order": "DESC",
+ "track_changes": 1
 }

landa/water_body_management/utils.py

@@ -99,6 +99,7 @@ def get_version_log_query(from_datetime: str):
 					"Fish Species",
 					"File",
 					"Water Body Management Local Organization",
+					"Water Body Rules",
 				]
 			)
 		)
@@ -173,7 +174,10 @@ def build_modified_change_log(entry, changed_data, event):
 
 
 def build_dependency_change_log(entry, changed_data):
-	"""Return a pretty dict with changes in dependencies (File and WBMLO)."""
+	"""
+	Return a pretty dict with changes in dependencies (File and WBMLO).
+	`entry` could be creation (Version), modification (Version) or deletion (Deleted Document) events.
+	"""
 	if (
 		cint(entry.deleted)
 		and entry.doctype == "File"
@@ -183,13 +187,16 @@ def build_dependency_change_log(entry, changed_data):
 		return []
 
 	if entry.doctype == "Water Body Management Local Organization":
-		ref_water_body = frappe.db.get_value(
-			"Water Body Management Local Organization",
-			entry.docname,
-			"water_body",
-		)
+		if cint(entry.deleted):
+			ref_water_body = changed_data.get("water_body")  # Deleted WBMLO
+		else:
+			ref_water_body = frappe.db.get_value(  # Created/Modified WBMLO
+				"Water Body Management Local Organization",
+				entry.docname,
+				"water_body",
+			)
 	else:
-		ref_water_body = entry.attached_to_name or changed_data.attached_to_name
+		ref_water_body = changed_data.attached_to_name if cint(entry.deleted) else entry.attached_to_name
 
 	key = "files" if entry.doctype == "File" else "organizations"
 	return {