# HG changeset patch
# User Lennard de Rijk <ljvderijk@gmail.com>
# Date 1256513269 25200
# Node ID cdc652533fcce365bc0aa879fd577fd7cd6a22d4
# Parent  cd08e035ac9e140c24c20fa1932ada53f4d7c909
Made Request Logic ID-based and added some helper methods.

diff -r cd08e035ac9e -r cdc652533fcc app/soc/logic/models/request.py
--- a/app/soc/logic/models/request.py	Sun Oct 25 16:24:34 2009 -0700
+++ b/app/soc/logic/models/request.py	Sun Oct 25 16:27:49 2009 -0700
@@ -34,30 +34,47 @@
   """
 
   def __init__(self, model=soc.models.request.Request,
-               base_model=None, scope_logic=linkable_logic):
+               base_model=None, id_based=True):
     """Defines the name, key_name and model for this entity.
     """
 
     super(Logic, self).__init__(model, base_model=base_model,
-                                scope_logic=scope_logic)
+                                id_based=id_based)
 
-  def getKeyValuesFromEntity(self, entity):
-    """See base.Logic.getKeyNameValues.
+  def isValidNewRequest(self, fields, role_logic):
+    """Returns True iff the given fields are for a valid new request.
+
+    Args:
+      fields: fields for the new Request
+      logic: the Logic for the Role that is being requested
     """
 
-    return [entity.scope.link_id, entity.role, entity.link_id]
+    # check for already outstanding or ignored requests
+    request_fields = {
+        'user': fields['user'],
+        'group': fields['group'],
+        'status': ['new', 'group_accepted', 'ignored']
+        }
 
-  def getKeyValuesFromFields(self, fields):
-    """See base.Logic.getKeyValuesFromFields.
-    """
+    request_entity = self.getForFields(request_fields, unique=True)
+
+    if request_entity:
+      return False
 
-    return [fields['scope_path'], fields['role'], fields['link_id']]
+    # check whether the user in the request already has the requested role
+    role_fields = {
+        'user': fields['user'],
+        'scope': fields['group'],
+        'status': ['active', 'inactive']
+        }
 
-  def getKeyFieldNames(self):
-    """See base.Logic.getKeyFieldNames.
-    """
+    role_entity = role_logic.getForFields(role_fields, unique=True)
 
-    return ['scope_path', 'role', 'link_id']
+    if role_entity:
+      return False
+
+    # seems to be a valid new request, so return True
+    return True
 
   def _onCreate(self, entity):
     """Sends out a message notifying users about the new invite/request.