fix: Fixes #187294757. Data is hidden when selecting multiple years if the default year has no student data to show

rpp-main
Nelson Jovel 2 years ago
parent a191271504
commit d52eebbc5c

@ -8,7 +8,7 @@ module Analyze
end end
def n_size(year_index) def n_size(year_index)
SurveyItemResponse.where(ell:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), SurveyItemResponse.where(ell:, survey_item: measure.student_survey_items, school:, grade: grades,
academic_year: academic_years[year_index]).select(:response_id).distinct.count academic_year: academic_years[year_index]).select(:response_id).distinct.count
end end
end end

@ -8,7 +8,7 @@ module Analyze
end end
def n_size(year_index) def n_size(year_index)
SurveyItemResponse.where(gender:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), SurveyItemResponse.where(gender:, survey_item: measure.student_survey_items, school:, grade: grades,
academic_year: academic_years[year_index]).select(:response_id).distinct.count academic_year: academic_years[year_index]).select(:response_id).distinct.count
end end
end end

@ -114,7 +114,7 @@ module Analyze
end end
def n_size(year_index) def n_size(year_index)
SurveyItemResponse.where(survey_item: measure.student_survey_items, school:, grade: grades(year_index), SurveyItemResponse.where(survey_item: measure.student_survey_items, school:, grade: grades,
academic_year: academic_years[year_index]).select(:response_id).distinct.count academic_year: academic_years[year_index]).select(:response_id).distinct.count
end end
@ -137,8 +137,8 @@ module Analyze
end end
end end
def grades(year_index) def grades
Respondent.by_school_and_year(school:, academic_year: academic_years[year_index]).enrollment_by_grade.keys Respondent.by_school_and_year(school:, academic_year: academic_years)&.enrollment_by_grade&.keys
end end
private private

@ -8,7 +8,7 @@ module Analyze
end end
def n_size(year_index) def n_size(year_index)
SurveyItemResponse.where(income:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), SurveyItemResponse.where(income:, survey_item: measure.student_survey_items, school:, grade: grades,
academic_year: academic_years[year_index]).select(:response_id).distinct.count academic_year: academic_years[year_index]).select(:response_id).distinct.count
end end
end end

@ -8,7 +8,7 @@ module Analyze
end end
def n_size(year_index) def n_size(year_index)
SurveyItemResponse.where(sped:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), SurveyItemResponse.where(sped:, survey_item: measure.student_survey_items, school:, grade: grades,
academic_year: academic_years[year_index]).select(:response_id).distinct.count academic_year: academic_years[year_index]).select(:response_id).distinct.count
end end
end end

@ -52,4 +52,3 @@ module Analyze
end end
end end
end end

@ -84,7 +84,7 @@ module Analyze
@graphs ||= [Analyze::Graph::AllData.new, @graphs ||= [Analyze::Graph::AllData.new,
Analyze::Graph::StudentsAndTeachers.new, Analyze::Graph::StudentsAndTeachers.new,
Analyze::Graph::StudentsByRace.new(races: selected_races), Analyze::Graph::StudentsByRace.new(races: selected_races),
Analyze::Graph::StudentsByGrade.new(grades: selected_grades), Analyze::Graph::StudentsByGrade.new(grades:),
Analyze::Graph::StudentsByGender.new(genders: selected_genders), Analyze::Graph::StudentsByGender.new(genders: selected_genders),
Analyze::Graph::StudentsByIncome.new(incomes: selected_incomes), Analyze::Graph::StudentsByIncome.new(incomes: selected_incomes),
Analyze::Graph::StudentsByEll.new(ells: selected_ells), Analyze::Graph::StudentsByEll.new(ells: selected_ells),
@ -159,7 +159,7 @@ module Analyze
end end
def grades def grades
@grades ||= SurveyItemResponse.where(school:, academic_year:) @grades ||= SurveyItemResponse.where(school:, academic_year: academic_years)
.where.not(grade: nil) .where.not(grade: nil)
.group(:grade) .group(:grade)
.select(:response_id) .select(:response_id)

Loading…
Cancel
Save