tweaks and documentation.

git-svn-id: svn://svn.open-ils.org/ILS-Contrib/servres/trunk@165 6d9bc8c9-1ec2-4278-b937-99fde70a366f

diff --git a/conifer/syrup/generics.py b/conifer/syrup/generics.py
index fac85ef..fa0122e 100644 (file)
--- a/conifer/syrup/generics.py
+++ b/conifer/syrup/generics.py
@@ -25,7 +25,7 @@ def generic_edit(form, request, obj_id):
         instance = None
     else:
         instance = get_object_or_404(form.Meta.model, pk=obj_id)
-    if request.method == 'GET':
+    if request.method != 'POST':
         form = form(instance=instance)
         return g.render('generic/edit.xhtml', **locals())
     else:

diff --git a/conifer/syrup/views.py b/conifer/syrup/views.py
index e3593fe..e403d0c 100644 (file)
--- a/conifer/syrup/views.py
+++ b/conifer/syrup/views.py
@@ -115,6 +115,8 @@ def _access_denied(message):
                     content=message,
                     _django_type=HttpResponseForbidden)
 
+# todo, these decorators could be refactored.
+
 # decorator
 def instructors_only(handler):
     def hdlr(request, course_id, *args, **kwargs):
@@ -343,13 +345,17 @@ def add_new_course_ajax_title(request):
 @instructors_only
 def edit_course_permissions(request, course_id):
     course = get_object_or_404(models.Course, pk=course_id)
+    # choices: make the access-choice labels more personalized than
+    # the ones in 'models'.
     choices = [
+        # note: I'm leaving ANON out for now, until we discuss it further.
         (u'CLOSE', _(u'No students: this site is closed.')),
         (u'STUDT', _(u'Students in my course -- I will provide section numbers')),
         (u'INVIT', _(u'Students in my course -- I will share an Invitation Code with them')),
         (u'LOGIN', _(u'All Reserves patrons'))]
     if models.course_sections.sections_tuple_delimiter is None:
-        del choices[1]          # no STUDT support.
+        # no course-sections support? Then STUDT cannot be an option.
+        del choices[1]
     choose_access = django.forms.Select(choices=choices)
         
     if request.method != 'POST':
@@ -549,7 +555,7 @@ def item_add(request, course_id, item_id):
     assert item_type in ('HEADING', 'URL', 'ELEC'), \
         _('Sorry, only HEADINGs, URLs and ELECs can be added right now.')
 
-    if request.method == 'GET':
+    if request.method != 'POST':
         item = models.Item()    # dummy object
         return g.render('item_add_%s.xhtml' % item_type.lower(),
                         **locals())
@@ -621,7 +627,7 @@ def item_edit(request, course_id, item_id):
     item_type = item.item_type
     template = 'item_add_%s.xhtml' % item_type.lower()
     parent_item = item.parent_heading
-    if request.method == 'GET':
+    if request.method != 'POST':
         return g.render(template, **locals())
     else:
         if 'file' in request.FILES: