Merge pull request #1158 from codeRIT/bh-1157

Added event sorting

Author: Colby Gallup

index.js

@@ -147,7 +147,27 @@ $('.day-first').click(function() {
 });
 $('.day-second').click(showSecondDayEvents);
 
-// Dynamic schedule code (sample API data)
+// Dynamic schedule code
+
+function compareEvents(a, b) {
+    // We can sort by start/end here because the ISO 8061
+    // timestamps given by the server are lexicographically
+    // sortable.
+
+    if (a.start < b.start) {         // if a starts before b...
+        return -1;                   //   ...then a goes before b
+    } else if (a.start > b.start) {  // if a starts after b...
+        return 1;                    //   ...then b goes before a
+    } else {
+        if (a.end < b.end) {         // if a ends before b...
+            return -1;               //   ...then a goes before b
+        } else if (a.end > b.end) {  // if a ends after b...
+            return 1;                //   ...then b goes before a
+        } else {
+            return 0;
+        }
+    }
+}
 
 function convertDate(date) {
     let output = '';
@@ -185,6 +205,9 @@ function handleEventData(events) {
         showSecondDayEvents();
     }
 
+    // need to sort events by start/end times instead of IDs
+    events.sort(compareEvents);
+
     events.forEach(event => {
         let startDate = new Date(event.start);  // convert ISO 8601 -> Date object