Change people sorting to active/inactive, then alpha within each (ignore start date)

maebeale · maebeale · commit eb5601a40fd3 · 2026-02-16T17:03:33.000-05:00
diff --git a/app/controllers/people_controller.rb b/app/controllers/people_controller.rb
@@ -181,7 +181,6 @@ def set_form_variables
                       .sort_by { |affiliation|
                         expired = affiliation.inactive? || (affiliation.end_date.present? && affiliation.end_date < Date.current)
                         [ expired ? 1 : 0,
-                          affiliation.start_date || Date.new(9999),
                           affiliation.organization&.name.to_s.downcase ]
                       }
       @person.affiliations.proxy_association.target.replace(sorted)

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,6 @@ def set_form_variables`
`181`	`181`	`.sort_by { \|affiliation\|`
`182`	`182`	`expired = affiliation.inactive? \|\| (affiliation.end_date.present? && affiliation.end_date < Date.current)`
`183`	`183`	`[ expired ? 1 : 0,`
`184`		`- affiliation.start_date \|\| Date.new(9999),`
`185`	`184`	`affiliation.organization&.name.to_s.downcase ]`
`186`	`185`	`}`
`187`	`186`	`@person.affiliations.proxy_association.target.replace(sorted)`