Add Intro Project Results management

stevenmirabito · stevenmirabito · commit 6d2aa764f60a · 2016-10-26T02:58:36.000-04:00
diff --git a/conditional/blueprints/member_management.py b/conditional/blueprints/member_management.py
@@ -7,7 +7,7 @@
 
 import structlog
 
-from flask import Blueprint, request, jsonify
+from flask import Blueprint, request, jsonify, abort
 
 from conditional.models.models import FreshmanAccount
 from conditional.models.models import FreshmanEvalData
@@ -24,6 +24,7 @@
 
 from conditional.blueprints.cache_management import clear_active_members_cache
 from conditional.blueprints.cache_management import clear_onfloor_members_cache
+from conditional.blueprints.intro_evals import display_intro_evals
 
 from conditional.util.ldap import ldap_is_eval_director
 from conditional.util.ldap import ldap_is_financial_director
@@ -241,6 +242,7 @@ def member_management_edituser(uid):
     db.session.commit()
     return jsonify({"success": True}), 200
 
+
 def edit_uid(uid, user_name, post_data):
     active_member = post_data['activeMember']
 
@@ -290,8 +292,8 @@ def edit_uid(uid, user_name, post_data):
 
 def edit_fid(uid, post_data):
     logger.info('backend', action="edit freshman account %s room: %s onfloor: %s eval_date: %s sig_missed %s" %
-        (uid, post_data['roomNumber'], post_data['onfloorStatus'],
-        post_data['evalDate'], post_data['sigMissed']))
+                                  (uid, post_data['roomNumber'], post_data['onfloorStatus'],
+                                   post_data['evalDate'], post_data['sigMissed']))
 
     name = post_data['name']
 
@@ -473,13 +475,13 @@ def member_management_upgrade_user():
     for fhm in FreshmanHouseMeetingAttendance.query.filter(FreshmanHouseMeetingAttendance.fid == fid):
         # Don't duplicate HM attendance records
         mhm = MemberHouseMeetingAttendance.query.filter(
-                  MemberHouseMeetingAttendance.meeting_id == fhm.meeting_id).first()
+            MemberHouseMeetingAttendance.meeting_id == fhm.meeting_id).first()
         if mhm is None:
             db.session.add(MemberHouseMeetingAttendance(
                 uid, fhm.meeting_id, fhm.excuse, fhm.attendance_status))
         else:
             logger.info('backend', action="duplicate house meeting attendance! fid: %s, uid: %s, id: %s" %
-					  (fid, uid, fhm.meeting_id))
+                                          (fid, uid, fhm.meeting_id))
         db.session.delete(fhm)
 
     if acct.onfloor_status:
@@ -496,3 +498,58 @@ def member_management_upgrade_user():
     clear_onfloor_members_cache()
 
     return jsonify({"success": True}), 200
+
+
+@member_management_bp.route('/manage/intro_project', methods=['GET'])
+def introductory_project():
+    log = logger.new(user_name=request.headers.get("x-webauth-user"),
+                     request_id=str(uuid.uuid4()))
+    log.info('api', action='show introductory project management')
+
+    user_name = request.headers.get('x-webauth-user')
+
+    if not ldap_is_eval_director(user_name):
+        return "must be eval director", 403
+
+    return render_template(request,
+                           'introductory_project.html',
+                           username=user_name,
+                           intro_members=display_intro_evals(internal=True))
+
+
+@member_management_bp.route('/manage/intro_project', methods=['POST'])
+def introductory_project_submit():
+    log = logger.new(user_name=request.headers.get("x-webauth-user"),
+                     request_id=str(uuid.uuid4()))
+    log.info('api', action='submit introductory project results')
+
+    user_name = request.headers.get('x-webauth-user')
+
+    if not ldap_is_eval_director(user_name):
+        return "must be eval director", 403
+
+    post_data = request.get_json()
+
+    if not isinstance(post_data, list):
+        abort(400)
+
+    for intro_member in post_data:
+        if not isinstance(intro_member, dict):
+            abort(400)
+
+        if 'uid' not in intro_member or 'status' not in intro_member:
+            abort(400)
+
+        if intro_member['status'] not in ['Passed', 'Pending', 'Failed']:
+            abort(400)
+
+        log.info('debug', action='setting status "' + intro_member['status'] + '" for ' + intro_member['uid'])
+
+        FreshmanEvalData.query.filter(FreshmanEvalData.uid == intro_member['uid']).update({
+            'freshman_project': intro_member['status']
+        })
+
+    db.session.flush()
+    db.session.commit()
+
+    return jsonify({"success": True}), 200
diff --git a/conditional/templates/intro_eval_slideshow.html b/conditional/templates/intro_eval_slideshow.html
@@ -39,8 +39,7 @@ <h3>{{m['technical_seminars']|length}}</h3>
             </div>
           </div>
         </div>
