Handle Site=None when getting absolute media URL (#79)

Author: Chadys

docs/customizing/customize-meta.rst

@@ -189,7 +189,7 @@ first make a property on the page model:
                 # structured data image fields. This will render the image in
                 # 1:1, 4:3, and 16:9 aspect ratios with very high resolution
                 # and return a list of URLs.
-                "image": get_struct_data_images(self.recipe_photo),
+                "image": get_struct_data_images(self.get_site(), self.recipe_photo),
 
                 "author": {
                     "@type": "Person",

wagtailseo/utils.py

@@ -4,6 +4,7 @@
 from datetime import time
 from json import JSONEncoder
 from typing import List
+from typing import Optional
 from typing import Union
 
 from django.conf import settings
@@ -28,7 +29,7 @@ def serialize_date(date: Union[date, datetime, time]) -> str:
     return date.isoformat()
 
 
-def get_absolute_media_url(site: Site) -> str:
+def get_absolute_media_url(site: Optional[Site]) -> str:
     """
     Returns an absolute base URL for media files.
 
@@ -40,7 +41,7 @@ def get_absolute_media_url(site: Site) -> str:
     :rtype: str
     :returns: The absolute base URL for media files of this site.
     """
-    if MEDIA_IS_ABSOLUTE:
+    if MEDIA_IS_ABSOLUTE or site is None:
         return ""
 
     return site.root_url