* Add a status to Characters.

[matthijs/projects/xerxes.git] / influences / views.py

diff --git a/influences/views.py b/influences/views.py
index 41a1ecbe08cbd3db595a3e9b69344a47f80136ab..51cfb495692fb9b6c6b71671eed1a0c10fe00206 100644 (file)
--- a/influences/views.py
+++ b/influences/views.py
@@ -1,24 +1,30 @@
+from django.contrib.auth.decorators import login_required
 from django.shortcuts import render_to_response
 from django.shortcuts import get_object_or_404
+from django.template import RequestContext
 from django.utils.translation import ugettext as _
 from django.contrib.auth.models import User
+from django.core.urlresolvers import reverse
+from django.http import HttpResponseRedirect, HttpResponseForbidden
+from django.views.generic.list_detail import object_detail, object_list
 from ee.influences.models import Character
 from ee.influences.models import Influence
 from ee.tools.forms import ContextModelForm
 
+
+
 class InfluenceForm(ContextModelForm):
     class Meta:
         model = Influence
-        fields = ('character', 'contact', 'description')
+        fields = ('character', 'contact', 'summary', 'description')
 
 class CharacterForm(ContextModelForm):
     class Meta:
         model = Character
         fields = ('name')
 
-def add(request, character_id):
-    message = None
-
+@login_required
+def add(request, character_id=None):
     initial = {}
     # Get the current user's characters
     chars = request.user.character_set.all()
@@ -30,29 +36,44 @@ def add(request, character_id):
 
     f = InfluenceForm(request=request, initial=initial)
     if (f.is_valid()):
-        f.save()
-        message = _("Influence added.")
-        f = None
+        influence = f.save(commit=False)
+       if (influence.character.player == request.user):
+               influence.save()
+               return HttpResponseRedirect(reverse('influences_influence_detail', args=[influence.id]))
+       else:
+               # TODO: Make this a bit more pretty. Perhaps throw an exception here and add some middleware to catch it?
+               return HttpResponseForbidden("Forbidden -- Trying to submit influence for somebody else's character")
      
     # Only allow characters of the current user
-    if f:
-        f.fields['character']._set_queryset(chars)
+    f.fields['character']._set_queryset(chars)
 
-    return render_to_response('influences/add.html', {'message' : message, 'form' : f})
+    return render_to_response('influences/add.html', {'form' : f}, RequestContext(request))
 
+@login_required
 def addCharacter(request):
-    message = None
     f = CharacterForm(request=request)
     if (f.is_valid()):
-        c = f.save(commit=False)
-        c.player = request.user
-        c.save()
-        message = _("Character added.")
-        f = None
+        character = f.save(commit=False)
+        character.player = request.user
+        character.save()
+        return HttpResponseRedirect(reverse('influences_character_detail', args=[character.id]))
 
-    return render_to_response('influences/add_character.html', {'message' : message, 'form' : f})
+    return render_to_response('influences/add_character.html', {'form' : f}, RequestContext(request))
 
+@login_required
 def index(request):
     characters = request.user.character_set.all()
     influences = Influence.objects.filter(character__in=characters)
-    return render_to_response('influences/index.html', {'characters' : characters, 'influences' : influences})
+    return render_to_response('influences/index.html', {'characters' : characters, 'influences' : influences}, RequestContext(request))
+
+@login_required
+def character_list(*args, **kwargs):
+    return object_list(*args, **kwargs)
+
+@login_required
+def character_detail(*args, **kwargs):
+    return object_detail(*args, **kwargs)
+
+@login_required
+def influence_detail(*args, **kwargs):
+    return object_detail(*args, **kwargs)