sportdb-sync 1.0.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 942eecead4fedb4294a6d73db750063aa253ec14
-  data.tar.gz: 30ba069b6309c65d8e8ba979d315e5c31d2ecbec
+  metadata.gz: e69405f0f6aab33c299ed7479747f1adb6e90282
+  data.tar.gz: ef45575530d5b0e899ff08f69089462486fd580a
 SHA512:
-  metadata.gz: 5bd714ea1d982ba37ba6b5490e58894b16bd0ca0b81fa44e2d56fca4e772c1ca395a6423b24b40c243a9d124ac4bba38c76cdf0e739c5148109d0f7b533221b4
-  data.tar.gz: fc54ed580b7a5ea80fd0c053343c5c2bd6c34b5e2d5744a09e018a708f6fef56badb53014751dd0043740acf37b005b9eb422f12f1ba18caca0ff40cf088a91d
+  metadata.gz: f969dc99e13d4b5b98fddf63925e44f24e799cc5ae7d6ee8db34738e1140c9bbe8657199f389586a9ef6a16b895e39207f3e34e98b264c978934a853ee08800e
+  data.tar.gz: 51a8e71e961fdd780561a038afe381026c5aca39838f210825b72f2aafc583bc85db62e5cf6190e32ec1256a983d410de26e1033b5ed9257020819e6894f4bd2

data/Rakefile

@@ -3,7 +3,7 @@ require './lib/sportdb/sync/version.rb'
 
 Hoe.spec 'sportdb-sync' do
 
-  self.version = SportDb::Sync::VERSION
+  self.version = SportDb::Module::Sync::VERSION
 
   self.summary = "sportdb-sync - sport.db sync helpers for leagues, seasons, clubs, match schedules and results, and more"
   self.description = summary
@@ -20,8 +20,8 @@ Hoe.spec 'sportdb-sync' do
   self.licenses = ['Public Domain']
 
   self.extra_deps = [
-    ['sportdb-config',  '>= 1.0.1'],
-    ['sportdb-models',  '>= 1.19.0'],
+    ['sportdb-catalogs',  '>= 1.0.0'],
+    ['sportdb-models',    '>= 2.0.2'],
   ]
 
   self.spec_extras = {

data/lib/sportdb/sync.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 
-require 'sportdb/config'
+require 'sportdb/catalogs'
 require 'sportdb/models'   ## add sql database support
 
 
@@ -16,4 +16,4 @@ require 'sportdb/sync/club'
 require 'sportdb/sync/sync'
 
 
-puts SportDb::Sync.banner   # say hello
+puts SportDb::Module::Sync.banner   # say hello

data/lib/sportdb/sync/club.rb

@@ -50,8 +50,8 @@ module SportDb
       #  finders
 
       def self.find_or_create( club )
-        ## note: assume "canonical uniquie" names/titles for now for clubs
-        rec = Model::Team.find_by( title: club.name )
+        ## note: assume "canonical uniquie" names for now for clubs
+        rec = Model::Team.find_by( name: club.name )
         if rec.nil?
 
           ## todo/fix:  move auto-key gen to structs for re(use)!!!!!!
@@ -61,7 +61,7 @@ module SportDb
 
           attribs = {
               key:        key,
-              title:      club.name,
+              name:       club.name,
               country_id: Sync::Country.find_or_create( club.country ).id,
               club:       true,
               national:   false  ## check -is default anyway - use - why? why not?
@@ -71,7 +71,7 @@ module SportDb
           attribs[:code] = club.code   if club.code   ## add code (abbreviation) if present
 
           if club.alt_names.empty? == false
-            attribs[:synonyms] = club.alt_names.join('|')
+            attribs[:alt_names] = club.alt_names.join('|')
           end
 
           rec = Model::Team.create!( attribs )

data/lib/sportdb/sync/country.rb

@@ -57,8 +57,8 @@ module SportDb
         attribs = {
           key:  country.key,
           name: country.name,
-          code: country.fifa,  ## fix:  uses fifa code now (should be iso-alpha3 if available)
-          fifa: country.fifa,
+          code: country.code,  ## fix:  uses fifa code now (should be iso-alpha3 if available)
+          ## fifa: country.fifa,
           area: 1,
           pop:  1
         }

data/lib/sportdb/sync/event.rb

@@ -79,8 +79,8 @@ module SportDb
           ##                               otherwise use 2017, 7, 1
           ##  start_at use year and 7,1 e.g. Date.new( 2017, 7, 1 )
           ## hack:  fix/todo1!!
-          ##   add "fake" start_at date for now
-          attribs[:start_at]  = if season.year?  ## e.g. assume 2018 etc.
+          ##   add "fake" start_date for now
+          attribs[:start_date]  = if season.year?  ## e.g. assume 2018 etc.
                                    Date.new( season.start_year, 1, 1 )
                                 else  ## assume 2014/15 etc.
                                    Date.new( season.start_year, 7, 1 )

data/lib/sportdb/sync/league.rb

@@ -40,11 +40,8 @@ module SportDb
     def self.find_or_create( league )
       rec = find( league )
       if rec.nil?
-         ## use title and not name - why? why not?
-         ##  quick fix:  change name to title
-
          attribs = { key:   league.key,
-                     title: league.name }
+                     name:  league.name }
 
         if league.country
            attribs[ :country_id ] = Sync::Country.find_or_create( league.country ).id

data/lib/sportdb/sync/season.rb

@@ -43,7 +43,7 @@ module SportDb
         rec = find( season )
         if rec.nil?
            attribs = { key:   season.key,
-                       title: season.name  }
+                       name:  season.name  }
            rec = Model::Season.create!( attribs )
         end
         rec

