# HG changeset patch
# User Sverre Rabbelier <srabbelier@gmail.com>
# Date 1236455178 0
# Node ID 711c8dcab67ee4e92c34058da0ff8d2400799b48
# Parent  52d90bbc98dccc3c72335c544bfc33cca0a3b081
Add a link to the full agreement text

Patch by: Sverre Rabbelier

diff -r 52d90bbc98dc -r 711c8dcab67e app/soc/views/models/mentor.py
--- a/app/soc/views/models/mentor.py	Sat Mar 07 19:45:38 2009 +0000
+++ b/app/soc/views/models/mentor.py	Sat Mar 07 19:46:18 2009 +0000
@@ -31,6 +31,7 @@
 from soc.views.helper import access
 from soc.views.helper import decorators
 from soc.views.helper import dynaform
+from soc.views.helper import redirects
 from soc.views.helper import params as params_helper
 from soc.views.helper import widgets
 from soc.views.models import organization as org_view
@@ -192,10 +193,8 @@
     if 'scope_path' in form.initial:
       scope_path = form.initial['scope_path']
     elif 'scope_path' in request.POST:
-      # TODO: do this nicely
       scope_path = request.POST['scope_path']
     else:
-      # TODO: is this always sufficient?
       form.fields['mentor_agreement'] = None
       return
 
@@ -204,9 +203,18 @@
     if not (entity and entity.scope and entity.scope.mentor_agreement):
       return
 
-    content = entity.scope.mentor_agreement.content
+    agreement = entity.scope.mentor_agreement
+
+    if not (entity and agreement):
+      return
 
-    form.fields['mentor_agreement'].widget.text = content
+    content = agreement.content
+    params = {'url_name': 'document'}
+
+    widget = form.fields['mentor_agreement'].widget
+    widget.text = content
+    widget.url = redirects.getPublicRedirect(agreement, params)
+
 
 view = View()
 
diff -r 52d90bbc98dc -r 711c8dcab67e app/soc/views/models/org_admin.py
--- a/app/soc/views/models/org_admin.py	Sat Mar 07 19:45:38 2009 +0000
+++ b/app/soc/views/models/org_admin.py	Sat Mar 07 19:46:18 2009 +0000
@@ -33,6 +33,7 @@
 from soc.views.helper import access
 from soc.views.helper import decorators
 from soc.views.helper import dynaform
+from soc.views.helper import redirects
 from soc.views.helper import params as params_helper
 from soc.views.helper import widgets
 from soc.views.models import organization as org_view
@@ -194,10 +195,8 @@
     if 'scope_path' in form.initial:
       scope_path = form.initial['scope_path']
     elif 'scope_path' in request.POST:
-      # TODO: do this nicely
       scope_path = request.POST['scope_path']
     else:
-      # TODO: is this always sufficient?
       form.fields['admin_agreement'] = None
       return
 
@@ -214,9 +213,17 @@
     if not (entity and entity.scope and entity.scope.org_admin_agreement):
       return
 
-    content = entity.scope.org_admin_agreement.content
+    agreement = entity.scope.org_admin_agreement
+
+    if not (entity and agreement):
+      return
 
-    form.fields['admin_agreement'].widget.text = content
+    content = agreement.content
+    params = {'url_name': 'document'}
+
+    widget = form.fields['admin_agreement'].widget
+    widget.text = content
+    widget.url = redirects.getPublicRedirect(agreement, params)
 
 
 view = View()
diff -r 52d90bbc98dc -r 711c8dcab67e app/soc/views/models/org_app.py
--- a/app/soc/views/models/org_app.py	Sat Mar 07 19:45:38 2009 +0000
+++ b/app/soc/views/models/org_app.py	Sat Mar 07 19:46:18 2009 +0000
@@ -161,21 +161,24 @@
     if 'scope_path' in form.initial:
       scope_path = form.initial['scope_path']
     elif 'scope_path' in request.POST:
-      # TODO: do this nicely
       scope_path = request.POST['scope_path']
     else:
-      # TODO: is this always sufficient?
       del form.fields['admin_agreement']
       return
 
     entity = program_logic.logic.getFromKeyName(scope_path)
 
-    if not entity or not entity.org_admin_agreement:
+    agreement = entity.org_admin_agreement
+
+    if not (entity and agreement):
       return
 
-    content = entity.org_admin_agreement.content
+    content = agreement.content
+    params = {'url_name': 'document'}
 
-    form.fields['admin_agreement'].widget.text = content
+    widget = form.fields['admin_agreement'].widget
+    widget.text = content
+    widget.url = redirects.getPublicRedirect(agreement, params)
 
   def _review(self, request, params, app_entity, status, **kwargs):
     """Sends out an email if an org_app has been accepted or rejected.
diff -r 52d90bbc98dc -r 711c8dcab67e app/soc/views/models/student.py
--- a/app/soc/views/models/student.py	Sat Mar 07 19:45:38 2009 +0000
+++ b/app/soc/views/models/student.py	Sat Mar 07 19:46:18 2009 +0000
@@ -178,10 +178,8 @@
     if 'scope_path' in form.initial:
       scope_path = form.initial['scope_path']
     elif 'scope_path' in request.POST:
-      # TODO do this nicely
       scope_path = request.POST['scope_path']
     else:
-      # TODO is this always sufficient?
       form.fields['student_agreement'] = None
       return
 
@@ -190,8 +188,14 @@
     if not (program and program.student_agreement):
       return
 
-    content = program.student_agreement.content
-    form.fields['student_agreement'].widget.text = content
+    agreement = program.student_agreement
+
+    content = agreement.content
+    params = {'url_name': 'document'}
+
+    widget = form.fields['student_agreement'].widget
+    widget.text = content
+    widget.url = redirects.getPublicRedirect(agreement, params)
 
 
 view = View()