From e634bbdbebc6313daad35ad6b700d2c723a7eaa9 Mon Sep 17 00:00:00 2001
From: Brett Parker <iDunno@sommitrealweird.co.uk>
Date: Sat, 16 Feb 2008 12:55:14 +0000
Subject: [PATCH] Add caching for submenus

---
 sommitrealweird/bpcms/context_processors.py | 73 +++++++++++----------
 1 file changed, 38 insertions(+), 35 deletions(-)

diff --git a/sommitrealweird/bpcms/context_processors.py b/sommitrealweird/bpcms/context_processors.py
index 768d481..0edc30a 100644
--- a/sommitrealweird/bpcms/context_processors.py
+++ b/sommitrealweird/bpcms/context_processors.py
@@ -45,51 +45,54 @@ def content_submenu(request):
     folderpath = re.sub(u'^%s' %(settings.BPCMS_ROOT), '', folderpath)
     # right - that should be any prefix got rid of, now traverse the tree till
     # we've got something
-    folder = None
-    folderpath = folderpath.split('/')
-    for part in folderpath:
-        try:
-            if folder:
-                f = Folder.objects.get(parent__exact=folder, slug__exact=part)
-            else:
-                f = Folder.objects.get(parent__isnull=True, slug__exact=part)
-            folder = f
-        except:
-            break
+    menu_items = cache.get('bpcms_contentmenu_%s' %(folderpath))
 
-    # We have the folder (in theory)
-    menu_items = []
-    if folder:
-        items = Document.objects.filter(folder__exact=folder, \
-            islive__exact=True)
-        for item in items:
-            if item.slug != 'index':
-                menu_items.append(
-                    {
-                        'title': item.title,
-                        'url': item.get_basic_url(),
-                    }
-                )
-        try:
-            folders = Folder.objects.filter(parent__exact=folder)
-        except:
-            pass
-        for folder in folders:
+    if not menu_items:
+        folder = None
+        folderpath = folderpath.split('/')
+        for part in folderpath:
             try:
-                items = Document.objects.filter(folder__exact=folder, \
-                    islive__exact=True, slug__exact='index')
-                for item in items:
+                if folder:
+                    f = Folder.objects.get(parent__exact=folder, slug__exact=part)
+                else:
+                    f = Folder.objects.get(parent__isnull=True, slug__exact=part)
+                folder = f
+            except:
+                break
+
+        # We have the folder (in theory)
+        menu_items = []
+        if folder:
+            items = Document.objects.filter(folder__exact=folder, \
+                islive__exact=True)
+            for item in items:
+                if item.slug != 'index':
                     menu_items.append(
                         {
                             'title': item.title,
-                            'url': folder.get_basic_url(),
+                            'url': item.get_basic_url(),
                         }
                     )
+            try:
+                folders = Folder.objects.filter(parent__exact=folder)
             except:
                 pass
+            for folder in folders:
+                try:
+                    items = Document.objects.filter(folder__exact=folder, \
+                        islive__exact=True, slug__exact='index')
+                    for item in items:
+                        menu_items.append(
+                            {
+                                'title': item.title,
+                                'url': folder.get_basic_url(),
+                            }
+                        )
+                except:
+                    pass
 
-    menu_items.sort( \
-        cmp=lambda x,y: cmp(x['title'].lower(), y['title'].lower()))
+        menu_items.sort( \
+            cmp=lambda x,y: cmp(x['title'].lower(), y['title'].lower()))
 
     return {
         'content_submenu': menu_items,
-- 
2.39.5