X-Git-Url: https://git.sommitrealweird.co.uk/sommitrealweird.git/blobdiff_plain/8e218bdb4f1958b28ce249511e5acdb63eeb326b..29f2eef0ba05968f84c3afad30e1fa105fc56ed3:/sommitrealweird/bpcms/context_processors.py

diff --git a/sommitrealweird/bpcms/context_processors.py b/sommitrealweird/bpcms/context_processors.py
index 768d481..1d254b1 100644
--- a/sommitrealweird/bpcms/context_processors.py
+++ b/sommitrealweird/bpcms/context_processors.py
@@ -4,6 +4,7 @@ from django.core.cache import cache
 import re
 
 def content_menu(request):
+
     menu_items = cache.get('bpcms_mainmenu')
 
     if not menu_items:
@@ -14,7 +15,7 @@ def content_menu(request):
                 menu_items.append(
                     {
                         'title': item.title,
-                        'url': item.get_basic_url(),
+                        'url': item.get_absolute_url(),
                     }
                 )
         folders = Folder.objects.filter(parent__isnull=True)
@@ -25,14 +26,14 @@ def content_menu(request):
                 menu_items.append(
                     {
                         'title': item.title,
-                        'url': folder.get_basic_url(),
+                        'url': folder.get_absolute_url(),
                     }
                 )
             except:
                 pass
 
         menu_items.sort( \
-            cmp=lambda x,y: cmp(x['title'].lower(), y['title'].lower()))
+            key=lambda x: x['title'].lower())
         cache.set('bpcms_mainmenu', menu_items, 3600)
 
     return {
@@ -43,60 +44,69 @@ def content_submenu(request):
     # find where we are in the tree first
     folderpath = request.path
     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
+    try:
+        menu_items = cache.get('bpcms_contentmenu_%s' %(folderpath))
+    except:
+        # fail on cache engine or bad key, ignore
+        return {
+            'content_submenu': {},
+        }
 
-    # 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:
+                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': item.get_absolute_url(),
+                        }
+                    )
+            folders = Folder.objects.filter(parent__exact=folder)
+            for folder in folders:
                 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(),
+                            'url': folder.get_absolute_url(),
                         }
                     )
-            except:
-                pass
 
-    menu_items.sort( \
-        cmp=lambda x,y: cmp(x['title'].lower(), y['title'].lower()))
+        menu_items.sort( \
+            key=lambda x: x['title'].lower())
+
+        cache.set('bpcms_contentmenu_%s' %("/".join(folderpath)), menu_items)
 
     return {
         'content_submenu': menu_items,
     }
 
 def content_breadcrumb(request):
-    breadcrumb = [{'url': settings.BPCMS_ROOT, 'title': settings.BPCMS_ROOT},]
+    if settings.BPCMS_ROOT == "/":
+        breadcrumb = [{'url': settings.BPCMS_ROOT, 'title': "Home"}]
+    else:
+        breadcrumb = [{'url': settings.BPCMS_ROOT, 'title': settings.BPCMS_ROOT},]
     # find where we are in the tree first
     folderpath = request.path
     folderpath = re.sub(u'^%s' %(settings.BPCMS_ROOT), '', folderpath)
@@ -114,10 +124,10 @@ def content_breadcrumb(request):
             try:
                 item = Document.objects.get(folder__exact=folder,
                     slug__exact=u'index', islive__exact=True)
-                breadcrumb.append({'url': folder.get_basic_url(),
+                breadcrumb.append({'url': folder.get_absolute_url(),
                     'title': item.title})
             except:
-                breadcrumb.append({'url': folder.get_basic_url(),
+                breadcrumb.append({'url': folder.get_absolute_url(),
                     'title': folder.title})
         except:
             # must have hit the end of the tree, lets see if we can get an
@@ -129,7 +139,7 @@ def content_breadcrumb(request):
                 else:
                     item = Document.objects.get(folder__isnull=True, \
                         slug__exact=part, islive__exact=True)
-                breadcrumb.append({'url': item.get_basic_url(), 'title': item.title})
+                breadcrumb.append({'url': item.get_absolute_url(), 'title': item.title})
             except:
                 pass
             break