data/lib/sportdb/sync/sync.rb

@@ -6,14 +6,14 @@ module SportDb
 
   class NationalTeam
     def self.find_or_create( team )
-      rec = Model::Team.find_by( title: team.name )
+      rec = Model::Team.find_by( name: team.name )
       if rec.nil?
         puts "add national team: #{team.key}, #{team.name}, #{team.country.name} (#{team.country.key})"
 
         ### note: key expected three or more lowercase letters a-z /\A[a-z]{3,}\z/
         attribs = {
           key:        team.key,   ## note: always use downcase fifa code for now!!!
-          title:      team.name,
+          name:       team.name,
           code:       team.code,
           country_id: Sync::Country.find_or_create( team.country ).id,
           club:       false,
@@ -21,7 +21,7 @@ module SportDb
         }
 
         if team.alt_names.empty? == false
-          attribs[:synonyms] = team.alt_names.join('|')
+          attribs[:alt_names] = team.alt_names.join('|')
         end
 
         rec = Model::Team.create!( attribs )
@@ -64,13 +64,20 @@ module SportDb
 
   class Round
     def self.find_or_create( round, event: )
-       rec = Model::Round.find_by( title: round.title, event_id: event.id )
+       rec = Model::Round.find_by( name: round.name, event_id: event.id )
        if rec.nil?
+         ## find last pos - check if it can be nil?
+         max_pos = Model::Round.where( event_id: event.id ).maximum( 'pos' )
+         max_pos = max_pos ? max_pos+1 : 1
+
          attribs = { event_id: event.id,
-                     title:    round.title,
-                     pos:      round.pos,
-                     start_at: event.start_at.to_date
+                     name:     round.name,
+                     pos:      max_pos
                    }
+
+         ## todo/fix:  check if round has (optional) start or end date and add!!!
+         ## attribs[ :start_date] = round.start_date.to_date
+
          rec = Model::Round.create!( attribs )
        end
        rec
@@ -80,12 +87,18 @@ module SportDb
 
   class Group
     def self.find_or_create( group, event: )
-       rec = Model::Group.find_by( title: group.title, event_id: event.id )
+       rec = Model::Group.find_by( name: group.name, event_id: event.id )
        if rec.nil?
+         ## find last pos - check if it can be nil?
+         max_pos = Model::Group.where( event_id: event.id ).maximum( 'pos' )
+         max_pos = max_pos ? max_pos+1 : 1
+
          attribs = { event_id: event.id,
-                     title:    group.title,
-                     pos:      group.pos
+                     name:     group.name,
+                     pos:      max_pos
                    }
+
+         ## todo/fix: check/add optional group key (was: pos before)!!!!
          rec = Model::Group.create!( attribs )
        end
        ## todo/fix: add/update teams in group too!!!!!
@@ -96,7 +109,7 @@ module SportDb
 
   class Stage
     def self.find( name, event: )
-      Model::Stage.find_by( title: name, event_id: event.id  )
+      Model::Stage.find_by( name: name, event_id: event.id  )
     end
     def self.find!( name, event: )
       rec = find( name, event: event  )
@@ -112,10 +125,8 @@ module SportDb
     def self.find_or_create( name, event: )
        rec = find( name, event: event )
        if rec.nil?
-         ## use title and not name - why? why not?
-         ##  quick fix:  change name to title
          attribs = { event_id: event.id,
-                     title:    name,
+                     name:     name,
                    }
          rec = Model::Stage.create!( attribs )
        end
@@ -125,21 +136,28 @@ module SportDb
 
 
 
-  class Match   ## todo/check:  add alias for Game class - why? why not?
+  class Match
+    ### todo/fix: rename to create!!   (add update support later) !!!!
+    ##    use update_by_round  or update_by_date or update_by_teams or such
+    ##      NO easy (unique always auto-id match) possible!!!!!!
     def self.create_or_update( match, event: )
        ## note: MUST find round, thus, use bang (!)
 
        ## todo/check: allow strings too - why? why not?
 