-        {% set freshman_project_passed = m['freshman_project'] == 'Passed' %}
-        <h4><span class="icon glyphicon glyphicon-{% if freshmen_project_passed %}ok passed{%else%}remove{% endif %}" aria-hidden="true"></span> Freshman Project</h4>
+        <h4><span class="icon glyphicon glyphicon-{% if m['freshman_project'] == 'Passed' %}ok passed{%else%}remove{% endif %}" aria-hidden="true"></span> Freshman Project</h4>
         
         <div class="actions" data-uid="{{m['uid']}}" data-cn="{{m['name']}}">
           <button class="pass" type="button">Pass</button>
diff --git a/conditional/templates/intro_evals.html b/conditional/templates/intro_evals.html
@@ -75,20 +75,20 @@ <h6 class="eval-uid">{{ m['uid'] }}</h6>
 
                         </div>
                         <div class="text-center">
-                            {% if m['freshman_project'] == "Pending" %}
-                            <div class="eval-info-label">
-                                <span class="glyphicon glyphicon-hourglass yellow eval-info-status"></span>Freshmen Project
-                                <span class="eval-info-number">Pending</span>
-                            </div>
-                            {% elif m['freshman_project'] == "Passed" %}
+                            {% if m['freshman_project'] == "Passed" %}
                             <div class="eval-info-label">
                                 <span class="glyphicon glyphicon-ok-sign green eval-info-status"></span>Freshmen Project
                                 <span class="eval-info-number">Passed</span>
                             </div>
+                            {% elif m['freshman_project'] == "Failed" %}
+                            <div class="eval-info-label">
+                                <span class="glyphicon glyphicon-remove-sign red eval-info-status"></span>Freshmen Project
+                                <span class="eval-info-number">Failed</span>
+                            </div>
                             {% else %}
                             <div class="eval-info-label">
-                                <span class="glyphicon glyphicon-ok-sign green eval-info-status"></span>Freshmen Project
-                                <span class="eval-info-number">Passed</span>
+                                <span class="glyphicon glyphicon-hourglass yellow eval-info-status"></span>Freshmen Project
+                                <span class="eval-info-number">Pending</span>
                             </div>
                             {% endif %}
 
