# HG changeset patch
# User nishanth
# Date 1267361870 -19800
# Node ID 3395960549e8c6fd02bb2630214b09cc1e05b048
# Parent  665eaf56e1d0b3638fcd495a6c8ba981b62cd95e
now a user cannot make two requests to another user for mentoring a task.

diff -r 665eaf56e1d0 -r 3395960549e8 taskapp/views/task.py
--- a/taskapp/views/task.py	Sun Feb 28 17:53:52 2010 +0530
+++ b/taskapp/views/task.py	Sun Feb 28 18:27:50 2010 +0530
@@ -3,7 +3,7 @@
 from django.http import HttpResponse
 from django.shortcuts import render_to_response, redirect
 
-from pytask.taskapp.models import User, Task, Comment, Claim, Credit
+from pytask.taskapp.models import User, Task, Comment, Claim, Credit, Request
 from pytask.taskapp.utilities.task import getTask
 from pytask.taskapp.forms.task import TaskCreateForm, AddMentorForm, AddTaskForm, ChoiceForm, AssignCreditForm, RemoveUserForm
 from pytask.taskapp.events.task import createTask, reqMentor, publishTask, addSubTask, addDep, addClaim, assignTask, updateTask, removeTask, removeUser, assignCredits, completeTask, closeTask
@@ -159,7 +159,10 @@
     is_guest = True if not user.is_authenticated() else False
     
     if (not is_guest) and user in task.mentors.all():
-        
+
+        pending_requests = Request.objects.filter(is_replied=False,is_valid=True,role="MT",task=task).order_by('creation_date').reverse()
+        user_pending_requests = pending_requests.filter(sent_by=user)
+
         ## now iam going for a brute force method
         user_list = list(User.objects.filter(is_active=True))
         for mentor in task.mentors.all():
@@ -170,10 +173,13 @@
 
         for a_user in task.assigned_users.all():
             user_list.remove(a_user)
+
+        for req in user_pending_requests:
+            user_list.remove(req.sent_to.all()[0])
             
         non_mentors = ((_.id,_.username) for _ in user_list)
         ## code till must be made elegant and not brute force like above
-        
+
         form = AddMentorForm(non_mentors)
         if request.method == "POST":
             uid = request.POST['mentor']
@@ -181,7 +187,7 @@
             reqMentor(task, new_mentor, user)
             return redirect(task_url)
         else:
-            return render_to_response('task/addmentor.html', {'user':user,'form':form, 'errors':errors})
+            return render_to_response('task/addmentor.html', {'user':user,'pending_requests':pending_requests,'form':form, 'errors':errors})
         
     else:
         return show_msg(user, 'You are not authorised to add mentors for this task', task_url, 'view the task')
diff -r 665eaf56e1d0 -r 3395960549e8 templates/task/addmentor.html
--- a/templates/task/addmentor.html	Sun Feb 28 17:53:52 2010 +0530
+++ b/templates/task/addmentor.html	Sun Feb 28 18:27:50 2010 +0530
@@ -1,8 +1,17 @@
 {% extends 'base.html' %}
 {% block content %}
-    Form here
     <form action="" method="post">
     {{form.as_table}}
     <input type="submit" value="submit">
     </form>
+    {% if pending_requests %}
+        Pending requests:<br />
+        {% for req in pending_requests %}
+            <a href="/user/view/uid={{req.sent_by.id}}">{{req.sent_by.username}}</a> requested 
+            {% for a_user in req.sent_to.all %}
+                <a href="/user/view/uid={{a_user.id}}">{{a_user.username}}</a> to act as a mentor
+            {% endfor %}
+            on {{req.creation_date|date:"D d M Y"}} at {{req.creation_date|time:"H:i"}}<br />
+        {% endfor %}
+    {% endif %}
 {% endblock %}