Do not fail if request is not in context

djangocms_page_meta/templatetags/page_meta_tags.py

@@ -5,6 +5,7 @@
 from classytags.core import Options, Tag
 from cms.utils import get_language_from_request
 from django import template
+from meta.views import Meta
 
 from ..utils import get_page_meta
 
@@ -20,8 +21,12 @@ class MetaFromPage(Tag):
     )
 
     def render_tag(self, context, page, varname):
-        language = get_language_from_request(context['request'])
-        meta = get_page_meta(page, language)
+        request = context.get('request')
+        if request:
+            language = get_language_from_request(request)
+            meta = get_page_meta(page, language)
+        else:
+            meta = Meta()
         context[varname] = meta
         return ''
 register.tag(MetaFromPage)

tests/test_general.py

@@ -3,17 +3,33 @@
 
 from copy import copy
 
+from classytags.core import Options
+from classytags.tests import DummyTokens
 from django.conf import settings
+from django.template.base import Parser
 from django.utils.functional import SimpleLazyObject
 
 from djangocms_page_meta import models
+from djangocms_page_meta.templatetags.page_meta_tags import MetaFromPage
 from djangocms_page_meta.utils import get_page_meta
 
 from . import BaseTest
 
 
 class PageMetaUtilsTest(BaseTest):
 
+    def test_context_no_request(self):
+        """
+        This is a weird and limited test to ensure that if request is not in context
+        no exception is thrown. In this case MetaFromPage.render_tag does not need
+        a full environment to work
+        """
+        context = {}
+        dummy_tokens = DummyTokens('myval', 'as', 'myname')
+        tag = MetaFromPage.render_tag(MetaFromPage(Parser(dummy_tokens), dummy_tokens), context, None, 'meta')
+        self.assertFalse(tag)
+        self.assertTrue(context['meta'])
+
     def test_page_meta_og(self):
         """
         Tests the OpenGraph meta tags