diff --git a/conditional/templates/introductory_project.html b/conditional/templates/introductory_project.html
@@ -0,0 +1,62 @@
+{% extends "nav.html" %}
+{% block title %}
+Introductory Project Results
+{% endblock %}
+{% block body %}
+<div class="container main">
+    <h3>Introductory Project Results</h3>
+    <div style="padding-top:20px"></div>
+    <form data-module="introductoryProject">
+        <!-- Intro Members -->
+        <div class="panel panel-default">
+            <div class="panel-heading">
+                <h3 class="panel-title">Introductory Members</h3>
+            </div>
+            <div class="panel-body table-fill">
+                <div class="hm-search">
+                    <input class="form-control" type="text" placeholder="Quick Select" data-module="ipSearch" data-target="table.intro-project">
+                </div>
+                <div class="table-responsive intro-project-container">
+                    <table class="table table-striped no-bottom-margin intro-project" data-module="table" data-sort-column="0" data-sort-order="asc" data-paginated="false">
+                        <thead>
+                            <tr>
+                                <th>Name</th>
+                                <th>Result</th>
+                            </tr>
+                        </thead>
+                        <tbody>
+                            {% for f in intro_members %}
+                            <tr data-uid="{{f['uid']}}">
+                                <td>
+                                    <img class="table-img mobile-hide" src="https://profiles.csh.rit.edu/image/{{f['uid']}}"> {{f['name']}}
+                                </td>
+                                <td>
+                                    <div class="btn-group">
+                                        <a href="#" class="btn {% if f['freshman_project'] == 'Passed' %}btn-success{% elif f['freshman_project'] == 'Failed' %}btn-danger{% else %}btn-warning{% endif %} dropdown-toggle btn-mp" data-toggle="dropdown" data-selected="{{f['freshman_project']}}" aria-expanded="false">
+                                            {{f['freshman_project']}}
+                                            <span class="caret"></span>
+                                        </a>
+                                        <ul class="dropdown-menu">
+                                            <li>
+                                                <a href="#" data-option="Passed"><span class="glyphicon glyphicon-ok-sign green"></span> Passed</a>
+                                            </li>
+                                            <li>
+                                                <a href="#" data-option="Pending"><span class="glyphicon glyphicon-hourglass yellow"></span> Pending</a>
+                                            </li>
+                                            <li>
+                                                <a href="#" data-option="Failed"><span class="glyphicon glyphicon-remove-sign red"></span> Failed</a>
+                                            </li>
+                                        </ul>
+                                    </div>
+                                </td>
+                            </tr>
+                            {% endfor %}
+                        </tbody>
+                    </table>
+                </div>
+            </div>
+        </div>
+        <input type="submit" role="button" class="btn btn-raised btn-primary" style="width:100%;" value="Save Results"/>
+    </form>
+</div>
+{% endblock %}
diff --git a/conditional/templates/nav.html b/conditional/templates/nav.html
@@ -51,6 +51,7 @@
                         <li><a href="/manage"><span class="glyphicon glyphicon-user"></span> Member Management</a></li>
                         {% if is_eval_director %}
 
+                        <li><a href="/manage/intro_project"><span class="glyphicon glyphicon-tower"></span> Introductory Project Results</a></li>
                         <li><a href="/slideshow/intro"><span class="glyphicon glyphicon-eye-open"></span> Introductory Evaluations Presentation</a></li>
                         <li><a href="/slideshow/spring"><span class="glyphicon glyphicon-eye-open"></span> Membership Evaluations Presentation</a></li>
 
diff --git a/frontend/javascript/modules/hmSearch.js b/frontend/javascript/modules/hmSearch.js
@@ -49,9 +49,7 @@ export default class HouseMeetingSearch {
       // Yes, prevent form submission
       event.preventDefault();
 
-      // Check the first visible table row's checkbox
-      this.api.table().body().firstElementChild
-        .querySelector("input[type=checkbox]").checked = true;
+      this._handleKeyAction();
 
       // Reset the table
       this.api.search('').draw();
@@ -65,6 +63,12 @@ export default class HouseMeetingSearch {
     }
   }
 
+  _handleKeyAction() {
+    // Check the first visible table row's checkbox
+    this.api.table().body().firstElementChild
+      .querySelector("input[type=checkbox]").checked = true;
+  }
+
   /*
    * Custom filtering function that will remove rows that are already selected
    */
