Luke Plant avatar Luke Plant committed 7462024

Removed unnecessary use of 'lazy'

Comments (0)

Files changed (1)

cciw/cciwmain/common.py

 from django.contrib.sites.models import Site
 from django.core.urlresolvers import reverse
 from django.http import HttpResponse
-from django.utils.functional import lazy
 from django.utils.safestring import mark_safe
 from django.views.generic.edit import FormView
 from django.views.generic.list import ListView
     else:
         request_path = request.path
 
-    # Memoized, lazy function to avoid queries we don't
-    # always need.
+    # As a callable, get_links will get called automatically by the template
+    # renderer *when needed*, so we avoid queries. We memoize in links_cache to
+    # avoid double queries
     links_cache = []
     def get_links():
         if len(links_cache) > 0:
             links_cache.extend(links)
             return links
 
-    context['menulinks'] = lazy(get_links, list)
+    context['menulinks'] = get_links
     context['GOOGLE_ANALYTICS_ACCOUNT'] = getattr(settings, 'GOOGLE_ANALYTICS_ACCOUNT', '')
     context['PRODUCTION'] = (settings.LIVEBOX and settings.PRODUCTION)
 
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.