# HG changeset patch
# User nishanth
# Date 1267008024 -19800
# Node ID 97953dc03ae318730946a7a7df9384c025f71b65
# Parent  9fc60a22101679e53e5921710e2e404e51b81da3# Parent  801cf8fca53a4465a4a77068201a6473add603a1
commited.

diff -r 801cf8fca53a -r 97953dc03ae3 taskapp/views/task.py
--- a/taskapp/views/task.py	Wed Feb 24 15:41:37 2010 +0530
+++ b/taskapp/views/task.py	Wed Feb 24 16:10:24 2010 +0530
@@ -193,18 +193,19 @@
     user = request.user
     task = getTask(tid)
     claims = Claim.objects.filter(task=task)
+
+    mentors = task.mentors.all()
+    claimed_users = task.claimed_users.all()
     
     is_guest = True if not user.is_authenticated() else False
-    if user in task.mentors.all():
-        is_mentor = True 
-    else:
-         is_mentor = False
+    is_mentor = True if user in mentors else False
 
-    task_claimable = True if task.status in ["OP", "RE", "CL"] else False
-    user_can_claim = True if  task_claimable and not ( is_guest or is_mentor ) and ( user not in task.claimed_users.all() )  else False
-    task_claimed = True if task.status == "CL" else False
+    task_claimable = True if task.status in ["OP", "WR"] else False
+    user_can_claim = True if  task_claimable and not ( is_guest or is_mentor ) and ( user not in claimed_users )  else False
+    task_claimed = True if claimed_users else False
     
-    context = {'is_mentor':is_mentor,
+    context = {'user':user,
+               'is_mentor':is_mentor,
                'task':task,
                'claims':claims,
                'user_can_claim':user_can_claim,
@@ -268,4 +269,4 @@
     and then give the user fields accordingly.
     """
     
-    return None
+    task = Task.objects.get(id=tid) 
diff -r 801cf8fca53a -r 97953dc03ae3 templates/task/claim.html
--- a/templates/task/claim.html	Wed Feb 24 15:41:37 2010 +0530
+++ b/templates/task/claim.html	Wed Feb 24 16:10:24 2010 +0530
@@ -8,11 +8,15 @@
             {{claim.message}}<br />
         {% endfor %}
     {% else %}
-        There are no claims for this task yet.<br />
+        {% if task_claimable%}
+            There are no claims for this task yet.<br />
+        {% else %}
+            This task cannot be claimed right now.
+        {% endif %}
         <a href="/task/view/tid={{task.id}}">Click here</a> to return to the task.
     {% endif %}
     {% if task_claimed and is_mentor %}
-        <a href="/task/assign/tid={{task.id}}">Assign task</a>
+        <a href="/task/assign/tid={{task.id}}">Select a user to assign the work.</a>
     {% endif %}
     {% if user_can_claim %}
     <hr />
diff -r 801cf8fca53a -r 97953dc03ae3 templates/task/view.html
--- a/templates/task/view.html	Wed Feb 24 15:41:37 2010 +0530
+++ b/templates/task/view.html	Wed Feb 24 16:10:24 2010 +0530
@@ -17,7 +17,7 @@
             edit task goes here and it should contain all those add subs and add deps depending on availability<br />
         {% endif %}
 
-        {%  if deps %}
+        {% if deps %}
             <br />The task has following dependencies
             {% for dep in deps %}
                 <a href="/task/view/tid={{dep.id}}">{{dep.title}}</a><br />
@@ -27,9 +27,9 @@
                 <a href="/task/remtask/tid={{task.id}}">remove an existing dependency</a>
             {% endif %}
         {% else %}
-            {%if can_mod_tasks %}
+            {% if can_mod_tasks %}
                 <a href="/task/addtask/tid={{task.id}}">add a subtask/dependency </a>
-            {%endif%}
+            {% endif %}
         {% endif %}
         
         <hr>
@@ -43,12 +43,15 @@
             {% for user in assigned_users %}
                 <a href="/user/view/uid={{user.id}}">{{user.username}}</a>|
             {% endfor %}
+            {% if is_mentor %}
+                <a href="/task/user/remove/">Remove an existing user</a>
             <br />
+            {% endif %}
         {% endif %}
-        {%if can_assign_credits%}
+        {% if can_assign_credits %}
             <a href="/task/assigncredits/tid={{task.id}}">Assign credits</a>
-        {%endif%}
-        {% if not is_guest %}
+        {% endif %}
+        {% if not is_guest and task_claimable %}
             <a href="/task/claim/tid={{task.id}}">View claims</a><br />
         {% endif %}