-       ## query for round - allow string or round rec
-       round_title = match.round.is_a?( String ) ? match.round : match.round.title
-       round_rec   = Model::Round.find_by!( event_id: event.id,
-                                            title:    round_title )
+       round_rec = if match.round
+                     ## query for round - allow string or round rec
+                     round_name  = match.round.is_a?( String ) ? match.round : match.round.name
+                     Model::Round.find_by!( event_id: event.id,
+                                            name:     round_name )
+                   else    # note: allow matches WITHOUT rounds too (e.g. England Football League 1888 and others)
+                     nil
+                   end
 
        ## todo/check: allow fallback with db lookup if NOT found in cache - why? why not?
        ##  or better use Sync::Team.find_or_create( team )  !!!!!!! to auto-create on first hit!
        ##    || Team.find_or_create( team1 )  -- note: does NOT work for string (only recs) - what to do?
-       ##    || Model::Team.find_by!( title: team1_name )
+       ##    || Model::Team.find_by!( name: team1_name )
        team1_name   = match.team1.is_a?( String ) ? match.team1 : match.team1.name
        team1_rec    = Team.cache[ team1_name ]
        team2_name   = match.team2.is_a?( String ) ? match.team2 : match.team2.name
@@ -147,32 +165,63 @@ module SportDb
 
        ## check optional group (e.g. Group A, etc.)
        group_rec = if match.group
-                     group_title = match.group.is_a?( String ) ? match.group : match.group.title
+                     group_name = match.group.is_a?( String ) ? match.group : match.group.name
                      Model::Group.find_by!( event_id: event.id,
-                                            title:    group_title )
+                                            name:     group_name )
                    else
                      nil
                    end
 
+       ## check optional stage (e.g. Regular, Play Off, Relegation, etc. )
+       stage_rec = if match.stage
+                     stage_name = match.stage.is_a?( String ) ? match.stage : match.stage.name
+                     Model::Stage.find_by!( event_id: event.id,
+                                            name:     stage_name )
+                   else
+                     nil
+                   end
+
+       ### todo/check: what happens if there's more than one match? exception raised??
+       rec = if round_rec
+               ## add match status too? allows [abandoned] and [replay] in same round
+               Model::Match.find_by( round_id: round_rec.id,
+                                     team1_id: team1_rec.id,
+                                     team2_id: team2_rec.id )
+             else
+               ## always assume new record for now
+               ##   check for date or such - why? why not?
+               nil
+             end
 
-       rec = Model::Game.find_by( round_id: round_rec.id,
-                                  team1_id: team1_rec.id,
-                                  team2_id: team2_rec.id )
        if rec.nil?
-         attribs = { round_id: round_rec.id,
+        ## find last pos - check if it can be nil?  yes, is nil if no records found
+         max_pos = Model::Match.where( event_id: event.id ).maximum( 'pos' )
+         max_pos = max_pos ? max_pos+1 : 1
+
+         attribs = { event_id: event.id,          ## todo/fix: change to data struct too?
                      team1_id: team1_rec.id,
                      team2_id: team2_rec.id,
-                     pos:      999,    ## make optional why? why not? - change to num?
-                     play_at:  match.date.to_date,
+                     pos:      max_pos,
+                     date:     match.date.to_date,  ## todo/fix: split and add date & time!!!!
                      score1:   match.score1,
                      score2:   match.score2,
                      score1i:  match.score1i,
-                     score2i:  match.score2i }
+                     score2i:  match.score2i,
+                     score1et: match.score1et,
+                     score2et: match.score2et,
+                     score1p:  match.score1p,
+                     score2p:  match.score2p,
+                     status:   match.status }
+
+         attribs[ :round_id ] = round_rec.id   if round_rec
          attribs[ :group_id ] = group_rec.id   if group_rec
+         attribs[ :stage_id ] = stage_rec.id   if stage_rec
 
-         rec = Model::Game.create!( attribs )
+         rec = Model::Match.create!( attribs )
        else
          # update - todo
+         puts "!! ERROR - match updates not yet supported (only inserts); sorry"
+         exit 1
        end
        rec
     end

data/lib/sportdb/sync/version.rb

@@ -2,11 +2,12 @@
 
 
 module SportDb
+module Module
 module Sync
 
   MAJOR = 1    ## todo: namespace inside version or something - why? why not??
-  MINOR = 0
-  PATCH = 0
+  MINOR = 1
+  PATCH = 1
   VERSION = [MAJOR,MINOR,PATCH].join('.')
 
   def self.version
