chore: Make sure 'hispanic' column only gets applied when using SIS race information

This commit is contained in:
Nelson Jovel 2023-12-12 10:53:07 -08:00
parent 2d6bbe0e5a
commit 41d942c214
2 changed files with 13 additions and 7 deletions

View file

@ -129,13 +129,12 @@ class SurveyItemValues
def races
@races ||= begin
hispanic = value_from(pattern: /Hispanic\s*Latino/i)&.downcase
race_codes ||= value_from(pattern: /Race\s*self\s*report/i)
race_codes ||= self_report = value_from(pattern: /Race\s*self\s*report/i)
race_codes ||= value_from(pattern: /^RACE$/i)
race_codes ||= value_from(pattern: %r{What is your race/ethnicity?(Please select all that apply) - Selected Choice}i)
race_codes ||= value_from(pattern: /Race Secondary/i)
race_codes ||= value_from(pattern: /Race-\s*SIS/i)
race_codes ||= value_from(pattern: /Race\s*-\s*Qcodes/i)
race_codes ||= sis ||= value_from(pattern: /Race-\s*SIS/i)
race_codes ||= sis ||= value_from(pattern: /Race\s*-\s*Qcodes/i)
race_codes ||= value_from(pattern: /RACE/i) || ""
race_codes ||= []
race_codes = race_codes.split(",")
@ -144,8 +143,14 @@ class SurveyItemValues
end.flatten
.reject(&:blank?)
.map { |race| Race.qualtrics_code_from(race) }.map(&:to_i)
race_codes = race_codes.reject { |code| code == 5 } if hispanic == "true" && race_codes.count == 1
race_codes = race_codes.push(4) if hispanic == "true"
# Only check the secondary hispanic column if we don't have self reported data and are relying on SIS data
if self_report.nil? && sis.present?
hispanic = value_from(pattern: /Hispanic\s*Latino/i)&.downcase
race_codes = race_codes.reject { |code| code == 5 } if hispanic == "true" && race_codes.count == 1
race_codes = race_codes.push(4) if hispanic == "true"
end
process_races(codes: race_codes)
end
end