[#8541] Instructor: home page: give menu option to edit a course (#8725)

TEAMMATES · Apr 10, 2018 · 25008d1 · 25008d1
1 parent 173b33d
commit 25008d1
Show file tree

Hide file tree

Showing 16 changed files with 136 additions and 0 deletions.
diff --git a/src/main/java/teammates/ui/pagedata/InstructorHomeCourseAjaxPageData.java b/src/main/java/teammates/ui/pagedata/InstructorHomeCourseAjaxPageData.java
@@ -111,6 +111,7 @@ private List<ElementTag> createCourseTableLinks(InstructorAttributes instructor,
         instructors.addNestedElement(edit);
 
         courses.addNestedElement(archive);
+        courses.addNestedElement(edit);
         courses.addNestedElement(delete);
 
         return Arrays.asList(students, instructors, sessions, courses);

diff --git a/src/test/resources/pages/instructorHomeCourseArchiveSuccessful.html b/src/test/resources/pages/instructorHomeCourseArchiveSuccessful.html
@@ -194,6 +194,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTML.html b/src/test/resources/pages/instructorHomeHTML.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLResponseRateFail.html b/src/test/resources/pages/instructorHomeHTMLResponseRateFail.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLResponseRatePass.html b/src/test/resources/pages/instructorHomeHTMLResponseRatePass.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLSortByDate.html b/src/test/resources/pages/instructorHomeHTMLSortByDate.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLSortById.html b/src/test/resources/pages/instructorHomeHTMLSortById.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLSortByName.html b/src/test/resources/pages/instructorHomeHTMLSortByName.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -697,6 +702,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLSortSessionsByEndDate.html b/src/test/resources/pages/instructorHomeHTMLSortSessionsByEndDate.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLSortSessionsByName.html b/src/test/resources/pages/instructorHomeHTMLSortSessionsByName.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete

diff --git a/src/test/resources/pages/instructorHomeHTMLSortSessionsByStartDate.html b/src/test/resources/pages/instructorHomeHTMLSortSessionsByStartDate.html
@@ -187,6 +187,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS1101&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS1101" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS1101&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete
@@ -333,6 +338,11 @@ <h5 class="inline-block">
                     Archive
                   </a>
                 </li>
+                <li>
+                  <a class="btn-tm-actions course-edit-for-test" data-original-title="Edit Course information and instructor list" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseEditPage?courseid=CHomeUiT.CS2104&user=CHomeUiT.instructor.tmms" title="">
+                    View / Edit
+                  </a>
+                </li>
                 <li>
                   <a class="btn-tm-actions course-delete-for-test course-delete-link" data-course-id="CHomeUiT.CS2104" data-original-title="Delete the course and its corresponding students and sessions" data-placement="left" data-toggle="tooltip" href="/page/instructorCourseDelete?courseid=CHomeUiT.CS2104&next=%2Fpage%2FinstructorHomePage&user=CHomeUiT.instructor.tmms&token=${sessionToken}" title="">
                     Delete