From 5f13c60923214b3e25a99e92cb81aefb7418196c Mon Sep 17 00:00:00 2001 From: Sriram Hariharan Date: Mon, 16 Jul 2018 01:31:11 -0500 Subject: [PATCH] more fixes --- content.js | 181 ++++++++++++++++++++++++-------------------------- grades.db | Bin 3657728 -> 3657728 bytes manifest.json | 2 +- popup.html | 2 +- popup.js | 22 +++++- styles.css | 56 +--------------- todo | 31 ++++----- 7 files changed, 121 insertions(+), 173 deletions(-) diff --git a/content.js b/content.js index 1411c6ca..a15e8c1e 100644 --- a/content.js +++ b/content.js @@ -12,6 +12,7 @@ var datetimearr = []; var chart; var description; var status; + const days = new Map([["M" ,"Monday"], ["T", "Tuesday"], ["W", "Wednesday"],["TH" ,"Thursday"], ["F", "Friday"]]); @@ -29,7 +30,7 @@ $(document).ready( function() { if(!($(this).find('td').hasClass("course_header")) && $(this).has('th').length == 0){ //if a course row, then add the extension button and do something if that course has been "saved" var thisForm = this; - $(this).append(''); + $(this).append(''); chrome.runtime.sendMessage({command: "isSingleConflict",dtarr: getDtarr(this)}, function(response) { if(response.isConflict){ //DO SOMETHING IF ALREADY CONTAINS @@ -82,6 +83,12 @@ $(document).ready( function() { } $("#snackbar").attr("class",""); }); + chrome.runtime.onMessage.addListener( + function(request, sender, sendResponse) { + if (request.command == "update"){ + update(); + } + }); }); function update(){ @@ -111,7 +118,6 @@ function update(){ function getDtarr(row){ var numlines = $(row).find('td[data-th="Days"]>span').length; - console.log("Numlines: "+numlines); var dtarr = []; for(var i=0; ispan:eq('+i+')').text(); @@ -131,7 +137,6 @@ function getDtarr(row){ } } } - console.log(dtarr); return dtarr; } @@ -154,7 +159,6 @@ function getCourseInfo(row){ profurl = $(this).find('td[data-th="Unique"] a').prop('href'); uniquenum = $(this).find('td[data-th="Unique"]').text(); status = $(this).find('td[data-th="Status"]').text(); - console.log(status); profname = $(this).find('td[data-th="Instructor"]').text().split(', ')[0]; profinit = $(this).find('td[data-th="Instructor"]').text().split(', ')[1]; if(profname.indexOf(" ") == 0){ @@ -169,20 +173,16 @@ function getCourseInfo(row){ $("#topbuttons").before('

'+makeLine(date,time,place)+''); // makeLine(date,time,place); } - console.log(datetimearr); return false; } }); if(typeof coursename == 'undefined'){ coursename = $("#details h2").text(); - console.log(profname+" "+profinit); profinit = profinit.substring(0,1); profurl = document.URL; } - //console.log(coursename); getDescription(); department = coursename.substring(0,coursename.search(/\d/)-2); - //console.log(department); course_nbr = coursename.substring(coursename.search(/\d/),coursename.indexOf(" ",coursename.search(/\d/))); } @@ -240,12 +240,10 @@ function convertTime(time){ function getDistribution(){ var query = "select * from agg"; query += " where dept like '%"+department+"%'"; - query += " and prof like '%"+profname+"%'"; + query += " and prof like '%"+profname.replace(/'/g, "")+"%'"; query += " and course_nbr like '%"+course_nbr+"%'"; query += "order by a1+a2+a3+b1+b2+b3+c1+c2+c3+d1+d2+d3+f desc"; -// console.log(query); var res = grades.exec(query)[0]; -// console.log(res); var output = ""; openDialog(department,coursename,"aggregate",profname,res); } @@ -271,12 +269,7 @@ function openDialog(dep,cls,sem,professor,res){ console.log(res); var title = null if(profname == "" && typeof res != 'undefined'){ - // for(var i = 0; i<5;i++){ - // $("#profdropdown-menu").append('
  • '+res.values[i][1]+'
    • '); - // } - // $("#profdropdown").css("visibility","visible"); title = res.values[0][1]; - } var modal = document.getElementById('myModal'); var span = document.getElementsByClassName("close")[0]; @@ -294,94 +287,90 @@ function openDialog(dep,cls,sem,professor,res){ color = "#F44336"; } $("#title").append(""+" #"+uniquenum+""); + var name; if(profname == ""){ name = "Undecided Professor "; - if(typeof res == 'undefined'){ - - } } else{ name = profinit+". "+profname.substring(0,1)+profname.substring(1).toLowerCase(); } - $("#profname").text("with "+ name); -//console.log(coursename); -span.onclick = function() { - $("#myModal").fadeOut(200); - $("#snackbar").attr("class",""); -} -chart = Highcharts.chart('chart', { - chart: { - type: 'column', - backgroundColor: ' #fefefe', - spacingLeft: 10 - }, - title: { - text: null - }, - subtitle: { - text: title - }, - legend: { - enabled: false - }, - xAxis: { - title: { - text: 'Grades' + span.onclick = function() { + $("#myModal").fadeOut(200); + $("#snackbar").attr("class",""); + } + chart = Highcharts.chart('chart', { + chart: { + type: 'column', + backgroundColor: ' #fefefe', + spacingLeft: 10 }, - categories: [ - 'A', - 'A-', - 'B+', - 'B', - 'B-', - 'C+', - 'C', - 'C-', - 'D+', - 'D', - 'D-', - 'F' - ], - crosshair: true - }, - yAxis: { - min: 0, title: { - text: 'Students' - } - }, - credits: { - enabled: false - }, - lang: { - noData: "The professor hasn't taught this class :(" - }, - tooltip: { - headerFormat: '{point.key}', - pointFormat: '', - footerFormat: '
    {point.y:.0f} Students
    ', - shared: true, - useHTML: true - }, - plotOptions: { - bar: { - pointPadding: 0.2, - borderWidth: 0 + text: null }, - series: { - animation: { - duration: 700 + subtitle: { + text: title + }, + legend: { + enabled: false + }, + xAxis: { + title: { + text: 'Grades' + }, + categories: [ + 'A', + 'A-', + 'B+', + 'B', + 'B-', + 'C+', + 'C', + 'C-', + 'D+', + 'D', + 'D-', + 'F' + ], + crosshair: true + }, + yAxis: { + min: 0, + title: { + text: 'Students' } - } - }, - series: [{ - name: 'Grades', - data: [{y: data[6], color: '#4CAF50'}, {y: data[7], color: '#8BC34A'}, {y: data[8], color: '#CDDC39'}, {y: data[9], color: '#FFEB3B'}, {y: data[10], color: '#FFC107'}, {y: data[11], color: '#FFA000'}, {y: data[12], color: '#F57C00'}, {y: data[13], color: '#FF5722'}, {y: data[14], color: '#FF5252'}, {y: data[15], color: '#E64A19'}, {y: data[16], color: '#F44336'}, {y: data[17], color: '#D32F2F'}] + }, + credits: { + enabled: false + }, + lang: { + noData: "The professor hasn't taught this class :(" + }, + tooltip: { + headerFormat: '{point.key}', + pointFormat: '', + footerFormat: '
    {point.y:.0f} Students
    ', + shared: true, + useHTML: true + }, + plotOptions: { + bar: { + pointPadding: 0.2, + borderWidth: 0 + }, + series: { + animation: { + duration: 700 + } + } + }, + series: [{ + name: 'Grades', + data: [{y: data[6], color: '#4CAF50'}, {y: data[7], color: '#8BC34A'}, {y: data[8], color: '#CDDC39'}, {y: data[9], color: '#FFEB3B'}, {y: data[10], color: '#FFC107'}, {y: data[11], color: '#FFA000'}, {y: data[12], color: '#F57C00'}, {y: data[13], color: '#FF5722'}, {y: data[14], color: '#FF5252'}, {y: data[15], color: '#E64A19'}, {y: data[16], color: '#F44336'}, {y: data[17], color: '#D32F2F'}] - }] + }] }, function(chart) { // on complete if(data.length == 0){ chart.renderer.text('Could not find distribution for this Instructor teaching this Course', 100, 120) @@ -398,13 +387,13 @@ chart = Highcharts.chart('chart', { } }); // When the user clicks anywhere outside of the modal, close it -window.onclick = function(event) { - if (event.target == modal) { - $("#myModal").fadeOut(fadetime); - $("#snackbar").attr("class",""); + window.onclick = function(event) { + if (event.target == modal) { + $("#myModal").fadeOut(fadetime); + $("#snackbar").attr("class",""); - } -} + } + } } function prettifyTitle(){ diff --git a/grades.db b/grades.db index 5ea5087707cf1c21cc249469dc9629a1c83206af..7605be1e20aecc1ce3afbb0afe2928ac890ebb58 100644 GIT binary patch delta 2776 zcmWmGX@HGY7y#h!yNt2TW-u5#gJE1dEwY6oipoy1rj!;^sf=A^6v>iOT4*n&jCQmu zN?Np73R#L)N((*b$8+BI-us^OedoOA$DOuvcRY;+xmnjnQHfh4Llh-OiX9ggwDM&# z9-Nku9mVO%C_R~;EbhD}Z(C|{pUk`?DT|Wn`%9`kvM49xP->bi%8b^gW}e6>47qhi z7(zt~rAVbTW&GAsEK51c6XvIrK_*#LP}fS#rXrQ7Ob%73N;Pt+P7U>~#lo6~TGXZv zb*V>v@@POq#cjk!+VN*>~29^p||snwQ5IQrE*#^XG}lROpn z@wDU_p5-~#@H{V=81BVdUgRa#QOtT?<`rJ$HMQ-<*Lj0C*}z6N@fL6Mj=JvRyS&Hy z;T%7ZZ018gVhdaOSaC1$6Sm32<@uEDe8%T|!I$jdE57C%wP%UnvXfoxW)I)-J$u=w zu7l@C`}u(b9OMv(`H`PEqVOE?C_nQHzw#Tu^9RQ`&Yx;~i+^!KezEvB|L`yWagtL} z6bo^oF-}lIaYxbOaY<4rMJlB!Ls`mEo;1?QFt3xCNfs5zrXrQ7Ob%73s`gy58o5;0 z|G0*vCbg(d9qKAPU#v%cd1on7PUCdW(Es>M ziOUn~S=@ne3Smd#p%koJjWWI=LOdCA}_IyV%ED?!$SXG_Td#?KU7HuE7LsSPdM!d5=!6Snaw+xd*o`NB~rLa)DM2Ve0u z-|#Iv*~M=5s2w4G$M@`IAN%=%103X#x{=~xew2?Af8q#7`I+DNox(p1$2iWP{AEHv z@dSVK5C8HXCpqQ!h?xHPkSIY(QYb|#r71&M$}xcQq>)Yrnfjl|l2jm@id14CmC2zB zRjEcU)u};E22qRJ)S)i*s81dZXh1(de!EGl{|8{8e#HO) delta 1176 zcmZ9~OKclO7y#g%*`3{4KXx6D*Iwu0wd43zuV3B7Bu&Dj=>dZZqKZT# zL8Y{caOt5E2SnJ45sg5Fs)$Hp+c1a2`W_nb%02W{rT|w z-+%u3XWJKUqc(WIr|Y;Nh)crGdUDxzAXxhAco2eED+@Z)%Il5Q%%-wF8Oq!vZdto4 zoj!&XH>Urj5XeIMpI<4VKlHb~M{M*^ATjA%>6DZaZ;0>cm-QpMtgUJx5xDPaSF;ZjLt_2b0q?v-_GO(`kZ5 zJfblx$BY_9Dncq%;Y+F8SMKI>CrT!MCcxJwD~#WJSE8`ncaxPe8I_q;;rC}vZ~v)$ z*5H-%*@)&>~TwaTICHtvHL_CI>Rba-WUNZE&-S2-#b$dtb6e0Kp#H=Tn z;oXRrs8a7vcCzlgWVr2-6qQ6p#Q2cS?_WW^?$Vq?MNytA!Z3MiJnS8>JrR!ZliigR z8QbNB?_3JTd8k<_Q6?WQz#MeGcEV@!+dqb#5EO#~iYQ?+o@Z8ue|k3Q75VwsdX?_W zL4>g(-ugn1@>@8UC*xUWrFr8Nlx!aimx!w1=Q6zgz14#px3w7O6*JwZ5;)Jy0JCa* z^9+sig%@iEw=#oibt28IJ&YBRi<}sDHtSLM<|5Omh>3)!`#Y~hlV5u!U_d!~a*zaI zFe$#dmVo-ab+u~p_g=SCnn=WnWar|BV4}SCCS2S%g(&~%jk0-WrEqqofB*&}kN^QP zPyhug@Bu#vfFRI-4nm*{gaHE)V1Ouy0TaYQH|PPqpbsQ~1(F~I`av2DfDFij9LNK& f$6cTZY)}GaPytm?1G|9(2Eh=hudfvL9R2)npw1-p diff --git a/manifest.json b/manifest.json index ef7f3c17..96f58499 100644 --- a/manifest.json +++ b/manifest.json @@ -4,7 +4,7 @@ "name": "UT Course Extension", "version": "0.1.0", "description": "Chrome Extension to show more information on the UT Course catalog", - "permissions": [ + "permissions": [ "tabs", "declarativeContent", "storage", "*://*.utdirect.utexas.edu/apps/registrar/course_schedule/*", diff --git a/popup.html b/popup.html index f3c9734c..6143a711 100644 --- a/popup.html +++ b/popup.html @@ -8,7 +8,7 @@ diff --git a/popup.js b/popup.js index 7a5a72a6..2589ad85 100644 --- a/popup.js +++ b/popup.js @@ -67,9 +67,6 @@ function updateConflicts(){ } $(document).ready(function() { - $("#clear").click(function(){ - clear(); - }); $("#courseList li").click(function(){ //GACKY FIX $(this).find("#listMoreInfo").click(function(){ @@ -87,6 +84,11 @@ $(document).ready(function() { }); } updateConflicts(); + chrome.tabs.query({active: true, currentWindow: true}, function(tabs) { + for(var i = 0; i