show record at the pick level (winsleague#183)

machadoit · Aug 8, 2016 · 50527ef · 50527ef
1 parent 7c49864
commit 50527ef
Show file tree

Hide file tree

Showing 7 changed files with 40 additions and 7 deletions.
diff --git a/app/imports/api/pool_team_picks/pool_team_picks.js b/app/imports/api/pool_team_picks/pool_team_picks.js
@@ -5,6 +5,7 @@ import { Factory } from 'meteor/dburles:factory';
 import log from '../../utils/log';
 
 import { LeagueTeams } from '../league_teams/league_teams';
+import { SeasonLeagueTeams } from '../season_league_teams/season_league_teams';
 import { PoolTeams } from '../pool_teams/pool_teams';
 import { Pools } from '../pools/pools';
 import { Seasons } from '../seasons/seasons';
@@ -100,6 +101,14 @@ PoolTeamPicks.schema = new SimpleSchema({
     decimal: true,
     defaultValue: 0,
   },
+  actualLosses: {
+    type: Number,
+    defaultValue: 0,
+  },
+  actualTies: {
+    type: Number,
+    defaultValue: 0,
+  },
   createdAt: {
     // Force value to be current date (on server) upon insert
     // and prevent updates thereafter.
@@ -129,6 +138,14 @@ PoolTeamPicks.schema = new SimpleSchema({
 
 PoolTeamPicks.attachSchema(PoolTeamPicks.schema);
 
+PoolTeamPicks.helpers({
+  record() {
+    if (this.actualTies > 0) {
+      return `${this.actualWins}-${this.actualLosses}-${this.actualTies}`;
+    }
+    return `${this.actualWins}-${this.actualLosses}`;
+  },
+});
 
 /* Access control */
 function isPoolTeamOwner(userId, poolTeamId) {

diff --git a/app/imports/api/pool_team_picks/server/hooks.js b/app/imports/api/pool_team_picks/server/hooks.js
@@ -6,7 +6,7 @@ PoolTeamPicks.hookOptions.after.update = { fetchPrevious: false };
 
 function updatePoolTeam(doc) {
   PoolTeamUpdater.updateTeamSummary(doc.poolTeamId);
-  PoolTeamUpdater.updatePoolTeamWins(doc.poolTeamId);
+  PoolTeamUpdater.updatePoolTeamRecord(doc.poolTeamId);
   PoolTeamUpdater.updatePoolTeamPickQuality(doc.poolTeamId);
 }
 

diff --git a/app/imports/api/pool_teams/server/updater.js b/app/imports/api/pool_teams/server/updater.js
@@ -13,15 +13,15 @@ export default {
 
     const poolTeamPicks = PoolTeamPicks.find({ leagueTeamId });
     poolTeamPicks.forEach(poolTeamPick => {
-      this.updatePoolTeamWins(poolTeamPick.poolTeamId);
+      this.updatePoolTeamRecord(poolTeamPick.poolTeamId);
       this.updatePoolTeamPickQuality(poolTeamPick.poolTeamId);
     });
 
     log.debug('Done finding PoolTeams who picked leagueTeamId:', leagueTeamId);
   },
 
-  updatePoolTeamWins(poolTeamId) {
-    log.info('Updating PoolTeam wins', poolTeamId);
+  updatePoolTeamRecord(poolTeamId) {
+    log.info('Updating PoolTeam record', poolTeamId);
 
     let totalWins = 0;
     let totalLosses = 0;
@@ -34,8 +34,15 @@ export default {
       const seasonId = poolTeamPick.seasonId;
       const leagueTeamId = poolTeamPick.leagueTeamId;
       const seasonLeagueTeam = SeasonLeagueTeams.findOne({ seasonId, leagueTeamId });
-      log.debug('Found seasonLeagueTeam', seasonLeagueTeam);
       if (seasonLeagueTeam) {
+        PoolTeamPicks.direct.update(poolTeamPick._id, {
+          $set: {
+            actualWins: seasonLeagueTeam.wins,
+            actualLosses: seasonLeagueTeam.losses,
+            actualTies: seasonLeagueTeam.ties,
+          },
+        });
+
         totalWins += seasonLeagueTeam.wins;
         totalLosses += seasonLeagueTeam.losses;
         totalGames += seasonLeagueTeam.totalGames();

diff --git a/app/imports/api/pool_teams/server/updater.tests.js b/app/imports/api/pool_teams/server/updater.tests.js
@@ -38,7 +38,7 @@ describe('Pool Teams Updater', () => {
         period: 'final',
       });
 
-      PoolTeamsUpdater.updatePoolTeamWins(poolTeam._id);
+      PoolTeamsUpdater.updatePoolTeamRecord(poolTeam._id);
 
       poolTeam = PoolTeams.findOne(poolTeam._id);
       log.debug('poolTeam:', poolTeam);

diff --git a/app/imports/api/season_league_teams/season_league_teams.js b/app/imports/api/season_league_teams/season_league_teams.js
@@ -99,6 +99,13 @@ SeasonLeagueTeams.helpers({
   totalGames() {
     return this.wins + this.losses + this.ties;
   },
+
+  record() {
+    if (this.ties > 0) {
+      return `${this.wins}-${this.losses}-${this.ties}`;
+    }
+    return `${this.wins}-${this.losses}`;
+  }
 });
 
 SeasonLeagueTeams.deny({

diff --git a/app/imports/ui/pages/pool-teams-show-page.html b/app/imports/ui/pages/pool-teams-show-page.html
@@ -10,6 +10,7 @@ <h3 id="PoolTeams_show_title">{{poolTeamName}}</h3>
         <tr>
           <th>Pick</th>
           <th>Team</th>
+          <th>Record</th>
           <th>Pick Quality</th>
           <th>&nbsp;</th>
         </tr>
@@ -19,6 +20,7 @@ <h3 id="PoolTeams_show_title">{{poolTeamName}}</h3>
           <tr id="{{poolTeamPick._id}}">
             <td class="PoolTeamPick">{{poolTeamPick.pickNumber}}</td>
             <td>{{leagueTeamName poolTeamPick.leagueTeamId}}</td>
+            <td>{{poolTeamPick.record}}</td>
             <td>{{roundedPickQuality poolTeamPick.pickQuality}}</td>
             <td>
               {{#if isCommissioner}}

diff --git a/app/server/migrations/v1_add_total_losses_to_pool_teams.js b/app/server/migrations/v1_add_total_losses_to_pool_teams.js
@@ -7,7 +7,7 @@ Migrations.add({
   name: 'Adds totalLosses field to PoolTeams and populates it',
   up: () => {
     PoolTeams.find().forEach(poolTeam => {
-      PoolTeamsUpdater.updatePoolTeamWins(poolTeam);
+      PoolTeamsUpdater.updatePoolTeamRecord(poolTeam);
     });
   },
   down: () => {