Speedup static collection during development

[snoepkast]

The speedup is achieved by only doing the collection once per request instead of once for every time the template tag is used. For reference see issue #428.

[cyberdelia]

Could this be a guard?

if self.request and self.request is request:
    return  # Prevent execution multiple times per request
self.request = request
found_files = OrdereDict()
[...]

[cyberdelia]

Looks good, thanks a lot
It would be lovely if the tests passed 😐

[snoepkast]

I don't get why my change would break this? Do you have any idea?

[cyberdelia]

@snoepkast My current guess, is that your changes in PipelineMixin have broke PipelineExtension, probably because you changed __init__.

[leonardoo]

@snoepkast the problem is the init like say @cyberdelia, you can convert init get request_var to something like this

@property
    def request_var(self):
        if not self._request_var:
            self._request_var = template.Variable('request')
        return self._request_var

and remove the init call in these methods

class StylesheetNode(PipelineMixin, template.Node):
    def __init__(self, name):
         super(StylesheetNode, self).__init__(name) # this call is the problem
         self.name = name
  class JavascriptNode(PipelineMixin, template.Node):
     def __init__(self, name):
         super(JavascriptNode, self).__init__(name) # this call is the problem
         self.name = name

[snoepkast]

Locally the tests run now, but apparently not on Travis.. But it fails on a native python 'open file' which has nothing to do with any of this?

[snoepkast]

Ahh now I see the PR fixing that problem (#459), so this code should work then.

[cyberdelia]

Merged in 840bf27

[Remiz]

Amazing work! Now everything is loading under a second for me.

Thank you so much for this PR guys!