georges/gexp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix merge not working for candidates with no scope or All Lebanon scope

Browse Source
This commit is contained in:
Georges Haddad
2026-04-15 10:29:41 +03:00
parent 88f047a654
commit 86ff96816a
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
routers/review.py
+12 -1
View File
@@ -233,8 +233,19 @@ def _get_merge_candidates(
if normalized_father and person.get("father_name"): if normalized_father and person.get("father_name"):
if normalize_arabic(person["father_name"]) != normalized_father: if normalize_arabic(person["father_name"]) != normalized_father:
continue continue
scope_values = [_normalize_scope(s) or s.strip() for s in (person.get("search_scopes") or "").split(",") if s.strip()] scope_values = [_normalize_scope(s) or s.strip() for s in (person.get("search_scopes") or "").split(",") if s.strip()]
same_scope = bool(normalized_scope and normalized_scope in scope_values)
same_scope = False
if not normalized_scope:
same_scope = True
elif not scope_values:
same_scope = True
elif normalized_scope == "كل لبنان" or "كل لبنان" in scope_values or _normalize_scope("كل لبنان") in scope_values:
same_scope = True
elif normalized_scope in scope_values:
same_scope = True
registry_match = bool( registry_match = bool(
normalized_registry normalized_registry
and person.get("registry_number") and person.get("registry_number")