Cosmetic changes

TEAMMATES · Apr 16, 2018 · b1f7dae · b1f7dae
1 parent 341d083
commit b1f7dae
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/src/main/java/teammates/common/datatransfer/questions/FeedbackMsqQuestionDetails.java b/src/main/java/teammates/common/datatransfer/questions/FeedbackMsqQuestionDetails.java
@@ -347,12 +347,10 @@ public int getNumOfChoicesForMsq(String courseId, FeedbackParticipantType genera
             } catch (EntityDoesNotExistException e) {
                 Assumption.fail("Course disappeared");
             }
-        } else {
-            List<InstructorAttributes> instructorList = InstructorsLogic.inst().getInstructorsForCourse(courseId);
-
-            return instructorList.size();
         }
-        return 0;
+        List<InstructorAttributes> instructorList = InstructorsLogic.inst().getInstructorsForCourse(courseId);
+
+        return instructorList.size();
     }
 
     private List<String> generateOptionList(String courseId) {
@@ -482,8 +480,8 @@ public String getNewQuestionSpecificEditFormHtml() {
              + "</div>";
     }
 
-    //Confusing Ternary flagged for the `else if` condition below.
-    //Note: Exclusion to this rule will be added in future PMD patch.
+    // Confusing Ternary flagged for the `else if` condition below.
+    // Note: Exclusion to this rule will be added in future PMD patch.
     @SuppressWarnings("PMD.ConfusingTernary")
     @Override
     public String getQuestionAdditionalInfoHtml(int questionNumber, String additionalInfoId) {