BOOK 1191 - Fix bug involving event statuses not being properly read

.eslintrc

@@ -22,7 +22,7 @@
       "vars": "all", "args": "after-used"
     }],
     "keyword-spacing": ["error", { "before": true, "after": true }],
-    "comma-dangle": "off",
+    "comma-dangle": "error",
     "indent": ["error", 2, {"SwitchCase": 1}],
     "quotes": ["error", "single"],
     "semi": ["error", "always"],

src/components/App/TitleBar/SportsbookToggle.jsx

@@ -69,7 +69,7 @@ class SportsbookToggle extends PureComponent {
 SportsbookToggle.propTypes = {
   bookMode: PropTypes.string,
   setMode: PropTypes.func,
-  navigateTo: PropTypes.func,
+  navigateTo: PropTypes.func
 };
 
 const mapStateToProps = (state) => {
@@ -99,7 +99,7 @@ const mapStateToProps = (state) => {
       } else if (objectType === ChainTypes.object_type.event.toString()) {
         // We want to have the 'first' bmgID on hand incase the user toggles to the sportsbook
         let bmg = EventPageSelector.getFirstBettingMarketGroupByEventId(state, {
-          eventId: blockchainObject,
+          eventId: blockchainObject
         });
 
         if (bmg) {
@@ -112,15 +112,15 @@ const mapStateToProps = (state) => {
   return {
     bookMode: state.getIn(['app', 'bookMode']),
     previousRoute: state.getIn(['routing', 'previousRoute']),
-    sportID, eventGroupID, eventID, bmgID,
+    sportID, eventGroupID, eventID, bmgID
   };
 };
 
 const mapDispatchToProps = (dispatch) => {
   return bindActionCreators(
     {
       setMode: AppActions.setBookMode,
-      navigateTo: NavigateActions.navigateTo,
+      navigateTo: NavigateActions.navigateTo
     },
     dispatch
   );

src/components/App/TitleBar/TitleBar.jsx

@@ -185,7 +185,7 @@ TitleBar.propTypes = {
 
 const mapStateToProps = (state) => {
   return {
-    loggedIn: state.getIn(['account', 'isLoggedIn']),
+    loggedIn: state.getIn(['account', 'isLoggedIn'])
   };
 };
 

src/components/BettingDrawers/Common/index.js

@@ -9,5 +9,5 @@ export {
   Overlay,
   Waiting,
   PlaceBetConfirm,
-  OverlayUtils,
+  OverlayUtils
 };

src/components/BettingWidgets/BackingBettingWidget/BackingBettingWidget.jsx

@@ -113,7 +113,7 @@ class BackingBettingWidget extends PureComponent {
 }
 
 BackingBettingWidget.propTypes = {
-  isLiveMarket: PropTypes.bool.isRequired,
+  isLiveMarket: PropTypes.bool.isRequired
 };
 
 const mapDispatchToProps = (dispatch) => {

src/components/BettingWidgets/BackingWidgetContainer/BackingWidgetContainer.jsx

@@ -34,7 +34,7 @@ class BackingWidgetContainer extends PureComponent {
             title={ this.props.eventName }
             eventID={ this.props.marketData.get('event_id') }
             eventTime={ this.props.eventTime }
-            eventStatus={ this.props.marketData.get('status') }
+            eventStatus={ this.props.eventStatus }
             eventRoute={ this.props.eventRoute }
             columnType={ this.props.marketData.get('description') }
             marketData={ this.props.marketData }

src/components/Login/Login.jsx

@@ -112,7 +112,7 @@ class Login extends PureComponent {
 
 const mapStateToProps = (state) => ({
   errors: state.getIn(['auth', 'loginErrors']),
-  status: state.getIn(['auth', 'loginLoadingStatus']),
+  status: state.getIn(['auth', 'loginLoadingStatus'])
 });
 
 const mapDispatchToProps = (dispatch) => bindActionCreators(

src/components/SportsBookEvent/SportsBookEvent.jsx

@@ -13,7 +13,7 @@ import {
   EventGroupPageSelector,
   MarketDrawerSelector,
   MyAccountPageSelector,
-  EventPageSelector,
+  EventPageSelector
 } from '../../selectors';
 
 class SportsBookEvent extends PureComponent {
@@ -103,7 +103,7 @@ const mapStateToProps = (state, ownProps) => {
 
   let props = {
     event,
-    oddsFormat: MyAccountPageSelector.oddsFormatSelector(state),
+    oddsFormat: MyAccountPageSelector.oddsFormatSelector(state)
   };
 
   // Populate other properties if betting market group exists
@@ -119,7 +119,7 @@ const mapStateToProps = (state, ownProps) => {
       unconfirmedBets: BettingMarketGroupPageSelector.getUnconfirmedBets(state, ownProps),
       loadingStatus: BettingMarketGroupPageSelector.getLoadingStatus(state, ownProps),
       canCreateBet: MarketDrawerSelector.canAcceptBet(state, ownProps),
-      sportName,
+      sportName
     });
   }
 

src/constants/BackingWidgetTypes.js

@@ -1,14 +1,14 @@
 const orderPriority = {
   TOP: 0,
   MIDDLE: 5,
-  BOTTOM: 10,
+  BOTTOM: 10
 };
 
 // This constant object maps onto BMG descriptions produced by BOS
 const BackingWidgetTypes = {
   MATCHODDS: 'MATCHODDS',
   MONEYLINE: 'MONEYLINE',
-  OVERUNDER: 'OVERUNDER',
+  OVERUNDER: 'OVERUNDER'
 };
 
 const BackingWidgetLayouts = {
@@ -18,22 +18,22 @@ const BackingWidgetLayouts = {
       default: 8
     },
     order: orderPriority.TOP, // Highest Priority, always show on the top.
-    numberOfMarkets: 3,
+    numberOfMarkets: 3
   },
   MONEYLINE: {
     columns: {
       eventFlag: 7,
       default: 12
     },
-    order: orderPriority.TOP, // Highest Priority, always show on the top.
+    order: orderPriority.TOP // Highest Priority, always show on the top.
   },
   OVERUNDER: {
     columns: {
       eventFlag: 12,
       default: 12
     },
-    order: orderPriority.MIDDLE, // Middle priority, show below the top.
-  },
+    order: orderPriority.MIDDLE // Middle priority, show below the top.
+  }
 };
 
 export {BackingWidgetTypes, BackingWidgetLayouts};

src/constants/BookieModes.js

@@ -1,5 +1,5 @@
 const BookieModes = {
   EXCHANGE: 'exchange',
-  SPORTSBOOK: 'sportsbook',
+  SPORTSBOOK: 'sportsbook'
 };
 export default BookieModes;

src/constants/LayoutConstants.js

@@ -4,8 +4,8 @@ const LayoutConstants = {
   betslipWidth: 360,
   splitPaneStyle: {
     top: '0px',
-    position: 'fixed',
-  },
+    position: 'fixed'
+  }
 };
 
 export default LayoutConstants;

src/constants/MessageId.js

@@ -9,7 +9,7 @@ const MessageId = {
       INCOMPLETE_BET: 'war-b-IB'
     },
     ERROR: {
-      INSUFFICIENT_FUNDS: 'err-b-IF',
+      INSUFFICIENT_FUNDS: 'err-b-IF'
     }
   },
   EXCHANGE: {

src/constants/index.js

@@ -45,5 +45,5 @@ export {
   BettingMarketResolutionTypes,
   BookieModes,
   BackingWidgetTypes,
-  LayoutConstants,
+  LayoutConstants
 };

src/reducers/AppReducer.js

@@ -18,7 +18,7 @@ const initialState = Immutable.fromJS({
   isTitleBarTransparent: true,
   gatewayAccount: {},
   showLicenseScreen: false,
-  bookMode: BookieModes.EXCHANGE,
+  bookMode: BookieModes.EXCHANGE
 
 });
 
@@ -117,7 +117,7 @@ export default function(state = initialState, action) {
 
     case ActionTypes.APP_SET_BOOK_MODE: {
       return state.merge({
-        bookMode: action.mode,
+        bookMode: action.mode
       });
     }
 

src/selectors/index.js

@@ -19,5 +19,5 @@ export {
   MyAccountPageSelector,
   QuickBetDrawerSelector,
   SidebarSelector,
-  SportPageSelector,
+  SportPageSelector
 };

src/utility/SportsbookUtils.js

@@ -241,20 +241,30 @@ const sortAndCenter = (bettingMarketGroups) => {
 };
 
 const isAbleToBet = (eventStatus) => {
+  let canBet = true; 
+
   if (eventStatus) {
-    switch (eventStatus[1]) {
+    let status = eventStatus;
+
+    // Err Handling, in case data does not come as an array.
+    if (eventStatus.constructor === Array) { 
+      status = eventStatus[1];
+    }
+
+    switch (status) {
       case EventStatus.FINISHED:
       case EventStatus.FROZEN:
       case EventStatus.COMPLETED:
       case EventStatus.SETTLED:
       case EventStatus.CANCELED:
-        return false;
+        canBet = false;
+        break;
       default:
-        return true;
+        break;
     }
   }
 
-  return true;
+  return canBet;
 };
 
 const hasBettingMarkets = (bettingMarketGroup) => {

src/utility/index.js

@@ -35,5 +35,5 @@ export {
   ObjectUtils,
   HelpAndSupportUtils,
   AuthUtils,
-  SportsbookUtils,
+  SportsbookUtils
 };