Add caching for submenus

diff --git a/sommitrealweird/bpcms/context_processors.py b/sommitrealweird/bpcms/context_processors.py
index 768d481b3ebce7d030e79f5cd091b11d20fd3cb1..0edc30af4e26242bed1dd99e0cfbcedc980256c7 100644 (file)
--- 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
     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:
             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,
                     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
             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,
 
     return {
         'content_submenu': menu_items,