Minor stylistic adjustments and comments

jadudm · jadudm · commit 0837a8ce93a8 · 2015-12-10T23:26:14.000Z
diff --git a/api/committee.py b/api/committee.py
@@ -8,11 +8,12 @@
 
 @app.route("/<username>/committee", methods = ["GET"])
 def committee_GET (username):
-  username = os.getenv("USER")
+  if username != authUser(request.environ):
+    return { "response": cfg["response"]["badUsername"] }
   
   ldap = getLDAPFaculty(username)
   
   return render_template ("committee.html", 
-                           username = os.getenv('USER'),
+                           username = username,
                            ldap = ldap
                            )
diff --git a/api/config.yaml b/api/config.yaml
@@ -37,7 +37,7 @@ flow:
   done:         "start"
 # END NO MAS
 
-contacts:
+contacts:         # I think this should move to the Parameters table, so it can be more easily configured in the UI by the committee.
   irbChairName: "Dan Huck"
   irbChairEmail: "huckd@berea.edu"
   
diff --git a/api/programs.py b/api/programs.py
@@ -3,7 +3,7 @@
 def getAllPrograms ():
   # This returns the program table
   progQ = (Programs.select()
-                   .order_by(+Programs.abbreviation))
+                   .order_by(+Programs.abbreviation)) # + is short for Sort ascending
   
   if progQ.exists():
     return progQ.select()
diff --git a/api/templates/create.html b/api/templates/create.html
@@ -73,34 +73,51 @@ <h1>URCPP Project Basics</h1>
     <div class = "input-group date">
 
     <div class = "col-sm-9">
-      <input id    = "startDate"
-	     name  = "startDate"  
-	     type  = "text" 
- 	     class = "form-control" 
- 	     value = "{% if proj.startDate is defined %}{{proj.startDate.strftime('%m-%d-%Y')}}{% endif %}" 
-             placeholder = "mm-dd-yyyy"
-	     onkeydown="return false" 
-	     required>
+      <input  id    = "startDate"
+	            name  = "startDate"  
+	            type  = "text" 
+ 	            class = "form-control" 
+ 	            value = "{% if proj.startDate is defined %}{{proj.startDate.strftime('%m-%d-%Y')}}{% endif %}" 
+              placeholder = "mm-dd-yyyy"
+	            onkeydown="return false" 
+	            required>
     </div>
 </div>
   </div>
 
   <script type = "text/javascript">
     $(function() {
-      $("#startDate").datepicker({dateFormat: "mm-dd-yy", startDate: "01-01-{{params.year}}", endDate: "12-31-{{params.year}}"});
-
+      $("#startDate").datepicker({
+        dateFormat: "mm-dd-yy", 
+        startDate: "01-01-{{params.year}}", 
+        endDate: "12-31-{{params.year}}"
+      });
     });
   </script>
   
   <div class = "row col-md-offset-1" style="font-size:1.4em;">
     <div class = "checkbox">
-      <label><input name = "isServiceToCommunity" {% if proj.isServiceToCommunity %} checked {% endif %} id = "isServiceToCommunity" type = "checkbox"  class="checkbox" value = "{{proj.isServiceToCommunity}}">Does your project provide a service to the community?</label>
+      <label><input name = "isServiceToCommunity" 
+                    {% if proj.isServiceToCommunity %} checked {% endif %} 
+                    id = "isServiceToCommunity" 
+                    type = "checkbox"  
+                    class="checkbox" 
+                    value = "{{proj.isServiceToCommunity}}">
+        Does your project provide a service to the community?
+      </label>
     </div>
   </div>
   
   <div class = "row col-md-offset-1" style="font-size:1.4em;">
     <div class = "checkbox">
-      <label><input name = "hasCommunityPartner" {% if proj.hasCommunityPartner %} checked {% endif %} id = "hasCommunityPartner" type = "checkbox"  class="checkbox" value = "{{proj.hasCommunityPartner}}">Does your project involve working with community partners? </label>
+      <label><input name = "hasCommunityPartner" 
+                    {% if proj.hasCommunityPartner %} checked {% endif %} 
+                    id = "hasCommunityPartner" 
+                    type = "checkbox"  
+                    class="checkbox" 
+                    value = "{{proj.hasCommunityPartner}}">
+        Does your project involve working with community partners? 
+      </label>
     </div>
   </div>
   <br />