diff --git a/frontend/javascript/modules/introductoryProject.js b/frontend/javascript/modules/introductoryProject.js
@@ -0,0 +1,82 @@
+import _ from "lodash";
+import FetchUtil from "../utils/fetchUtil";
+
+export default class IntroductoryProjectForm {
+  constructor(form) {
+    this.form = form;
+    this.endpoint = '/manage/intro_project';
+
+    this.render();
+  }
+
+  render() {
+    // Prevent the form from submitting if the user hits the enter key
+    ['keyup', 'keypress'].forEach(keyevent =>
+      this.form.addEventListener(keyevent, event => {
+        let keyCode = event.keyCode || event.which;
+        if (keyCode === 13) {
+          event.preventDefault();
+          return false;
+        }
+      }, true));
+
+    this.form.querySelectorAll('tbody > tr .btn-group').forEach(control => {
+      control.querySelectorAll('[data-option]').forEach(option => {
+        option.addEventListener('click', e => {
+          e.preventDefault();
+
+          let toggle = control.querySelector('.dropdown-toggle');
+
+          ["btn-success", "btn-danger", "btn-warning"]
+            .forEach(classToRemove =>
+              toggle.classList.remove(classToRemove));
+
+          const caret = document.createElement('span');
+          caret.classList.add('caret');
+          toggle.text = option.dataset.option + " ";
+          toggle.appendChild(caret);
+          toggle.dataset.selected = option.dataset.option;
+
+          if (option.dataset.option === "Passed") {
+            toggle.classList.add("btn-success");
+          } else if (option.dataset.option === "Failed") {
+            toggle.classList.add("btn-danger");
+          } else {
+            toggle.classList.add("btn-warning");
+          }
+        });
+      });
+    });
+
+    // Form submit handler
+    this.form.querySelectorAll("input[type=submit]").forEach(submitBtn => {
+      submitBtn.addEventListener("click", e => {
+        e.preventDefault();
+
+        let payload = [];
+
+        this.form.querySelectorAll("tbody > tr").forEach(freshman => {
+          const uid = freshman.dataset.uid;
+          const status = freshman.querySelector('.dropdown-toggle')
+            .dataset.selected;
+
+          // Quick sanity check
+          if (!_.isNil(uid) && !_.isNil(status) &&
+            (status === "Passed" || status === "Pending" ||
+            status === "Failed")) {
+            payload.push({
+              uid: uid,
+              status: status
+            });
+          }
+        });
+
+        FetchUtil.postWithWarning(this.endpoint, payload, {
+          warningText: "Are you sure you want to update the introductory " +
+          "project results?",
+          successText: "Introductory project results have been submitted."
+        });
+      });
+    });
+  }
+}
diff --git a/frontend/javascript/modules/ipSearch.js b/frontend/javascript/modules/ipSearch.js
@@ -0,0 +1,34 @@
+import HouseMeetingSearch from "./hmSearch";
+
+export default class IntroductoryProjectSearch extends HouseMeetingSearch {
+  constructor(input) {
+    super(input);
+    HouseMeetingSearch._alreadySelectedFilter = (settings, data, dataIndex) => {
+      // Only apply the filter if we're currently searching
+      if (typeof settings.oPreviousSearch.sSearch !== "undefined" &&
+          settings.oPreviousSearch.sSearch !== "") {
+        return !(settings.aoData[dataIndex].anCells[1]
+          .querySelector('.dropdown-toggle').dataset.selected === "Passed");
+      }
+
+      return true;
+    };
+  }
+
+  _handleKeyAction() {
+    // Set the status of the first visible table row's selector to Passed
+    let toggle = this.api.table().body().firstElementChild
+      .querySelector(".dropdown-toggle");
+
+    ["btn-success", "btn-danger", "btn-warning"]
+      .forEach(classToRemove =>
+        toggle.classList.remove(classToRemove));
+
+    const caret = document.createElement('span');
+    caret.classList.add('caret');
+    toggle.text = "Passed ";
+    toggle.appendChild(caret);
+    toggle.classList.add("btn-success");
+    toggle.dataset.selected = "Passed";
+  }
+}
diff --git a/frontend/stylesheets/pages/_management.scss b/frontend/stylesheets/pages/_management.scss
@@ -140,3 +140,14 @@
   text-transform: uppercase;
   font-size: 10px;
 }
+
+.intro-project-container {
+  overflow-x: visible !important;
+  overflow-y: visible !important;
+}
+
+.intro-project {
+  .btn-group {
+    margin-top: -22px;
+  }
+}
