Produits Entr'ouvert » Combo

M'est avis qu'on devrait faire usage de django.template.context.make_context comme les implémentations de render dans les backends de template :

def make_context(context, request=None, **kwargs):
    """ 
    Create a suitable Context from a plain dict and optionally an HttpRequest.
    """ 
    if context is not None and not isinstance(context, dict):
        raise TypeError('context must be a dict rather than %s.' % context.__class__.__name__)
    if request is None:
        context = Context(context, **kwargs)
    else:
        # The following pattern is required to ensure values from
        # context override those from template context processors.
        original_context = context
        context = RequestContext(request, **kwargs)
        if original_context:
            context.push(original_context)
    return context

Ça veut dire passer explicitement request à get_templated_url, et juste un dico dans context.

Ouaip. J'avoue que ces contextes qui "ressemblent" à des dict mais en fait non, ça me perd souvent ; avec l'impression brumeuse que Django a plusieurs fois changé d'idée sur le sujet.

Sinon comme résolution, on pourrait aussi dans get_template_url, si context dispose d'une méthode flatten(), faire un context = context.flatten() afin d'en faire un vrai dico. Comme ça, on se fiche que context soit un dico ou un "vrai" context Django, ça marche...

Genre :

--- a/combo/utils/urls.py
+++ b/combo/utils/urls.py
@@ -36,6 +36,9 @@ def get_templated_url(url, context=None):
         return url
     template_vars = Context(use_l10n=False)
     if context:
+        if hasattr(context, 'flatten'):
+            # it's a django Context, dictionnarize it:
+            context = context.flatten()
         template_vars.update(context)
         template_vars['user_email'] = ''
         template_vars['user_nameid'] = ''

Ceci dit, j'ai ajouté le test ci-dessous dans tests/test_templated_url.py et il marche très bien avec le code actuel... je rate quoi ?

+    # requestcontext
+    context = RequestContext(request, {'request': request, 'foo': 'bar'})
+    assert get_templated_url('{{ foo }}', context=ctx) == 'bar'
+    assert get_templated_url('[foo]', context=ctx) == 'bar'

En django 1.11 ?

Benjamin Dauvergne a écrit :

En django 1.11 ?

Oui, j'ai revérifié plusieurs fois aussi, j'arrive pas à bien piger ... tox -e django111

Sur nanterre-test-web1 ça marche pas (shell dans tenant agents) :

In [8]: get_templated_url('[foo]', RequestContext(HttpRequest(), {'foo': 'zoo'}))
---------------------------------------------------------------------------
TemplateError                             Traceback (most recent call last)
<ipython-input-8-820f9a5e7469> in <module>()
----> 1 get_templated_url('[foo]', RequestContext(HttpRequest(), {'foo': 'zoo'}))

/usr/lib/python2.7/dist-packages/combo/utils/urls.pyc in get_templated_url(url, context)
     62             raise TemplateError('unknown variable %s', varname)
     63         return force_text(template_vars[varname])
---> 64     return re.sub(r'(\[.+?\])', repl, url)

/usr/lib/python2.7/re.pyc in sub(pattern, repl, string, count, flags)
    153     a callable, it's passed the match object and must return
    154     a replacement string to be used.""" 
--> 155     return _compile(pattern, flags).sub(repl, string, count)
    156 
    157 def subn(pattern, repl, string, count=0, flags=0):

/usr/lib/python2.7/dist-packages/combo/utils/urls.pyc in repl(matchobj)
     60             return '['
     61         if varname not in template_vars:
---> 62             raise TemplateError('unknown variable %s', varname)
     63         return force_text(template_vars[varname])
     64     return re.sub(r'(\[.+?\])', repl, url)

TemplateError: unknown variable foo

In [9]: django.version
---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
<ipython-input-9-271baf40d83f> in <module>()
----> 1 django.version

NameError: name 'django' is not defined

In [10]: import django

In [11]: django.VERSION
Out[11]: (1, 11, 20, u'final', 0)

Frédéric Péters a écrit :

Anyway, patch avec le flatten et le test de Benj, et hop.

Et un seul 'n' à dictionarize :)