@@ -22,4 +23,5 @@ module Sync
   end
 
 end # module Sync
+end # module Module
 end # module SportDb

data/test/helper.rb

@@ -1,6 +1,6 @@
 ## note: use the local version of sportdb gems
 $LOAD_PATH.unshift( File.expand_path( '../sportdb-formats/lib' ))
-$LOAD_PATH.unshift( File.expand_path( '../sportdb-config/lib' ))
+$LOAD_PATH.unshift( File.expand_path( '../sportdb-catalogs/lib' ))
 $LOAD_PATH.unshift( File.expand_path( '../sportdb-models/lib' ))
 
 

data/test/test_country.rb

@@ -13,8 +13,8 @@ class TestCountry < MiniTest::Test
   Country = SportDb::Sync::Country
 
   def test_find  # note: find uses "data" structs
-    at  = SportDb::Import::Country.new( key: 'at',  name: 'Austria', fifa: 'AUT' )
-    eng = SportDb::Import::Country.new( key: 'eng', name: 'England', fifa: 'ENG' )
+    at  = SportDb::Import::Country.new( key: 'at',  name: 'Austria', code: 'AUT' )
+    eng = SportDb::Import::Country.new( key: 'eng', name: 'England', code: 'ENG' )
 
     rec  = Country.find_or_create( at )
     rec2 = Country.find_or_create( at )
@@ -28,23 +28,23 @@ class TestCountry < MiniTest::Test
     rec = Country.search_or_create!( 'at' )  ## try (iso-alpha2) key
     assert_equal 'Austria', rec.name
     assert_equal 'at',      rec.key
-    assert_equal 'AUT',     rec.fifa
+    assert_equal 'AUT',     rec.code
 
     rec = Country.search_or_create!( 'aut' )  ## try fifa code
     assert_equal 'Austria', rec.name
     assert_equal 'at',      rec.key
-    assert_equal 'AUT',     rec.fifa
+    assert_equal 'AUT',     rec.code
 
 
     rec = Country.search_or_create!( 'eng' )
     assert_equal 'England', rec.name
     assert_equal 'eng',     rec.key
-    assert_equal 'ENG',     rec.fifa
+    assert_equal 'ENG',     rec.code
 
     rec = Country.search_or_create!( 'eng' )
     assert_equal 'England', rec.name
     assert_equal 'eng',     rec.key
-    assert_equal 'ENG',     rec.fifa
+    assert_equal 'ENG',     rec.code
   end
 
 end  # class TestCountry

data/test/test_league.rb

@@ -14,25 +14,25 @@ class TestLeague < MiniTest::Test
 
   def test_search
     rec = League.search_or_create!( 'eng' )
-    assert_equal 'Premier League',  rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal 'Premier League',  rec.name
     assert_equal 'eng.1',           rec.key
     assert_equal 'eng',             rec.country.key
     assert_equal 'England',         rec.country.name
-    # assert_equal true,              rec.club
+    # assert_equal true,              rec.clubs
 
     rec = League.search!( 'eng' )
-    assert_equal 'Premier League',  rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal 'Premier League',  rec.name
     assert_equal 'eng.1',           rec.key
     assert_equal 'eng',             rec.country.key
     assert_equal 'England',         rec.country.name
-    ## assert_equal true,              rec.club
+    ## assert_equal true,              rec.clubs
 
     ## try 2nd call (just lookup)
     rec = League.search_or_create!( 'eng' )
-    assert_equal 'Premier League', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal 'Premier League', rec.name
     assert_equal 'eng.1',          rec.key
     assert_equal 'eng',            rec.country.key
     assert_equal 'England',        rec.country.name
-    ## assert_equal true,                 rec.club
+    ## assert_equal true,                 rec.clubs
   end
 end # class TestLeague

data/test/test_misc.rb

@@ -14,15 +14,15 @@ class TestMisc < MiniTest::Test
 
   def test_season
     rec = Season.search_or_create( '2017-18' )
-    assert_equal '2017/18', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal '2017/18', rec.name
     assert_equal '2017/18', rec.key
 
     rec = Season.search_or_create( '2017/2018' )
-    assert_equal '2017/18', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal '2017/18', rec.name
     assert_equal '2017/18', rec.key
 
     rec = Season.search_or_create( '2017/8' )
-    assert_equal '2017/18', rec.title    ## fix: add name alias (rename title to name!!)
+    assert_equal '2017/18', rec.name
     assert_equal '2017/18', rec.key
   end
 

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: sportdb-sync
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Gerald Bauer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-11 00:00:00.000000000 Z
+date: 2020-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: sportdb-config
+  name: sportdb-catalogs
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.1
+        version: 1.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.1
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: sportdb-models
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.19.0
+        version: 2.0.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.19.0
+        version: 2.0.2
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement