From a6b303b3474f6bd24808dc393622f0e887d99255 Mon Sep 17 00:00:00 2001
From: Peter <peterlgirouard@gmail.com>
Date: Wed, 17 Jul 2024 21:05:13 -0500
Subject: [PATCH] Sort standings on home view and standings page by division
 name desc

---
 src/components/ExtendedStandingsTable.vue |  6 +++++-
 src/views/HomeView.vue                    | 13 +++++++------
 src/views/StandingsView.vue               | 13 +++++++------
 3 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/src/components/ExtendedStandingsTable.vue b/src/components/ExtendedStandingsTable.vue
index cb75ba5..b339408 100644
--- a/src/components/ExtendedStandingsTable.vue
+++ b/src/components/ExtendedStandingsTable.vue
@@ -1,7 +1,7 @@
 <template>
   <div class="extended-standings-table table-responsive-xl">
     <table class="table table-sm table-striped">
-      <template v-for="(division, index) in divisions" :key=index>
+      <template v-for="division in divisions" :key=division.name>
         <thead class="thead-dark">
           <tr>
             <th>{{ division.name }}</th>
@@ -71,11 +71,15 @@ export default {
   },
   computed: {
     divisions(): DivisionStub[] {
+      if (this.teamsByDivision.length <= 0 || this.teamsByDivision[0].length <= 0) return []
+
       const namedDivisions: DivisionStub[] = []
       for (let i = 0; i < this.teamsByDivision.length; i++) {
         namedDivisions.push({ name: this.teamsByDivision[i][0].divisionName, teamStandings: this.teamsByDivision[i] })
       }
 
+      namedDivisions.sort((a, b) => a.name < b.name ? -1 : 1)
+
       return namedDivisions
     }
 
diff --git a/src/views/HomeView.vue b/src/views/HomeView.vue
index 67892d6..b7d67a7 100644
--- a/src/views/HomeView.vue
+++ b/src/views/HomeView.vue
@@ -77,12 +77,13 @@ export default {
       this.wildcardTeams = this.teamStandings.filter(ts => ts.isWildcardTeam)
 
       const teamStandingsByDivisionAbbreviation: { [key: string]: TeamStanding[] } = {}
-      this.teamStandings.forEach(ts => {
-        if (!teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation]) {
-          teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation] = []
-        }
-        teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation].push(ts)
-      })
+      this.teamStandings.sort((a, b) => a.divisionName < b.divisionName ? -1 : 1)
+        .forEach(ts => {
+          if (!teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation]) {
+            teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation] = []
+          }
+          teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation].push(ts)
+        })
 
       this.standingsByDivision = Object.values(teamStandingsByDivisionAbbreviation)
 
diff --git a/src/views/StandingsView.vue b/src/views/StandingsView.vue
index ff71ffe..9520d7c 100644
--- a/src/views/StandingsView.vue
+++ b/src/views/StandingsView.vue
@@ -75,12 +75,13 @@ export default {
       this.wildcardTeams = this.teamStandings.filter(ts => ts.isWildcardTeam)
 
       const teamStandingsByDivisionAbbreviation: { [key: string]: TeamStanding[] } = {}
-      this.teamStandings.forEach(ts => {
-        if (!teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation]) {
-          teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation] = []
-        }
-        teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation].push(ts)
-      })
+      this.teamStandings.sort((a, b) => a.divisionName < b.divisionName ? -1 : 1)
+        .forEach(ts => {
+          if (!teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation]) {
+            teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation] = []
+          }
+          teamStandingsByDivisionAbbreviation[ts.divisionAbbreviation].push(ts)
+        })
 
       this.standingsByDivision = Object.values(teamStandingsByDivisionAbbreviation)
     },