Source:
<code>public ActionResult execute() { String section = getRequestParamValue(Const.ParamsNames.SECTION_NAME); String sectionDetailValue = getRequestParamValue(Const.ParamsNames.SECTION_NAME_DETAIL); SectionDetail sectionDetail = SectionDetail.NOT_APPLICABLE; if (section != null && sectionDetailValue != null && !sectionDetailValue.isEmpty()) { <START> if (SectionDetail.containsSectionDetail(sectionDetailValue) == null) { <END> throw new InvalidHttpParameterException("Section detail is invalid."); } sectionDetail = SectionDetail.valueOf(sectionDetailValue); } String courseId = getNonNullRequestParamValue(Const.ParamsNames.COURSE_ID); String feedbackSessionName = getNonNullRequestParamValue(Const.ParamsNames.FEEDBACK_SESSION_NAME); String questionId = getRequestParamValue(Const.ParamsNames.FEEDBACK_QUESTION_ID); boolean isMissingResponsesShown = getBooleanRequestParamValue( Const.ParamsNames.FEEDBACK_RESULTS_INDICATE_MISSING_RESPONSES); boolean isStatsShown = getBooleanRequestParamValue(Const.ParamsNames.FEEDBACK_RESULTS_SHOWSTATS); String simulateExcessDataForTesting = getRequestParamValue("simulateExcessDataForTesting"); String fileContent = ""; InstructorAttributes instructor = logic.getInstructorForGoogleId(courseId, userInfo.id); try { if ("true".equals(simulateExcessDataForTesting)) { throw new ExceedingRangeException("This session has more responses than that can be downloaded at one go."); } if (section == null || "All".equals(section)) { fileContent = logic.getFeedbackSessionResultSummaryAsCsv( courseId, feedbackSessionName, instructor.email, isMissingResponsesShown, isStatsShown, questionId); } else { fileContent = logic.getFeedbackSessionResultSummaryInSectionAsCsv( courseId, feedbackSessionName, instructor.email, section, sectionDetail, questionId, isMissingResponsesShown, isStatsShown); } } catch (EntityDoesNotExistException e) { throw new EntityNotFoundException(e); } catch (ExceedingRangeException e) { throw new RequestExceedingRangeException(e); } return new CsvResult(fileContent); }</code><technical_language>I honestly feel SectionDetail.containsSectionDetail return true/false instead. returns here</technical_language>

Target:
public ActionResult execute() { String section = getRequestParamValue(Const.ParamsNames.SECTION_NAME); String sectionDetailValue = getRequestParamValue(Const.ParamsNames.SECTION_NAME_DETAIL); SectionDetail sectionDetail = SectionDetail.NOT_APPLICABLE; if (section != null && sectionDetailValue != null && !sectionDetailValue.isEmpty()) { if (!SectionDetail.containsSectionDetail(sectionDetailValue)) { throw new InvalidHttpParameterException("Section detail is invalid."); } sectionDetail = SectionDetail.valueOf(sectionDetailValue); } String courseId = getNonNullRequestParamValue(Const.ParamsNames.COURSE_ID); String feedbackSessionName = getNonNullRequestParamValue(Const.ParamsNames.FEEDBACK_SESSION_NAME); String questionId = getRequestParamValue(Const.ParamsNames.FEEDBACK_QUESTION_ID); boolean isMissingResponsesShown = getBooleanRequestParamValue( Const.ParamsNames.FEEDBACK_RESULTS_INDICATE_MISSING_RESPONSES); boolean isStatsShown = getBooleanRequestParamValue(Const.ParamsNames.FEEDBACK_RESULTS_SHOWSTATS); String fileContent = ""; InstructorAttributes instructor = logic.getInstructorForGoogleId(courseId, userInfo.id); try { if (section == null) { fileContent = logic.getFeedbackSessionResultSummaryAsCsv( courseId, feedbackSessionName, instructor.email, isMissingResponsesShown, isStatsShown, questionId); } else { fileContent = logic.getFeedbackSessionResultSummaryInSectionAsCsv( courseId, feedbackSessionName, instructor.email, section, sectionDetail, questionId, isMissingResponsesShown, isStatsShown); } } catch (EntityDoesNotExistException e) { throw new EntityNotFoundException(e); } catch (ExceedingRangeException e) { throw new RequestExceedingRangeException(e); } return new CsvResult(fileContent); }