Treat Nones from private proxy like handle error (fixes #523)

Author: DavidMStraub

gramps_webapi/api/resources/util.py

@@ -26,7 +26,7 @@
 import os
 from hashlib import sha256
 from http import HTTPStatus
-from typing import Any, cast, Optional, Union, Literal
+from typing import Any, Literal, Optional, Union, cast
 
 import gramps
 import gramps.gen.lib
@@ -87,15 +87,21 @@
 def get_person_by_handle(db_handle: DbReadBase, handle: Handle) -> Union[Person, dict]:
     """Safe get person by handle."""
     try:
-        return db_handle.get_person_from_handle(handle)
+        person = db_handle.get_person_from_handle(handle)
+        if person is None:
+            return {}
+        return person
     except HandleError:
         return {}
 
 
 def get_place_by_handle(db_handle: DbReadBase, handle: Handle) -> Union[Place, dict]:
     """Safe get place by handle."""
     try:
-        return db_handle.get_place_from_handle(handle)
+        place = db_handle.get_place_from_handle(handle)
+        if place is None:
+            return {}
+        return place
     except HandleError:
         return {}
 
@@ -106,6 +112,8 @@ def get_family_by_handle(
     """Get a family and optional extended attributes."""
     try:
         obj = db_handle.get_family_from_handle(handle)
+        if obj is None:
+            return {}
     except HandleError:
         return {}
     args = args or {}
@@ -271,6 +279,8 @@ def get_event_profile_for_handle(
     """Get event profile given a handle."""
     try:
         obj = db_handle.get_event_from_handle(handle)
+        if obj is None:
+            return {}
     except HandleError:
         return {}
     return get_event_profile_for_object(
@@ -496,6 +506,8 @@ def get_person_profile_for_handle(
     """Get person profile given a handle."""
     try:
         obj = db_handle.get_person_from_handle(handle)
+        if obj is None:
+            return {}
     except HandleError:
         return {}
     return get_person_profile_for_object(db_handle, obj, args, locale=locale)
@@ -581,6 +593,8 @@ def get_family_profile_for_handle(
     """Get family profile given a handle."""
     try:
         obj = db_handle.get_family_from_handle(handle)
+        if obj is None:
+            return {}
     except HandleError:
         return {}
     return get_family_profile_for_object(db_handle, obj, args, locale=locale)
@@ -613,6 +627,8 @@ def get_citation_profile_for_handle(
     """Get citation profile given a handle."""
     try:
         obj = db_handle.get_citation_from_handle(handle)
+        if obj is None:
+            return {}
     except HandleError:
         return {}
     return get_citation_profile_for_object(db_handle, obj, args, locale=locale)
@@ -634,6 +650,8 @@ def get_media_profile_for_handle(
     """Get media profile given a handle."""
     try:
         obj = db_handle.get_media_from_handle(handle)
+        if obj is None:
+            return {}
     except HandleError:
         return {}
     return get_media_profile_for_object(db_handle, obj, args, locale=locale)