# HG changeset patch # User nishanth # Date 1267260746 -19800 # Node ID e747da8bc110b0596ce4ed1d4fc6e7841a047a63 # Parent f34e3a3e1439b66819895065655ac3fe741da4fa notifications work for approving and rejecting credits. diff -r f34e3a3e1439 -r e747da8bc110 taskapp/events/request.py --- a/taskapp/events/request.py Sat Feb 27 10:48:43 2010 +0530 +++ b/taskapp/events/request.py Sat Feb 27 14:22:26 2010 +0530 @@ -1,6 +1,7 @@ from datetime import datetime from pytask.taskapp.events.task import addCredits, addMentor from pytask.taskapp.events.user import changeRole +from pytask.taskapp.utilities.notification import create_notification def reply_to_request(request_obj, reply, replied_by): """ @@ -18,15 +19,21 @@ request_obj.save() if request_obj.role == "PY": - if reply: - ## note that we are not checking if he is stilla mentor - ## since we are not allowing removing mentors - ## if we allow, we have complications like removing unreplied requests made by this mentor and stuff. - ## but we check if this user is still an admin and put a notification if that you are no longer an admin and hence cannot do this. - addCredits(request_obj.task, request_obj.sent_by, request_obj.receiving_user, request_obj.pynts) - print "send yes notifications appropriately" - else: - print "send a no notificvaton" + ## note that we are not doing any check. we make requests invalid when an event like closing task happens. + task = request_obj.task + pynts = request_obj.pynts + receiving_user = request_obj.receiving_user + requested_by = request_obj.sent_by + for a_mentor in task.mentors.all(): + if reply: + addCredits(task, request_obj.sent_by, request_obj.receiving_user, pynts) + print "send yes notifications appropriately" + #def create_notification(role, sent_to, sent_from=None, reply=None, task=None, remark=None, receiving_user=None, pynts=None, requested_by): + create_notification("PY", a_mentor, replied_by, True, task, receiving_user, pynts, requested_by) + else: + print "send a no notificvaton" + create_notification("PY", a_mentor, replied_by, False, task, receiving_user, pynts, requested_by, request_obj.remarks) + elif request_obj.role == "MT": ## add him as a mentor to the task if reply: diff -r f34e3a3e1439 -r e747da8bc110 taskapp/models.py --- a/taskapp/models.py Sat Feb 27 10:48:43 2010 +0530 +++ b/taskapp/models.py Sat Feb 27 14:22:26 2010 +0530 @@ -172,13 +172,13 @@ role = models.CharField(max_length = 2, choices = NOTIFY_CHOICES, blank = False) sent_to = models.ForeignKey(User, related_name = "%(class)s_sent_to", blank = False) - sent_from = models.ForeignKey(User, related_name = "%(class)s_sent_from", blank = True) - task = models.ForeignKey(Task, related_name = "%(class)s_sent_for", blank = False) + sent_from = models.ForeignKey(User, related_name = "%(class)s_sent_from", null = True, blank = True) + task = models.ForeignKey(Task, related_name = "%(class)s_sent_for", null = True, blank = True) sub = models.CharField(max_length = 100) message = models.TextField() - remark = models.CharField(max_length = 100) + remarks = models.CharField(max_length = 100) sent_date = models.DateTimeField() is_read = models.BooleanField(default = False) diff -r f34e3a3e1439 -r e747da8bc110 taskapp/utilities/notification.py --- a/taskapp/utilities/notification.py Sat Feb 27 10:48:43 2010 +0530 +++ b/taskapp/utilities/notification.py Sat Feb 27 14:22:26 2010 +0530 @@ -2,19 +2,46 @@ from django.contrib.auth.models import User from pytask.taskapp.models import Notification -def create_notification(to,subject,message): +def create_notification(role, sent_to, sent_from=None, reply=None, task=None, receiving_user=None, pynts=None, requested_by=None, remarks=None): """ creates a notification based on the passed arguments. to - a list of users to which the notification is to be sent subject - subject of the notification message to be sent message - message body of the notification """ + notification = Notification(sent_date = datetime.now()) + notification.role = role + notification.sent_to = sent_to notification.save() - notification.sent_to = to - notification.sub = subject - notification.message = message - notification.save() + + if role == "PY": + + notification.sent_from = sent_from + notification.task = task + notification.pynts = pynts + + task_url= '%s'%(task.id, task.title) + credits_url = '%s'%(task.id, "click here") + mentor_url = '%s'%(requested_by.id, requested_by.username) + admin_url = '%s'%(sent_from.id, sent_from.username) + user_url = '%s'%(receiving_user.id, receiving_user.username) + + if reply: + notification.sub = "Approved request for assign of credits" + notification.message = """ Request made by %s to assign %s pynts to %s for the task %s has been approved by %s
+ %s if you want the view/assign pynts page of the task.
"""%(mentor_url, pynts, user_url, task_url, admin_url, credits_url) + + else: + notification.sub = "Rejected request for assign of credits" + notification.message = """ Request made by %s to assign %s pynts to %s for the task %s has been rejected by %s.
"""%(mentor_url, pynts, user_url, task_url, admin_url) + if remarks: + notification.remarks = remarks + notification.message += "Reason: %s
"%remarks + notification.message += "
" + + notification.save() + def mark_notification_read(notification_id): """ @@ -54,10 +81,5 @@ except Notification.DoesNotExist: return None - try: - notify_obj.sent_to.get(id=user.id) - except User.DoesNotExist: - return None - - if not notify_obj.is_deleted: + if notify_obj.sent_to == user and ( not notify_obj.is_deleted ): return notify_obj diff -r f34e3a3e1439 -r e747da8bc110 templates/user/view_notification.html --- a/templates/user/view_notification.html Sat Feb 27 10:48:43 2010 +0530 +++ b/templates/user/view_notification.html Sat Feb 27 14:22:26 2010 +0530 @@ -1,10 +1,10 @@ {% extends 'base.html' %} {% block content %} - Subject: {{notification.sub}}
- Sent time: {{notification.sent_date}}
- {% autoescape off %} - {{notification.message}} - {% endautoescape %} + sent at {{notification.sent_date}}
+ Sub: {{notification.sub}}
+ +
+ {{notification.message|safe}}
{% endblock %}