Move donation banner to separate html file

openlibrary/plugins/upstream/utils.py

@@ -1434,25 +1434,6 @@ def jsdef_get(obj, key, default=None):
     return obj.get(key, default)
 
 
-@public
-def get_donation_include() -> str:
-    ia_host = get_ia_host(allow_dev=True)
-    # The following allows archive.org staff to test banners without
-    # needing to reload openlibrary services
-    # Note donate.js also reads the ?ymd=YYYYMMDD parameter and the
-    # `variant` parameter.
-    if ia_host != "archive.org":
-        script_src = f"https://{ia_host}/includes/donate.js"
-    else:
-        script_src = "/cdn/archive.org/donate.js"
-
-    html = f"""
-        <div id="donato"></div>
-        <script src="{websafe(script_src)}" data-platform="ol"></script>
-    """
-    return html
-
-
 @public
 def get_ia_host(allow_dev: bool = False) -> str:
     if allow_dev:

openlibrary/templates/site/alert.html

@@ -1,5 +1,5 @@
 $if not is_bot():
-  $:get_donation_include()
+  $:render_template('site/donation_banner')
 
 $ lang = get_lang() or 'en'
 $ supported_languages = get_supported_languages()

openlibrary/templates/site/donation_banner.html

@@ -0,0 +1,13 @@
+$ ia_host = get_ia_host(allow_dev=True)
+
+$# The following allows archive.org staff to test banners without
+$# needing to reload openlibrary services
+$# Note donate.js also reads the ?ymd=YYYYMMDD parameter and the
+$# `variant` parameter.
+$if ia_host != "archive.org":
+    $ script_src = "https://%s/includes/donate.js" % ia_host
+$else:
+    $ script_src = "/cdn/archive.org/donate.js"
+
+<div id="donato"></div>
+<script src="$script_src" data-platform="ol"></script>