From a191271504e3179c02a34c118872ca2ad8a50103 Mon Sep 17 00:00:00 2001 From: Nelson Jovel Date: Tue, 26 Mar 2024 15:32:54 -0700 Subject: [PATCH] Revert "fix: Fixes #187294757. Data is hidden when selecting multiple years if the default year has no student data to show" This reverts commit b9bdd707b5b2795ffa1f5cb15c9d665cf9b0d650. --- app/presenters/analyze/graph/column/ell_column/ell_count.rb | 2 +- .../analyze/graph/column/gender_column/gender_count.rb | 2 +- .../analyze/graph/column/grouped_bar_column_presenter.rb | 6 +++--- .../analyze/graph/column/income_column/income_count.rb | 2 +- .../analyze/graph/column/sped_column/sped_count.rb | 2 +- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/app/presenters/analyze/graph/column/ell_column/ell_count.rb b/app/presenters/analyze/graph/column/ell_column/ell_count.rb index 66c0ca79..fa20518a 100644 --- a/app/presenters/analyze/graph/column/ell_column/ell_count.rb +++ b/app/presenters/analyze/graph/column/ell_column/ell_count.rb @@ -8,7 +8,7 @@ module Analyze end def n_size(year_index) - SurveyItemResponse.where(ell:, survey_item: measure.student_survey_items, school:, grade: grades, + SurveyItemResponse.where(ell:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), academic_year: academic_years[year_index]).select(:response_id).distinct.count end end diff --git a/app/presenters/analyze/graph/column/gender_column/gender_count.rb b/app/presenters/analyze/graph/column/gender_column/gender_count.rb index 874c7cde..97460248 100644 --- a/app/presenters/analyze/graph/column/gender_column/gender_count.rb +++ b/app/presenters/analyze/graph/column/gender_column/gender_count.rb @@ -8,7 +8,7 @@ module Analyze end def n_size(year_index) - SurveyItemResponse.where(gender:, survey_item: measure.student_survey_items, school:, grade: grades, + SurveyItemResponse.where(gender:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), academic_year: academic_years[year_index]).select(:response_id).distinct.count end end diff --git a/app/presenters/analyze/graph/column/grouped_bar_column_presenter.rb b/app/presenters/analyze/graph/column/grouped_bar_column_presenter.rb index 8277f38d..1206fccc 100644 --- a/app/presenters/analyze/graph/column/grouped_bar_column_presenter.rb +++ b/app/presenters/analyze/graph/column/grouped_bar_column_presenter.rb @@ -114,7 +114,7 @@ module Analyze end def n_size(year_index) - SurveyItemResponse.where(survey_item: measure.student_survey_items, school:, grade: grades, + SurveyItemResponse.where(survey_item: measure.student_survey_items, school:, grade: grades(year_index), academic_year: academic_years[year_index]).select(:response_id).distinct.count end @@ -137,8 +137,8 @@ module Analyze end end - def grades - Respondent.by_school_and_year(school:, academic_year: academic_years)&.enrollment_by_grade&.keys + def grades(year_index) + Respondent.by_school_and_year(school:, academic_year: academic_years[year_index]).enrollment_by_grade.keys end private diff --git a/app/presenters/analyze/graph/column/income_column/income_count.rb b/app/presenters/analyze/graph/column/income_column/income_count.rb index a61bdc61..642f7308 100644 --- a/app/presenters/analyze/graph/column/income_column/income_count.rb +++ b/app/presenters/analyze/graph/column/income_column/income_count.rb @@ -8,7 +8,7 @@ module Analyze end def n_size(year_index) - SurveyItemResponse.where(income:, survey_item: measure.student_survey_items, school:, grade: grades, + SurveyItemResponse.where(income:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), academic_year: academic_years[year_index]).select(:response_id).distinct.count end end diff --git a/app/presenters/analyze/graph/column/sped_column/sped_count.rb b/app/presenters/analyze/graph/column/sped_column/sped_count.rb index 61975708..72f1b78f 100644 --- a/app/presenters/analyze/graph/column/sped_column/sped_count.rb +++ b/app/presenters/analyze/graph/column/sped_column/sped_count.rb @@ -8,7 +8,7 @@ module Analyze end def n_size(year_index) - SurveyItemResponse.where(sped:, survey_item: measure.student_survey_items, school:, grade: grades, + SurveyItemResponse.where(sped:, survey_item: measure.student_survey_items, school:, grade: grades(year_index), academic_year: academic_years[year_index]).select(:response_id).distinct.count end end