Template Namespace and `url`

[rpetrello]

I've just noticed this in TG2.1a3, verified in a recent quickstarted project:

It seems that any @exposed() controller that uses the key url in its namespace, e.g.:

    @expose('genshibug.templates.index')
    def index(self):
        """Handle the front-page."""
        return dict(page='index', url='http://google.com')

...the value for url is always overwritten w/ <routes.util.URLGenerator object at 0x10422c8d0>. Is this expected behavior? Maybe some form of misconfiguration?

[percious]

yes, this is expected behavior because pylons writes the url function into the namespace before rendering the template. We cannot change the way this functions because it will break API for many people.

[mramm]

This is not done by pylons, it's done in the tg rendering code, and it seems like it's the reverse of what *should* be happening.

The standard variables should be overwritten by the ones that come from the user's return dictionary, not the other way round.

I think this is a very small fix.

[rick446]

Actually, Mark, you are wrong ;-) . Pylons (pylons.templating) is the culprit here:

def render_genshi(template_name, extra_vars=None, cache_key=None, 
                  cache_type=None, cache_expire=None, method='xhtml'):
    """Render a template with Genshi
    
    Accepts the cache options ``cache_key``, ``cache_type``, and
    ``cache_expire`` in addition to method which are passed to Genshi's
    render function.
    
    """
    # Create a render callable for the cache function
    def render_template():
        # Pull in extra vars if needed
        globs = extra_vars or {}
        
        # Second, get the globals
        globs.update(pylons_globals())

        # Grab a template reference
        template = globs['app_globals'].genshi_loader.load(template_name)
        
        return literal(template.generate(**globs).render(method=method,
                                                         encoding=None))
    
    return cached_template(template_name, render_template, cache_key=cache_key,
                           cache_type=cache_type, cache_expire=cache_expire,
                           ns_options=('method'), method=method)