[svn] My crusade to make our project more Djangoic.

Got rid of the url tags in template, replaced them with get_absolute_url method calls where possible. Adding get_absolute_url in models enables direct link to the public model views in admin. The use of get_absolute_url, which is the correct Django way of doing things, eliminates any need for the redundant "href" fields we were using. Those fields now need to be deleted from the models and from the parsers.

Made the context processor to pass settings to all templates actually work, although this was a little uglier than expected. I had to put in a new render_response to replace render_to_response. This is because Django uses Context, not RequestContext by default. I wish they would change this, it's annoying. Anyway, I deleted all the manual settings passing in the views.

I also eliminated a couple of unnecessary methods and stuff like that.
Copied from http://cucc@cucc.survex.com/svn/trunk/expoweb/troggle/, rev. 8244 by aaron @ 2/16/2009 8:31 AM

1 files changed, 4 insertions, 6 deletions

diff --git a/expo/views_other.py b/expo/views_other.py
index e413297..d890a6e 100644
--- a/expo/views_other.py
+++ b/expo/views_other.py
@@ -1,4 +1,3 @@
-from django.shortcuts import render_to_response

 from troggle.expo.models import Cave, Expedition, Person, LogbookEntry, PersonExpedition

 import troggle.settings as settings

 from django import forms

@@ -9,6 +8,7 @@ from troggle.parsers.survex import LoadAllSurvexBlocks
 import randSent

 

 from django.core.urlresolvers import reverse

+from troggle.alwaysUseRequestContext import render_response # see views_logbooks for explanation on this.

 

 def stats(request):

     statsDict={}

@@ -16,7 +16,7 @@ def stats(request):
     statsDict['caveCount'] = int(Cave.objects.count())

     statsDict['personCount'] = int(Person.objects.count())

     statsDict['logbookEntryCount'] = int(LogbookEntry.objects.count())

-    return render_to_response('statistics.html', statsDict)

+    return render_response(request,'statistics.html', statsDict)

 

 def frontpage(request):

     message = "no test message"  #reverse('personn', kwargs={"name":"hkjhjh"}) 

@@ -29,7 +29,7 @@ def frontpage(request):
 

     #'randSent':randSent.randomLogbookSentence(),

     expeditions =  Expedition.objects.order_by("-year")

-    return render_to_response('index.html', {'expeditions':expeditions, 'settings':settings, 'all':'all', "message":message})

+    return render_response(request,'index.html', {'expeditions':expeditions, 'all':'all', "message":message})

     

 def calendar(request,year):

     week=['S','S','M','T','W','T','F']

@@ -37,6 +37,4 @@ def calendar(request,year):
 	expedition=Expedition.objects.get(year=year)

 	PersonExpeditions=expedition.personexpedition_set.all()

 	

-	dictToPass=locals()

-	dictToPass.update({'settings':settings})

-    return render_to_response('calendar.html', dictToPass)

+    return render_response(request,'calendar.html', locals())