]> gitweb.michael.orlitzky.com - dead/htsn-import.git/blobdiff - src/TSN/XML/Odds.hs
Make the Odds team rotation numbers optional.
[dead/htsn-import.git] / src / TSN / XML / Odds.hs
index 8061a3e23db2ec2c2eb6c3311ecebcf735fd239e..934c3d340508f89a3d0c40b770ce49c3fd9e975d 100644 (file)
@@ -18,7 +18,6 @@ module TSN.XML.Odds (
   -- * WARNING: these are private but exported to silence warnings
   OddsCasinoConstructor(..),
   OddsConstructor(..),
-  OddsGame_TeamConstructor(..),
   OddsGameConstructor(..),
   OddsGameLineConstructor(..) )
 where
@@ -66,10 +65,11 @@ import TSN.Codegen (
 import TSN.DbImport ( DbImport(..), ImportResult(..), run_dbmigrate )
 import TSN.Picklers ( xp_date_padded, xp_time, xp_time_stamp )
 import TSN.Team ( Team(..) )
-import TSN.XmlImport ( XmlImport(..), XmlImportFk(..) )
+import TSN.XmlImport ( XmlImport(..), XmlImportFkTeams(..) )
 import Xml (
+  Child(..),
   FromXml(..),
-  FromXmlFk(..),
+  FromXmlFkTeams(..),
   ToDb(..),
   pickle_unpickle,
   unpickleable,
@@ -166,7 +166,7 @@ data OddsGameHomeTeamXml =
                                         --   so we ignore the probable
                                         --   upper bound of three
                                         --   characters.
-    xml_home_team_rotation_number :: Int,
+    xml_home_team_rotation_number :: Maybe Int,
     xml_home_team_abbr            :: String,
     xml_home_team_name            :: String,
     xml_home_team_starter         :: Maybe (Int, String), -- ^ (id, name)
@@ -187,8 +187,8 @@ instance FromXml OddsGameHomeTeamXml where
   from_xml OddsGameHomeTeamXml{..} =
     Team {
       team_id   = xml_home_team_id,
-      team_abbreviation = Just xml_home_team_abbr,
-      team_name = Just xml_home_team_name }
+      abbreviation = Just xml_home_team_abbr,
+      name = Just xml_home_team_name }
 
 -- | This allows us to insert the XML representation
 --   'OddsGameHomeTeamXml' directly.
@@ -209,7 +209,7 @@ data OddsGameAwayTeamXml =
                                         --   on lengthless text fields, so
                                         --   we ignore the probable upper
                                         --   bound of three characters
-    xml_away_team_rotation_number :: Int,
+    xml_away_team_rotation_number :: Maybe Int,
     xml_away_team_abbr            :: String,
     xml_away_team_name            :: String,
     xml_away_team_starter         :: Maybe (Int, String), -- ^ (id, name)
@@ -238,16 +238,6 @@ instance FromXml OddsGameAwayTeamXml where
 instance XmlImport OddsGameAwayTeamXml where
 
 
--- * OddsGame_OddsGameTeam
-
--- | Database mapping between games and their home/away teams.
---
-data OddsGame_Team =
-  OddsGame_Team {
-    ogt_odds_games_id :: DefaultKey OddsGame,
-    ogt_away_team_id  :: DefaultKey Team,
-    ogt_home_team_id  :: DefaultKey Team }
-
 
 -- * OddsGameOverUnderXml
 
@@ -290,10 +280,12 @@ data OddsGameLine =
 data OddsGame =
   OddsGame {
     db_odds_id           :: DefaultKey Odds,
+    db_away_team_id      :: DefaultKey Team,
+    db_home_team_id      :: DefaultKey Team,
     db_game_id           :: Int,
     db_game_time         :: UTCTime, -- ^ Contains both the date and time.
-    db_away_team_rotation_number :: Int,
-    db_home_team_rotation_number :: Int,
+    db_away_team_rotation_number :: Maybe Int,
+    db_home_team_rotation_number :: Maybe Int,
     db_away_team_starter_id :: Maybe Int,
     db_away_team_starter_name :: Maybe String,
     db_home_team_starter_id :: Maybe Int,
@@ -325,19 +317,25 @@ instance ToDb OddsGameXml where
   --
   type Db OddsGameXml = OddsGame
 
-instance FromXmlFk OddsGameXml where
+
+instance Child OddsGameXml where
   -- | Each 'OddsGameXml' is contained in an 'Odds'. In other words
   --   the foreign key for 'OddsGame' points to an 'Odds'.
   --
   type Parent OddsGameXml = Odds
 
+
+instance FromXmlFkTeams OddsGameXml where
   -- | To convert from the XML representation to the database one, we
-  --   drop the home/away teams and the casino lines, but retain the
-  --   home/away rotation numbers and the starters.
+  --   drop the casino lines, but retain the home/away rotation
+  --   numbers and the starters. The foreign keys to 'Odds' and the
+  --   home/away teams are passed in.
   --
-  from_xml_fk fk OddsGameXml{..} =
+  from_xml_fk_teams fk fk_away fk_home OddsGameXml{..} =
     OddsGame {
-      db_odds_id   = fk,
+      db_odds_id = fk,
+      db_away_team_id = fk_away,
+      db_home_team_id = fk_home,
       db_game_id   = xml_game_id,
 
       db_game_time = UTCTime
@@ -365,7 +363,7 @@ instance FromXmlFk OddsGameXml where
 
 -- | This lets us insert the XML representation 'OddsGameXml' directly.
 --
-instance XmlImportFk OddsGameXml
+instance XmlImportFkTeams OddsGameXml
 
 
 -- * OddsGameWithNotes
@@ -491,6 +489,12 @@ mkPersist tsn_codegen_config [groundhog|
         - name: db_odds_id
           reference:
             onDelete: cascade
+        - name: db_away_team_id
+          reference:
+            onDelete: cascade
+        - name: db_home_team_id
+          reference:
+            onDelete: cascade
 
 - entity: OddsGameLine
   dbName: odds_games_lines
@@ -504,20 +508,6 @@ mkPersist tsn_codegen_config [groundhog|
           reference:
             onDelete: cascade
 
-- entity: OddsGame_Team
-  dbName: odds_games__teams
-  constructors:
-    - name: OddsGame_Team
-      fields:
-        - name: ogt_odds_games_id
-          reference:
-            onDelete: cascade
-        - name: ogt_away_team_id
-          reference:
-            onDelete: cascade
-        - name: ogt_home_team_id
-          reference:
-            onDelete: cascade
 |]
 
 instance DbImport Message where
@@ -527,36 +517,25 @@ instance DbImport Message where
       migrate (undefined :: Odds)
       migrate (undefined :: OddsCasino)
       migrate (undefined :: OddsGame)
-      migrate (undefined :: OddsGame_Team)
       migrate (undefined :: OddsGameLine)
 
   dbimport m = do
     -- Insert the root "odds" element and acquire its primary key (id).
     odds_id <- insert_xml m
 
-    forM_ (xml_games m) $ \g -> do
-      -- Next, we insert the home and away teams. We do this before
-      -- inserting the game itself because the game has two foreign keys
-      -- pointing to "teams".
-      away_team_id <- insert_xml_or_select (xml_away_team g)
-      home_team_id <- insert_xml_or_select (xml_home_team g)
-
-      -- Now insert the game, keyed to the "odds",
-      game_id <- insert_xml_fk odds_id g
-
-      -- Insert a record into odds_games__teams mapping the
-      -- home/away teams to this game. Use the full record syntax
-      -- because the types would let us mix up the home/away teams.
-      insert_ OddsGame_Team {
-                ogt_odds_games_id = game_id,
-                ogt_away_team_id = away_team_id,
-                ogt_home_team_id = home_team_id }
-
-      -- Finaly, we insert the lines. The over/under entries for this
+    forM_ (xml_games m) $ \game -> do
+      -- First we insert the home and away teams.
+      away_team_id <- insert_xml_or_select (xml_away_team game)
+      home_team_id <- insert_xml_or_select (xml_home_team game)
+
+      -- Now insert the game, keyed to the "odds" and its teams.
+      game_id <- insert_xml_fk_teams odds_id away_team_id home_team_id game
+
+      -- Finally, we insert the lines. The over/under entries for this
       -- game and the lines for the casinos all wind up in the same
       -- table, odds_games_lines. We can insert the over/under entries
       -- freely with empty away/home lines:
-      forM_ (xml_over_under_casinos g) $ \c -> do
+      forM_ (xml_over_under_casinos game) $ \c -> do
         -- Start by inderting the casino.
         ou_casino_id <- insert_xml_or_select c
 
@@ -573,7 +552,7 @@ instance DbImport Message where
       -- ...but then when we insert the home/away team lines, we
       -- prefer to update the existing entry rather than overwrite it
       -- or add a new record.
-      forM_ (xml_away_team_casinos $ xml_away_team g) $ \c -> do
+      forM_ (xml_away_team_casinos $ xml_away_team game) $ \c -> do
         -- insert, or more likely retrieve the existing, casino
         a_casino_id <- insert_xml_or_select c
 
@@ -585,7 +564,7 @@ instance DbImport Message where
           Ogl_Odds_Casinos_Id ==. a_casino_id
 
       -- Repeat all that for the home team.
-      forM_ (xml_home_team_casinos $ xml_home_team g) $ \c ->do
+      forM_ (xml_home_team_casinos $ xml_home_team game) $ \c ->do
         h_casino_id <- insert_xml_or_select c
         let home_line = home_away_line c
         update [Ogl_Home_Line =. home_line] $ -- WHERE
@@ -640,7 +619,7 @@ pickle_home_team =
     xpWrap (from_tuple, to_tuple) $
       xp6Tuple
         (xpElem "HomeTeamID" xpText)
-        (xpElem "HomeRotationNumber" xpInt)
+        (xpElem "HomeRotationNumber" (xpOption xpInt))
         (xpElem "HomeAbbr" xpText)
         (xpElem "HomeTeamName" xpText)
         (-- This is an ugly way to get both the HStarter ID attribute
@@ -666,7 +645,7 @@ pickle_away_team =
     xpWrap (from_tuple, to_tuple) $
       xp6Tuple
         (xpElem "AwayTeamID" xpText)
-        (xpElem "AwayRotationNumber" xpInt)
+        (xpElem "AwayRotationNumber" (xpOption xpInt))
         (xpElem "AwayAbbr" xpText)
         (xpElem "AwayTeamName" xpText)
         (-- This is an ugly way to get both the AStarter ID attribute
@@ -849,8 +828,7 @@ test_on_delete_cascade = testGroup "cascading delete tests"
       let b = undefined :: Odds
       let c = undefined :: OddsCasino
       let d = undefined :: OddsGame
-      let e = undefined :: OddsGame_Team
-      let f = undefined :: OddsGameLine
+      let e = undefined :: OddsGameLine
       actual <- withSqliteConn ":memory:" $ runDbConn $ do
                   runMigration silentMigrationLogger $ do
                     migrate a
@@ -858,7 +836,6 @@ test_on_delete_cascade = testGroup "cascading delete tests"
                     migrate c
                     migrate d
                     migrate e
-                    migrate f
                   _ <- dbimport odds
                   deleteAll b
                   count_a <- countAll a
@@ -866,7 +843,6 @@ test_on_delete_cascade = testGroup "cascading delete tests"
                   count_c <- countAll c
                   count_d <- countAll d
                   count_e <- countAll e
-                  count_f <- countAll f
                   return $ sum [count_a, count_b, count_c,
-                                count_d, count_e, count_f ]
+                                count_d, count_e ]
       actual @?= expected