From 11b7a51deda96565479ed8063cdddce96e28d24f Mon Sep 17 00:00:00 2001
From: Abhinav Chadaga <abhinav.chadaga@utexas.edu>
Date: Sat, 17 Feb 2024 15:22:18 -0600
Subject: [PATCH] add course button onclick handlers

---
 src/stories/injected/CoursePopup2.stories.ts  |  2 +-
 .../CoursePopup2/CourseHeadingAndActions.tsx  | 41 ++++++++++++++-----
 .../injected/CoursePopup2/CoursePopup2.tsx    |  2 +-
 3 files changed, 33 insertions(+), 12 deletions(-)

diff --git a/src/stories/injected/CoursePopup2.stories.ts b/src/stories/injected/CoursePopup2.stories.ts
index 2d88e3b4..c81fb06c 100644
--- a/src/stories/injected/CoursePopup2.stories.ts
+++ b/src/stories/injected/CoursePopup2.stories.ts
@@ -11,7 +11,7 @@ const exampleCourse: Course = new Course({
     number: '314',
     fullName: 'CS314 Data Structures',
     courseName: 'Data Structures',
-    department: 'CS',
+    department: 'C S',
     creditHours: 3,
     status: Status.OPEN,
     instructors: [new Instructor({ fullName: 'Michael Scott', firstName: 'Michael', lastName: 'Scott' })],
diff --git a/src/views/components/injected/CoursePopup2/CourseHeadingAndActions.tsx b/src/views/components/injected/CoursePopup2/CourseHeadingAndActions.tsx
index 1085e64c..00f2c742 100644
--- a/src/views/components/injected/CoursePopup2/CourseHeadingAndActions.tsx
+++ b/src/views/components/injected/CoursePopup2/CourseHeadingAndActions.tsx
@@ -1,5 +1,8 @@
 import React from 'react';
+import addCourse from 'src/pages/background/lib/addCourse';
+import openNewTab from 'src/pages/background/util/openNewTab';
 import { Course } from 'src/shared/types/Course';
+import { UserSchedule } from 'src/shared/types/UserSchedule';
 import Add from '~icons/material-symbols/add';
 import CalendarMonth from '~icons/material-symbols/calendar-month';
 import CloseIcon from '~icons/material-symbols/close';
@@ -15,6 +18,8 @@ import Text from '../../common/Text/Text';
 interface CourseHeadingAndActionsProps {
     /* The course to display */
     course: Course;
+    /* The active schedule */
+    activeSchedule: UserSchedule;
     /* The function to call when the popup should be closed */
     onClose: () => void;
 }
@@ -25,8 +30,8 @@ interface CourseHeadingAndActionsProps {
  * @param {CourseHeadingAndActionsProps} props - The component props.
  * @returns {JSX.Element} The rendered component.
  */
-const CourseHeadingAndActions = ({ course, onClose }: CourseHeadingAndActionsProps) => {
-    const { courseName, department, number, uniqueId, instructors, flags, schedule } = course;
+const CourseHeadingAndActions = ({ course, onClose, activeSchedule }: CourseHeadingAndActionsProps) => {
+    const { courseName, department, number: courseNumber, uniqueId, instructors, flags, schedule, semester } = course;
     const instructorString = instructors
         .map(instructor => {
             const { firstName, lastName } = instructor;
@@ -39,12 +44,28 @@ const CourseHeadingAndActions = ({ course, onClose }: CourseHeadingAndActionsPro
         navigator.clipboard.writeText(uniqueId.toString());
     };
 
-    const handleOpenRateMyProf = () => {
-        instructors.forEach(instructor => {
+    const handleOpenRateMyProf = async () => {
+        const openTabs = instructors.map(instructor => {
             const { fullName } = instructor;
             const url = `https://www.ratemyprofessors.com/search/professors/1255?q=${fullName}`;
-            window.open(url, '_blank')?.focus();
+            return openNewTab(url);
         });
+
+        await Promise.all(openTabs);
+    };
+
+    const handleOpenCES = () => {
+        // TODO: not implemented
+    };
+
+    const handleOpenPastSyllabi = async () => {
+        const firstInstructor = instructors[0];
+        const url = `https://utdirect.utexas.edu/apps/student/coursedocs/nlogon/?year=&semester=${semester}&department=${department}&course_number=${courseNumber}&course_title=${courseName}&unique=${uniqueId}&instructor_first=${firstInstructor.firstName}&instructor_last=${firstInstructor.lastName}&course_type=In+Residence&search=Search`;
+        await openNewTab(url);
+    };
+
+    const handleAddCourse = async () => {
+        await addCourse(activeSchedule.name, course);
     };
 
     return (
@@ -56,12 +77,12 @@ const CourseHeadingAndActions = ({ course, onClose }: CourseHeadingAndActionsPro
                     </Text>
                     <Text variant='h1' className='flex-1 whitespace-nowrap'>
                         {' '}
-                        ({department} {number})
+                        ({department} {courseNumber})
                     </Text>
                     <Button color='ut-burntorange' variant='single' icon={Copy} onClick={handleCopy}>
                         {uniqueId}
                     </Button>
-                    <button className='btn bg-transparent p-0'>
+                    <button className='btn bg-transparent p-0' onClick={onClose}>
                         <CloseIcon className='h-7 w-7' />
                     </button>
                 </div>
@@ -98,13 +119,13 @@ const CourseHeadingAndActions = ({ course, onClose }: CourseHeadingAndActionsPro
                 <Button variant='outline' color='ut-blue' icon={Reviews} onClick={handleOpenRateMyProf}>
                     RateMyProf
                 </Button>
-                <Button variant='outline' color='ut-teal' icon={Mood}>
+                <Button variant='outline' color='ut-teal' icon={Mood} onClick={handleOpenCES}>
                     CES
                 </Button>
-                <Button variant='outline' color='ut-orange' icon={Description}>
+                <Button variant='outline' color='ut-orange' icon={Description} onClick={handleOpenPastSyllabi}>
                     Past Syllabi
                 </Button>
-                <Button variant='filled' color='ut-green' icon={Add}>
+                <Button variant='filled' color='ut-green' icon={Add} onClick={handleAddCourse}>
                     Add Course
                 </Button>
             </div>
diff --git a/src/views/components/injected/CoursePopup2/CoursePopup2.tsx b/src/views/components/injected/CoursePopup2/CoursePopup2.tsx
index 2452f0d6..407fd0fa 100644
--- a/src/views/components/injected/CoursePopup2/CoursePopup2.tsx
+++ b/src/views/components/injected/CoursePopup2/CoursePopup2.tsx
@@ -13,7 +13,7 @@ interface CoursePopup2Props {
 const CoursePopup2 = ({ course, activeSchedule, onClose }: CoursePopup2Props) => (
     <Popup overlay className='px-6' onClose={onClose}>
         <div className='flex flex-col'>
-            <CourseHeadingAndActions course={course} onClose={onClose} />
+            <CourseHeadingAndActions course={course} onClose={onClose} activeSchedule={activeSchedule} />
         </div>
     </Popup>
 );